pydio / pydio-core

Pydio 8 official repository
https://pydio.com
GNU Affero General Public License v3.0
869 stars 289 forks source link

Share User Add Multiple Group #1427

Closed vinc76 closed 6 years ago

vinc76 commented 6 years ago

Hello to the team We are in Core version. We use user shares. We have created several groups that list users. If we assign a first group and validate: no problem. We add a second user group and we validate: The group then added takes the name of the first group added. The second group is not added. We checked in the shares that the user did there is only one group and not both.

I do not speak English very well, my apologies, but if you can help us, thank you, Vincent PS: ADDGROUPENOTVALIDE.JPEG: This is the capture before validation ADDGROUPAFTERVALIDE.JPEG: result after saving addgroupaftervalide addgroupnotvalide

cdujeu commented 6 years ago

Hi, can you provide the exact version of Pydio? Thx

vinc76 commented 6 years ago

Hi, Sorry i forgot, we are in Pydio version 8.2.0 of MAY 22, 2018

Ubuntu 16.04.4/ Apache 2.4 / MariaDB 10 / PHP 7.0 Thx, Vincent

cdujeu commented 6 years ago

hi again - just trying to reproduce that but I can't Do you just use pydio "internal" users or do you use external directories connectors (like ldap ..) ?

vinc76 commented 6 years ago

Hello,

we only use users and internal group at Pydio. Here's a screenshot of groups / users: capture

cdujeu commented 6 years ago

So the EDI group ID is EDIWORKSPACE right?

vinc76 commented 6 years ago

EDI is Label and EDIWORSPACE is Group ID capture

cdujeu commented 6 years ago

ok i'll try to reproduce with similar groups

cdujeu commented 6 years ago

ok now i can reproduce strange behaviors with sub groups - i'll keep you informed

vinc76 commented 6 years ago

Thank's for your help

cdujeu commented 6 years ago

ok should be fixed by https://github.com/pydio/pydio-core/commit/2b80cddda914daad9134e17ea33f098994308cad

vinc76 commented 6 years ago

I'll do a test tomorrow at the latest Thank's a lot

vinc76 commented 6 years ago

I just did the test, I did not do a thorough test based on the users in the group. But it seems that this is corrected. Here is the result in image. capture