LockMode issues in transient services
authorAndreas Müller <a.mueller@bgbm.org>
Mon, 4 Aug 2014 11:46:15 +0000 (11:46 +0000)
committerAndreas Müller <a.mueller@bgbm.org>
Mon, 4 Aug 2014 11:46:15 +0000 (11:46 +0000)
eu.etaxonomy.taxeditor.editor/src/main/java/eu/etaxonomy/taxeditor/editor/name/operation/DeleteTaxonOperation.java
eu.etaxonomy.taxeditor.editor/src/main/java/eu/etaxonomy/taxeditor/editor/view/dataimport/transientServices/TransientAgentService.java
eu.etaxonomy.taxeditor.editor/src/main/java/eu/etaxonomy/taxeditor/editor/view/dataimport/transientServices/TransientClassificationService.java
eu.etaxonomy.taxeditor.editor/src/main/java/eu/etaxonomy/taxeditor/editor/view/dataimport/transientServices/TransientDescriptionService.java
eu.etaxonomy.taxeditor.editor/src/main/java/eu/etaxonomy/taxeditor/editor/view/dataimport/transientServices/TransientNameService.java
eu.etaxonomy.taxeditor.editor/src/main/java/eu/etaxonomy/taxeditor/editor/view/dataimport/transientServices/TransientOccurenceService.java
eu.etaxonomy.taxeditor.editor/src/main/java/eu/etaxonomy/taxeditor/editor/view/dataimport/transientServices/TransientReferenceService.java
eu.etaxonomy.taxeditor.editor/src/main/java/eu/etaxonomy/taxeditor/editor/view/dataimport/transientServices/TransientTaxonService.java
eu.etaxonomy.taxeditor.editor/src/main/java/eu/etaxonomy/taxeditor/editor/view/dataimport/transientServices/TransientTermService.java
eu.etaxonomy.taxeditor.navigation/src/main/java/eu/etaxonomy/taxeditor/navigation/key/polytomous/operation/DeleteOperation.java
eu.etaxonomy.taxeditor.navigation/src/main/java/eu/etaxonomy/taxeditor/navigation/navigator/operation/DeleteOperation.java

index 8bbc1800583f625562baf3e5ff1682c5c7b09a16..621e02ee5ed6c3a3217dfb9bf76355d58abf01c0 100644 (file)
@@ -15,7 +15,6 @@ import eu.etaxonomy.cdm.api.conversation.IConversationEnabled;
 import eu.etaxonomy.cdm.api.service.DeleteResult;\r
 import eu.etaxonomy.cdm.api.service.ITaxonService;\r
 import eu.etaxonomy.cdm.api.service.config.TaxonDeletionConfigurator;\r
-import eu.etaxonomy.cdm.api.service.exception.DataChangeNoRollbackException;\r
 import eu.etaxonomy.cdm.model.taxon.Classification;\r
 import eu.etaxonomy.cdm.model.taxon.Taxon;\r
 import eu.etaxonomy.cdm.model.taxon.TaxonNode;\r
index 80e0d90e6f50d63ddee16cefbde1b55b775a3c72..0f8bd8ad8bd01b3060dd220739ab9ffb05902b75 100644 (file)
@@ -15,7 +15,7 @@ import java.util.Map;
 import java.util.Set;
 import java.util.UUID;
 
-import org.hibernate.LockMode;
+import org.hibernate.LockOptions;
 import org.hibernate.Session;
 import org.hibernate.criterion.Criterion;
 import org.hibernate.envers.query.criteria.AuditCriterion;
