Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#2 | 1650796-mailcomment-filter-token-2.patch | 1.06 KB | ezra-g |
#1 | 1650796-mailcomment-filter-token.patch | 1.33 KB | ezra-g |
Comments
Comment #1
ezra-g CreditAttribution: ezra-g commentedThis patch resolves the issue for me.
Comment #2
ezra-g CreditAttribution: ezra-g commentedThis should work with any filter mode.
Comment #3
Dane Powell CreditAttribution: Dane Powell commentedThanks for the patch, I'll check it out as soon as I can!
Comment #4
Alex72RM CreditAttribution: Alex72RM commented@ezra-g
comment #2 doesn't work ($message variable isn't available at that point).
Instead code from #1 is working!
Comment #5
Dane Powell CreditAttribution: Dane Powell commentedTrying to trigger automated testing.
Comment #6
Dane Powell CreditAttribution: Dane Powell commentedPatch applies cleanly, but comment #4 indicates work is needed.
Comment #7
Dane Powell CreditAttribution: Dane Powell commentedI 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...
Comment #8
Dane Powell CreditAttribution: Dane Powell commentedNevermind, I had to revert those changes- neither of those patches worked.
http://drupalcode.org/project/mailcomment.git/commit/d8c28e3