# |
Mar 10th 2018, 12:44 |
bravo-kernel |
right, I will ask that as well |
# |
Mar 10th 2018, 12:44 |
admad |
the change looks good, though would be nice to have an actual test with custom foreign key |
# |
Mar 10th 2018, 12:44 |
bravo-kernel |
Much much obliged, thanks |
# |
Mar 10th 2018, 12:43 |
bravo-kernel |
thanks |
# |
Mar 10th 2018, 12:43 |
bravo-kernel |
:thumbsup: |
# |
Mar 10th 2018, 12:43 |
admad |
yes |
# |
Mar 10th 2018, 12:43 |
bravo-kernel |
so `?` as the key |
# |
Mar 10th 2018, 12:43 |
bravo-kernel |
aha |
# |
Mar 10th 2018, 12:43 |
admad |
that's how query string should always be set :slightly_smiling_face: to avoid any potential clash with route params |
# |
Mar 10th 2018, 12:43 |
bravo-kernel |
haha |
# |
Mar 10th 2018, 12:43 |
bravo-kernel |
whoa, what's that? |
# |
Mar 10th 2018, 12:42 |
admad |
then better to use `'?' => [$association->foreignKey() => $entity->id]` |
# |
Mar 10th 2018, 12:42 |
bravo-kernel |
yes, like this example `"self": "/cultures?country_id=2"` |
# |
Mar 10th 2018, 12:37 |
admad |
is the 'fk' => $entity->id supposed to endup in URLs query string? |
# |
Mar 10th 2018, 12:30 |
bravo-kernel |
@admad would you mind taking a quick look at this one-line PR? https://github.com/FriendsOfCake/crud-json-api/pull/26/files. Tests pass and looks ok but I would prefer confirmation before merging |
# |
Mar 10th 2018, 12:17 |
admad |
thanks |
# |
Mar 10th 2018, 12:07 |
steinkel |
sure, I'll do that |
# |
Mar 10th 2018, 12:06 |
admad |
and please submit to awesome list if it's not already there :slightly_smiling_face: |
# |
Mar 10th 2018, 12:06 |
gsitex |
@admad :+1: |
# |
Mar 10th 2018, 12:06 |
admad |
@steinkel perhaps the plugin should be suggested in error message here https://github.com/cakephp/app/blob/master/config/requirements.php#L31 |
# |
Mar 10th 2018, 12:05 |
gsitex |
I am not a fan of modifying files in `vendor` folder. But it's a hack anyway. |
# |
Mar 10th 2018, 12:05 |
gsitex |
@steinkel `use IntlCalendar as IC;` fixed the issue ;) |
# |
Mar 10th 2018, 12:00 |
steinkel |
ok, if you get issues feel free to throw us a ticket, thanks |
# |
Mar 10th 2018, 11:59 |
gsitex |
@steinkel ty, I have just installed `cakedc/intl`. I will try to figure out how it works. |
# |
Mar 10th 2018, 11:58 |
steinkel |
@gsitex there must be another class in the include path with the same name |
# |
Mar 10th 2018, 11:57 |
steinkel |
I've used it to bypass some shared hosting restrictions |
# |
Mar 10th 2018, 11:57 |
gsitex |
``` PHP Fatal error: Cannot use IntlCalendar as IntlCalendar because the name is already in use in /home/cigar/prices.cigarworld.com.au/vendor/cakedc/intl/src/IntlGregorianCalendar.php on line 14 ``` |
# |
Mar 10th 2018, 11:56 |
steinkel |
@gsitex @admad cakedc/intl works if you don't need multiple languages |
# |
Mar 10th 2018, 11:45 |
admad |
i feel ya |
# |
Mar 10th 2018, 11:44 |
gsitex |
@admad It's complicated. |
# |
Mar 10th 2018, 11:40 |
admad |
@gsitex no chance explaining to the client he's using a shitty hoster? |
# |
Mar 10th 2018, 11:38 |
gsitex |
@admad ty, I will try that. I really don't want to use L*** Framework instead of Cake |
# |
Mar 10th 2018, 11:37 |
admad |
@gsitex https://github.com/cakedc/intl not sure in what state that plugin is though |
# |
Mar 10th 2018, 11:09 |
turkles |
gives me errors saying the id column doesn't exist, but it does.. and I can see it in the query if I debug, but fails execution. foreign key is a different value than id.. |
# |
Mar 10th 2018, 11:08 |
turkles |
$this->query=$this->Factions->find()->where(['Factions.id' => $this->fac]); foreach($updatelist as $this->target_type) { if (empty($this->factionUpdate[$this->fac][$this->target_type])) $this->factionUpdate[$this->fac][$this->target_type]=0; $this->query->contain($this->target_type,function ($q) { return $q->where([$this->target_type.'.id' => $this->factionUpdate[$this->fac][$this->target_type]]); }); } |
# |
Mar 10th 2018, 11:08 |
turkles |
Now can anyone help me with constraints in a contain in 3.5? |
# |
Mar 10th 2018, 11:07 |
turkles |
https://help.dreamhost.com/hc/en-us/articles/214893957-How-do-I-enable-PHP-extensions-and-loaders- |
# |
Mar 10th 2018, 11:07 |
turkles |
yup |
# |
Mar 10th 2018, 10:56 |
gsitex |
I will try that. |
# |
Mar 10th 2018, 10:56 |
gsitex |
`extension = intl.so` in phprc? |
# |
Mar 10th 2018, 10:44 |
turkles |
my host allows phprc files for that, but has nothing in cPanel |