# |
Apr 11th 2019, 14:20 |
neothermic |
@admad that's like trying to do a continuity edit by taking each chapter of a book in isolation :P |
# |
Apr 11th 2019, 14:19 |
admad |
well can run them on each folder/package separtely |
# |
Apr 11th 2019, 14:19 |
neothermic |
(which means we've had to do horrible hacks to work around ClassRegistry::init's limitations WRT Plugins) |
# |
Apr 11th 2019, 14:19 |
admad |
hehe |
# |
Apr 11th 2019, 14:19 |
neothermic |
(and we do that a lot) |
# |
Apr 11th 2019, 14:18 |
neothermic |
apparently if you have two files which declare the same class and have no namespaces, they freak out. |
# |
Apr 11th 2019, 14:18 |
admad |
though you'll have to use an older version of the tools (if they exist) for old PHP :slightly_smiling_face: |
# |
Apr 11th 2019, 14:18 |
neothermic |
they don't like our codebase |
# |
Apr 11th 2019, 14:18 |
admad |
@neothermic using a static analysis tool like psalm or phpstan would help you find and fix such problems. |
# |
Apr 11th 2019, 14:16 |
neothermic |
lots of silent bugs, like doing `$foo = '';` and later `$foo[] = $something;` |
# |
Apr 11th 2019, 14:15 |
neothermic |
thankfully I've been working on that on the side too |
# |
Apr 11th 2019, 14:15 |
admad |
@ricksaccous |
# |
Apr 11th 2019, 14:15 |
neothermic |
we won't be able to move to 4 directly without first updating PHP too :P |
# |
Apr 11th 2019, 14:14 |
ricksaccous |
from what i understand anyway |
# |
Apr 11th 2019, 14:14 |
ricksaccous |
so you can bounce to 4 :slightly_smiling_face: |
# |
Apr 11th 2019, 14:14 |
dereuromark |
https://github.com/dereuromark/cakephp-shim/blob/2.x/Model/ShimModel.php#L312-L338 => saveFieldById() is the proper one for 2.x, and the other one can throw soft deprecation notice to help upgrading. Just have to leverage the right planning and tooling to make things easier for you. |
# |
Apr 11th 2019, 14:14 |
ricksaccous |
4 is non-deprecated 3 |
# |
Apr 11th 2019, 14:13 |
neothermic |
I suspect I'll get to 3 and you'll all be in 4 ;) |
# |
Apr 11th 2019, 14:13 |
ricksaccous |
weeeee |
# |
Apr 11th 2019, 14:13 |
ricksaccous |
join us in 3 we're waiting for you :slightly_smiling_face: |
# |
Apr 11th 2019, 14:12 |
neothermic |
1.3 to 2.x took ~3 months and that was with the upgrade console too |
# |
Apr 11th 2019, 14:12 |
neothermic |
this codebase is.. fun |
# |
Apr 11th 2019, 14:12 |
dereuromark |
nah, the original one is bad, it uses a stateful ->id usually |
# |
Apr 11th 2019, 14:11 |
admad |
or make all your table extends an `AppTable` which implements that method |
# |
Apr 11th 2019, 14:11 |
dereuromark |
actually, if you were pro-active you could have fixed it already in 2.x to use the shim version there, wich works future-proof :slightly_smiling_face: |
# |
Apr 11th 2019, 14:11 |
neothermic |
oh, the shim puts it back? |
# |
Apr 11th 2019, 14:11 |
dereuromark |
problem solved |
# |
Apr 11th 2019, 14:11 |
dereuromark |
or use the shim plugin :slightly_smiling_face: |
# |
Apr 11th 2019, 14:10 |
neothermic |
that's painful. if saveField is gone from 3.0, I have 112 places to fix to migrate this codebase :P |
# |
Apr 11th 2019, 13:38 |
conehead |
just seen the service layer plugin the first time. I really like the idea of having a service layer. Might try it out when I finished migrating! |
# |
Apr 11th 2019, 13:28 |
ricksaccous |
you can do it! |
# |
Apr 11th 2019, 13:20 |
admad |
time to dig the trenches |
# |
Apr 11th 2019, 13:20 |
neon1024 |
Moral is wavering! |
# |
Apr 11th 2019, 13:20 |
admad |
hah |
# |
Apr 11th 2019, 13:20 |
neon1024 |
Day three! |
# |
Apr 11th 2019, 13:20 |
neon1024 |
In that horrible zone where it works locally, the test suite passes, the dev versions of the consuming front-ends work, but prod is not happy |
# |
Apr 11th 2019, 13:20 |
admad |
marching forward or retreating in the battle? :slightly_smiling_face: |
# |
Apr 11th 2019, 13:19 |
neon1024 |
I’m battling with league/fractal at the moment |
# |
Apr 11th 2019, 13:19 |
neon1024 |
Then there are the form resource bits I think |
# |
Apr 11th 2019, 13:19 |
neon1024 |
Then everything in my fork is covered off in the plugin I think |
# |
Apr 11th 2019, 13:18 |
neon1024 |
I’d like to get the pagination in ideally |