6.0.0-git
2019-04-21

[#12102] Selected user list of share advanced is unsorted
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 2013-03-08 (2235 days ago)
Due
Updated 2013-03-13 (2230 days ago)
Assigned 2013-03-13 (2230 days ago)
Resolved 2013-03-13 (2230 days ago)
Milestone
Patch No

History
2013-03-13 14:41:30 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.
2013-03-13 13:58:21 gerard (dot) breiner (at) ias (dot) u-psud (dot) fr Comment #3 Reply to this comment

[Show Quoted Text - 10 lines]
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

2013-03-13 09:38:11 Jan Schneider Comment #2
State ⇒ Feedback
Priority ⇒ 1. Low
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.
2013-03-08 13:39:43 gerard (dot) breiner (at) ias (dot) u-psud (dot) fr Comment #1
Type ⇒ Bug
State ⇒ Unconfirmed
Priority ⇒ 2. Medium
Summary ⇒ Selected user list of share advanced is unsorted
Due ⇒ 2013-03-08
Queue ⇒ Kronolith
Milestone ⇒
Patch ⇒ No
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

Saved Queries