# |
Jul 9th 2019, 10:51 |
info315 |
https://github.com/it-novum/openITCOCKPIT/commit/f597a69c3bc8c67dc174c943675e7153440d17da I did it manually now and it was a pain in the ass |
# |
Jul 9th 2019, 10:07 |
challgren |
Whoops is great! |
# |
Jul 9th 2019, 10:07 |
dereuromark |
that already exists :slightly_smiling_face: at least for error handling to click and jump to IDE |
# |
Jul 9th 2019, 09:59 |
neon1024 |
A nice PHP Storm plugin would allow me to click a URL in my code, such as an AJAX call, and it would match the routing, open the controller and take me to the method |
# |
Jul 9th 2019, 09:50 |
challgren |
I <3 PHPStorm |
# |
Jul 9th 2019, 09:47 |
challgren |
Crude but it works |
# |
Jul 9th 2019, 09:45 |
challgren |
replace regex `allowEmptyString\($1null, $2\)` |
# |
Jul 9th 2019, 09:45 |
challgren |
find regex `allowEmptyString\((\'[A-z0-9]*\', )((false)|\'(create|update)\')` |
# |
Jul 9th 2019, 09:45 |
challgren |
Here’s a regex to fix it |
# |
Jul 9th 2019, 09:35 |
info315 |
brb |
# |
Jul 9th 2019, 09:35 |
info315 |
no one have a message, most have a `$when` |
# |
Jul 9th 2019, 09:35 |
challgren |
Do all your methods have a message? |
# |
Jul 9th 2019, 09:34 |
info315 |
@challgren yes |
# |
Jul 9th 2019, 09:34 |
challgren |
Your trying to fix the deprecated warning for the parameter flop? |
# |
Jul 9th 2019, 09:34 |
conehead |
I don't get the point. You can replace all that you want with one command in PHPStorm |
# |
Jul 9th 2019, 09:33 |
info315 |
> All I did for the allowEmptyString was search for it and fix everyone didn't really take too long I need to fix more than 360 lines... Thats annoying especially because i needed to do this with 3.7 because of allowEmpty was replaced |
# |
Jul 9th 2019, 09:33 |
conehead |
``` Basically i want to string replace all `allowEmptyString` with `allowEmptyStringLegacy` ``` I ment you could do this. Select your folder on the left and press `ctrl + shift + r` |
# |
Jul 9th 2019, 09:32 |
info315 |
@conehead Nop my IDE (phpStrom) can not fix deprecated argument order |
# |
Jul 9th 2019, 09:21 |
mehov |
https://github.com/cakephp/cakephp/issues/8777 |
# |
Jul 9th 2019, 09:17 |
challgren |
All I did for the allowEmptyString was search for it and fix everyone didn't really take too long |
# |
Jul 9th 2019, 09:17 |
dereuromark |
i thought you guys and rector will take over^^ |
# |
Jul 9th 2019, 09:16 |
challgren |
Yeah I think @dereuromark has been slacking |
# |
Jul 9th 2019, 09:15 |
conehead |
And I think usually you should be able to perform such tasks with https://github.com/dereuromark/upgrade But currently it seems to only support up to version 3.7 |
# |
Jul 9th 2019, 09:14 |
neon1024 |
:point_up: |
# |
Jul 9th 2019, 09:13 |
conehead |
@info315 Can't your IDE do this? |
# |
Jul 9th 2019, 09:05 |
s.ahmad |
@challgren Thanks :slightly_smiling_face: |
# |
Jul 9th 2019, 09:05 |
challgren |
IDK I suck at Validation |
# |
Jul 9th 2019, 09:05 |
info315 |
Basically i want to string replace all `allowEmptyString` with `allowEmptyStringLegacy` |
# |
Jul 9th 2019, 09:05 |
challgren |
`Validator::addDefaultProvider('custom', 'App\Model\Validation');` wouldnt that work? |
# |
Jul 9th 2019, 09:04 |
info315 |
@challgren Yep i found this as well, but I'm looking for a way where i don't need to modify all files manualy |
# |
Jul 9th 2019, 09:04 |
challgren |
@s.ahmad https://book.cakephp.org/3.0/en/appendices/3-8-migration-guide.html |
# |
Jul 9th 2019, 09:03 |
s.ahmad |
Hey what's new in cake3.8 |
# |
Jul 9th 2019, 09:03 |
challgren |
@info315 https://book.cakephp.org/3.0/en/core-libraries/validation.html#adding-validation-providers |
# |
Jul 9th 2019, 08:59 |
martin |
or authentication middleware |
# |
Jul 9th 2019, 08:46 |
challgren |
Maybe someone else has a better way but thats what I would do |
# |
Jul 9th 2019, 08:44 |
info315 |
Is this the only possible method? |
# |
Jul 9th 2019, 08:43 |
challgren |
Yep |
# |
Jul 9th 2019, 08:43 |
info315 |
But than i would need to modify all my tables to extends my `AppTable` instead of `Table` right? |
# |
Jul 9th 2019, 08:42 |
challgren |
@info315 You could write a “AppTable” class and define it there |
# |
Jul 9th 2019, 08:40 |
slackebot |
``` |
# |
Jul 9th 2019, 08:40 |
info315 |
Is it possible to overwrite the default `Validator` class with an `AppValidator` or some like that? I found this in the docs, but i don't know how to use it. https://book.cakephp.org/3.0/en/core-libraries/validation.html#creating-reusable-validators What I'm trying to achieve is the ability to use this in my Tables: ``` public function validationDefault(Validator $validator) { $validator ->methodFromMyAppValidator('id') |