• Type: Bug
• Resolution: Fixed
• Priority: Blocker
• Fix Version/s: 2.3.0
• Affects Version/s: 2.2.0
• Labels:

  None

[LINKS-122] Can not limit an application to a group that has a # in the id

Collapse comment: Laurence Aumeunier added a comment - 09/Mar/18 15:25

Laurence Aumeunier added a comment - 09/Mar/18 15:25

Widget edition.group should be review too, as it splits on "#" (see #getValue)

Expand comment: Laurence Aumeunier added a comment - 09/Mar/18 15:25

Laurence Aumeunier added a comment - 09/Mar/18 15:25 Widget edition.group should be review too, as it splits on "#" (see #getValue )

Collapse comment: Raphaël Franchet added a comment - 09/Mar/18 11:52, Edited by Raphaël Franchet - 09/Mar/18 11:55

Raphaël Franchet added a comment - 09/Mar/18 11:52 - edited

A search returned me the following occurences

• org.ametys.core.group.GroupIdentity#stringToGroupIdentity
• org.ametys.core.user.UserIdentity#stringToUserIdentity
• org.ametys.plugins.core.group.GroupSearchAction#_searchGroupsByContext
• org.ametys.plugins.core.user.UserSearchAction#_searchUsersByContext

In all cases, we user String#split and we could change this by StringUtils.substringBeforeLast("#") StringUtils.substringAfterLast("#") to allow the '#' character in login/groupId (as we know that '#' is forbiddent in user and group populations)

Expand comment: Raphaël Franchet added a comment - 09/Mar/18 11:52, Edited by Raphaël Franchet - 09/Mar/18 11:55

Raphaël Franchet added a comment - 09/Mar/18 11:52 - edited A search returned me the following occurences org.ametys.core.group.GroupIdentity#stringToGroupIdentity org.ametys.core.user.UserIdentity#stringToUserIdentity org.ametys.plugins.core.group.GroupSearchAction#_searchGroupsByContext org.ametys.plugins.core.user.UserSearchAction#_searchUsersByContext In all cases, we user String#split and we could change this by StringUtils.substringBeforeLast("#") StringUtils.substringAfterLast("#") to allow the '#' character in login/groupId (as we know that '#' is forbiddent in user and group populations)