diff --git a/source/java/org/alfresco/repo/action/scheduled/ScheduledPersistedActionServiceImpl.java b/source/java/org/alfresco/repo/action/scheduled/ScheduledPersistedActionServiceImpl.java index adf753f269..7dcd48e20a 100644 --- a/source/java/org/alfresco/repo/action/scheduled/ScheduledPersistedActionServiceImpl.java +++ b/source/java/org/alfresco/repo/action/scheduled/ScheduledPersistedActionServiceImpl.java @@ -76,8 +76,7 @@ public class ScheduledPersistedActionServiceImpl implements ScheduledPersistedAc protected static final String JOB_SCHEDULE_NODEREF = "ScheduleNodeRef"; protected static final String JOB_ACTION_NODEREF = "ActionNodeRef"; - // TODO: Should be private NodeRef scheduledActionRootNodeRef; - protected static NodeRef SCHEDULED_ACTION_ROOT_NODE_REF; + protected NodeRef SCHEDULED_ACTION_ROOT_NODE_REF; protected static final Set ACTION_TYPES = new HashSet(Arrays .asList(new QName[] { ActionModel.TYPE_ACTION_SCHEDULE })); diff --git a/source/java/org/alfresco/repo/action/scheduled/ScheduledPersistedActionServiceTest.java b/source/java/org/alfresco/repo/action/scheduled/ScheduledPersistedActionServiceTest.java index 352ff61f75..c885ebc7df 100644 --- a/source/java/org/alfresco/repo/action/scheduled/ScheduledPersistedActionServiceTest.java +++ b/source/java/org/alfresco/repo/action/scheduled/ScheduledPersistedActionServiceTest.java @@ -117,17 +117,13 @@ public class ScheduledPersistedActionServiceTest extends TestCase testAction = new TestAction(actionService .createAction(SleepActionExecuter.NAME)); runtimeActionService.createActionNodeRef( - // - testAction, - ScheduledPersistedActionServiceImpl.SCHEDULED_ACTION_ROOT_NODE_REF, + testAction, serviceImpl.SCHEDULED_ACTION_ROOT_NODE_REF, ContentModel.ASSOC_CONTAINS, QName.createQName("TestAction")); testAction2 = new TestAction(actionService .createAction(SleepActionExecuter.NAME)); runtimeActionService.createActionNodeRef( - // - testAction2, - ScheduledPersistedActionServiceImpl.SCHEDULED_ACTION_ROOT_NODE_REF, + testAction2, serviceImpl.SCHEDULED_ACTION_ROOT_NODE_REF, ContentModel.ASSOC_CONTAINS, QName.createQName("TestAction2")); testAction3 = new TestAction(actionService @@ -182,8 +178,7 @@ public class ScheduledPersistedActionServiceTest extends TestCase // Persist the 3rd action runtimeActionService.createActionNodeRef( // - testAction3, - ScheduledPersistedActionServiceImpl.SCHEDULED_ACTION_ROOT_NODE_REF, + testAction3, serviceImpl.SCHEDULED_ACTION_ROOT_NODE_REF, ContentModel.ASSOC_CONTAINS, QName.createQName("TestAction3")); assertEquals(null, schedule.getPersistedAtNodeRef()); @@ -673,9 +668,7 @@ public class ScheduledPersistedActionServiceTest extends TestCase .getPersistedAtNodeRef()); runtimeActionService.createActionNodeRef( - // - testAction3, - ScheduledPersistedActionServiceImpl.SCHEDULED_ACTION_ROOT_NODE_REF, + testAction3, serviceImpl.SCHEDULED_ACTION_ROOT_NODE_REF, ContentModel.ASSOC_CONTAINS, QName.createQName("TestAction3")); assertNotNull(schedule.getActionNodeRef());