Opened 5 years ago
Closed 5 years ago
#1484972 closed Feature Patches (fixed)
performance issues
| Reported by: | alec | Owned by: | |
|---|---|---|---|
| Priority: | 5 | Milestone: | 0.2-alpha |
| Component: | Core functionality | Version: | 0.1.1 |
| Severity: | normal | Keywords: | performance |
| Cc: |
Description
- If a mail is opened for preview this same request should also mark it to be read.
This will save one request between client and server and one db
connection attempt and one IMAP connection request on the server.
- Qoutadisplay action can be used without imap connection because that action is called by set_quota function (app.js) which sets quota values when is called (move_del.inc and check_recent.inc).
Attachments (2)
Change History (6)
Changed 5 years ago by alec
comment:1 Changed 5 years ago by alec
Sorry, quotadisplay.patch does not fixes second issue, because quotadisplay action is in mail task and imap connection is established, also func.inc is called but shouldn't. It must be fixed in other way, maybe that action should be outside mail task.
comment:2 Changed 5 years ago by JohnDoh
Perhaps make the fix for point one a config option? I dont think messages should be marked as read when I preview them. I might just glance at them in preview mode but not want them marked as read until I have done something about them later.
comment:3 Changed 5 years ago by alec
added fixed quotadisplay.patch - quotadisplay action is not needed anymore
comment:4 Changed 5 years ago by alec
- Milestone changed from later to 0.1.2
- Resolution set to fixed
- Status changed from new to closed
Fixed in [6d2714b3]
patch for point 1.