| # |
Jul 5th 2017, 18:44 |
 savant |
`allow_failures` is the travis.ci key |
| # |
Jul 5th 2017, 18:44 |
WyriHaximus |
7.2 is nearing stable |
| # |
Jul 5th 2017, 18:43 |
jeremyharris |
someone more familiar with travis.yml could add it to the build matrix (allowing failures for the time being). if there are a bunch of issues it might just be noise until a stable php release happens |
| # |
Jul 5th 2017, 18:43 |
WyriHaximus |
Note that is is 7.2-dev so it can be expected to be unstable but it is a good indicator :) |
| # |
Jul 5th 2017, 18:43 |
WyriHaximus |
https://travis-ci.org/php-api-clients/twitter/jobs/250218773#L175 |
| # |
Jul 5th 2017, 18:42 |
jeremyharris |
cool |
| # |
Jul 5th 2017, 18:42 |
WyriHaximus |
yes |
| # |
Jul 5th 2017, 18:42 |
jeremyharris |
on travis? |
| # |
Jul 5th 2017, 18:42 |
WyriHaximus |
you can test against nightly which is 7.2 at the moment |
| # |
Jul 5th 2017, 18:42 |
jeremyharris |
that’ll probably reveal the errors we need to fix for the current + future versions |
| # |
Jul 5th 2017, 18:41 |
jeremyharris |
since it’s not released yet it’s not on travis yet |
| # |
Jul 5th 2017, 18:41 |
savant |
if it doesnt, then thats a bug and we will fix it asap |
| # |
Jul 5th 2017, 18:40 |
savant |
@mcf it should be fine to work on 7.2 |
| # |
Jul 5th 2017, 18:37 |
mcf |
@admad do you think a CakePHP issue would be a better place to start a discussion about PHP 7.2? I'm happy to open one. |
| # |
Jul 5th 2017, 18:36 |
mcf |
You can just use null if you don't need anything too fancy. |
| # |
Jul 5th 2017, 18:33 |
jarard01 |
if i want to create a form which submits some params for a search, what do i set the 'context' of the form to be? |
| # |
Jul 5th 2017, 17:59 |
mcf |
BTW - the underlying Chronos problem seems to be caused by a known PHP memory corruption bug according to the PHP team. If their suspicions are correct they have told me that the only fix is to use PHP 7.2. There is no workaround for 7.0 or 7.1. |
| # |
Jul 5th 2017, 17:54 |
mcf |
It's a show stopper for me, so I will have to use PHP 7.2 if it does prove that it solves the problem (so far so good with my preliminary testing as I said). |
| # |
Jul 5th 2017, 17:53 |
mcf |
And the Chronos issue is very problematic for those unfortunate enough to hit it. |
| # |
Jul 5th 2017, 17:53 |
mcf |
Just wanted to see if there was any plan yet. |
| # |
Jul 5th 2017, 17:52 |
mcf |
Yes. I'm working through an issue with the Query |
| # |
Jul 5th 2017, 17:48 |
admad |
@mcf have you tried running cake on 7.2? |
| # |
Jul 5th 2017, 17:18 |
mcf |
And it looks like (after preliminary testing) that the only fix is to upgrade to CakePHP 7.2... |
| # |
Jul 5th 2017, 17:18 |
mcf |
The last alpha release comes out tomorrow and I'm working through this issue: https://github.com/cakephp/chronos/issues/147 |
| # |
Jul 5th 2017, 17:15 |
mcf |
Not sure if this is the place to ask, perhaps a github issue would be better.. but can anyone comment on CakePHP's plans for PHP 7.2? |
| # |
Jul 5th 2017, 17:11 |
savant |
@jeremyharris get me a :rubberduck: emoji |
| # |
Jul 5th 2017, 16:47 |
jeremyharris |
np! :beers: |
| # |
Jul 5th 2017, 16:46 |
pedroseco |
ok that’s a wrap for today :slightly_smiling_face: Thanks for the help! |
| # |
Jul 5th 2017, 16:46 |
pedroseco |
i will try with the example I did since probably is the most common one. |
| # |
Jul 5th 2017, 16:46 |
jeremyharris |
if you can articulate what it doesn’t say, create an issue and ping me and I can write it |
| # |
Jul 5th 2017, 16:46 |
jeremyharris |
contributions or suggestions welcome |
| # |
Jul 5th 2017, 16:46 |
pedroseco |
those auto magic associations get me confused sometimes |
| # |
Jul 5th 2017, 16:45 |
jeremyharris |
yeah, cake 3 is smarter when it comes to BTM. Just pass the entities you want to link to each other, as they are |
| # |
Jul 5th 2017, 16:45 |
pedroseco |
not too sure if the Book explains this part very well. It’s a litle bit confusing |
| # |
Jul 5th 2017, 16:45 |
jeremyharris |
heh sweet! |
| # |
Jul 5th 2017, 16:45 |
pedroseco |
lol |
| # |
Jul 5th 2017, 16:45 |
pedroseco |
what a long day! |
| # |
Jul 5th 2017, 16:45 |
pedroseco |
that’s it! ahah! thanks … was driving me crazy… |
| # |
Jul 5th 2017, 16:43 |
jeremyharris |
in fact, you might just be able to change file_id to id on this line: https://gist.github.com/pedroseco/7f960a7399c0ce4aca3dc68051d7fdfa#file-gistfile1-txt-L4 |
| # |
Jul 5th 2017, 16:43 |
jeremyharris |
the only thing you should need in _joinData is the extra column |
| # |
Jul 5th 2017, 16:43 |
jeremyharris |
yeah let the join table figure out the keys itself |