Patch attached which separates the condition into two conditions in the template
somatick → created an issue.
Oops I didn't notice the .txt extension when I saved locally. Same patch
Patch attached which adds current display to the container class name
somatick → created an issue.
See attached patch.
It did not appear to be possible to disable a view in a normal install hook, the suggested approach was to use hook_install_tasks from the profile itself
somatick → created an issue.
After further discussion it was decided to disable the publish and unpublish actions on the content audit view by default
> I am happy for now to remove the publish un-publish ones from the content audit completely. That is definitely a workflow task! The audit is more for transparence, decision making, and tagging ...
The latest patch includes the above fix as well as the removal of the publish/unpublish options
Ok phew patch #4 applied.
The view now appears to be working, but testing some of the individual actions there are some other issues/strange interactions exposed.
Using the 'Modify field values' action, produces an error from the rabbit hole module, which could be related to this issue:
https://www.drupal.org/project/rabbit_hole/issues/3383171
🐛
Error: Call to undefined method Drupal\entity_browser\Form\EntityBrowserForm::getEntity()
Needs review
Rabbit hole is required by the main sector project so if it is to be patched or updated will likely need to be done there.
Also choosing a publish or unpublish action if content moderation is enabled fails and produces a message "Cannot directly (un)publish moderated entities" - which comes from core, however there don't seem to be any suitable actions available for modifying the status of moderable entities.
Sheesh. I think the encoding was bung with that one. Third time's the charm.
Pasted too hastily and missed a line, use this patch.
Patch with updated view config yml attached
somatick → created an issue.