From b2011a8a5206ad34b2be27e792cd7e1e0182fc7f Mon Sep 17 00:00:00 2001 From: Roxana Lucanu Date: Mon, 27 Jan 2020 08:43:09 +0000 Subject: [PATCH] Revert "Merge branch 'merge-2.7/RM-7065_MNT-20822' into 'release/V2.7'" This reverts merge request !1353 --- .../rm-service-context.xml | 1 - .../action/impl/RetainAction.java | 5 - .../capability/RMAfterInvocationProvider.java | 15 -- .../disposition/DispositionServiceImpl.java | 64 ++------- .../DispositionScheduleInheritanceTest.java | 130 ------------------ 5 files changed, 11 insertions(+), 204 deletions(-) delete mode 100644 rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/integration/disposition/DispositionScheduleInheritanceTest.java diff --git a/rm-community/rm-community-repo/config/alfresco/module/org_alfresco_module_rm/rm-service-context.xml b/rm-community/rm-community-repo/config/alfresco/module/org_alfresco_module_rm/rm-service-context.xml index c44b1e0ad8..32ea68a220 100644 --- a/rm-community/rm-community-repo/config/alfresco/module/org_alfresco_module_rm/rm-service-context.xml +++ b/rm-community/rm-community-repo/config/alfresco/module/org_alfresco_module_rm/rm-service-context.xml @@ -103,7 +103,6 @@ - diff --git a/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/action/impl/RetainAction.java b/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/action/impl/RetainAction.java index 1820a83421..8116c18715 100644 --- a/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/action/impl/RetainAction.java +++ b/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/action/impl/RetainAction.java @@ -38,11 +38,6 @@ import org.alfresco.service.cmr.repository.NodeRef; */ public class RetainAction extends RMDispositionActionExecuterAbstractBase { - /** - * Action name - */ - public static final String NAME = "retain"; - @Override protected void executeRecordFolderLevelDisposition(Action action, NodeRef recordFolder) { diff --git a/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/capability/RMAfterInvocationProvider.java b/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/capability/RMAfterInvocationProvider.java index 4cb07cfba8..0eb210f6e4 100644 --- a/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/capability/RMAfterInvocationProvider.java +++ b/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/capability/RMAfterInvocationProvider.java @@ -440,21 +440,6 @@ public class RMAfterInvocationProvider extends RMSecurityCommon { this.numberFound = numberFound; } - - @Override - public boolean hasMore() - { - /* - Overriding for MNT-20822 - Changing code to correct the hasMore field in the search api response. - Basing off the implementation in SolrJSONResultSet with changes due to - numberFound being equal to skip + count rather than an accurate number - of possible results - */ - return getUnFilteredResultSet().length() >= getResultSetMetaData() - .getSearchParameters() - .getMaxItems(); - } } BitSet inclusionMask = new BitSet(returnedObject.length()); diff --git a/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/disposition/DispositionServiceImpl.java b/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/disposition/DispositionServiceImpl.java index 1fcfc72d25..a16bdefd28 100644 --- a/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/disposition/DispositionServiceImpl.java +++ b/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/disposition/DispositionServiceImpl.java @@ -57,9 +57,6 @@ import org.alfresco.repo.policy.annotation.BehaviourBean; import org.alfresco.repo.policy.annotation.BehaviourKind; import org.alfresco.repo.security.authentication.AuthenticationUtil; import org.alfresco.repo.security.authentication.AuthenticationUtil.RunAsWork; -import org.alfresco.repo.transaction.AlfrescoTransactionSupport; -import org.alfresco.repo.transaction.AlfrescoTransactionSupport.TxnReadState; -import org.alfresco.repo.transaction.RetryingTransactionHelper.RetryingTransactionCallback; import org.alfresco.service.cmr.dictionary.DictionaryService; import org.alfresco.service.cmr.repository.ChildAssociationRef; import org.alfresco.service.cmr.repository.NodeRef; @@ -68,7 +65,6 @@ import org.alfresco.service.cmr.repository.Period; import org.alfresco.service.namespace.NamespaceService; import org.alfresco.service.namespace.QName; import org.alfresco.service.namespace.RegexQNamePattern; -import org.alfresco.service.transaction.TransactionService; import org.alfresco.util.ParameterCheck; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -119,9 +115,6 @@ public class DispositionServiceImpl extends ServiceBaseImpl /** Freeze Service */ private FreezeService freezeService; - /** Transaction service */ - private TransactionService transactionService; - /** Disposition properties */ private Map dispositionProperties = new HashMap(4); @@ -199,14 +192,6 @@ public class DispositionServiceImpl extends ServiceBaseImpl this.freezeService = freezeService; } - /** - * @param transactionService transaction service - */ - public void setTransactionService(TransactionService transactionService) - { - this.transactionService = transactionService; - } - /** * Behavior to initialize the disposition schedule of a newly filed record. * @@ -423,7 +408,7 @@ public class DispositionServiceImpl extends ServiceBaseImpl NodeRef dsNodeRef = getAssociatedDispositionScheduleImpl(nodeRef); if (dsNodeRef != null) { - // Create disposition schedule object + // Cerate disposition schedule object ds = new DispositionScheduleImpl(serviceRegistry, nodeService, dsNodeRef); } } @@ -712,7 +697,7 @@ public class DispositionServiceImpl extends ServiceBaseImpl * @param nodeRef node reference * @param dispositionActionDefinition disposition action definition */ - private DispositionAction initialiseDispositionAction(final NodeRef nodeRef, DispositionActionDefinition dispositionActionDefinition) + private DispositionAction initialiseDispositionAction(NodeRef nodeRef, DispositionActionDefinition dispositionActionDefinition) { List childAssocs = nodeService.getChildAssocs(nodeRef, ASSOC_NEXT_DISPOSITION_ACTION, ASSOC_NEXT_DISPOSITION_ACTION, 1, true); if (childAssocs != null && !childAssocs.isEmpty()) @@ -721,7 +706,7 @@ public class DispositionServiceImpl extends ServiceBaseImpl } // Create the properties - final Map props = new HashMap(10); + Map props = new HashMap(10); Date asOfDate = calculateAsOfDate(nodeRef, dispositionActionDefinition); @@ -733,23 +718,14 @@ public class DispositionServiceImpl extends ServiceBaseImpl props.put(PROP_DISPOSITION_AS_OF, asOfDate); } - DispositionAction da; - // check if current transaction is a READ ONLY one and if true create the node in a READ WRITE transaction - if (AlfrescoTransactionSupport.getTransactionReadState().equals(TxnReadState.TXN_READ_ONLY)) - { - da = - transactionService.getRetryingTransactionHelper().doInTransaction(new RetryingTransactionCallback() - { - public DispositionAction execute() throws Throwable - { - return createDispositionAction(nodeRef, props); - } - }, false, true); - } - else - { - da = createDispositionAction(nodeRef, props); - } + // Create a new disposition action object + NodeRef dispositionActionNodeRef = this.nodeService.createNode( + nodeRef, + ASSOC_NEXT_DISPOSITION_ACTION, + ASSOC_NEXT_DISPOSITION_ACTION, + TYPE_DISPOSITION_ACTION, + props).getChildRef(); + DispositionAction da = new DispositionActionImpl(serviceRegistry, dispositionActionNodeRef); // Create the events List events = dispositionActionDefinition.getEvents(); @@ -761,24 +737,6 @@ public class DispositionServiceImpl extends ServiceBaseImpl return da; } - /** Creates a new disposition action object - * - * @param nodeRef node reference - * @param props properties of the disposition action to be created - * @return the disposition action object - */ - private DispositionAction createDispositionAction(final NodeRef nodeRef, Map props) - { - NodeRef dispositionActionNodeRef = nodeService.createNode( - nodeRef, - ASSOC_NEXT_DISPOSITION_ACTION, - ASSOC_NEXT_DISPOSITION_ACTION, - TYPE_DISPOSITION_ACTION, - props).getChildRef(); - - return new DispositionActionImpl(serviceRegistry, dispositionActionNodeRef); - } - /** * Compute the "disposition as of" date (if necessary) for a disposition action and a node. * diff --git a/rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/integration/disposition/DispositionScheduleInheritanceTest.java b/rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/integration/disposition/DispositionScheduleInheritanceTest.java deleted file mode 100644 index 319cfb0e3b..0000000000 --- a/rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/integration/disposition/DispositionScheduleInheritanceTest.java +++ /dev/null @@ -1,130 +0,0 @@ -/* - * #%L - * Alfresco Records Management Module - * %% - * Copyright (C) 2005 - 2020 Alfresco Software Limited - * %% - * This file is part of the Alfresco software. - * - - * If the software was purchased under a paid Alfresco license, the terms of - * the paid license agreement will prevail. Otherwise, the software is - * provided under the following open source license terms: - * - - * Alfresco is free software: you can redistribute it and/or modify - * it under the terms of the GNU Lesser General Public License as published by - * the Free Software Foundation, either version 3 of the License, or - * (at your option) any later version. - * - - * Alfresco is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU Lesser General Public License for more details. - * - - * You should have received a copy of the GNU Lesser General Public License - * along with Alfresco. If not, see . - * #L% - */ -package org.alfresco.module.org_alfresco_module_rm.test.integration.disposition; - -import static org.alfresco.module.org_alfresco_module_rm.test.util.CommonRMTestUtils.DEFAULT_DISPOSITION_DESCRIPTION; -import static org.alfresco.module.org_alfresco_module_rm.test.util.CommonRMTestUtils.DEFAULT_DISPOSITION_INSTRUCTIONS; -import static org.alfresco.util.GUID.generate; - -import java.io.Serializable; -import java.util.HashMap; -import java.util.Map; - -import org.alfresco.module.org_alfresco_module_rm.action.impl.CutOffAction; -import org.alfresco.module.org_alfresco_module_rm.action.impl.RetainAction; -import org.alfresco.module.org_alfresco_module_rm.disposition.DispositionSchedule; -import org.alfresco.module.org_alfresco_module_rm.test.util.BaseRMTestCase; -import org.alfresco.module.org_alfresco_module_rm.test.util.CommonRMTestUtils; -import org.alfresco.service.cmr.repository.NodeRef; -import org.alfresco.service.namespace.QName; - -/** - * Inherit disposition schedule on record level when moving categories. - * - * @author Roxana Lucanu - * @since 2.5 - */ -public class DispositionScheduleInheritanceTest extends BaseRMTestCase -{ - /** - * Given a root record category with a retention schedule - * and another root record category - * When moving the second record category into the first one (with the disposition) - * Then records under the second record category inherit the retention schedule of the parent record category - *

