Skip to content

Use case-insensitive sort for categories#7402

Merged
Alkarex merged 1 commit intoFreshRSS:edgefrom
abackstrom:category-sorting-natcase
Mar 5, 2025
Merged

Use case-insensitive sort for categories#7402
Alkarex merged 1 commit intoFreshRSS:edgefrom
abackstrom:category-sorting-natcase

Conversation

@abackstrom
Copy link

Changes proposed in this pull request:

  • Use strnatcasecmp() inside listSortedCategories()

How to test the feature manually:

  1. Create categories A, b, and C
  2. Observe that categories are sorted case-insensitively on the feed index and Subscription Management pages

Pull request checklist:

  • clear commit messages
  • code manually tested
  • unit tests written (optional if too hard)
  • documentation updated

Additional information can be found in the documentation.

@abackstrom abackstrom force-pushed the category-sorting-natcase branch from 97dca61 to 50ecf37 Compare March 5, 2025 13:14
@Alkarex Alkarex added this to the 1.27.0 milestone Mar 5, 2025
@Alkarex Alkarex merged commit f40acd7 into FreshRSS:edge Mar 5, 2025
1 check passed
@Alkarex
Copy link
Member

Alkarex commented Mar 5, 2025

Thanks 👍🏻
Please add a line for you in https://github.com/FreshRSS/FreshRSS/blob/edge/CREDITS.md

