When you update the node which contains the user_reference field, you receive an error message if you leave the field blank.
If the field is empty then foreach loop will not create any element in the array $form_values['uid']. So count() gets a non-existing array.

This patch declares the $form_values['uid'] as an empty array before foreach loop.

CommentFileSizeAuthor
#6 access-7.x-1.x-dev-undefined-index-uid-2410067-6.patch584 bytesluenemann
#3 access-7.x-1.x-dev-undefined-index-uid.patch584 bytesadci_contributor
nodeaccess-7.x-1.4-undefined-index-uid.patch586 bytesadci_contributor
after.png12.71 KBadci_contributor
before.png25.2 KBadci_contributor
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vlad.pavlovic’s picture

vlad.pavlovic CreditAttribution: vlad.pavlovic commented
Version: 7.x-1.4 » 7.x-1.x-dev

Please re-roll this patch against dev version.

vlad.pavlovic’s picture

vlad.pavlovic CreditAttribution: vlad.pavlovic commented
Status: Active » Needs review
adci_contributor’s picture

adci_contributor CreditAttribution: adci_contributor commented
FileSize
access-7.x-1.x-dev-undefined-index-uid.patch584 bytes

Rerolled :)

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented
Status: Needs review » Reviewed & tested by the community

Still applies, thanks @adci_contributor

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet at The University of British Columbia commented

bump

luenemann’s picture

luenemann
Primary language German
Location Südbaden, Germany
CreditAttribution: luenemann commented
Status: Reviewed & tested by the community » Needs review
FileSize
access-7.x-1.x-dev-undefined-index-uid-2410067-6.patch584 bytes

reroll of #2410067-3: Undefined index 'uid' in nodeaccess_insert_user_reference(). moved the fix from
nodeaccess_insert_user_reference()
to
nodeaccess_delete_user_reference()

Rerolled

luenemann’s picture

luenemann
Primary language German
Location Südbaden, Germany
CreditAttribution: luenemann commented
LeDucDuBleuet’s picture

LeDucDuBleuet CreditAttribution: LeDucDuBleuet as a volunteer commented
Status: Needs review » Closed (outdated)
Related issues: +#2099871: User reference case wont work.

This issue is fixed as well with the patch I submitted in : #2099871 User reference case wont work.

Since the other issue goes at the root of the problem for User Reference permissions, I propose we close this one as outdated and work on the other.

Thank you.