Ticket #4271 defect closed fixed

Opened 3 years ago

Last modified 3 years ago

twisted.mail.imap4 docs state incorrect meaning of UNSEEN value

Reported by: bobf Owned by:
Priority: normal Milestone:
Component: mail Keywords:
Cc: Branch: branches/imap4-unseen-docs-4271
Author: exarkun Launchpad Bug:

Description

The docs for IMAP4Client.select() ( http://twistedmatrix.com/documents/current/api/twisted.mail.imap4.IMAP4Client.html#select) state: "UNSEEN: An integer indicating the number of messages not flagged \Seen in this mailbox."

However, the RFC ( http://www.faqs.org/rfcs/rfc3501.html) states:

"OK [UNSEEN <n>]

The message sequence number of the first unseen message in the mailbox. If this is missing, the client can not make any assumptions about the first unseen message in the mailbox, and needs to issue a SEARCH command if it wants to find it."

The behaviour of IMAP4Client matches the RFC, not the Twisted docs (i.e. if I have one unread message, UNSEEN is the sequence number of that unread message, not "1").

Change History

1

Changed 3 years ago by exarkun

  • branch set to branches/imap4-unseen-docs-4271
  • branch_author set to exarkun

(In [28231]) Branching to 'imap4-unseen-docs-4271'

2

Changed 3 years ago by exarkun

  • owner exarkun deleted
  • keywords review added

3

Changed 3 years ago by khorn

  • owner set to exarkun
  • keywords review removed

Looks good to me...merge.

4

Changed 3 years ago by exarkun

  • status changed from new to closed
  • resolution set to fixed

(In [28643]) Merge imap4-unseen-docs-4271

Author: exarkun Reviewer: khorn Fixes: #4271

Correct the twisted.mail.imap4 API documentation about the meaning of the UNSEEN flag in responses.

5

Changed 2 years ago by <automation>

  • owner exarkun deleted
Note: See TracTickets for help on using tickets.