[PR #1438] [MERGED] feat(frontend/backend): ensure that the sort fields include the sent and arrival date across all message lists #1514

New issue

Closed

opened 2026-02-25 21:38:30 +03:00 by kerem · 0 comments

kerem commented

2026-02-25 21:38:30 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/cypht-org/cypht/pull/1438
Author: @mercihabam
Created: 1/29/2025
Status: ✅ Merged
Merged: 1/30/2025
Merged by: @mercihabam

Base: master ← Head: sentdate-filter

📝 Commits (7)

972745c WIP: add a sort parameter to the combined message lists' requests
c6c1e38 ensure the sort order correcly applies to the end-user facing list
ece7af7 do not discriminate EWS
83cff7a extend the sort field for combined pages to have all necessary fields
c7e66c9 a fix for the advanced search module
34534c0 keep the sorting client-side for the search page
8fbd326 fix unit tests

📊 Changes

13 files changed (+155 additions, -67 deletions)

View changed files

📝 modules/advanced_search/modules.php (+2 -2)
📝 modules/core/hm-mailbox.php (+1 -1)
📝 modules/core/js_modules/actions/search.js (+1 -2)
➕ modules/core/js_modules/actions/sortCombinedLists.js (+15 -0)
📝 modules/core/js_modules/route_handlers.js (+6 -0)
📝 modules/core/message_list_functions.php (+15 -6)
📝 modules/core/site.css (+0 -3)
📝 modules/core/site.js (+11 -21)
📝 modules/imap/functions.php (+59 -7)
📝 modules/imap/handler_modules.php (+37 -18)
📝 modules/imap/hm-imap.php (+1 -1)
📝 modules/imap/site.js (+2 -1)
📝 tests/phpunit/modules/core/output_modules.php (+5 -5)

📄 Description

No description provided

_{🔄 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/1438 **Author:** [@mercihabam](https://github.com/mercihabam) **Created:** 1/29/2025 **Status:** ✅ Merged **Merged:** 1/30/2025 **Merged by:** [@mercihabam](https://github.com/mercihabam) **Base:** `master` ← **Head:** `sentdate-filter` --- ### 📝 Commits (7) - [`972745c`](https://github.com/cypht-org/cypht/commit/972745c3576812bf0507b1f97b9d2eb648591e12) WIP: add a sort parameter to the combined message lists' requests - [`c6c1e38`](https://github.com/cypht-org/cypht/commit/c6c1e38af5fe94cb3d65a9683582b577e64b91ad) ensure the sort order correcly applies to the end-user facing list - [`ece7af7`](https://github.com/cypht-org/cypht/commit/ece7af7cc878b3b55f5f44424a1b10be2dfdaf52) do not discriminate EWS - [`83cff7a`](https://github.com/cypht-org/cypht/commit/83cff7a6339e1a1e1c72ca223b597350020040cc) extend the sort field for combined pages to have all necessary fields - [`c7e66c9`](https://github.com/cypht-org/cypht/commit/c7e66c9b53474afab9c122c306ebd6d6cd298944) a fix for the advanced search module - [`34534c0`](https://github.com/cypht-org/cypht/commit/34534c022a25edc3870e2874043d05df1c4d859b) keep the sorting client-side for the search page - [`8fbd326`](https://github.com/cypht-org/cypht/commit/8fbd32689d0de9d034413e505ebd39f0f7f6773f) fix unit tests ### 📊 Changes **13 files changed** (+155 additions, -67 deletions) <details> <summary>View changed files</summary> 📝 `modules/advanced_search/modules.php` (+2 -2) 📝 `modules/core/hm-mailbox.php` (+1 -1) 📝 `modules/core/js_modules/actions/search.js` (+1 -2) ➕ `modules/core/js_modules/actions/sortCombinedLists.js` (+15 -0) 📝 `modules/core/js_modules/route_handlers.js` (+6 -0) 📝 `modules/core/message_list_functions.php` (+15 -6) 📝 `modules/core/site.css` (+0 -3) 📝 `modules/core/site.js` (+11 -21) 📝 `modules/imap/functions.php` (+59 -7) 📝 `modules/imap/handler_modules.php` (+37 -18) 📝 `modules/imap/hm-imap.php` (+1 -1) 📝 `modules/imap/site.js` (+2 -1) 📝 `tests/phpunit/modules/core/output_modules.php` (+5 -5) </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>