[PR #434] [MERGED] Expose default sort order even on combined message views #813

New issue

Closed

opened 2026-02-25 21:36:18 +03:00 by kerem · 0 comments

kerem commented

2026-02-25 21:36:18 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/cypht-org/cypht/pull/434
Author: @kroky
Created: 9/24/2020
Status: ✅ Merged
Merged: 10/20/2020
Merged by: @jasonmunro

Base: master ← Head: bugfix/date-sort

📝 Commits (1)

542de99 expose default sort order even on combined message views, so imap message list can include the right timestamp to sort by

📊 Changes

1 file changed (+8 additions, -8 deletions)

View changed files

📝 modules/imap/handler_modules.php (+8 -8)

📄 Description

This is because imap message list can include the right timestamp to sort by. If we don't expose the list_sort parameter which is the default sort order in combined message views, we end up with using internal "arrival" date by default even if the default sort order is changed to Send date... see also #418

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/cypht-org/cypht/pull/434 **Author:** [@kroky](https://github.com/kroky) **Created:** 9/24/2020 **Status:** ✅ Merged **Merged:** 10/20/2020 **Merged by:** [@jasonmunro](https://github.com/jasonmunro) **Base:** `master` ← **Head:** `bugfix/date-sort` --- ### 📝 Commits (1) - [`542de99`](https://github.com/cypht-org/cypht/commit/542de99323012a7b3af70f6928d92773e91e31cd) expose default sort order even on combined message views, so imap message list can include the right timestamp to sort by ### 📊 Changes **1 file changed** (+8 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `modules/imap/handler_modules.php` (+8 -8) </details> ### 📄 Description This is because imap message list can include the right timestamp to sort by. If we don't expose the list_sort parameter which is the default sort order in combined message views, we end up with using internal "arrival" date by default even if the default sort order is changed to Send date... see also #418 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>