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.
The diff module has a default implementation which expects cck types to store their values in $item['value']. This is not the case for email, which uses $item['email'].
Ideally email would use 'value' however until then I have written a patch that implements hook_content_diff_values() instead and uses the 'email' value.
Comment | File | Size | Author |
---|---|---|---|
email_diff.patch | 644 bytes | serenecloud |
Comments
Comment #1
serenecloud CreditAttribution: serenecloud commentedThis patch is 6 weeks old and nothing has happened - can a maintainer please review?
Comment #2
kerberos CreditAttribution: kerberos commentedWe're have this issue as well. Haven't tried the patch yet.
Comment #3
kerberos CreditAttribution: kerberos commentedApplied the patch and it works like a charm.
-Daniel
Comment #4
patchak CreditAttribution: patchak commentedHey there, I just installed the latest dev version of this module, but still I can't seem to see the diff for the email field, would there be anything special to do?
For what version of diff is this done?
I use the : 6.x-2.0 and the latest dev from this module, and still no luck seeing the diffs!
Thanks for any help,
Alexis
Comment #5
Countzero CreditAttribution: Countzero commentedThanks for this patch. Works perfectly.
By the way, it's really bad practice, but I just pasted the code in a custom module for easier maintenance. I had to use a patch for the link module too, plus one custom code for a special field of mine, so it started to become unbearable.
Comment #6
Encarte CreditAttribution: Encarte commentedComment #7
Alan D. CreditAttribution: Alan D. commentedCross-posting #1685698: Provide field support for diff - this provides Diff 7.x-3.x support.