Use asort instead of ksort
marcingy - August 28, 2008 - 01:17
| Project: | Admin Role |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | by design |
Description
If a role is deleted and then an admin role is selected ksort causes the incorrect array value to be set in adminrole_adminrole. Instead asort should be used as this maintains the assocaited key order. I'd role a patch but.. as this is basically a one letter change on line 67 it doesn't seem worth it ;-)
#1
#2
On further investigation more changes are required so here is a patch
#3
#4
I can't reproduce this bug. Please retest with drupal 6 dev and provide more detailed instructions to reproduce
#5
#6
ksort maintains key => value associations so I don't see a bug here.