Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.As the title says. Patch forthcoming...
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 1224996-removed-values-preview-changes.patch | 1.64 KB | katbailey |
| #1 | 1224996-diff-added-and-removed-values.patch | 2.09 KB | katbailey |
Comments
Comment #1
katbailey commentedPatch attached.
Here's a screenshot: https://skitch.com/katbailey/fk9e6/diff-new-and-removed WIthout this patch those changes wouldn't show up.
Comment #2
realityloop commentedAdded to dev, thanks for the patch.
Comment #3
katbailey commentedYay - 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
$langcodekey 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.
Comment #4
realityloop commentedApplied in dev, thanks again