As the title says. Patch forthcoming...

Comments

katbailey’s picture

Status: Active » Needs review
StatusFileSize
new2.09 KB

Patch attached.
Here's a screenshot: https://skitch.com/katbailey/fk9e6/diff-new-and-removed WIthout this patch those changes wouldn't show up.

realityloop’s picture

Status: Needs review » Fixed

Added to dev, thanks for the patch.

katbailey’s picture

Status: Fixed » Needs review
StatusFileSize
new1.64 KB

Yay - just saw it made it into the 2.0 release, awesome! :-)
What's not awesome is that I hadn't tested it properly with preview changes, and when I did the changes involving removed field values were not showing up. The problem is that the new version of the node still has the $langcode key set for the field, even though it's an empty array. This gets blown away when the node gets saved, which is why it works fine when you do a full revision diff.
Here's a follow-up patch that checks that it's not empty.

realityloop’s picture

Status: Needs review » Fixed

Applied in dev, thanks again

Status: Fixed » Closed (fixed)

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