- Issue created by @tedbow
- πΊπΈUnited States tedbow Ithaca, NY, USA
This seems like a similar problem space to π Media Library dialogs triggered from page data do not have buttons yet Active because that is changing on the way
entity_form_fields
is set - Merge request !784Draft: #3512017 filter entity_form_fields before saving hash β (Open) created by tedbow
- πΊπΈUnited States tedbow Ithaca, NY, USA
So far all I have done in the merge request is bring over my attempts in π Auto save shows a change when there is not one Active to try to make
entity_form_fields
not show a difference after entity is published and there no additional changes. That was before I split that problem in to this issueThe current changes do reduced number of changes in
entity_form_fields
but didn't eliminate them - πΊπΈUnited States tedbow Ithaca, NY, USA
I did some further manual testing.
it seems that with the current MR and the standard profile the parts of
entity_form_fields
that still change are field_tags and field_image, both entity reference fields.So I
- Installed the standard profile and enabled xb_dev_standard
- deleted both the tags and images field from Article
- created an article
- opened it in XB
- changed the title
- published the change
- waited for 10 seconds to see if "review 1 change" showed
In the MR there doesn't result in "review 1 change" but 0.x does
In 0.x the difference between the call from
AutoSaveManager::recordInitialClientSideRepresentation
andAutoSaveManager::save
is in save() there are 2 extra items underentity_form_fields
,advanced__active_tab
andrevision
- π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
π ImageItem::defaultStorageSettings should override display_default Active is the core bug
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
Reviewed & RTBC'd the core issue.
- πΊπΈUnited States tedbow Ithaca, NY, USA
Obviously we need tests because nothing is failing now because of this. Also should be stable blocker
I wonder if we want a more complex e2e test where we say start with no media item select load a media item and then remove it again. This should return to the start state and not show a change but I wonder if it actually would
- π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
Without the core fix or a workaround tests fail in the media library branch - exactly as we'd expect them to. So this may end up being solved by π Media Library dialogs triggered from page data do not have buttons yet Active
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
π Media Library dialogs triggered from page data do not have buttons yet Active landed.
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
FYI: thanks to @longwave, π ImageItem::defaultStorageSettings should override display_default Active already landed in core, but it hasn't shipped in a release yet.
- πΊπΈUnited States tedbow Ithaca, NY, USA
I think π Media Library dialogs triggered from page data do not have buttons yet Active caused π "There are no users matching "/" whe publishing a node Active . Think we need to postpone on that because it changes the entity fields and will always show a change after publish