- * relates to https://issues.alfresco.com/jira/browse/RM-7065 - */ - public void testRetentionScheduleInheritance_RM7065() - { - doBehaviourDrivenTest(new BehaviourDrivenTest() - { - NodeRef category1; - NodeRef subcategory2; - NodeRef record; - - @Override - public void given() - { - // create root category1 - category1 = filePlanService.createRecordCategory(filePlan, generate()); - - // create record level disposition schedule for category1 - createDispositionSchedule(category1); - - // create subcategory1 under category1 - NodeRef subcategory1 = filePlanService.createRecordCategory(category1, generate()); - - // create root category2 - NodeRef category2 = filePlanService.createRecordCategory(filePlan, generate()); - - // create subcategory2 under category2 - subcategory2 = filePlanService.createRecordCategory(category2, generate()); - - // create folder under subcategory2 - folder = recordFolderService.createRecordFolder(subcategory2, generate()); - - // file record in folder and complete it - record = utils.createRecord(folder, generate(), generate()); - utils.completeRecord(record); - } - - @Override - public void when() throws Exception - { - // move subcategory2 under category1 - fileFolderService.move(subcategory2, category1, null); - } - - @Override - public void then() throws Exception - { - dispositionService.getDispositionSchedule(record); - // check for the lifecycle aspect - assertTrue("Record " + record + " doesn't have the disposition lifecycle aspect.", nodeService.hasAspect(record, ASPECT_DISPOSITION_LIFECYCLE)); - } - }); - } - - private void createDispositionSchedule(NodeRef category) - { - DispositionSchedule ds = utils.createDispositionSchedule(category, DEFAULT_DISPOSITION_INSTRUCTIONS, DEFAULT_DISPOSITION_DESCRIPTION, true, false, false); - - // CUTOFF immediately - Map cutOff = new HashMap(3); - cutOff.put(PROP_DISPOSITION_ACTION_NAME, CutOffAction.NAME); - cutOff.put(PROP_DISPOSITION_DESCRIPTION, generate()); - cutOff.put(PROP_DISPOSITION_PERIOD, CommonRMTestUtils.PERIOD_IMMEDIATELY); - dispositionService.addDispositionActionDefinition(ds, cutOff); - - // RETAIN immediately - Map retain = new HashMap(3); - retain.put(PROP_DISPOSITION_ACTION_NAME, RetainAction.NAME); - retain.put(PROP_DISPOSITION_DESCRIPTION, generate()); - retain.put(PROP_DISPOSITION_PERIOD, CommonRMTestUtils.PERIOD_IMMEDIATELY); - dispositionService.addDispositionActionDefinition(ds, retain); - } -} \ No newline at end of file