@Alkarex Alkarex modified the milestones: 1.27.0, 1.26.1 Mar 5, 2025
@abackstrom abackstrom deleted the category-sorting-natcase branch March 5, 2025 19:56
Sh4kE added a commit to Sh4kE/freshrss-helm-chart that referenced this pull request Mar 15, 2025
This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [freshrss/freshrss](https://freshrss.org/) ([source](https://github.com/FreshRSS/FreshRSS)) | patch | `1.26.0-alpine` -> `1.26.1-alpine` |

---

### Release Notes

<details>
<summary>FreshRSS/FreshRSS (freshrss/freshrss)</summary>

### [`v1.26.1`](https://github.com/FreshRSS/FreshRSS/blob/HEAD/CHANGELOG.md#2025-03-13-FreshRSS-1261)

[Compare Source](FreshRSS/FreshRSS@1.26.0...1.26.1)

-   Features
    -   Add cURL version to page about system information [#&#8203;7409](FreshRSS/FreshRSS#7409)
-   Bug fixing
    -   Fix regression with cURL HTTP headers breaking conditional HTTP requests [#&#8203;7403](FreshRSS/FreshRSS#7403), [FreshRSS/simplepie#33](FreshRSS/simplepie#33)
    -   Fix regression with saving states of user queries [#&#8203;7400](FreshRSS/FreshRSS#7400)
    -   Fix regression with dynamic OPML [#&#8203;7394](FreshRSS/FreshRSS#7394)
    -   Fix update of the user’s last activity on login action [#&#8203;7406](FreshRSS/FreshRSS#7406)
    -   Fix setting category option *Maximum number of articles to keep per feed* [#&#8203;7416](FreshRSS/FreshRSS#7416)
    -   Fix priority field when processing a new feed from an extension [#&#8203;7354](FreshRSS/FreshRSS#7354)
-   Deployment
    -   Fix regression with 64-bit timestamps on 32-bit platforms [#&#8203;7375](FreshRSS/FreshRSS#7375)
    -   Fix back-compatibility with cURL 7.51 (we require cURL 7.52+ for `CURLPROXY_HTTPS`) [#&#8203;7409](FreshRSS/FreshRSS#7409)
-   UI
    -   Use case-insensitive sort for categories [#&#8203;7402](FreshRSS/FreshRSS#7402)
    -   Improve dark mode of *Origine* theme [#&#8203;7413](FreshRSS/FreshRSS#7413)
    -   Added API password indicator [#&#8203;7340](FreshRSS/FreshRSS#7340)
-   I18n
    -   Fix (es, fa, sk): do not translate XPath code [#&#8203;7404](FreshRSS/FreshRSS#7404)
    -   Fix date bug in Finish [#&#8203;7423](FreshRSS/FreshRSS#7423)
    -   Add Portuguese from Portugal [#&#8203;7329](FreshRSS/FreshRSS#7329)
    -   Improve Hungarian [#&#8203;7391](FreshRSS/FreshRSS#7391)
-   Misc.
    -   Improve PHP code [#&#8203;7339](FreshRSS/FreshRSS#7339)
    -   Update dev dependencies [#&#8203;7386](FreshRSS/FreshRSS#7386), [#&#8203;7387](FreshRSS/FreshRSS#7387), [#&#8203;7388](FreshRSS/FreshRSS#7388)

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOS4yMDAuMCIsInVwZGF0ZWRJblZlciI6IjM5LjIwMC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->

Reviewed-on: https://gitea.sh4ke.rocks/lickler/freshrss/pulls/35
Co-authored-by: Michael Wittig <michael.wittig@posteo.de>
Co-committed-by: Michael Wittig <michael.wittig@posteo.de>
Sh4kE added a commit to Sh4kE/freshrss-helm-chart that referenced this pull request Mar 15, 2025
This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [freshrss](https://github.com/FreshRSS/FreshRSS) | patch | `1.26.0` -> `1.26.1` |

---

### Release Notes

<details>
<summary>FreshRSS/FreshRSS (freshrss)</summary>

### [`v1.26.1`](https://github.com/FreshRSS/FreshRSS/blob/HEAD/CHANGELOG.md#2025-03-13-FreshRSS-1261)

[Compare Source](FreshRSS/FreshRSS@1.26.0...1.26.1)

-   Features
    -   Add cURL version to page about system information [#&#8203;7409](FreshRSS/FreshRSS#7409)
-   Bug fixing
    -   Fix regression with cURL HTTP headers breaking conditional HTTP requests [#&#8203;7403](FreshRSS/FreshRSS#7403), [FreshRSS/simplepie#33](FreshRSS/simplepie#33)
    -   Fix regression with saving states of user queries [#&#8203;7400](FreshRSS/FreshRSS#7400)
    -   Fix regression with dynamic OPML [#&#8203;7394](FreshRSS/FreshRSS#7394)
    -   Fix update of the user’s last activity on login action [#&#8203;7406](FreshRSS/FreshRSS#7406)
    -   Fix setting category option *Maximum number of articles to keep per feed* [#&#8203;7416](FreshRSS/FreshRSS#7416)
    -   Fix priority field when processing a new feed from an extension [#&#8203;7354](FreshRSS/FreshRSS#7354)
-   Deployment
    -   Fix regression with 64-bit timestamps on 32-bit platforms [#&#8203;7375](FreshRSS/FreshRSS#7375)
    -   Fix back-compatibility with cURL 7.51 (we require cURL 7.52+ for `CURLPROXY_HTTPS`) [#&#8203;7409](FreshRSS/FreshRSS#7409)
-   UI
    -   Use case-insensitive sort for categories [#&#8203;7402](FreshRSS/FreshRSS#7402)
    -   Improve dark mode of *Origine* theme [#&#8203;7413](FreshRSS/FreshRSS#7413)
    -   Added API password indicator [#&#8203;7340](FreshRSS/FreshRSS#7340)
-   I18n
    -   Fix (es, fa, sk): do not translate XPath code [#&#8203;7404](FreshRSS/FreshRSS#7404)
    -   Fix date bug in Finish [#&#8203;7423](FreshRSS/FreshRSS#7423)
    -   Add Portuguese from Portugal [#&#8203;7329](FreshRSS/FreshRSS#7329)
    -   Improve Hungarian [#&#8203;7391](FreshRSS/FreshRSS#7391)
-   Misc.
    -   Improve PHP code [#&#8203;7339](FreshRSS/FreshRSS#7339)
    -   Update dev dependencies [#&#8203;7386](FreshRSS/FreshRSS#7386), [#&#8203;7387](FreshRSS/FreshRSS#7387), [#&#8203;7388](FreshRSS/FreshRSS#7388)

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOS4yMDAuMCIsInVwZGF0ZWRJblZlciI6IjM5LjIwMC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->

Reviewed-on: https://gitea.sh4ke.rocks/lickler/freshrss/pulls/37
Co-authored-by: Michael Wittig <michael.wittig@posteo.de>
Co-committed-by: Michael Wittig <michael.wittig@posteo.de>
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.

2 participants