Tickets :: [#12102] Selected user list of share advanced is unsorted

6.0.0-alpha12

6/8/25

Summary	Selected user list of share advanced is unsorted
Queue	Kronolith
Queue Version	4.0.3
Type	Bug
State	Not A Bug
Priority	1. Low
Owners
Requester	gerard.breiner (at) ias (dot) u-psud (dot) fr
Created	03/08/2013 (4475 days ago)
Due
Updated	03/13/2013 (4470 days ago)
Assigned	03/13/2013 (4470 days ago)
Resolved	03/13/2013 (4470 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

03/13/2013 02:41:30 PM	Jan Schneider	Comment #4 State ⇒ Not A Bug	Reply to this comment
If the argument passed to a method is not the same inside this method, you have a badly broken PHP installation.

03/13/2013 01:58:21 PM	gerard (dot) breiner (at) ias (dot) u-psud (dot) fr	Comment #3	Reply to this comment
[Show Quoted Text - 10 lines][Hide Quoted Text] I find out that at the lines 68 and 192 the function listUsers is called with the value ³true² that means we want a sorted list but the issue is that the value ³true² is not transmit to the variable $sort of listUsers. This function is defined in pear/php/Horde/Auth/Ldap.php. I have no idea what you mean by that. Horde_Auth_Ldap::listUsers() is called in the kronolith code if you use an LDAP authentication backend, so true is passed directly to the method, and further passed to _sort() further down. I don't see how the user list could not be sorted if using LDAP. Yet, it is?. I do understand that listUsers is define as function listUsers($sort = false). Kronolith call it by listUsers(true), so $sort should be "true" but $sort remains "false" . For testing that true is not passed to the method, I modify temporarily in Ldap.php $sort = true then I noticed my ldap user list is sorted. Therefore it is a fact that true is not passed to the method even though I agree with you it should be. Best regards. Gérard

03/13/2013 09:38:11 AM	Jan Schneider	Comment #2 Priority ⇒ 1. Low State ⇒ Feedback	Reply to this comment
I find out that at the lines 68 and 192 the function listUsers is called with the value ³true² that means we want a sorted list but the issue is that the value ³true² is not transmit to the variable $sort of listUsers. This function is defined in pear/php/Horde/Auth/Ldap.php. I have no idea what you mean by that. Horde_Auth_Ldap::listUsers() is called in the kronolith code if you use an LDAP authentication backend, so true is passed directly to the method, and further passed to _sort() further down. I don't see how the user list could not be sorted if using LDAP.

03/08/2013 01:39:43 PM	gerard (dot) breiner (at) ias (dot) u-psud (dot) fr	Comment #1 Priority ⇒ 2. Medium Type ⇒ Bug Summary ⇒ Selected user list of share advanced is unsorted Due ⇒ 03/08/2013 Queue ⇒ Kronolith Milestone ⇒ Patch ⇒ No State ⇒ Unconfirmed	Reply to this comment
It appears that the select user list of shore advanced is unsorted. The template that is in charge of this is kronolith/templates/chunks/permissions.inc I find out that at the lines 68 and 192 the function listUsers is called with the value ³true² that means we want a sorted list but the issue is that the value ³true² is not transmit to the variable $sort of listUsers. This function is defined in pear/php/Horde/Auth/Ldap.php. In the waiting of an answer to this issue I modified permissions.inc as follow : -- <?php foreach ($auth->listUsers(true) as $user): ?> ++ <?php $userslist = $auth->listUsers(); sort($userslist); foreach ($userslist as $user): ?> Many thanks. Best regards Gérard Breiner