New Mail menu item not decrementing

[jalopeura]

If I open a mail and read it, the New Mail menu item decrements the count of new mails. However, if I use the Tracker add-on to mark it read, it does not.

If I click on the menu item, the new mail window shows up and it's empty. But the mailbox icon is full and the count in the menu item is not 0.

hrev41256

[czeidler]

Could you point me to the tracker add-on? it needs some changes too. I introduced another attribute that set the real status, e.g. the string can't be unread and forwarded...

[anevilyak]

Replying to czeidler:

Could you point me to the tracker add-on? it needs some changes too. I introduced another attribute that set the real status, e.g. the string can't be unread and forwarded...

​http://dev.haiku-os.org/browser/haiku/trunk/src/add-ons/tracker/mark_as

[czeidler]

ok that was easy :) will fix it soon... @jalopeura you can try to fix it yourself there is a function for that in libmail.so...

[jalopeura]

I added a patch to fix the Mark As add-ons. It still changes MAIL:status, but now it also changes MAIL:read.

Once the new add-ons have been installed, the menu problem can be fixed by running the following query:

(((MAIL:status=="Read")&&(MAIL:read==0))&&(BEOS:TYPE=="text_x-email"))

Then "Select all" and use the add-on to mark them read.

[czeidler]

Thanks for the patch! sorry was not very clear in my previous post. There is a helper function in: headers/private/mail/mail_util.h

write_read_attr

think its better to use this function. Probably you have to include the private header path in the Jamfile...

[jalopeura]

I discovered that the code in my previous patch would overwrite B_READ with B_SEEN. So I fixed that.

Note that using "Mark As..." and selecting "Read" will still overwrite B_READ with B_SEEN. I can fix that if desired, but it will require moving some code around a little bit.

[jalopeura]

Sorry, I hadn't seen the last comment (comment8) when I posted (comment9).

I'm stuck on Windows (doing some word processing) at the moment, but I'll look at those functions later on today.

[jalopeura]

Patch

[jalopeura]

Okay, it's now using the mail_util functions.

I noticed while doing this that users can add additional statuses by making a <settings>/Mail/status directory and creating empty files in it. It would be nice to make it so these were not empty files, but contained the value to put into MAIL:read. That way users could map each of their custom status strings to a read_flags value. Perhaps empty files would mean to not change MAIL:read. But I didn't want to make such a drastic change in a bugfix.

[czeidler]

Sorry again! was a little bit busy when writing the last comment missed an important part :( the right solution is to go over the mail_daemon: https://dev.haiku-os.org/browser/haiku/trunk/headers/os/mail/MailDaemon.h

there is a MarkAsRead method which send a message to the daemon. The daemon dispatch it to the right add-on and the add-on do the work depending on the protocol... This is necessary because for example the IMAP add-on also sets the read status on the server. This would not happen in the previous solution.

To your proposal, think it is a good idea. I think its better to use the status file as a template for the attributes. For example, you can just at a MAIL:read attribute to the status template file...

[axeld]

(In #7748) Duplicate of #7458 (which also has a patch set, and more information).

[mmu_man]

Applied as hrev42972 with some changes to use the BMailDaemon::MarkAsRead() method. Maybe we'd want to add other methods to mark as new again as well ? It seems to work here but I didn't test all cases nor with an IMAP server, so please test.