fix #8683 TaxonDaoHibernateImpl.countTaxaByName rename parameter authorship authorshi...
authorAndreas Kohlbecker <a.kohlbecker@bgbm.org>
Fri, 15 Nov 2019 08:57:29 +0000 (09:57 +0100)
committerAndreas Kohlbecker <a.kohlbecker@bgbm.org>
Fri, 15 Nov 2019 08:57:29 +0000 (09:57 +0100)
cdmlib-persistence/src/main/java/eu/etaxonomy/cdm/persistence/dao/hibernate/taxon/TaxonDaoHibernateImpl.java
cdmlib-persistence/src/main/java/eu/etaxonomy/cdm/persistence/dao/taxon/ITaxonDao.java
cdmlib-remote/src/main/java/eu/etaxonomy/cdm/remote/controller/TaxonListController.java
cdmlib-services/src/main/java/eu/etaxonomy/cdm/api/service/ITaxonService.java
cdmlib-services/src/main/java/eu/etaxonomy/cdm/api/service/TaxonServiceImpl.java

index 5fc9b1c4a8cb41c4a4bf45cf18a2750e44723d71..bb401eeb862d10277cfc62ea01a6a15dd98f162d 100755 (executable)
@@ -887,8 +887,9 @@ public class TaxonDaoHibernateImpl
     }
 
     @Override
