Project

General

Profile

Actions

Bug #24303

closed

Subgroups field in usergroup edit form should be renamed in "Is subgroup of"

Added by Nico de Haen over 13 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
Start date:
2010-12-06
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
4.5
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

The subgroup selector in the backend usergroup edit form has the misleading name "Subgroups". It could be interpreted in the sense that the selected groups become subgroups of the current group record. In fact the current group record becomes a subgroup of the selected groups. This would be clearer if the labe would say: "Is subgroup of" or "Parent groups".

Even the help text is wrong:
Select backend user groups which are automatically included for members of this group.

Screenshot attached
(issue imported from #M16694)


Files

Usergroup.png (25.4 KB) Usergroup.png Administrator Admin, 2010-12-06 22:29
Actions #1

Updated by Urs Braem over 12 years ago

  • Target version deleted (0)

+1
This should really be fixed

Actions #2

Updated by Urs Braem almost 12 years ago

push

Actions #3

Updated by Ernesto Baschny almost 12 years ago

It's not really a parent / child relationship. In fact when you add "subgroups" (name is indeed misleading) you are adding all the permissions from these other groups as well.

So at the end the field might be called: "Add permissions from other groups" or something, as this is what really will happen.

What do you think?

Actions #4

Updated by Urs Braem over 11 years ago

Hi Ernesto

excuse my late reply, I didn't get a notification, strange.

"Add permissions from" or similar would be a better label than "subgroups", yes.

AFAIK only positive permissions are added, and there is no way to take away a permission by adding another BE user group.
So the explicit label would enlighten that too, I guess, if that's correct? That would be good.

Cheers
Urs

Actions #5

Updated by Urs Braem over 10 years ago

  • Target version set to 6.2.0

Request to include this into 6.2 LTS

Actions #6

Updated by Mathias Schreiber over 9 years ago

  • Target version changed from 6.2.0 to 7.1 (Cleanup)
  • Is Regression set to No
  • Sprint Focus set to On Location Sprint
Actions #7

Updated by Zbigniew Jacko over 9 years ago

I'm working on it.

Actions #8

Updated by Gerrit Code Review over 9 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36396

Actions #9

Updated by Gerrit Code Review over 9 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36396

Actions #10

Updated by Gerrit Code Review over 9 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36396

Actions #11

Updated by Zbigniew Jacko over 9 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #12

Updated by Anja Leichsenring over 8 years ago

  • Sprint Focus deleted (On Location Sprint)
Actions #13

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF