Report every editorial attribute in the revision diff

diff_against gained external_url, published_at and user. The view renders
each metadata section only when that attribute changed, and says so once
when none did.

Metadata now precedes the locale pointer, which speaks only of translations:
above the metadata it read as "nothing changed" on a revision that had moved
the page.

1 files changed, 6 insertions, 0 deletions

diff --git a/app/models/page.rb b/app/models/page.rb
index 3240057f..c2dc227b 100644
--- a/app/models/page.rb
+++ b/app/models/page.rb
@@ -277,6 +277,12 @@ class Page < ApplicationRecord
 
     text_diffs.merge(
       address:       address_diff_against(other),
+      external_url:  { from: other.external_url, to: external_url,
+                       changed: external_url.presence != other.external_url.presence },
+      published_at:  { from: other.published_at, to: published_at,
+                       changed: published_at != other.published_at },
+      user:          { from: other.user, to: user,
+                       changed: user_id != other.user_id },
       tags:          { added: tag_list.to_a - other.tag_list.to_a, removed: other.tag_list.to_a - tag_list.to_a },
       template_name: { from: other.template_name, to: template_name, changed: template_name != other.template_name },
       assets:        { added: assets.to_a - other.assets.to_a, removed: other.assets.to_a - assets.to_a }