From fe1bb60bece8c5ab07eb31640bc4ebcd3a6c5898 Mon Sep 17 00:00:00 2001 From: srosse <none@none> Date: Tue, 27 Nov 2012 14:48:09 +0100 Subject: [PATCH] OO-427: remove some of the delete methods specific for the templates which seem unnecessary --- .../org/olat/portfolio/manager/EPStructureManager.java | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/main/java/org/olat/portfolio/manager/EPStructureManager.java b/src/main/java/org/olat/portfolio/manager/EPStructureManager.java index 5956f4f2bd5..70849fb9d97 100755 --- a/src/main/java/org/olat/portfolio/manager/EPStructureManager.java +++ b/src/main/java/org/olat/portfolio/manager/EPStructureManager.java @@ -1006,12 +1006,13 @@ public class EPStructureManager extends BasicManager { if(map == null) { return;//nothing to delete } + + //already delete in removeStructureRecursively: deletePortfolioMapTemplateRecursively((EPStructureElement)map); removeStructureRecursively(map); - deletePortfolioMapTemplateRecursively((EPStructureElement)map); - dbInstance.deleteObject(map); + //already delete in removeStructureRecursively: dbInstance.deleteObject(map); } - private void deletePortfolioMapTemplateRecursively(EPStructureElement element) { + /*private void deletePortfolioMapTemplateRecursively(EPStructureElement element) { element.getInternalArtefacts().clear(); element.setRoot(null); element.setRootMap(null); @@ -1020,7 +1021,7 @@ public class EPStructureManager extends BasicManager { deletePortfolioMapTemplateRecursively((EPStructureElement)subLink.getChild()); } links.clear(); - } + }*/ public void removeStructureRecursively(PortfolioStructure struct){ List<PortfolioStructure> children = loadStructureChildren(struct); -- GitLab