This adds a small check to check for the publish/unpublish state prior to execution.

Issue originally found in the Context Control Center over at https://www.drupal.org/node/3568673 .

CommentFileSizeAuthor
#2 scheduler_content_moderation_integration-undefined-publish-state-3568673.patch2.08 KBrobloach

Comments

robloach created an issue. See original summary.

robloach’s picture

robloach
he/him
commented
Status: Active » Needs review
StatusFileSize
new scheduler_content_moderation_integration-undefined-publish-state-3568673.patch2.08 KB

Here is the patch originally uploaded to the Context Control Center module.

robloach’s picture

robloach
he/him
commented
Issue summary: View changes
robloach’s picture

robloach
he/him
commented
Related issues: +#3579841: Can't Apply Scheduler Patch
smustgrave’s picture

smustgrave commented
Status: Needs review » Needs work

Can you put into an MR please.

  • smustgrave committed 7bcd3da7 on 3.0.x 
    feat: #3579842 Publish/Unpublish State Safety Check

By: robloach
By:...
smustgrave’s picture

smustgrave commented
Status: Needs work » Patch (to be ported)

Went ahead and did that. Will get into 4.0.x soon.

smustgrave’s picture

smustgrave commented
Status: Patch (to be ported) » Fixed

Pushed to #3584553: Convert to OOP hooks and prepare for D12

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

kristen pol’s picture

kristen pol
she/her
Primary language English
Location Santa Cruz, CA, USA
commented

Thanks! Once it's in an official release, I can remove my patch. Any ETA?

smustgrave’s picture

smustgrave commented

If you got time want to review the current items in review? Would love to include them

kristen pol’s picture

kristen pol
she/her
Primary language English
Location Santa Cruz, CA, USA
commented

Pasting from slack:

I looked at all 4

one is super long and I don't have time for review unfortunately

I reviewed one (with ai help) and added some feedback and moved to needs work

I merged 3.0.x into the mr branches

unfortunately I'm super slammed o_O

Status: Fixed » Closed (fixed)

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