When a gmail user changes the subject of a message before replying, the "in-reply-to" headers are removed, which means that Mailcomment needs to look for the "view original" token. This is stripped from the body before _mailcomment_get_signature() process it.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ezra-g’s picture

Status: Active » Needs review
FileSize
1.33 KB

This patch resolves the issue for me.

ezra-g’s picture

This should work with any filter mode.

Dane Powell’s picture

Thanks for the patch, I'll check it out as soon as I can!

Alex72RM’s picture

@ezra-g

comment #2 doesn't work ($message variable isn't available at that point).

Instead code from #1 is working!

Dane Powell’s picture

Trying to trigger automated testing.

Dane Powell’s picture

Status: Needs review » Needs work

Patch applies cleanly, but comment #4 indicates work is needed.

Dane Powell’s picture

Priority: Normal » Major
Status: Needs work » Fixed

I think I fixed this:
http://drupalcode.org/project/mailcomment.git/commit/0962533
http://drupalcode.org/project/mailcomment.git/commit/ec5b87a

I haven't had time to test it, so please test and report back. I really need to get those simpletests working...

Dane Powell’s picture

Nevermind, I had to revert those changes- neither of those patches worked.
http://drupalcode.org/project/mailcomment.git/commit/d8c28e3

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.