[imapfilter-devel] IMAPFilter 2.0.1

[Hans-Werner Hilse]

Hi,

On Sat, 30 Jun 2007 18:49:01 +0300
Lefteris Chatzimparmpas <lefcha at hellug.gr> wrote:

> > Indeed, options.uid is set to _nil_, i.e. is not set at all. And in
> > fact, there are no "UID <uid>" parameters issued upon FETCH and COPY.
> > If I read the code right, it just relies on what 
> > get_option_boolean("uid") returns -- and it returns false for an unset
> > option, I think.
> 
> Oh, crud, the line where I set uid in lua.c just isn't there... :-(
> 
> Hmm... Yes, you're right... There is the problem...
> 
> But options.uid = true should bring back UIDs...  What is the error that
> is caused when you enable it?

The error was on my part (in my test.lua script I was using 
'msgid=next(messages)', not 'nr, msgid=next(messages)'). Just setting
"options.uid=true" is enough. And sure enough this makes the caching
work now.

If this option should be configurable in the future (and I think it
will be, since it is a member of the "options" table), it might still be
worth thinking about having the cache-disable function in place --
depending on whether options.uid is set or not. Example attached as a
patch.

Another thing I stumbled upon is a missing "expunge" function for the
mailbox object. I attached a patch for this, too. It might be wanted in
the case the automatic expunge is disabled. Seems to work fine for me.
I can now do the EXPUNGE manually after working on the messages.

Just for your information: I would really like to have IDLE support.
I'm hacking on it, maybe I'll have something ready until tomorrow. Is
it feasible to include it if it's a clean patch?

-hwh
-------------- next part --------------
A non-text attachment was scrubbed...
Name: imapfilter-2.0.1-add-expunge-method.patch
Type: text/x-patch
Size: 702 bytes
Desc: not available
URL: <http://lists.hellug.gr/pipermail/imapfilter-devel/attachments/20070630/785c0023/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: imapfilter-2.0.1-make-caching-dependent-on-uid.patch
Type: text/x-patch
Size: 1480 bytes
Desc: not available
URL: <http://lists.hellug.gr/pipermail/imapfilter-devel/attachments/20070630/785c0023/attachment-0001.bin>