@julienmoumne opened this Issue on March 5th 2011 Member

In my box test_setUserAccess_multipleCallDistinctAccessMultipleUser() fails because

$access1 = array( 'user1' => array(1,2), 'user2' => array(2,3) );
$wanted1 = array( 'user1' => array(1,2), 'user2' => array(3,2) );

This happens because getUsersSitesFromAccess() doesn't order column idsite.

Should we :

  • order idsite
  • sort the array before comparison
@robocoder commented on March 5th 2011 Contributor

Interesting. Presumably, we can't expect login to be sorted either. I would add an "ORDER BY login,idsite" clause to the SQL.

@mattab commented on March 5th 2011 Member

sounds good

@julienmoumne commented on March 6th 2011 Member

(In [4039]) fixes #2151 - ordering getUsersSitesFromAccess() SQL query, updating unit tests, should all API queries have an order by clause ?

This Issue was closed on March 6th 2011
Powered by GitHub Issue Mirror