Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UserList filterByGroup false #6476

Open
ounziw opened this issue Feb 28, 2018 · 2 comments
Open

UserList filterByGroup false #6476

ounziw opened this issue Feb 28, 2018 · 2 comments
Labels
Status:Available Reviewed issue, it’s real, we’d review a pull request. Type:Bug Existing functionality not performing as expected.

Comments

@ounziw
Copy link
Contributor

ounziw commented Feb 28, 2018

The documentation says UserList filterByGroup false will exclude the members who belong to the group,

http://documentation.concrete5.org/developers/users-groups/searching-and-sorting-users-userlist-object
$list = new \Concrete\Core\User\UserList();
...
$group = \Group::getByName('Administrators');
$list->filterByGroup($group, false); // Return all non-admins

but actually exlude members who belong to the group and no other group.

Sample Case:
User_A belongs to Group_X
User_B belongs to Group_X and Group_Y
User_C belongs to Group_Y

$list->filterByGroup("Group_X", false) will exlude only User_A. (= User_B is included.)

@katzueno
Copy link
Contributor

I agree if this is true.

@aembler aembler added Type:Bug Existing functionality not performing as expected. Status:Available Reviewed issue, it’s real, we’d review a pull request. labels Mar 1, 2018
@stale
Copy link

stale bot commented Jan 13, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions!

@stale stale bot added the Stale Issues that have been inactive for more than 180 days and will soon be closed label Jan 13, 2020
@KorvinSzanto KorvinSzanto removed the Stale Issues that have been inactive for more than 180 days and will soon be closed label Jan 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status:Available Reviewed issue, it’s real, we’d review a pull request. Type:Bug Existing functionality not performing as expected.
Projects
None yet
Development

No branches or pull requests

4 participants