Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
If content has:
- unpublish enabled,
- unpublish required,
And has been unpublished by cron, the node is:
- unpublished,
- 'Unpublish on' date is empty,
When editing node manually, the scheduler will require providing 'Unpublish on' date.
However, when updating node from 'admin/content' page (mass update) any action that forces publishing selected content (such as: Publish, Promote, make sticky) should require providing 'Unpublish on' date - same as for manual edit (above).
Comment | File | Size | Author |
---|---|---|---|
#4 | 2919534-4.interdiff-3-4.txt | 1.99 KB | jonathan1055 |
#4 | 2919534-4.prevent-bulk-publish.patch | 1.22 KB | jonathan1055 |
|
Comments
Comment #2
Michael Dajewski CreditAttribution: Michael Dajewski as a volunteer commentedCreated patch with 7.x-1.5
Following tests pass: 'Scheduler API', 'Scheduler date functionalities', 'Scheduler functionality'.
It did take some time to make it work with test case ID: 8 in: scheduler.test
Comment #3
SpaghettiBolognese CreditAttribution: SpaghettiBolognese at Emble commentedThe patchs works as expected, thanks. re-rolled because of the invalid paths.
Comment #4
jonathan1055 CreditAttribution: jonathan1055 as a volunteer commentedThanks @Michael Dajewski and @SpaghettiBolognese, you are right, this is a problem that should be fixed. I've made a few minor changes, removed a redundant line and redundant logic checks, but basically this patch is the same as yours.
However, this is also a problem in Scheduler 8.x so we'll need to fix that first. Probably should add test coverage too, atleast in 8.x
Comment #5
jonathan1055 CreditAttribution: jonathan1055 as a volunteer commentedNeeds review, then we get automatic testing.
Comment #6
jonathan1055 CreditAttribution: jonathan1055 as a volunteer commentedMoving to the 2.x branch