josegonzalez / cakephp-entity Goto Github PK

Hello there,

I see in the repo that there were many releases (3.0.0 being the latest) since 1.0.0 and I was wondering if packagist could be updated accordingly. I would like to freeze the version of the library to a specific one and not be tied to a dev-master branch just in case of anything changes in the future.

Is there any chance this could be done??

hi
in readme say requirement to php 5.3 but short array and JsonSerializable as of php 5.4.

if your target is 5.3 , i change and downgrade to 5.3 , send pull request?

Hello!

The headline on this repo home page says:
CakePHP2: Deprecated Table/Entity wrapper around old Model system

Is it "deprecated" because something else for Cake 2 has replaced this? Or because Cake3 is out?

Although the documentation says by default the find still returns array,
When I extended my models from Table, it's by default returning entities.
The only problem with this is that:
When I try a find('threaded') it return erros:

Warning (4096): Argument 1 passed to Hash::get() must be of the type array, object given, called in /var/www/html/sabhio/lib/Cake/Utility/Hash.php on line 982 and defined [CORE/Cake/Utility/Hash.php, line 43]
Warning (4096): Argument 1 passed to Hash::get() must be of the type array, object given, called in /var/www/html/sabhio/lib/Cake/Utility/Hash.php on line 983 and defined [CORE/Cake/Utility/Hash.php, line 43]
Warning (2): array_merge() [function.array-merge]: Argument #1 is not an array [CORE/Cake/Utility/Hash.php, line 988]

In Table::convertToEntity there is a a check on a set id field

if (is_null($data) || empty($data[$this->alias]['id'])) {

My sight in this plugin isn't deep enough to understand if this limitation is needed, but if we change this to

if (is_null($data) || empty($data[$this->alias][$this->primaryKey])) {

this works in first short tests and we don't have to change our models to use id fields