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
Comment #2
jamesyao commentedFor an urgent fix in PROD site, i will temporally use the patch #2 to turn off the bulk publish validation, and improve it soon.
Comment #3
jamesyao commentedComment #4
jamesyao commentedCorrected typos.
Comment #5
jamesyao commentedImproved 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).
Comment #6
joseph.olstad@jamesyao, great work on this patch!
Comment #7
jamesyao commentedThanks @joseph.olstad for your suggestions!
Comment #8
joseph.olstadlooks great, works great, it's going in!
Comment #10
joseph.olstadCommit c4926e56 authored just now by jamesyao
is in, next tag a release.
Comment #11
joseph.olstadRelease to follow shortly
Comment #12
joseph.olstadhttps://www.drupal.org/project/safedelete/releases/1.0.32
Comment #13
jamesyao commentedThanks @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.
Comment #15
joseph.olstadreview newest patch
Comment #16
jamesyao commentedAll is good. I reviewed and tested it.
Thanks @joseph.olstad.
Comment #17
joseph.olstadThanks for all your great work on this project @jamesyao ! :)
Comment #18
joseph.olstadlater patches above are for special case.
Comment #20
joseph.olstadReroll of 06