Problem/Motivation

ChangedItem is similar to TimestampItem and CreatedItem, but their default formatter is not available to ChangedItem.

I have classed this is a bug, because currently there is no way to display ChangedItem values when viewing entities.

Proposed resolution

Alter TimestampFormatter's plugin definition to allow it to be used with ChangedItem too.

Remaining tasks

None.

User interface changes

None.

API changes

None.

CommentFileSizeAuthor
#11 drupal_2415937_11.patch4.27 KBxano
#9 drupal_2415937_9.patch3.63 KBxano
#9 drupal_2415937_9_test_only.patch2.65 KBxano
#7 changed-date.png9.91 KBtstoeckler
#7 changed-ago.png13.1 KBtstoeckler
#2 drupal_2415937_2.patch1005 bytesxano
#1 drupal_2415937_1.patch487 bytesxano

Comments

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
Title: Add a formater for ChangedItem » Add a formatter for ChangedItem
Status: Active » Needs review
StatusFileSize
new drupal_2415937_1.patch487 bytes
xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
StatusFileSize
new drupal_2415937_2.patch1005 bytes

Let's do the same for TimestampAgoFormatter and fix a bug there while we're at it.

Xano queued 2: drupal_2415937_2.patch for re-testing.

Xano queued 2: drupal_2415937_2.patch for re-testing.

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
commented
Status: Needs review » Reviewed & tested by the community

Tested this out by adding some display options to node and it works beautifully. Nice!

I found two nice bugs along the way, will open in a minute.

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
commented

Here we go:
#2427195: The output of TimestampAgoFormatter should never be render cached
#2427203: $options in setDisplayOptions() should be optional

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
commented
StatusFileSize
new changed-ago.png13.1 KB
new changed-date.png9.91 KB

Well, why not share some screenshots, if I had to jump through all those hoops already :-)

A node showing the changed field formatted with the Time ago formatter

A node showing the changed field formatted with the Default timestamp formatter

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs tests 
+++ b/core/lib/Drupal/Core/Field/Plugin/Field/FieldFormatter/TimestampAgoFormatter.php
@@ -23,7 +23,7 @@
- *     "updated",
+ *     "changed",

Given that we had to make this change I think we should include some tests.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
Status: Needs work » Needs review
StatusFileSize
new drupal_2415937_9_test_only.patch2.65 KB
new drupal_2415937_9.patch3.63 KB

The last submitted patch, 9: drupal_2415937_9_test_only.patch, failed testing.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
Issue tags: -Needs tests
StatusFileSize
new drupal_2415937_11.patch4.27 KB

I added pass asserts and modules are now enabled more selectively.

Xano queued 11: drupal_2415937_11.patch for re-testing.

Xano queued 11: drupal_2415937_11.patch for re-testing.

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
commented
Status: Needs review » Reviewed & tested by the community

Wow, impressive test!

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

This issue is a normal bug fix, and doesn't include any disruptive changes, so it is allowed per https://www.drupal.org/core/beta-changes. Committed 1fadbc0 and pushed to 8.0.x. Thanks!

  • alexpott committed 1fadbc0 on 8.0.x 
    Issue #2415937 by Xano: Add a formatter for ChangedItem

Status: Fixed » Closed (fixed)

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