From f9d7e5f6a2e562659448558fa18196d05831fe68 Mon Sep 17 00:00:00 2001
From: Alan Orth <alan.orth@gmail.com>
Date: Thu, 28 Dec 2023 10:26:11 +0300
Subject: [PATCH] src/main/java: minor refactor

Use isEmpty() instead of checking size.
---
 .../io/github/ilri/cgspace/ctasks/CountryCodeTagger.java    | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/main/java/io/github/ilri/cgspace/ctasks/CountryCodeTagger.java b/src/main/java/io/github/ilri/cgspace/ctasks/CountryCodeTagger.java
index 6891426..4a18290 100644
--- a/src/main/java/io/github/ilri/cgspace/ctasks/CountryCodeTagger.java
+++ b/src/main/java/io/github/ilri/cgspace/ctasks/CountryCodeTagger.java
@@ -91,7 +91,7 @@ public class CountryCodeTagger extends AbstractCurationTask {
                 itemService.getMetadataByMetadataString(item, config.iso3166Field);
 
         // skip items that don't have country metadata
-        if (itemCountries.size() == 0) {
+        if (itemCountries.isEmpty()) {
             alpha2Result.setResult(itemHandle + ": no countries, skipping.");
             alpha2Result.setStatus(Curator.CURATE_SKIP);
         } else {
@@ -134,7 +134,7 @@ public class CountryCodeTagger extends AbstractCurationTask {
             List<MetadataValue> itemAlpha2CountryCodes =
                     itemService.getMetadataByMetadataString(item, config.iso3166Alpha2Field);
 
-            if (itemAlpha2CountryCodes.size() == 0) {
+            if (itemAlpha2CountryCodes.isEmpty()) {
                 List<String> newAlpha2Codes = new ArrayList<String>();
                 for (MetadataValue itemCountry : itemCountries) {
                     // check ISO 3166-1 countries
@@ -158,7 +158,7 @@ public class CountryCodeTagger extends AbstractCurationTask {
                     }
                 }
 
-                if (newAlpha2Codes.size() > 0) {
+                if (!newAlpha2Codes.isEmpty()) {
                     try {
                         itemService.addMetadata(
                                 Curator.curationContext(),