-    public long countTaxaByName(Class<? extends TaxonBase> clazz, String genusOrUninomial, String infraGenericEpithet, String specificEpithet, String infraSpecificEpithet, String authorship, Rank rank) {
-        checkNotInPriorView("TaxonDaoHibernateImpl.countTaxaByName(Boolean accepted, String genusOrUninomial,  String infraGenericEpithet, String specificEpithet,     String infraSpecificEpithet, String authorship, Rank rank)");
+    public long countTaxaByName(Class<? extends TaxonBase> clazz, String genusOrUninomial, String infraGenericEpithet, String specificEpithet,
+            String infraSpecificEpithet, String authorshipCache, Rank rank) {
+        checkNotInPriorView("TaxonDaoHibernateImpl.countTaxaByName(Boolean accepted, String genusOrUninomial,  String infraGenericEpithet, String specificEpithet,     String infraSpecificEpithet, String authorshipCache, Rank rank)");
         Criteria criteria = null;
 
         criteria = getCriteria(clazz);
@@ -921,10 +922,10 @@ public class TaxonDaoHibernateImpl
             criteria.add(Restrictions.eq("name.infraSpecificEpithet", infraSpecificEpithet));
         }
 
-        if(authorship == null) {
+        if(authorshipCache == null) {
             criteria.add(Restrictions.eq("name.authorshipCache", ""));
-        } else if(!authorship.equals("*")) {
-            criteria.add(Restrictions.eq("name.authorshipCache", authorship));
+        } else if(!authorshipCache.equals("*")) {
+            criteria.add(Restrictions.eq("name.authorshipCache", authorshipCache));
         }
 
         if(rank != null) {
index 118f22043a1668f47dcb6abd55998c068ff148ba..355d341e8420d6a504e43bd90f68e627cb682470 100644 (file)
@@ -140,9 +140,11 @@ public interface ITaxonDao
      * @param specificEpithet
      * @param infraspecificEpithet
      * @param rank
+     * @param authorshipCache
      * @return a count of TaxonBase instances
      */
-    public long countTaxaByName(Class <? extends TaxonBase> clazz, String uninomial, String infragenericEpithet,String specificEpithet, String infraspecificEpithet, String authorship, Rank rank);
+    public long countTaxaByName(Class <? extends TaxonBase> clazz, String uninomial, String infragenericEpithet,String specificEpithet,
+            String infraspecificEpithet, String authorshipCache, Rank rank);
 
     /**
      * Returns a list of TaxonBase instances where the
@@ -161,11 +163,13 @@ public interface ITaxonDao
      * @param specificEpithet
      * @param infraspecificEpithet
      * @param rank
+     * @param authorshipCache
      * @param pageSize The maximum number of taxa returned (can be null for all matching taxa)
      * @param pageNumber The offset (in pageSize chunks) from the start of the result set (0 - based)
      * @return a list of TaxonBase instances
      */
-    public List<TaxonBase> findTaxaByName(Class<? extends TaxonBase> clazz, String uninomial, String infragenericEpithet, String specificEpithet, String infraspecificEpithet, String authorship, Rank rank, Integer pageSize, Integer pageNumber);
+    public List<TaxonBase> findTaxaByName(Class<? extends TaxonBase> clazz, String uninomial, String infragenericEpithet, String specificEpithet,
+            String infraspecificEpithet, String authorshipCache, Rank rank, Integer pageSize, Integer pageNumber);
 
     /**
      * Find taxa by searching for Taxa and Synonyms where the
index 239ca8610465f7e5f5734b3e2bb279d1631daffc..8df96f672cd2c5d5679d2525b24b93e2443c6ac2 100644 (file)
@@ -590,7 +590,7 @@ public class TaxonListController extends AbstractIdentifiableListController<Taxo
             @RequestParam(value = "infragenericEpithet", required = false) String infragenericEpithet,
             @RequestParam(value = "specificEpithet", required = false) String specificEpithet,
             @RequestParam(value = "infraspecificEpithet", required = false) String infraspecificEpithet,
-            @RequestParam(value = "authorship", required = false) String authorship,
+            @RequestParam(value = "authorshipCache", required = false) String authorshipCache,
             @RequestParam(value = "rankUuid", required = false) UUID rankUuid,
             @RequestParam(value = "pageNumber", required = false) Integer pageNumber,
             @RequestParam(value = "pageSize", required = false) Integer pageSize,
@@ -609,7 +609,8 @@ public class TaxonListController extends AbstractIdentifiableListController<Taxo
              rank = findRank(rankUuid);
         }
 
-        Pager<TaxonBase> result = service.findTaxaByName(null, genusOrUninomial, infragenericEpithet, specificEpithet, infraspecificEpithet, authorship, rank, pageSize, pageNumber);
+        Pager<TaxonBase> result = service.findTaxaByName(null, genusOrUninomial, infragenericEpithet, specificEpithet, infraspecificEpithet,
+                authorshipCache, rank, pageSize, pageNumber);
 
         return result;
     }
index 2038c634111ade4625ed2ad4792fe421e3023d99..d6203bac2d7c0fc8c3b5640cf88d4ea386922f58 100644 (file)
@@ -56,7 +56,6 @@ import eu.etaxonomy.cdm.model.taxon.TaxonRelationship;
 import eu.etaxonomy.cdm.model.taxon.TaxonRelationshipType;
 import eu.etaxonomy.cdm.model.term.DefinedTerm;
 import eu.etaxonomy.cdm.persistence.dao.common.Restriction;
-import eu.etaxonomy.cdm.persistence.dao.initializer.IBeanInitializer;
 import eu.etaxonomy.cdm.persistence.dto.UuidAndTitleCache;
 import eu.etaxonomy.cdm.persistence.query.MatchMode;
 import eu.etaxonomy.cdm.persistence.query.OrderHint;
@@ -469,7 +468,7 @@ public interface ITaxonService
      * @param pageNumber The offset (in pageSize chunks) from the start of the result set (0 - based)
      * @return a list of TaxonBase instances
      */
-    public Pager<TaxonBase> findTaxaByName(Class<? extends TaxonBase> clazz, String uninomial, String infragenericEpithet, String specificEpithet, String infraspecificEpithet, String authorship, Rank rank, Integer pageSize, Integer pageNumber);
+    public Pager<TaxonBase> findTaxaByName(Class<? extends TaxonBase> clazz, String uninomial, String infragenericEpithet, String specificEpithet, String infraspecificEpithet, String authorshipCache, Rank rank, Integer pageSize, Integer pageNumber);
 
     /**
      * Returns a list of TaxonBase instances where the
@@ -482,11 +481,13 @@ public interface ITaxonService
      * @param specificEpithet
      * @param infraspecificEpithet
      * @param rank
+     * @param authorshipCache
      * @param pageSize The maximum number of taxa returned (can be null for all matching taxa)
      * @param pageNumber The offset (in pageSize chunks) from the start of the result set (0 - based)
      * @return a List of TaxonBase instances
      */
-    public List<TaxonBase> listTaxaByName(Class<? extends TaxonBase> clazz, String uninomial, String infragenericEpithet, String specificEpithet, String infraspecificEpithet, String authorship, Rank rank, Integer pageSize, Integer pageNumber);
+    public List<TaxonBase> listTaxaByName(Class<? extends TaxonBase> clazz, String uninomial, String infragenericEpithet, String specificEpithet,
+            String infraspecificEpithet, String authorshipCache, Rank rank, Integer pageSize, Integer pageNumber);
 
     /**
      * Returns a list of IdentifiableEntity instances (in particular, TaxonName and TaxonBase instances)
@@ -794,7 +795,7 @@ public interface ITaxonService
      */
     public long countSynonyms(boolean onlyAttachedToTaxon);
 
-    
+
 
     public Map<String, List<TaxonName>> findIdenticalTaxonNameIds(Reference sec1, Reference sec2, List<String> propertyPaths);
 
index f0fae59aeebe8229237e38a9e0e8bea8e0b90fed..d0c96f26e2126080851abac3dd152ada70bc80d5 100644 (file)
@@ -505,24 +505,26 @@ public class TaxonServiceImpl
 
 
     @Override
-    public Pager<TaxonBase> findTaxaByName(Class<? extends TaxonBase> clazz, String uninomial, String infragenericEpithet, String specificEpithet,     String infraspecificEpithet, String authorship, Rank rank, Integer pageSize,Integer pageNumber) {
-        long numberOfResults = dao.countTaxaByName(clazz, uninomial, infragenericEpithet, specificEpithet, infraspecificEpithet, authorship, rank);
+    public Pager<TaxonBase> findTaxaByName(Class<? extends TaxonBase> clazz, String uninomial, String infragenericEpithet, String specificEpithet,
+            String infraspecificEpithet, String authorshipCache, Rank rank, Integer pageSize,Integer pageNumber) {
+        long numberOfResults = dao.countTaxaByName(clazz, uninomial, infragenericEpithet, specificEpithet, infraspecificEpithet, authorshipCache, rank);
 
         List<TaxonBase> results = new ArrayList<>();
         if(numberOfResults > 0) { // no point checking again
-            results = dao.findTaxaByName(clazz, uninomial, infragenericEpithet, specificEpithet, infraspecificEpithet, authorship, rank, pageSize, pageNumber);
+            results = dao.findTaxaByName(clazz, uninomial, infragenericEpithet, specificEpithet, infraspecificEpithet, authorshipCache, rank, pageSize, pageNumber);
         }
 
         return new DefaultPagerImpl<>(pageNumber, numberOfResults, pageSize, results);
     }
 
     @Override
-    public List<TaxonBase> listTaxaByName(Class<? extends TaxonBase> clazz, String uninomial, String infragenericEpithet, String specificEpithet, String infraspecificEpithet, String authorship, Rank rank, Integer pageSize,Integer pageNumber) {
-        long numberOfResults = dao.countTaxaByName(clazz, uninomial, infragenericEpithet, specificEpithet, infraspecificEpithet, authorship, rank);
+    public List<TaxonBase> listTaxaByName(Class<? extends TaxonBase> clazz, String uninomial, String infragenericEpithet, String specificEpithet,
+            String infraspecificEpithet, String authorshipCache, Rank rank, Integer pageSize,Integer pageNumber) {
+        long numberOfResults = dao.countTaxaByName(clazz, uninomial, infragenericEpithet, specificEpithet, infraspecificEpithet, authorshipCache, rank);
 
         List<TaxonBase> results = new ArrayList<>();
         if(numberOfResults > 0) { // no point checking again
-            results = dao.findTaxaByName(clazz, uninomial, infragenericEpithet, specificEpithet, infraspecificEpithet, authorship, rank, pageSize, pageNumber);
+            results = dao.findTaxaByName(clazz, uninomial, infragenericEpithet, specificEpithet, infraspecificEpithet, authorshipCache, rank, pageSize, pageNumber);
         }
 
         return results;