@@ -132,8 +132,8 @@ public class TransientAgentService implements IAgentService {
      * @see eu.etaxonomy.cdm.api.service.IService#lock(eu.etaxonomy.cdm.model.common.ICdmBase, org.hibernate.LockMode)
      */
     @Override
-    public void lock(AgentBase t, LockMode lockMode) {
-        defaultAgentService.lock(t, lockMode);
+    public void lock(AgentBase t, LockOptions lockOptions) {
+        defaultAgentService.lock(t, lockOptions);
     }
 
     /**
@@ -143,8 +143,8 @@ public class TransientAgentService implements IAgentService {
      * @see eu.etaxonomy.cdm.api.service.IService#refresh(eu.etaxonomy.cdm.model.common.ICdmBase, org.hibernate.LockMode, java.util.List)
      */
     @Override
-    public void refresh(AgentBase t, LockMode lockMode, List<String> propertyPaths) {
-        defaultAgentService.refresh(t, lockMode, propertyPaths);
+    public void refresh(AgentBase t, LockOptions lockOptions, List<String> propertyPaths) {
+        defaultAgentService.refresh(t, lockOptions, propertyPaths);
     }
 
     /**
index 7f907cdf397f7f946453f728d082cdcbad08350c..89116165b5973eff378cb68cb553ec010b99373d 100644 (file)
@@ -15,7 +15,7 @@ import java.util.Map;
 import java.util.Set;
 import java.util.UUID;
 
-import org.hibernate.LockMode;
+import org.hibernate.LockOptions;
 import org.hibernate.Session;
 import org.hibernate.criterion.Criterion;
 import org.hibernate.envers.query.criteria.AuditCriterion;
@@ -143,8 +143,8 @@ public class TransientClassificationService implements IClassificationService {
      * @see eu.etaxonomy.cdm.api.service.IService#lock(eu.etaxonomy.cdm.model.common.ICdmBase, org.hibernate.LockMode)
      */
     @Override
-    public void lock(Classification t, LockMode lockMode) {
-        defaultClassificationService.lock(t, lockMode);
+    public void lock(Classification t, LockOptions lockOptions) {
+        defaultClassificationService.lock(t, lockOptions);
     }
 
     /**
@@ -154,8 +154,8 @@ public class TransientClassificationService implements IClassificationService {
      * @see eu.etaxonomy.cdm.api.service.IService#refresh(eu.etaxonomy.cdm.model.common.ICdmBase, org.hibernate.LockMode, java.util.List)
      */
     @Override
-    public void refresh(Classification t, LockMode lockMode, List<String> propertyPaths) {
-        defaultClassificationService.refresh(t, lockMode, propertyPaths);
+    public void refresh(Classification t, LockOptions lockOptions, List<String> propertyPaths) {
+        defaultClassificationService.refresh(t, lockOptions, propertyPaths);
     }
 
     /**
index 2f46507ad575c24adc7ccde23b2e85edf4a40482..3a5e1fdfbb6da399e4e5cad36fe0d9ae0ba172eb 100644 (file)
@@ -15,7 +15,7 @@ import java.util.Map;
 import java.util.Set;
 import java.util.UUID;
 
-import org.hibernate.LockMode;
+import org.hibernate.LockOptions;
 import org.hibernate.Session;
 import org.hibernate.criterion.Criterion;
 import org.hibernate.envers.query.criteria.AuditCriterion;
@@ -120,8 +120,8 @@ public class TransientDescriptionService implements IDescriptionService {
      * @see eu.etaxonomy.cdm.api.service.IService#lock(eu.etaxonomy.cdm.model.common.ICdmBase, org.hibernate.LockMode)
      */
     @Override
-    public void lock(DescriptionBase t, LockMode lockMode) {
-        defaultDescriptionService.lock(t, lockMode);
+    public void lock(DescriptionBase t, LockOptions lockOptions) {
+        defaultDescriptionService.lock(t, lockOptions);
     }
 
     /**
@@ -131,8 +131,8 @@ public class TransientDescriptionService implements IDescriptionService {
      * @see eu.etaxonomy.cdm.api.service.IService#refresh(eu.etaxonomy.cdm.model.common.ICdmBase, org.hibernate.LockMode, java.util.List)
      */
     @Override
-    public void refresh(DescriptionBase t, LockMode lockMode, List<String> propertyPaths) {
-        defaultDescriptionService.refresh(t, lockMode, propertyPaths);
+    public void refresh(DescriptionBase t, LockOptions lockOptions, List<String> propertyPaths) {
+        defaultDescriptionService.refresh(t, lockOptions, propertyPaths);
     }
 
     /**
index 54c6911f367b1cf23601f69a41b24a7a4ecde54d..b1c12d16b8919a8071896a902794bfd3dd1ea12b 100644 (file)
@@ -18,7 +18,7 @@ import java.util.UUID;
 
 import org.apache.lucene.index.CorruptIndexException;
 import org.apache.lucene.queryParser.ParseException;
-import org.hibernate.LockMode;
+import org.hibernate.LockOptions;
 import org.hibernate.Session;
 import org.hibernate.criterion.Criterion;
 import org.hibernate.envers.query.criteria.AuditCriterion;
@@ -131,8 +131,8 @@ public class TransientNameService implements INameService {
      * @see eu.etaxonomy.cdm.api.service.IService#lock(eu.etaxonomy.cdm.model.common.ICdmBase, org.hibernate.LockMode)
      */
     @Override
-    public void lock(TaxonNameBase t, LockMode lockMode) {
-        defaultNameService.lock(t, lockMode);
+    public void lock(TaxonNameBase t, LockOptions lockOptions) {
+        defaultNameService.lock(t, lockOptions);
     }
 
     /**
@@ -142,8 +142,8 @@ public class TransientNameService implements INameService {
      * @see eu.etaxonomy.cdm.api.service.IService#refresh(eu.etaxonomy.cdm.model.common.ICdmBase, org.hibernate.LockMode, java.util.List)
      */
     @Override
-    public void refresh(TaxonNameBase t, LockMode lockMode, List<String> propertyPaths) {
-        defaultNameService.refresh(t, lockMode, propertyPaths);
+    public void refresh(TaxonNameBase t, LockOptions lockOptions, List<String> propertyPaths) {
+        defaultNameService.refresh(t, lockOptions, propertyPaths);
     }
 
     /**
index 77402df6a1a8f7cd0277cf3ec93bb8f51f651e96..0a0fea8517eccce946e4e51449b635ad2c57735b 100644 (file)
@@ -19,7 +19,7 @@ import java.util.UUID;
 
 import org.apache.lucene.index.CorruptIndexException;
 import org.apache.lucene.queryParser.ParseException;
-import org.hibernate.LockMode;
+import org.hibernate.LockOptions;
 import org.hibernate.Session;
 import org.hibernate.criterion.Criterion;
 import org.hibernate.envers.query.criteria.AuditCriterion;
@@ -128,8 +128,8 @@ public class TransientOccurenceService implements IOccurrenceService {
      * @see eu.etaxonomy.cdm.api.service.IService#lock(eu.etaxonomy.cdm.model.common.ICdmBase, org.hibernate.LockMode)
      */
     @Override
-    public void lock(SpecimenOrObservationBase t, LockMode lockMode) {
-        defaultService.lock(t, lockMode);
+    public void lock(SpecimenOrObservationBase t, LockOptions lockOptions) {
+        defaultService.lock(t, lockOptions);
     }
 
     /**
@@ -139,8 +139,8 @@ public class TransientOccurenceService implements IOccurrenceService {
      * @see eu.etaxonomy.cdm.api.service.IService#refresh(eu.etaxonomy.cdm.model.common.ICdmBase, org.hibernate.LockMode, java.util.List)
      */
     @Override
-    public void refresh(SpecimenOrObservationBase t, LockMode lockMode, List<String> propertyPaths) {
-        defaultService.refresh(t, lockMode, propertyPaths);
+    public void refresh(SpecimenOrObservationBase t, LockOptions lockOptions, List<String> propertyPaths) {
+        defaultService.refresh(t, lockOptions, propertyPaths);
     }
 
     /**
index 4aa2dabc68fdea0173650f19ace1e6cd01e9dbf9..528813557d1bf74e84c94f9ec343977621f8c47f 100644 (file)
@@ -15,7 +15,7 @@ import java.util.Map;
 import java.util.Set;
 import java.util.UUID;
 
-import org.hibernate.LockMode;
+import org.hibernate.LockOptions;
 import org.hibernate.Session;
 import org.hibernate.criterion.Criterion;
 import org.hibernate.envers.query.criteria.AuditCriterion;
@@ -121,8 +121,8 @@ public class TransientReferenceService implements IReferenceService {
      * @see eu.etaxonomy.cdm.api.service.IService#lock(eu.etaxonomy.cdm.model.common.ICdmBase, org.hibernate.LockMode)
      */
     @Override
-    public void lock(Reference t, LockMode lockMode) {
-        defaultReferenceService.lock(t, lockMode);
+    public void lock(Reference t, LockOptions lockOptions) {
+        defaultReferenceService.lock(t, lockOptions);
     }
 
     /**
@@ -132,8 +132,8 @@ public class TransientReferenceService implements IReferenceService {
      * @see eu.etaxonomy.cdm.api.service.IService#refresh(eu.etaxonomy.cdm.model.common.ICdmBase, org.hibernate.LockMode, java.util.List)
      */
     @Override
-    public void refresh(Reference t, LockMode lockMode, List<String> propertyPaths) {
-        defaultReferenceService.refresh(t, lockMode, propertyPaths);
+    public void refresh(Reference t, LockOptions lockOptions, List<String> propertyPaths) {
+        defaultReferenceService.refresh(t, lockOptions, propertyPaths);
     }
 
     /**
index 9d00256fb3892cac1bbdff8a6a9d8e0b85c6df91..c5745650597dc1fd1140dbc9c5b150ec742ed1bf 100644 (file)
@@ -19,7 +19,7 @@ import java.util.UUID;
 
 import org.apache.lucene.index.CorruptIndexException;
 import org.apache.lucene.queryParser.ParseException;
-import org.hibernate.LockMode;
+import org.hibernate.LockOptions;
 import org.hibernate.Session;
 import org.hibernate.criterion.Criterion;
 import org.hibernate.envers.query.criteria.AuditCriterion;
@@ -144,8 +144,8 @@ public class TransientTaxonService implements ITaxonService {
      * @see eu.etaxonomy.cdm.api.service.IService#lock(eu.etaxonomy.cdm.model.common.ICdmBase, org.hibernate.LockMode)
      */
     @Override
-    public void lock(TaxonBase t, LockMode lockMode) {
-        defaultTaxonService.lock(t, lockMode);
+    public void lock(TaxonBase t, LockOptions lockOptions) {
+        defaultTaxonService.lock(t, lockOptions);
     }
 
     /**
@@ -155,8 +155,8 @@ public class TransientTaxonService implements ITaxonService {
      * @see eu.etaxonomy.cdm.api.service.IService#refresh(eu.etaxonomy.cdm.model.common.ICdmBase, org.hibernate.LockMode, java.util.List)
      */
     @Override
-    public void refresh(TaxonBase t, LockMode lockMode, List<String> propertyPaths) {
-        defaultTaxonService.refresh(t, lockMode, propertyPaths);
+    public void refresh(TaxonBase t, LockOptions lockOptions, List<String> propertyPaths) {
+        defaultTaxonService.refresh(t, lockOptions, propertyPaths);
     }
 
     /**
@@ -1343,33 +1343,21 @@ public class TransientTaxonService implements ITaxonService {
         return defaultTaxonService.createAllInferredSynonyms(taxon, tree, doWithMisappliedNames);
     }
 
-    /* (non-Javadoc)
-     * @see eu.etaxonomy.cdm.api.service.ITaxonService#deleteSynonym(eu.etaxonomy.cdm.model.taxon.Synonym, eu.etaxonomy.cdm.api.service.config.SynonymDeletionConfigurator)
-     */
     @Override
     public DeleteResult deleteSynonym(Synonym arg0, SynonymDeletionConfigurator arg1) {
         return defaultTaxonService.deleteSynonym(arg0, arg1);
     }
 
-    /* (non-Javadoc)
-     * @see eu.etaxonomy.cdm.api.service.ITaxonService#deleteSynonym(eu.etaxonomy.cdm.model.taxon.Synonym, eu.etaxonomy.cdm.model.taxon.Taxon, eu.etaxonomy.cdm.api.service.config.SynonymDeletionConfigurator)
-     */
     @Override
     public DeleteResult deleteSynonym(Synonym arg0, Taxon arg1, SynonymDeletionConfigurator arg2) {
         return defaultTaxonService.deleteSynonym(arg0, arg1, arg2);
     }
 
-    /* (non-Javadoc)
-     * @see eu.etaxonomy.cdm.api.service.ITaxonService#deleteTaxon(eu.etaxonomy.cdm.model.taxon.Taxon, eu.etaxonomy.cdm.api.service.config.TaxonDeletionConfigurator, eu.etaxonomy.cdm.model.taxon.Classification)
-     */
     @Override
     public DeleteResult deleteTaxon(Taxon arg0, TaxonDeletionConfigurator arg1, Classification arg2) {
         return defaultTaxonService.deleteTaxon(arg0, arg1, arg2);
     }
 
-    /* (non-Javadoc)
-     * @see eu.etaxonomy.cdm.api.service.ITaxonService#getSynonymsByHomotypicGroup(eu.etaxonomy.cdm.model.taxon.Taxon, java.util.List)
-     */
     @Override
     public List<List<Synonym>> getSynonymsByHomotypicGroup(Taxon taxon, List<String> propertyPaths) {
         return defaultTaxonService.getSynonymsByHomotypicGroup(taxon, propertyPaths);
index 4264a12bb327fd47e57efeee5f9e0c861a776cc2..578205af6b9fbf2ea94365554d1161b249e4dd86 100644 (file)
@@ -18,7 +18,7 @@ import java.util.Map;
 import java.util.Set;
 import java.util.UUID;
 
-import org.hibernate.LockMode;
+import org.hibernate.LockOptions;
 import org.hibernate.Session;
 import org.hibernate.criterion.Criterion;
 import org.hibernate.envers.query.criteria.AuditCriterion;
@@ -118,8 +118,8 @@ public class TransientTermService implements ITermService {
      * @see eu.etaxonomy.cdm.api.service.IService#lock(eu.etaxonomy.cdm.model.common.ICdmBase, org.hibernate.LockMode)
      */
     @Override
-    public void lock(DefinedTermBase t, LockMode lockMode) {
-        defaultTermService.lock(t, lockMode);
+    public void lock(DefinedTermBase t, LockOptions lockOptions) {
+        defaultTermService.lock(t, lockOptions);
     }
 
     /**
@@ -139,8 +139,8 @@ public class TransientTermService implements ITermService {
      * @see eu.etaxonomy.cdm.api.service.IService#refresh(eu.etaxonomy.cdm.model.common.ICdmBase, org.hibernate.LockMode, java.util.List)
      */
     @Override
-    public void refresh(DefinedTermBase t, LockMode lockMode, List<String> propertyPaths) {
-        defaultTermService.refresh(t, lockMode, propertyPaths);
+    public void refresh(DefinedTermBase t, LockOptions lockOptions, List<String> propertyPaths) {
+        defaultTermService.refresh(t, lockOptions, propertyPaths);
     }
 
     /**
index 377bc61dacce8f56966c40dcdf2de98c80c12b84..59001c82a44e2a50680d99f55e81b19c31a1f027 100644 (file)
@@ -20,7 +20,6 @@ import org.eclipse.jface.dialogs.MessageDialog;
 import eu.etaxonomy.cdm.api.conversation.IConversationEnabled;
 import eu.etaxonomy.cdm.api.service.DeleteResult;
 import eu.etaxonomy.cdm.api.service.IPolytomousKeyService;
-import eu.etaxonomy.cdm.api.service.exception.ReferencedObjectUndeletableException;
 import eu.etaxonomy.cdm.model.description.PolytomousKey;
 import eu.etaxonomy.taxeditor.operation.AbstractPersistentPostOperation;
 import eu.etaxonomy.taxeditor.operation.IPostOperationEnabled;
index 481b6d92848e1981f89ae8d8185f36dd76da7ef8..16bdd88787c12e5c748f4b49487505fcf949c728 100644 (file)
@@ -27,8 +27,6 @@ import eu.etaxonomy.cdm.api.service.DeleteResult;
 import eu.etaxonomy.cdm.api.service.IClassificationService;
 import eu.etaxonomy.cdm.api.service.ITaxonNodeService;
 import eu.etaxonomy.cdm.api.service.config.TaxonDeletionConfigurator;
-import eu.etaxonomy.cdm.api.service.exception.DataChangeNoRollbackException;
-import eu.etaxonomy.cdm.api.service.exception.ReferencedObjectUndeletableException;
 import eu.etaxonomy.cdm.model.common.CdmBase;
 import eu.etaxonomy.cdm.model.taxon.Classification;
 import eu.etaxonomy.cdm.model.taxon.ITaxonTreeNode;
@@ -103,7 +101,7 @@ public class DeleteOperation extends AbstractPersistentPostOperation{
                                //((TaxonNode) treeNode).delete();
 
                                element = ((TaxonNode)taxonNode).getTaxon();
-                               
+
                                DeleteResult result = service.deleteTaxonNode((TaxonNode)taxonNode, config);
                                if (result.isError()){
                                        //TODO:Error message!