Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sorting User Index (in admin section) by Alias (A-Z) or Alias (Z-A) functions incorrectly #226

Open
SpeedyD opened this issue Dec 19, 2021 · 0 comments · Fixed by #735
Open
Labels
bug Something isn't working
Milestone

Comments

@SpeedyD
Copy link
Contributor

SpeedyD commented Dec 19, 2021

Bug as reported:

(Main branch, last commit parent: f447da64)

URL of page: <siteurl>/admin/users?name=&rank_id=0&sort=alias or <siteurl>/admin/users?name=&rank_id=0&sort=alias-reverse
Description of bug: Attempting to use 'Sort by Alias (A-Z)' or 'Sort by Alias (Z-A)' on the Admin version of the User Index shows results that do not match any sorting method. It seems similar to 'Oldest First' and 'Newest First' at first glance, but some user names do nto match up.
How was the bug caused (exact steps to reproduce): Go to the Admin Home, go to the User Index from there, pick either 'Sort by Alias (A-Z)' or 'Sort by Alias (Z-A)' and click Search.

Further details

Unlike the public User Index, this one actually does something.. but it's unclear what. At first glance, I assumed it was matching with Oldest First (if sorting by A-Z) or Newest first (if sorting by Z-A), but on closer look didn't match even that.

@itinerare itinerare added the bug Something isn't working label Dec 19, 2021
@itinerare itinerare added this to the v2.1.0 milestone Dec 19, 2021
@itinerare itinerare linked a pull request Dec 6, 2023 that will close this issue
itinerare pushed a commit that referenced this issue Dec 10, 2023
* fix issue #728

* fix #96

* fix #113

* Update birthday selector on register

* fix #225

* fix #226

* fix #346

* fix #558

* fix #460

* fix #459

* fix #91

* Apply fixes from #91 to prompt submissions

* refactor: fix blade formatting

* remove character item retention due to return input wackiness

* refactor: fix PHP styling

* array_unique on slugs for gallery

* add missing set active on user sidebars

* add comment history and bugfix for likes

* refactor: fix blade formatting

* tweak function docs and sales comments checks

* refactor: fix blade formatting

* Capitalise 'carbon' in UserService

* refactor: fix PHP styling

---------

Co-authored-by: Ne-wt <Ne-wt@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants