Skip to content

Commit

Permalink
Debt/met 6032 Refactoring metis common library (#681)
Browse files Browse the repository at this point in the history
* MET-6032 Code refactoring

* MET-6032 Fixing code in Country class
  • Loading branch information
JoanaCMS authored Aug 8, 2024
1 parent 59547e5 commit b1d2365
Show file tree
Hide file tree
Showing 10 changed files with 36 additions and 12 deletions.
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
package eu.europeana.metis.core.common;
package eu.europeana.metis.utils;

import java.util.Arrays;
import java.util.Comparator;
Expand Down Expand Up @@ -120,6 +120,28 @@ public static Country getCountryFromIsoCode(String isoCode) {
return null;
}

/**
* Method that returns the enum with the corresponding given string value
* @param countryName The string value to match the enum values with
* @return The enum country values that matches the given string value
*/
public static Country fromCountryNameToIsoCode(String countryName){
Country result = null;
for(Country country : values()){
if(country.getName().equals(countryName)){
result = country;
break;
}
}

if(result == null){
throw new IllegalArgumentException("Country name "+countryName+" not found");
}

return result;

}

/**
* Provides the countries sorted by the {@link #getName()} field
*
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
import com.fasterxml.jackson.databind.DeserializationContext;
import com.fasterxml.jackson.databind.JsonNode;
import com.fasterxml.jackson.databind.deser.std.StdDeserializer;
import eu.europeana.metis.utils.Country;
import java.io.IOException;

/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
import com.fasterxml.jackson.core.JsonGenerator;
import com.fasterxml.jackson.databind.SerializerProvider;
import com.fasterxml.jackson.databind.ser.std.StdSerializer;
import eu.europeana.metis.utils.Country;
import java.io.IOException;

/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
import dev.morphia.annotations.Index;
import dev.morphia.annotations.IndexOptions;
import dev.morphia.annotations.Indexes;
import eu.europeana.metis.core.common.Country;
import eu.europeana.metis.utils.Country;
import eu.europeana.metis.core.common.CountryDeserializer;
import eu.europeana.metis.core.common.CountrySerializer;
import eu.europeana.metis.core.common.Language;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
import static org.junit.jupiter.api.Assertions.assertEquals;

import eu.europeana.metis.core.dataset.Dataset;
import eu.europeana.metis.utils.Country;
import java.util.Locale;
import org.junit.jupiter.api.Test;

Expand All @@ -20,4 +21,4 @@ void testTransformationParametersConstruction() {
assertEquals(dataset.getCountry().getName(), transformationParameters.getEdmCountry());
assertEquals(dataset.getLanguage().name().toLowerCase(Locale.US), transformationParameters.getEdmLanguage());
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,8 @@
import com.fasterxml.jackson.annotation.JsonProperty;
import eu.europeana.metis.authentication.rest.client.AuthenticationClient;
import eu.europeana.metis.authentication.user.MetisUserView;
import eu.europeana.metis.core.common.Country;
import eu.europeana.metis.core.common.CountrySerializer;
import eu.europeana.metis.utils.Country;
import eu.europeana.metis.core.common.Language;
import eu.europeana.metis.core.dataset.Dataset;
import eu.europeana.metis.core.dataset.DatasetSearchView;
Expand Down Expand Up @@ -585,7 +586,7 @@ public ResponseListWrapper<Dataset> getAllDatasetsByOrganizationName(
*
* @param authorization the String provided by an HTTP Authorization header <p> The expected input should follow the rule Bearer
* accessTokenHere </p>
* @return The list of countries that are serialized based on {@link eu.europeana.metis.core.common.CountrySerializer}
* @return The list of countries that are serialized based on {@link CountrySerializer}
* @throws GenericMetisException which can be one of:
* <ul>
* <li>{@link UserUnauthorizedException} if the user is unauthorized.</li>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@
import eu.europeana.metis.authentication.rest.client.AuthenticationClient;
import eu.europeana.metis.authentication.user.AccountRole;
import eu.europeana.metis.authentication.user.MetisUserView;
import eu.europeana.metis.core.common.Country;
import eu.europeana.metis.utils.Country;
import eu.europeana.metis.core.common.Language;
import eu.europeana.metis.core.dataset.Dataset;
import eu.europeana.metis.core.dataset.DatasetSearchView;
Expand Down Expand Up @@ -1143,4 +1143,4 @@ private List<Dataset> getDatasets() {
}


}
}
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
import eu.europeana.cloud.common.model.dps.TaskErrorsInfo;
import eu.europeana.metis.authentication.user.AccountRole;
import eu.europeana.metis.authentication.user.MetisUserView;
import eu.europeana.metis.core.common.Country;
import eu.europeana.metis.utils.Country;
import eu.europeana.metis.core.common.Language;
import eu.europeana.metis.core.dao.WorkflowExecutionDao.ExecutionDatasetPair;
import eu.europeana.metis.core.dataset.Dataset;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
package eu.europeana.metis.core.service;

import eu.europeana.cloud.common.model.dps.TaskState;
import eu.europeana.metis.core.common.Country;
import eu.europeana.metis.utils.Country;
import eu.europeana.metis.core.common.Language;
import eu.europeana.metis.core.dataset.Dataset;
import eu.europeana.metis.core.dataset.Dataset.PublicationFitness;
Expand All @@ -17,9 +17,7 @@
import eu.europeana.metis.core.workflow.plugins.ExecutablePluginType;
import eu.europeana.metis.core.workflow.plugins.ExecutionProgress;
import eu.europeana.metis.core.workflow.plugins.HTTPHarvestPluginMetadata;
import eu.europeana.metis.core.workflow.plugins.IndexToPreviewPlugin;
import eu.europeana.metis.core.workflow.plugins.IndexToPreviewPluginMetadata;
import eu.europeana.metis.core.workflow.plugins.IndexToPublishPlugin;
import eu.europeana.metis.core.workflow.plugins.IndexToPublishPluginMetadata;
import eu.europeana.metis.core.workflow.plugins.MediaProcessPluginMetadata;
import eu.europeana.metis.core.workflow.plugins.NormalizationPluginMetadata;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
import eu.europeana.cloud.common.model.dps.TaskErrorsInfo;
import eu.europeana.metis.authentication.user.AccountRole;
import eu.europeana.metis.authentication.user.MetisUserView;
import eu.europeana.metis.core.common.Country;
import eu.europeana.metis.utils.Country;
import eu.europeana.metis.core.common.Language;
import eu.europeana.metis.core.dao.WorkflowExecutionDao.ExecutionDatasetPair;
import eu.europeana.metis.core.dataset.Dataset;
Expand Down

0 comments on commit b1d2365

Please sign in to comment.