Problem/Motivation

The publish and unpublish processing in Scheduler 8.x-1.x contained an extra "unnecessary" call to $node->save() which has now been removed in 2.x. However, if 3rd-party modules have implemented the old hook_scheduler_publish_action and hook_scheduler_unpublish_action functions, these would not have needed to include a save, due to the extra one in Scheduler.

Proposed resolution

Therefore if processing has been done by one of these lagacy hooks, it would be safe to add back the save() so that the 3rd-party functionality continues to work without requiring as save() to be added in the hook implementaion.

Issue fork scheduler-3273938

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

jonathan1055 created an issue. See original summary.

  • jonathan1055 committed 2bb5e79 on 2.x
    Issue #3273938 by jonathan1055: Maintain backwards compatibility with 8....
jonathan1055’s picture

Status: Active » Fixed

Done.

Status: Fixed » Closed (fixed)

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