Closed (fixed)
Project:
Scheduler
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
6 Nov 2016 at 11:47 UTC
Updated:
20 Feb 2019 at 09:19 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
jonathan1055 commentedSome of the test files contain calls to
t(' ')in places where it should not be used. These are:After the above have been removed this leaves just one place where we have
t(' ')in our test files, this is in calls to drupalPostForm() and judging by the API docs and real core test examples it is ok to use it here.This patch removes the calls in 1-3 above.
Comment #4
jonathan1055 commentedIn hook_node_presave the
isAllowed()3rd-party hook should be called for unpublishing, not just for publishing.Comment #5
jonathan1055 commentedAdded test coverage to SchedulerApiTest.php
Comment #8
jonathan1055 commentedIn many tests we had defined the node storage as
$this->container->get('entity.manager')->getStorage('node');. However, it has also been a property created in SchedulerTestBase for a while, so this patch removes the repeated re-definitions and re-uses$this->nodeStorageinstead.Comment #10
jonathan1055 commentedThis patch implements another two @TODOs in tests:
$this->nodetype, which is created in SchedulerTestBase, instead of re-defining withNodeType::load('page')in the test files.$this->nodetype->get('name')instead of the hard-coded'Basic page'.Comment #12
jonathan1055 commentedFurther tidy-up of test files and re-using properties for efficiency:
'page'to$this->type$this->nodetype->get('type')to$this->type$typeNamefor the string 'Basic Page' to avoid repeat hard-coding$this->nodetype->get('name')to$this->typeNameComment #14
jonathan1055 commentedThe schedulerManager function
runCronis better namedrunLightweightCronas that describes what it does.Comment #16
jonathan1055 commentedFixed typo in the function call.
Comment #18
jonathan1055 commentedIn 7.x Scheduler implemented
hook_scheduler_api()which allowed other modules to react to Scheduler publishing or unpublishing content. In 8.x we have events, so any implementations of this hook will be ignored and should be replaced by event listeners. This patch gives a warning in the dblog if any third-party modules still try to implementhook_scheduler_api()in their 8.x code.Comment #20
jonathan1055 commentedSlight text changes to the status report.
Comment #22
jonathan1055 commentedImproved messages and text in API test module
Comment #26
jonathan1055 commentedScheduler 8.x-1.0 was released on 14th November 2017. All of the commits above (apart from the last one in #25 updating the Travis testing core version) were in for 8.x-1.0 hence renaming and marking this issue fixed.