Item10819: New groups UI mangles nested groups when {RenderLoggedInButUnknownUsers} enabled

Whenever we manipulate a group containing other groups using the new groups UI, it seems that USERINFO has tried to expand details about a group, and we end up with a group that looks like UnknownUser (AcaciaAdminGroup), JoeUser, JillUser where it should be AcaciaAdminGroup, JoeUser, JillUser

-- PaulHarvey - 01 Jun 2011

I guess I need to confirm this on 1.1.3, we're running trunk

-- PaulHarvey - 01 Jun 2011

This doesn't apply to release branch

-- PaulHarvey - 01 Jun 2011

The fault was caused by enabling {RenderLoggedInButUnknownUsers} to work around the fact that cUIDs like BaseUserMapping_333 interpret the _ to mean an escape code which generates some octets that create illegal UTF sequences (when you're running UTF).

I lowered to normal because this setting is not documented and therefore not really supported I guess...

-- PaulHarvey - 03 Jun 2011

basicaly, it might be easiest to complete the addusertogroup feature to support adding groups.

which is now possible, due to the work done to store the unexpended list.

-- SvenDowideit - 03 Jun 2011

I think this no longer applies and can be closed. Groups are not expanded.

-- GeorgeClark - 28 Oct 2011

This is still broken. Any missing group or user renders as UnknownUser (TheMissingUser). This also breaks the API, it's impossible to remove a user or group rendered that way. This seems to be an architectural issue. The render setting RenderLoggedInButUnknowUsers is actually applied in the Users->getWikiName() routine. This is not a render function. This decision really ought to be made in Render.pm, when it's actually rendering the wikiname. Part of the problem is that getWikiName always returns something, so Render can't know when to render an unknown wikiname.

This probably needs a proposal. But getWikiName should either return undef when the wikiname is unknown, or it needs a flag to request that it return undef when no wikiname is known.

-- GeorgeClark - 13 Dec 2017

Actually this setting doesn't even do what it suggests. It claims it "obscures" the identity of the user when a WikiName does not exist. However the actual implementation, it returns "UnknownUser ($cUID)" when enabled, and "$cUID" when not enabled. So it is not actually obscuring anything. Maybe this ought to just be removed.

-- GeorgeClark - 13 Dec 2017

Commenting is disabled while not logged in