Closed (fixed)
Project:
Comment Alter
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
23 Oct 2013 at 14:13 UTC
Updated:
23 Nov 2013 at 20:10 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
boobaaAttached is a patch that provides the functionality (without tests). However, latest stable release of diff.module (7.x-3.2) does not recognise node title changes, though #2109905: Extending node comparison to optionally include status, sticky, promoted, revision timestamp and title. addresses this. Practically this means that node title does get changed, but this change is not displayed in the comment that changed it, at least with diff-7.x-3.2.
Comment #2
dsnopekWhat about using the Title module? It creates a surrogate field to set the title, which might allow Comment Alter to work without changing any of its code. I haven't tested it, but I think it's worth giving a try!
Comment #3
dsnopekHere is an alternate patch, that integrates with the Title module. It still depends on a 3.x-dev version of Diff to show it (because Diff ignores hidden fields and Title's field is hidden).
This patch is definitely smaller and really only adds 6 lines (ignoring comments), which means we'll have less to maintain.
What do you think?
Comment #4
boobaaThis is what I think: thanks, committed! :)
Comment #5
dsnopekGreat, thanks! :-) Now we need to pressure the Diff module team into making a 3.3 release. ;-)