Menu

#7 warning from quoted text

open
nanotube
None
5
2009-04-02
2009-04-02
No

As you requested, I'm attaching the files that captured the debug inormation

Discussion

  • Yaakov Stern

    Yaakov Stern - 2009-04-02
     
  • Yaakov Stern

    Yaakov Stern - 2009-04-02
     
  • nanotube

    nanotube - 2009-04-02

    Hi,

    Aha, ok, i see what's going on here. you are /forwarding/ a message, not /quoting/ a message (which would happen when you reply). forwarded messages are not considered "quoted" text.

    the presumption is that if you are forwarding a message that contained an attachment, you want to include the attachment as well...

    at least that was my logic when i was coding it. :)

    what are your thoughts on the matter?

     
  • nanotube

    nanotube - 2009-04-02
    • assigned_to: nobody --> nanotube
     
  • Yaakov Stern

    Yaakov Stern - 2009-04-02

    I get it, and it makes sense. Let me live with it now that I understand it, I'll let you know whether a) there is still a problem with responding, and b) whether an option to cover forwarding would be useful.
    Thanks for being so reponsive

     
  • nanotube

    nanotube - 2009-04-02

    OK sounds good

    play with it and let me know what you think

    i'll think on it as well... maybe a separate "include forwarded text" checkbox may make sense...

     
  • Yaakov Stern

    Yaakov Stern - 2009-04-07
     
  • Yaakov Stern

    Yaakov Stern - 2009-04-07

    OK, I'm back. I just got a warning when I replied, not forwarded. The body text is attached as "new shot1" below.

     
  • nanotube

    nanotube - 2009-04-07

    Hi,

    your screenshot is of the "raw mail body"

    could you post a shot of the "processed mail body" as well, for the same email reply?

    the pattern-matching gets done on the "processed mail body", so i'd like to see whether the "blockquote type='cite'" element was properly removed...

     
  • Nobody/Anonymous

    I now realize it was my mistake. The program was catching a word in the subject line and I thought that I had that option "unchecked". Sorry.
    Back to the question you raised previously, I think it would be nice to have an option to have an option to ignore quoted text in forwarded mail in addition to /reply/ mail, but I leave that to you.
    Thanks again for you attention

     
  • nanotube

    nanotube - 2009-04-07

    Aha, ok. :)
    I'll definitely put the 'forwarded mail' option feature on the list of things to do. No promises as to when i'll get to it, though... but i'll keep this bug open to serve as a reminder.

     

Log in to post a comment.