Re: Broken IMAP search with nnir.el

[Andreas Seltenreich <andreas+ding@gate450.dyndns.org>]

[-- Attachment #1: Type: text/plain, Size: 782 bytes --]

I wrote:

> Simon Josefsson writes:
>
>> I think it might be easier to have nnir never re-use article numbers.
>> But I'm not familiar enough with nnir to fix this.
>
> I'm afraid this would break solid groups, which work ok for engines
> that support ordering of returned articles by delivery-date or
> something comparable instead of "Score".

Actually, the same trick that nnweb uses to avoid caching anomalies
with ephemeral groups could be applied to nnir as well.  nnweb uses
message-unique-id to generate names for ephemeral groups, and pushing
such an id into nnir's query parameters has the same effect.

Tassilo: Could you try to reproduce the problem with the attached
patch?  If the problem is actually caching related, the symptoms
should disappear.

regards,
andreas


[-- Attachment #2: Type: text/x-patch, Size: 674 bytes --]

*** nnir.el	17 Jul 2006 17:14:52 +0200	7.19
--- nnir.el	25 Mär 2007 22:21:01 +0200	
***************
*** 294,299 ****
--- 294,300 ----
  (require 'nnoo)
  (require 'gnus-group)
  (require 'gnus-sum)
+ (require 'message)
  (eval-and-compile
    (require 'gnus-util))
  (eval-when-compile
***************
*** 610,615 ****
--- 611,617 ----
      (if extra-parms
          (setq parms (nnir-read-parms query))
        (setq parms (list (cons 'query query))))
+     (add-to-list 'parms (cons 'unique-id (message-unique-id)) t)
      (gnus-group-read-ephemeral-group
       (concat "nnir:" (prin1-to-string parms)) '(nnir "") t
       (cons (current-buffer)