Getting an undefined index on any new taxonomy field i add to a content type
Notice: Undefined index: target_id in Drupal\permissions_by_term\Service\TermHandler->getTidsByNid() (line 68 of modules/contrib/permissions_by_term/src/Service/TermHandler.php).
Drupal\permissions_by_term\Service\TermHandler->getTidsByNid('33') (Line: 72)
* PHP 7.4, Drupal 9.1.2
* Add an existing taxonomy reference field to a content type, for instance: field_tags
* Create node
* Editing the node gives the notice
Add a check to see if the term's target_id key is actually in the "termTargetIdsForField" array. See attached patch.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 3192523-3.patch | 577 bytes | aritrika.d |
| permissions_by_term_fix_undefined_index.patch | 577 bytes | j0sbeir |
Comments
Comment #2
j0sbeir commentedComment #3
aritrika.d commentedComment #4
bdanin commentedThis patch appears to have worked well for me. Thanks.
Comment #5
nattyweb commentedI have this same issue with Drupal 8.9.12 and Permissions by Term 8.x-2.28. Would this same patch work there?
Thanks
Martin
Comment #6
bdanin commented@nattyweb, this is the same setup I'm running (currently on 8.9.11, but would assume 8.9.12 is the same) and the patch appears to be working. I haven't tested it extensively, but let us know if it works for you, and if so, then we can probably set this as RTBC.
Comment #7
nattyweb commentedI can confirm this did indeed fix the problem on Drupal 8.9.12 and Permissions by Term 8.x-2.28.
I checked before applying to make sure the code looked the same at the relevant place in /src/Service/TermHandler.php, which it did.
Thank you!
Comment #8
bdanin commentedComment #10
jepster_Thanks for sharing your patch!
It has been applied to PbT version 8.x-2.29 and 3.1.6.
Comment #11
jepster_