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

  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.

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

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

Download all attachments as: .zip

Change History (6)

Changed 5 years ago by alec

patch for point 1.

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.

Changed 5 years ago by alec

patch for point 2.

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]

Note: See TracTickets for help on using tickets.