You are here: Foswiki>Tasks Web>Item10819 (13 Dec 2017, GeorgeClark)Edit Attach

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

pencil
Priority: Normal
Current State: Confirmed
Released In: n/a
Target Release: minor
Applies To: Extension
Component: TopicUserMappingContrib
Branches:
Reported By: PaulHarvey
Waiting For: PaulHarvey
Last Change By: GeorgeClark
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
 

ItemTemplate edit

Summary New groups UI mangles nested groups when {RenderLoggedInButUnknownUsers} enabled
ReportedBy PaulHarvey
Codebase 1.1.3, trunk
SVN Range
AppliesTo Extension
Component TopicUserMappingContrib
Priority Normal
CurrentState Confirmed
WaitingFor PaulHarvey
Checkins
TargetRelease minor
ReleasedIn n/a
CheckinsOnBranches
trunkCheckins
masterCheckins
ItemBranchCheckins
Release02x01Checkins
Release02x00Checkins
Release01x01Checkins
Topic revision: r9 - 13 Dec 2017, GeorgeClark
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License    Legal Imprint    Privacy Policy