Make this work with Media entities

gilmar.lima → made their first commit to this issue’s fork.

I've created a patch for the issue mentioned. We have also encountered the same issue and it is related to the getLatestRevision function in the AdminModeratedContent class, where the query to retrieve the latest revision has the storage type hardcoded for Nodes.

Ok I've pushed that into the 3.0.x branch

Please re-roll for the 2.0.x branch

joseph.olstad → closed merge request !12

Support media entities.

new patch

maybe this one

demo , uploading a simple patch

Thanks @joseph.olstad. The patch #20 looks good to me. It works on Drupal Core 9.4.11 & PHP 8.0.

Thanks everyone!

Automatically closed - issue fixed for 2 weeks with no activity.

I think this should be re-opened, I tried with the latest version and found the following issues:

1. The actions don't exist for the Bulk Form for media. I manually copied the "system.action.publish_latest" and "system.action.unpublish_latest" to media specific ones to get this to work from within my config sync. However, there should be examples in the module's config/optional folder.

2. Both action plugins for "PublishLatestRevisionAction" and "UnpublishCurrentRevisionAction" had checks in the access function for whether the entity was a node, therefore it would return an Access Denied when applying the action. I updated this to check for either nodes or media.

3. There should be similar settings for the module for "Enable dialog on media edit form" and "Enable dialog on admin media listing". This affects the dialog pop ups and the code is all hard coded to look for node forms.

I have attached a patch for 1 & 2 which at least makes the module functional for media entities. #3 is a larger lift I don't have time for at the moment.

Hi there, normally we'd create a new issue however since you've already pushed something I'll re-open this one. What is this change about?

Thanks  @nsclacca, nice work!

https://www.drupal.org/project/moderated_content_bulk_publish/releases/2... →

Automatically closed - issue fixed for 2 weeks with no activity.

New support request is opened.

Perhaps some improved documentation on setting this up is needed.

Automatically closed - issue fixed for 2 weeks with no activity.