| # |
Jul 11th 2017, 14:36 |
 jeremyharris |
:) |
| # |
Jul 11th 2017, 14:36 |
jeremyharris |
uh oh |
| # |
Jul 11th 2017, 14:36 |
neothermic |
`run your tests` yeaaaaahhhh... |
| # |
Jul 11th 2017, 14:36 |
jeremyharris |
you only have to read a small doc at a time, run your tests, move on |
| # |
Jul 11th 2017, 14:35 |
jeremyharris |
that’s what I did. most of the minor versions go really quick, but it ensures you don’t miss anything |
| # |
Jul 11th 2017, 14:35 |
neothermic |
so from 2.x to 3.0, and then up each 3.x to latest? |
| # |
Jul 11th 2017, 14:35 |
jeremyharris |
I recently upgraded a small to medium app from 2.6 to 3.4, minor at a time |
| # |
Jul 11th 2017, 14:34 |
jeremyharris |
probably best to go minor versions at a time. it’s a lot less to think about and the migration docs are pretty solid on them |
| # |
Jul 11th 2017, 14:34 |
neothermic |
Yeah, I'm just debating how far into 3.x I go from 2.x; I possibly wont' go right to 3.6 if there's depreciations (for 4.0) in it |
| # |
Jul 11th 2017, 14:33 |
jeremyharris |
yeah, much easier to fix the deprecations along the way. the 3.x change will be massive though, so plan time for it. well worth it after the fact |
| # |
Jul 11th 2017, 14:32 |
neothermic |
So I've got a cleaner upgrade path |
| # |
Jul 11th 2017, 14:32 |
neothermic |
but this is why I also want to get past 2.8 :slightly_smiling_face: |
| # |
Jul 11th 2017, 14:28 |
neothermic |
Yeah, the 2.x -> 3.x change is something we're going to have to tackle in our codebase at some point before 2.x is moonlighted |
| # |
Jul 11th 2017, 14:27 |
inoas |
there is no reason to change everything... the 2.x -> 3.x break and the middleware change is already a huge change alltogether |
| # |
Jul 11th 2017, 14:26 |
inoas |
$options schema validator |
| # |
Jul 11th 2017, 14:26 |
inoas |
NullInflector, AutoTables warning/exception, strict types, etc |
| # |
Jul 11th 2017, 14:26 |
inoas |
and I would just focus on making it less "trappy" |
| # |
Jul 11th 2017, 14:26 |
inoas |
I like how cakephp3 is |
| # |
Jul 11th 2017, 14:26 |
jeremyharris |
yeah that’s been the best thing the framework has done in its maturing - move away from too much magic |
| # |
Jul 11th 2017, 14:26 |
inoas |
hmic well at leas that's a "4.0 must have" feature... unless all that gibberish |
| # |
Jul 11th 2017, 14:25 |
hmic |
like the autotables warning lately |
| # |
Jul 11th 2017, 14:25 |
hmic |
there should be an option in debug: "show me all the magic i'm consuming" |
| # |
Jul 11th 2017, 14:24 |
neothermic |
Yeah, sometimes magic is handy. sometimes you work against the magic :) |
| # |
Jul 11th 2017, 14:18 |
jeremyharris |
was a Good Idea At The Time |
| # |
Jul 11th 2017, 14:18 |
jeremyharris |
if I recall correctly I’m responsible for some of that mess :P |
| # |
Jul 11th 2017, 14:17 |
neothermic |
yikes :slightly_smiling_face: |
| # |
Jul 11th 2017, 14:15 |
jeremyharris |
here’s the hunk if you want to figure it out: https://api.cakephp.org/1.3/source-class-FormHelper.html#255-282 |
| # |
Jul 11th 2017, 14:14 |
admad |
yes, 1.x did weird internal magic to decide which action/url to use based on request data |
| # |
Jul 11th 2017, 14:14 |
neothermic |
that might explain some of the redundant ones in this code |
| # |
Jul 11th 2017, 14:13 |
neothermic |
so in 1.3 you had to explicitly add an action? |
| # |
Jul 11th 2017, 14:13 |
dereuromark |
didnt have it |
| # |
Jul 11th 2017, 14:13 |
dereuromark |
one of my most important fixes back in the day when moving to 2.x, so no, 1.3 |
| # |
Jul 11th 2017, 14:12 |
neothermic |
out of 120 instances where we have the word `action` somewhere in a `create` |
| # |
Jul 11th 2017, 14:12 |
neothermic |
got 58 occurrences to check ¬_¬ |
| # |
Jul 11th 2017, 14:10 |
jeremyharris |
@neothermic I couldn’t remember if it was or not, but looking at the code it seems it is set by default if url is empty |
| # |
Jul 11th 2017, 14:09 |
neothermic |
so I have to evaluate if a) I need to move it into the url bit, or b) just cull it as it's redundant |
| # |
Jul 11th 2017, 14:09 |
neothermic |
it should, but we have things like `?php echo $this->Form->create('EnquirySubject', array('action' => 'add'));?` |
| # |
Jul 11th 2017, 14:09 |
neothermic |
hey, no need to slap |
| # |
Jul 11th 2017, 14:09 |
hmic |
should be seemless |
| # |
Jul 11th 2017, 14:08 |
neothermic |
(working on the 2.8 update from 2.7.x) |
| # |
Jul 11th 2017, 14:08 |
hmic |
slaps neothermic for asking about 1.3 still :P |