Jingga/phpOMS
1
0
Fork 0
mirror of https://github.com/Karaka-Management/phpOMS.git synced 2026-01-11 09:48:40 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

fix relation bug

Browse Source
This commit is contained in:
Dennis Eichhorn 2020-12-19 20:27:58 +01:00
parent f5e12ba33d
commit fe58dd74e8
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
DataStorage/Database/DataMapperAbstract.php
View File

@ -2003,12 +2003,12 @@ class DataMapperAbstract implements DataMapperInterface
? $mapper::get($values, RelationType::ALL, $depth)
: $mapper::get($values, RelationType::ALL, $depth, static::$hasMany[$member]['by']);
if (!($isPublic = $refProp->isPublic())) {
if (!$refProp->isPublic()) {
$refProp->setAccessible(true);
$refProp->setValue($obj, !\is_array($objects) && !isset(static::$hasMany[$member]['conditional'])
$refProp->setValue($obj, (!\is_array($objects) && (!isset(static::$hasMany[$member]['conditional']) || \is_array($values)) // @todo: \is_array($values) is weird, was necessary for the itemmanagement list at some point, but only suddenly????!!!!
? [$mapper::getObjectId($objects) => $objects]
: $objects
);
));
$refProp->setAccessible(false);
} else {
$obj->{$member} = !\is_array($objects) && !isset(static::$hasMany[$member]['conditional'])
@ -2138,6 +2138,7 @@ class DataMapperAbstract implements DataMapperInterface
* @return mixed
*
* @todo: in the future we could pass not only the $id ref but all of the data as a join!!! and save an additional select!!!
* @todo: only the belongs to model gets populated the children of the belongsto model are always null models. either this function needs to call the get for the children, it should call get for the belongs to right away like the has many, or i find a way to recursevily load the data for all sub models and then populate that somehow recursively, probably too complex.
*
* @since 1.0.0
*/