Skip to content

Add an option to let users change the sort order#79

Open
dregad wants to merge 5 commits into
masterfrom
i76-sort-order
Open

Add an option to let users change the sort order#79
dregad wants to merge 5 commits into
masterfrom
i76-sort-order

Conversation

@dregad

@dregad dregad commented Jun 4, 2026

Copy link
Copy Markdown
Member

Fixes #76

@Diablordk Please test and provide feedback.

@dregad dregad mentioned this pull request Jun 4, 2026
@Diablordk

Copy link
Copy Markdown

Thanks, Will publish on testing enviroment next week. Feedback to come after.

dregad added 5 commits June 5, 2026 16:56
It lets the admin change the order in which the Snippets are listed in
the selection lists: alphabetical, global or personal first.

Fixes #76
Required to use DbQuery class.
This allows users to override the default sort order set by the
administrator.

Fixes #76
By passing the whole list received from gpc to load_by_id(), invalid IDs
are automatically filtered out.

So we do this first, and check for empty list afterwards, which avoids
displaying an empty action page in case there are no valid snippet IDs.
@dregad

dregad commented Jun 11, 2026

Copy link
Copy Markdown
Member Author

Branch rebased to latest master (fixing PHP warning)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sort Order

2 participants