[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [sup-talk] Hook before-add-message, any way to know if this message is in a topic I replied to ?



This one time, at SUP Camp, Tero Tilus said, On 2011-01-03 21:46:
> Wael Nasreddine, 2010-12-23 14:04:
> > I added this method to my before-add-message hook and I call it at the
> > very end of the before-add-hook (so it's executed at the end)

> > ------- CUT
> > def show_if_participated(message)
> >   return if message.has_label? :inbox

> >   my_emails = ['wael.nasreddine@gmail.com', 'anotheremail@example.com']
> >   ts = ThreadSet.new Index.instance, $config[:thread_by_subject]
> >   ts.load_thread_for_message message
> >   message_thread = ts.thread_for message
> >   return unless message_thread
> >   message.add_label :inbox if my_emails.any? { |email|
> > message_thread.authors.include? email }
> > end
> > ------- CUT

> > However it's not working, I remove ~/.sup/xapian and ran sup-sync,
> > this email does not have the inbox label however it's one of the
> > thread that I replied to in a mailing list, what's wrong with my
> > function?

> On the surface it looks like it should work.  Couple of questions
> raise when I look at the code.  Does the thread loading actually work
> like that?  I might have been awfully wrong.  Does #authors return
> email addresses or Person instances?
I'm not sure, someone else might know about this.


> Did your reply come from one of
> the addresses listed in my_emails?
Of course, I made sure all my aliases are listed in this Array (BTW
any way to get all the addresses listed in the config file into this
array?)

> Did you reply only once and was
> that message the last one of it's thread to be indexed?  Your first
> mail to particular thread obviously won't trigger
> show_if_participated(), but subsequent mails will.

Well This email ended up in the list_sup_talk and out of the inbox,
even though I did reply couple of times to it

> Does your hook look like this?  (Judging by your explanation, it most
> probably does)

Yes exactly like this, I'll attach the before-add-hook in another
personal email to you since it has many addresses to people I know
won't like to be spammed (@list: sorry)

>   ...
>   def show_if_participated(message)
>   end
>   ...
>   show_if_participated(message)
>   ...

-- 
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
          Waël Nasreddine
website    http://wael.nasreddine.com
Blog       http://emxyptlk.com
TechnoGate http://technogate.fr
mobile     06.32.94.70.13
landline   09.70.444.236
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
_______________________________________________
sup-talk mailing list
sup-talk@rubyforge.org
http://rubyforge.org/mailman/listinfo/sup-talk