Closed (fixed)
Project:
Content Construction Kit (CCK)
Version:
6.x-2.x-dev
Component:
General
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
7 Dec 2008 at 18:05 UTC
Updated:
21 Dec 2008 at 22:05 UTC
In file content.diff.inc there's a loop for all CCK fields in the nodes being compared and generates a row in $results even when field is not explicitly supported by this loop.
When a CCK field implements diff support, it generates its own results that are mixed by module_invoke_all used in diff module.
The attached patch fixes this. It generates results for diff module only for fields directly supported.
| Comment | File | Size | Author |
|---|---|---|---|
| content.diff.inc_.patch | 422 bytes | markus_petrux |
Comments
Comment #1
markus_petrux commentedPlease, see the following issue in Money's queue for an example where this bug arised:
#341365: Implement hook_diff() for improved revision management
Comment #2
yched commentedFixed in 6.x-2.x (slightly different fix). Thanks !
I also removed the 'links module' code branches, they don't belong in CCK.