diff --git a/source/java/org/alfresco/repo/web/scripts/RepoStore.java b/source/java/org/alfresco/repo/web/scripts/RepoStore.java index 9b04d0d2a8..cd487ba2e6 100644 --- a/source/java/org/alfresco/repo/web/scripts/RepoStore.java +++ b/source/java/org/alfresco/repo/web/scripts/RepoStore.java @@ -700,8 +700,6 @@ public class RepoStore implements Store, TenantDeployer { public Object doWork() throws Exception { - // Run this in an isolated transaction to avoid flushing an uncommitted webscript transaction and - // causing deadlocks or trying to flush a 'dead' failed transaction return retryingTransactionHelper.doInTransaction(new RetryingTransactionCallback() { public Object execute() throws Exception @@ -714,7 +712,7 @@ public class RepoStore implements Store, TenantDeployer } return source; } - }, true, true); + }, true); } }, AuthenticationUtil.getSystemUserName()); } diff --git a/source/java/org/alfresco/repo/web/scripts/RepositoryContainer.java b/source/java/org/alfresco/repo/web/scripts/RepositoryContainer.java index 625bb15f04..3ff95da1e6 100644 --- a/source/java/org/alfresco/repo/web/scripts/RepositoryContainer.java +++ b/source/java/org/alfresco/repo/web/scripts/RepositoryContainer.java @@ -169,7 +169,7 @@ public class RepositoryContainer extends AbstractRuntimeContainer implements Ten */ public Map getTemplateParameters() { - // This must be in an isolated read-only transaction, in case we are handling failure of another transaction + // Ensure we have a transaction - we might be generating the status template after the main transaction failed return retryingTransactionHelper.doInTransaction(new RetryingTransactionCallback>() { public Map execute() throws Throwable @@ -180,7 +180,7 @@ public class RepositoryContainer extends AbstractRuntimeContainer implements Ten addRepoParameters(params); return params; } - }, true, true); + }, true); } /**