Title tells everything: Comment Alter should provide interface to change node titles as well.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | comment_alter-title-2118637-3.patch | 1.59 KB | dsnopek |
| #1 | comment_alter-2118637-1.patch | 5.64 KB | boobaa |
Title tells everything: Comment Alter should provide interface to change node titles as well.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | comment_alter-title-2118637-3.patch | 1.59 KB | dsnopek |
| #1 | comment_alter-2118637-1.patch | 5.64 KB | boobaa |
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. ;-)