Closed (fixed)
Project:
Diff
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
30 Apr 2008 at 10:57 UTC
Updated:
25 Aug 2010 at 20:20 UTC
Hey there I use diff module to work on a wiki-like site.
When I add some infos to a cck field, then I go to see diff with the new revision, I see that the changes are there on the revision, but diff says that there are no visible changes??
Is diff working with cck fields??
Thanks,
Patchak
Comments
Comment #1
moshe weitzman commentedyes it does. anyone else seeing this?
Comment #2
will_in_wi commentedIt appears that diff cannot do this atm, so I am changing this to a feature request.
Comment #3
moshe weitzman commenteddiff does this alreaady. in D6, the code is in cck itself.
Comment #4
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.
Comment #5
Anonymous (not verified) commentedIt definitely diffs taxonomy terms correctly. But with actual CCK fields, I'm seeing the same as patchak - no visible changes. Going into each revision shows the field values are definitely different.
Is anyone else seeing this? Interested if this is just a configuration problem on patchak and my parts.
Comment #6
Taras Zavaliy commentedIn D6 version the problem is only with CCK Content Taxonomy fields. These fields are not shown in diff's, and correspondig terms are not tracked correctly. Only the last versions of terms are shown.
Don't know if it is hard to repair. I suppose we need to write an additional .inc file for the Content Taxonomy...
After some testing: It diff's ordinary CCK fields, but not Content taxonomy fields. And Date fields aren't showed at all.
Comment #7
jmroth commentedDiffing CCK date fields actually works, but only for the start date. A changed end date is not shown in the diff. Workaround: create 2 separate fields for start and end. :-\
Comment #8
ronino commentedjmroth, I created a hook_diff() in my module to also show end date changes in the diff output. Maybe something like this should go into the date module?
Comment #9
realityloop commentedPatch posted http://drupal.org/node/448898#comment-3213876
Comment #10
higherform commentedWith php5.3.0 + PF6.17 + Diff 6.x-2.x-dev (2010-Jul-11) + CCK 6.x-3.x-dev, the following patch fixed diff display of CCK fields:
http://drupal.org/node/714762
Comment #11
yhahn commentedThis patch was committed in the referenced issue.