]> git.mxchange.org Git - jjobs-ejb.git/commitdiff
Please cherry-pick:
authorRoland Häder <roland@mxchange.org>
Wed, 1 Apr 2020 17:29:24 +0000 (19:29 +0200)
committerRoland Häder <roland@mxchange.org>
Thu, 2 Apr 2020 16:34:35 +0000 (18:34 +0200)
- no need for SearchCountryByCodeI18nKey anymore when you can use AllCountries
  plus Objects.equals()

Signed-off-by: Roland Häder <roland@mxchange.org>
src/java/org/mxchange/jcountry/model/data/JobsAdminCountrySingletonBean.java

index 274d6576150ccc8265c117b4c4d7fc2f8709f270..5265d00160650d8c2e374607b602e66919ee00b9 100644 (file)
  *
  * You should have received a copy of the GNU Affero General Public License
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
-*/
+ */
 package org.mxchange.jcountry.model.data;
 
 import java.text.MessageFormat;
 import java.util.Date;
+import java.util.Objects;
+import javax.ejb.EJB;
 import javax.ejb.Stateless;
-import javax.persistence.NoResultException;
-import javax.persistence.Query;
 import org.mxchange.jcountry.exceptions.CountryAlreadyAddedException;
 import org.mxchange.jjobs.enterprise.BaseJobsEnterpriseBean;
 
@@ -37,6 +37,12 @@ public class JobsAdminCountrySingletonBean extends BaseJobsEnterpriseBean implem
         */
        private static final long serialVersionUID = 15_846_983_298_691_208L;
 
+       /**
+        * Remote country EJB
+        */
+       @EJB (lookup = "java:global/jfinancials-ejb/country!org.mxchange.jcountry.model.data.CountrySingletonBeanRemote")
+       private CountrySingletonBeanRemote countryBean;
+
        /**
         * Default constructor
         */
@@ -107,22 +113,14 @@ public class JobsAdminCountrySingletonBean extends BaseJobsEnterpriseBean implem
                // Default is not found
                boolean isAdded = false;
 
-               // Get query instance
-               final Query query = this.getEntityManager().createNamedQuery("SearchCountryByCodeI18nKey", CountryData.class); //NOI18N
-
-               // Assign all parameters
-               query.setParameter("code", country.getCountryCode()); //NOI18N
-               query.setParameter("key", country.getCountryI18nKey()); //NOI18N
-
-               // Try to get a single result
-               try {
-                       // Get single result
-                       final Country foundCountry = (Country) query.getSingleResult();
-
-                       // Found it?
-                       isAdded = (foundCountry instanceof Country);
-               } catch (final NoResultException ex) {
-                       // Not found, don't log this
+               // Try to match code/i18n key (should be both unique!)
+               for (final Country currentCountry : this.countryBean.allCountries()) {
+                       // Is it matching
+                       if (Objects.equals(country, currentCountry)) {
+                               // Yes, then set flag and abort loop
+                               isAdded = true;
+                               break;
+                       }
                }
 
                // Return result