# |
Jun 18th 2019, 14:07 |
ricksaccous |
let me see if that was the problem all along |
# |
Jun 18th 2019, 14:07 |
ricksaccous |
basically i had my entity implementing identityInterface but just left the methods blank |
# |
Jun 18th 2019, 14:07 |
ricksaccous |
cool |
# |
Jun 18th 2019, 14:07 |
ricksaccous |
getOriginalData just works |
# |
Jun 18th 2019, 14:07 |
ricksaccous |
oh when i take my own implementation out |
# |
Jun 18th 2019, 14:05 |
ricksaccous |
hehe |
# |
Jun 18th 2019, 14:05 |
ricksaccous |
i realize i have getOriginalData in my entity but never implemented it |
# |
Jun 18th 2019, 14:05 |
ricksaccous |
i see i see |
# |
Jun 18th 2019, 14:05 |
jeremyharris |
so using a custom identity class would solve the problem |
# |
Jun 18th 2019, 14:04 |
jeremyharris |
I think it’s related to this: https://github.com/cakephp/authentication/issues/260 |
# |
Jun 18th 2019, 14:04 |
admad |
IDK, never used the plugin. I just checked the methods the identitydecorator class provides :slightly_smiling_face: |
# |
Jun 18th 2019, 14:04 |
jeremyharris |
I haven’t used the helper, just looking at the code atm |
# |
Jun 18th 2019, 14:03 |
jeremyharris |
shouldn’t the helper do that though admad? |
# |
Jun 18th 2019, 14:03 |
ricksaccous |
no |
# |
Jun 18th 2019, 14:03 |
admad |
@ricksaccous did you try `getOriginalData()` ? |
# |
Jun 18th 2019, 14:03 |
ricksaccous |
I'm guessing it's the map resolver that's adding this stuff? |
# |
Jun 18th 2019, 14:02 |
ricksaccous |
@jeremyharris https://gist.github.com/Modicrumb/8b98834d94e53c972d657007dfa56ef8 |
# |
Jun 18th 2019, 14:00 |
ricksaccous |
okay |
# |
Jun 18th 2019, 14:00 |
jeremyharris |
did you provide a custom identity class for authentication? can you gist your middleware? |
# |
Jun 18th 2019, 13:58 |
ricksaccous |
so i've opted to just not use it for now |
# |
Jun 18th 2019, 13:57 |
ricksaccous |
also this causes the IdentityHelper provided to throw an error |
# |
Jun 18th 2019, 13:57 |
ricksaccous |
how do i just grab the identity portion of it? |
# |
Jun 18th 2019, 13:57 |
ricksaccous |
and it's morphing my identity object |
# |
Jun 18th 2019, 13:57 |
ricksaccous |
so in middle ware i'm loading authentication, then authorization, then the requestAuthorization middleware |
# |
Jun 18th 2019, 13:56 |
slackebot1 |
<ricksaccous> |
# |
Jun 18th 2019, 13:54 |
ricksaccous |
I'm having a bit of difficulty on something that seems pretty simple |
# |
Jun 18th 2019, 13:52 |
neothermic |
there's also clear concepts that this codebase was never actually finished... |
# |
Jun 18th 2019, 13:47 |
jeremyharris |
I even worked on cake back in the svn days and even that history is preserved |
# |
Jun 18th 2019, 13:46 |
jeremyharris |
silly me for thinking that 2.x was a new repo |
# |
Jun 18th 2019, 13:45 |
jeremyharris |
oh nice |
# |
Jun 18th 2019, 13:45 |
ndm |
https://github.com/cakephp/cakephp/tree/1.2.12 ;) |
# |
Jun 18th 2019, 13:45 |
neothermic |
possibly? but the fact that the function they were doing these returns in was void too... I suspect caffeine induced awakeness |
# |
Jun 18th 2019, 13:44 |
jeremyharris |
I think some other framework might return view vars, so maybe they did it out of habit |
# |
Jun 18th 2019, 13:44 |
neothermic |
best part was, they were doing these `return $this-set()` inside of a function that was `@return void` too |
# |
Jun 18th 2019, 13:43 |
neothermic |
no idea what the dev was doing when they returned those |
# |
Jun 18th 2019, 13:43 |
neothermic |
it was void too |
# |
Jun 18th 2019, 13:43 |
neothermic |
sweet |
# |
Jun 18th 2019, 13:43 |
jeremyharris |
@neothermic I couldn’t find a repo, but you can look at the source via the api docs: https://api.cakephp.org/1.2/ |
# |
Jun 18th 2019, 13:34 |
neothermic |
hmm, odd question, but is there a repo copy of 1.x anywhere? I just want to check to see if the reason why this 2.x code tries to `return` on a `$this->set()` might've been because 1.x returned _something_... |
# |
Jun 18th 2019, 13:29 |
peppejaripappalardo |
thx... |
# |
Jun 18th 2019, 13:18 |
a.didier |
i need to make this plugins (https://github.com/loadsys/JSON-API) works with a 2.10.17... But the plugins seems to have been made for a custom version of cake or a very old one... |