Under some circumstances that I haven't quite nailed down, entities have a UID field but the field is empty; attempts to load using that value results in a Notice. For whatever reason, this doesn't seem to happen if I am using a highly privileged account (but shows up while testing more locked-down roles).
Notice: Trying to get property 'entity' of non-object in Drupal\eck\Entity\EckEntity->getOwner() (line 32 of modules/contrib/eck/src/Entity/EckEntity.php).
Drupal\eck\Entity\EckEntity->getOwner() (Line: 41)
Drupal\eck\Entity\EckEntity->getOwnerId() (Line: 79)
Drupal\eck\EckEntityAccessControlHandler->checkAccess(Object, 'view', Object) (Line: 105)
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | eck-get_owner_notices_empty_uid-3020141-upd.patch | 760 bytes | heathergaye |
| #5 | interdiff-3038397-2-5.txt | 657 bytes | manuel.adan |
| #5 | eck-get_owner_notices_empty_uid-3020141-5.patch | 773 bytes | manuel.adan |
| #2 | eck-3020141-null-entity-get-owner.patch | 518 bytes | elfin_john |
Comments
Comment #2
elfin_john commentedComment #3
romixuaComment #4
romixuaComment #5
manuel.adan#2 does not prevent NULL response from getOwner, but silences the notify in case uid is empty. A quick way is:
This automatically takes the first value (uid is not multivalue anyway) and evaluates to NULL if the field is empty. The same happens in the getOwnerId method. If the uid field is empty, getOwner and getOwnerId should also return NULL.
Focusing this issue on that. About why the uid field was empty, more indications are needed to help on that; open a new issue if something else found.
Comment #6
heathergaye commentedRe-roll of previous patch, which no longer applies.
Comment #7
pifagorComment #9
pifagor