amateescu / revision_tree

Sandbox for Drupal's revision tree and related work
GNU General Public License v2.0
0 stars 0 forks source link

Bypass workspace form and entity restrictions #13

Open pmelab opened 5 years ago

pmelab commented 5 years ago

Forms and non-managed entities are by default blocked in non-default workspaces. We had to bypass this for some cases by setting the entities to "internal" or manually re-enabling forms.

TODO:

Resolution: delivery module

amateescu commented 5 years ago

This should be done as a core patch that will depend on the conflict API issue/patch. A possible implementation would be that in the current code of the EntityChanged constraint, add this logic: if there is at least one "conflict resolver" class, follow the conflict management process, otherwise fallback to the current logic of the constraint.

pmelab commented 5 years ago

On the media library form: There are a lot of candidates, and not all of them are patchable. Every views bulk operation form is blocked for example. And there might be a lot of contrib modules with utility forms in modals and the likes. They all have to marked as workspace_safe. I'm not sure this is a satisfying option.

amateescu commented 5 years ago

We have https://www.drupal.org/project/drupal/issues/2986005 for the VBO part, and all the other forms need to be considered on a case-by-case basis. There's really no way to know if submitting a form in a non-default workspace can have (user facing) consequence in the Live workspace.