Problem/Motivation

When reverting a (node) revision, the user can chose between reverting only the translatable fields or all fields. While I'm still not happy about the whole kind-of-per-language revision UI, it's especially bad when using paragraphs or another embedded entity, I guess field_collection works basically the same way. IEF when using entity reference revision fields the same I guess.

1. If you chose to only revert this translation, then nothing happens, because the paragraph/reference field is not translatable and the whole field has been skipped.

2. If you revert all fields, then the the reference is reverted back to that old reference, which updates this translation but also reverts all translations back to that state, including deleting translations that were added after that.

Proposed resolution

Not sure, maybe add an event or hook, so that for example entity_reference_revisions can replicate what \Drupal\node\Form\NodeRevisionRevertTranslationForm::prepareRevertedRevision() did for any referenced paragraph.

Remaining tasks

User interface changes

API changes

Data model changes

Comments

Berdir created an issue. See original summary.

Version: 8.5.x-dev » 8.6.x-dev

Drupal 8.5.0-alpha1 will be released the week of January 17, 2018, which means new developments and disruptive changes should now be targeted against the 8.6.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Berdir’s picture

Title: Node revision revert behavior is weird when e.g. using paragraphs fields and can not alterd » Node revision revert behavior is weird when e.g. using paragraphs fields and can not altered
Berdir’s picture

This is now a duplicate of #2960253: [meta] Allow Paragraphs widget/field and similar use cases to to be considered translatable combined with the related ERR/Paragraphs patches, the paragraphs patch now also confirms test coverage that reverting revisions just works :)