trying to use the value of publish_on in a hook_presave this is not available. it because this value is set to NULL previously. it could be improved with an aditional Event before this publish_on to null.
In scheduler.module :
// If required, set the created date to match published date.
if ($entity->getThirdPartySetting('scheduler', 'publish_touch', $config->get('default_publish_touch'))) {
$node->setCreatedTime($node->publish_on->value);
}
$node->publish_on->value = NULL;
$node->setPublished(TRUE);
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | 2843425-4.add_pre_publish_immediately_event.patch | 7.52 KB | jonathan1055 |
Comments
Comment #2
apupiales commentedThe attached patch is a posible solution
Comment #3
jonathan1055 commentedHi apupiales,
From your patch it looks likes your problem is only when the node is being published immediately, that is - when a past date is entered and the config option for dates in the past 'publish immediately' is selected. Is that correct?
Given that we have 'PRE' versions of the events for both of the cron actions, I can see that it might be helpful to also have a 'PRE' event for the immediate action. I had not considered this before, but it makes for a complete set of events.
Thanks for the patch, I will test it. As this is a feature not really a bug I have changed the issue status.
Jonathan
Comment #4
jonathan1055 commentedI have modified the API test to cover the new PRE_PUBLISH_IMMEDIATELY event.
Comment #6
jonathan1055 commentedThanks apupiales. I have credited you on the commit.