Title tells everything: Comment Alter should provide interface to change node titles as well.

Comments

boobaa’s picture

Status: Active » Needs review
StatusFileSize
new5.64 KB

Attached 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.

dsnopek’s picture

What 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!

dsnopek’s picture

Issue summary: View changes
StatusFileSize
new1.59 KB

Here 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?

boobaa’s picture

Status: Needs review » Fixed

This is what I think: thanks, committed! :)

dsnopek’s picture

Great, thanks! :-) Now we need to pressure the Diff module team into making a 3.3 release. ;-)

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.