Ticket #1484972 (closed Patches: fixed)

Opened 5 months ago

Last modified 5 months ago

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

1. 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. 2. 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

preview_and_read_in_one_action.patch (3.8 kB) - added by alec 5 months ago.
patch for point 1.
quotadisplay.patch (6.8 kB) - added by alec 5 months ago.
patch for point 2.

Change History

Changed 5 months ago by alec

patch for point 1.

Changed 5 months 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.

Changed 5 months 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.

Changed 5 months ago by alec

patch for point 2.

Changed 5 months ago by alec

added fixed quotadisplay.patch - quotadisplay action is not needed anymore

Changed 5 months ago by alec

  • status changed from new to closed
  • resolution set to fixed
  • milestone changed from later to 0.1.2

Fixed in r1285

Note: See TracTickets for help on using tickets.