Two problems in subuser_load_all:

  1. If we set the children argument to FALSE to check get the parent user for a subuser we get the user id of the subuser itself. This is because we need to take the value from the other endpoint.
  2. If we have more than one relation in the system where the enpoint user is allowed, for example a 'node -> belongs_to -> user', subuser_load_all would return also the node ids in the $user array.

I attach a patch right away, as I got an issue number.

CommentFileSizeAuthor
#4 1256938-load_all.patch2.11 KBboombatower
#2 subuser-loadall-1256938-2.patch1.24 KBroderik
#1 subuser-subuser_load_all_fix-1256938-1.patch2.45 KBmarcusx

Comments

marcusx’s picture

marcusx commented
Status: Active » Needs review
StatusFileSize
new subuser-subuser_load_all_fix-1256938-1.patch2.45 KB

Patch attached...

roderik’s picture

roderik
Primary language Dutch
Location Amsterdam,NL / Budapest,HU
commented
StatusFileSize
new subuser-loadall-1256938-2.patch1.24 KB

Implemented caching using drupal_static() - because with #1213566: Add View/Edit/Delete Functionality, it can be called more than twice for the same account.

boombatower’s picture

boombatower commented
Status: Needs review » Fixed

Rewrote quite a bit to make variables clearer. Also, took advantage of ->entityCondition('bundle', 'subuser') instead of if condition in loop. Thanks.

Looks like reload_load() fails currently if you pass it a vid.

boombatower’s picture

boombatower commented
StatusFileSize
new 1256938-load_all.patch2.11 KB

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

  • Commit dd806c5 on 7.x-2.x, 8.x-1.x by boombatower: 
    Issue #1256938 by marcusx, roderik, and boombatower: Correct...