Problem/Motivation

A user case below is requested by the clients.

A draft page A refers to another draft page B.
The draft page B has a link to the draft page A.
A draft page C also has two links to the draft page A and B.

The client wants to perform the bulk publish action on the draft page A, B, and C in moderated content page (en/admin/content).

To allow the client work effectively, I temporarily turn off the bulk verification in the path #2. Later i will modify the logic as
function safedelete_moderated_content_bulk_publish_verify_publish($hookObject) {
$has_bulk_pulish_verify = \Drupal::config('safedelete.settings')->get('bulk_pulish_verify');
if (!($has_bulk_pulish_verify)) {
return;
}

....

}

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Comments

jamesyao created an issue. See original summary.

jamesyao’s picture

For an urgent fix in PROD site, i will temporally use the patch #2 to turn off the bulk publish validation, and improve it soon.

jamesyao’s picture

StatusFileSize
new2.69 KB
jamesyao’s picture

StatusFileSize
new2.7 KB

Corrected typos.

jamesyao’s picture

StatusFileSize
new6.97 KB

Improved the logic to allow the non-admin authenticated users who are able to access system.admin_content or content_moderation.admin_moderated_content pages to disable the bulk publish validation(unpublished pages).

joseph.olstad’s picture

@jamesyao, great work on this patch!

jamesyao’s picture

Thanks @joseph.olstad for your suggestions!

joseph.olstad’s picture

Status: Active » Reviewed & tested by the community

looks great, works great, it's going in!

  • joseph.olstad committed c4926e5 on 1.0.x
    Issue #3320470 by jamesyao, joseph.olstad: Make the bulk publish...
joseph.olstad’s picture

Commit c4926e56 authored just now by jamesyao

is in, next tag a release.

joseph.olstad’s picture

Assigned: jamesyao » Unassigned
Status: Reviewed & tested by the community » Fixed

Release to follow shortly

joseph.olstad’s picture

jamesyao’s picture

Thanks @joseph.olstad for adding the patch.05 into the build.

As the client requested, I disabled the checkbox (Enforce the bulk publish verification to check if the linked pages are published) and set its default value as True to avoid the admin user to make a mistake.

Status: Fixed » Closed (fixed)

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

joseph.olstad’s picture

Status: Closed (fixed) » Needs review

review newest patch

jamesyao’s picture

Status: Needs review » Reviewed & tested by the community

All is good. I reviewed and tested it.

Thanks @joseph.olstad.

joseph.olstad’s picture

Thanks for all your great work on this project @jamesyao ! :)

joseph.olstad’s picture

Status: Reviewed & tested by the community » Fixed

later patches above are for special case.

Status: Fixed » Closed (fixed)

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

joseph.olstad’s picture

Reroll of 06