Problem/Motivation
TitleFormatter currently extends from BasicStringFormatter. However StringFormatter would be a much better starting point as it already has one of the settings that TitleFormatter is adding.
The code in StringFormatter has some improvements
- Doesn't force the link render array to string in
viewElements(). - Correctly handles revisions.
Unfortunately the setting in StringFormatter has a different name: link_to_entity instead of linked.
Proposed resolution
- Change
TitleFormatterto extendStringFormatter. - Write an update hook to fix the setting name in existing display settings.
Remaining tasks
User interface changes
Slight differences in formatter settings.
API changes
None
Data model changes
Change of formatter setting name link_to_entity instead of linked.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | manage_display.title-formatter.3342452-3.patch | 5.23 KB | adamps |
Comments
Comment #2
adamps commentedComment #3
adamps commentedComment #5
adamps commentedComment #6
adamps commentedComment #7
adamps commented