diff --git a/CHANGELOG.md b/CHANGELOG.md index 9013656..a5ee362 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -15,6 +15,7 @@ Moved to maven-portal-bom 3.6.3 [#20599] Get List of Records [#22002] Integrated with ValidationReport and status [#21990] Provide the (first version of) edit mode +[#22040] Revisited the "Abstract Relazione di Scavo" ## [v1.2.0] - 2020-12-18 diff --git a/src/main/java/org/gcube/portlets/user/geoportaldataentry/server/ConvertToServiceModel.java b/src/main/java/org/gcube/portlets/user/geoportaldataentry/server/ConvertToServiceModel.java index e758a53..d6ee1ac 100644 --- a/src/main/java/org/gcube/portlets/user/geoportaldataentry/server/ConvertToServiceModel.java +++ b/src/main/java/org/gcube/portlets/user/geoportaldataentry/server/ConvertToServiceModel.java @@ -8,6 +8,7 @@ import java.util.Date; import java.util.List; import java.util.Map; +import org.gcube.application.geoportal.common.model.legacy.AbstractRelazione; import org.gcube.application.geoportal.common.model.legacy.AccessPolicy; import org.gcube.application.geoportal.common.model.legacy.Concessione; import org.gcube.application.geoportal.common.model.legacy.LayerConcessione; @@ -19,392 +20,383 @@ import org.gcube.vomanagement.usermanagement.model.GCubeUser; import org.slf4j.Logger; import org.slf4j.LoggerFactory; - /** * The Class ConvertToServiceModel. * * @author Francesco Mangiacrapa at ISTI-CNR (francesco.mangiacrapa@isti.cnr.it) * - * Oct 21, 2020 + * Oct 21, 2020 */ public class ConvertToServiceModel { - + /** The Constant LOG. */ private static final Logger LOG = LoggerFactory.getLogger(ConvertToServiceModel.class); - + public static final String HOURS_MINUTES_SEPARATOR = ConstantsGeoPortalDataEntryApp.HOURS_MINUTES_SEPARATOR; - + public static final String DATE_FORMAT = ConstantsGeoPortalDataEntryApp.DATE_FORMAT; - + public static final String TIME_FORMAT = ConstantsGeoPortalDataEntryApp.TIME_FORMAT; /** * To concessione. * - * @param gdb the gdb + * @param gdb the gdb * @param user the user * @return the concessione - * @throws Exception + * @throws Exception the exception */ public static Concessione toConcessione(GenericDatasetBean gdb, GCubeUser user) throws Exception { - + Map> mapFields = gdb.getFormDataEntryFields(); - + Concessione concessione = new Concessione(); - - + // List authors = mapFields.get("Autore"); // if(authors!=null) { // concessione.setAuthors(authors); // } - - + List authors = mapFields.get("Nome Autore, Email, Ruolo"); - if(authors!=null) { + if (authors != null) { concessione.setAuthors(authors); } List contributors = mapFields.get("Contributore"); - if(contributors!=null) { + if (contributors != null) { for (String contributor : contributors) { concessione.setContributore(contributor); } } - - //concessione.setCreationTime(Instant.now()); + + // concessione.setCreationTime(Instant.now()); concessione.setCreationUser(user.getUsername()); - + List dataInizProgettoList = mapFields.get("Data inizio Progetto"); - if(dataInizProgettoList!=null && dataInizProgettoList.size()>0) { + if (dataInizProgettoList != null && dataInizProgettoList.size() > 0) { String inizioProgetto = dataInizProgettoList.get(0); LocalDateTime theLDT = toLocalDateTime(inizioProgetto); concessione.setDataInizioProgetto(theLDT); } - + List dataFineProgettoList = mapFields.get("Data fine Progetto"); - if(dataFineProgettoList!=null && dataFineProgettoList.size()>0) { + if (dataFineProgettoList != null && dataFineProgettoList.size() > 0) { String fineProgetto = dataFineProgettoList.get(0); LocalDateTime theLDT = toLocalDateTime(fineProgetto); concessione.setDataFineProgetto(theLDT); } - - + List descrizioneLst = mapFields.get("Descrizione del contenuto"); - if(descrizioneLst!=null && descrizioneLst.size()>0) { + if (descrizioneLst != null && descrizioneLst.size() > 0) { concessione.setDescrizioneContenuto(descrizioneLst.get(0)); } - + List editors = mapFields.get("Editore"); - if(editors!=null && editors.size()>0) { + if (editors != null && editors.size() > 0) { concessione.setEditore(editors.get(0)); } - + List fontiFinanziamento = mapFields.get("Fonte del finanziamento"); - if(fontiFinanziamento!=null) { + if (fontiFinanziamento != null) { concessione.setFontiFinanziamento(fontiFinanziamento); } - + List licenzaLst = mapFields.get("ID Licenza"); - if(licenzaLst!=null && licenzaLst.size()>0) { + if (licenzaLst != null && licenzaLst.size() > 0) { concessione.setLicenzaID(licenzaLst.get(0)); } - + List introduzioneLst = mapFields.get("Introduzione"); - if(introduzioneLst!=null && introduzioneLst.size()>0) { + if (introduzioneLst != null && introduzioneLst.size() > 0) { concessione.setIntroduzione(introduzioneLst.get(0)); } - + List nomeLst = mapFields.get("Nome del progetto"); - if(nomeLst!=null && nomeLst.size()>0) { + if (nomeLst != null && nomeLst.size() > 0) { concessione.setNome(nomeLst.get(0)); } - + List paroleChiaveLibereLst = mapFields.get("Parola chiave a scelta libera"); - //LOG.debug("Parola chiave a scelta libera: "+paroleChiaveLibereLst); - if(paroleChiaveLibereLst!=null){ + // LOG.debug("Parola chiave a scelta libera: "+paroleChiaveLibereLst); + if (paroleChiaveLibereLst != null) { concessione.setParoleChiaveLibere(paroleChiaveLibereLst); } - + List paroleChiaveICCDLst = mapFields.get("Parola chiave relativa alla cronologia"); - //LOG.debug("Parola chiave relativa alla cronologia: "+paroleChiaveICCDLst); - if(paroleChiaveICCDLst!=null){ + // LOG.debug("Parola chiave relativa alla cronologia: "+paroleChiaveICCDLst); + if (paroleChiaveICCDLst != null) { concessione.setParoleChiaveICCD(paroleChiaveICCDLst); } - + List risorsaCorrelataLst = mapFields.get("Risorsa correlata"); - if(risorsaCorrelataLst!=null){ + if (risorsaCorrelataLst != null) { concessione.setRisorseCorrelate(risorsaCorrelataLst); } - + List responsabileLst = mapFields.get("Responsabile"); - if(responsabileLst!=null && responsabileLst.size()>0){ + if (responsabileLst != null && responsabileLst.size() > 0) { concessione.setResponsabile(responsabileLst.get(0)); } - + List soggettoLst = mapFields.get("Soggetto"); - if(soggettoLst!=null){ + if (soggettoLst != null) { concessione.setSoggetto(soggettoLst); } - + List titolareCopyrightLst = mapFields.get("Titolare Copyright"); - if(titolareCopyrightLst!=null){ + if (titolareCopyrightLst != null) { concessione.setTitolareCopyright(titolareCopyrightLst); } - + List titolareLicenzaLst = mapFields.get("Titolare Licenza"); - if(titolareLicenzaLst!=null){ + if (titolareLicenzaLst != null) { concessione.setTitolareLicenza(titolareLicenzaLst); } - + List titolareDatiLst = mapFields.get("Titolare dei dati"); - if(titolareDatiLst!=null){ + if (titolareDatiLst != null) { concessione.setTitolari(titolareDatiLst); } - + List latList = mapFields.get("Latitudine Centroide"); - if(latList!=null && latList.size()>0){ + if (latList != null && latList.size() > 0) { try { String theLat = latList.get(0); - if(theLat!=null && !theLat.isEmpty()) { + if (theLat != null && !theLat.isEmpty()) { Double centroidLat = Double.parseDouble(theLat); concessione.setCentroidLat(centroidLat); } - }catch (Exception e) { - throw new Exception("Unable to parse "+latList.get(0)+" as valid latitude"); + } catch (Exception e) { + throw new Exception("Unable to parse " + latList.get(0) + " as valid latitude"); } } - + List longList = mapFields.get("Longitudine Centroide"); - if(longList!=null && longList.size()>0){ + if (longList != null && longList.size() > 0) { try { String theLong = longList.get(0); - if(theLong!=null && !theLong.isEmpty()) { + if (theLong != null && !theLong.isEmpty()) { Double centroidLong = Double.parseDouble(theLong); concessione.setCentroidLong(centroidLong); } - }catch (Exception e) { - throw new Exception("Unable to parse "+longList.get(0)+" as valid longitude"); + } catch (Exception e) { + throw new Exception("Unable to parse " + longList.get(0) + " as valid longitude"); } } - + return concessione; - + } - + /** * To relazione scavo. * - * @param abstractRelazioneScavo the abstract relazione scavo * @param gdb the gdb * @return the relazione scavo */ - public static RelazioneScavo toRelazioneScavo(RelazioneScavo abstractRelazioneScavo, GenericDatasetBean gdb){ - + public static RelazioneScavo toRelazioneScavo(GenericDatasetBean gdb) { + Map> mapFields = gdb.getFormDataEntryFields(); - - RelazioneScavo relazioneScavo = abstractRelazioneScavo == null?new RelazioneScavo():abstractRelazioneScavo; - + + RelazioneScavo relazioneScavo = new RelazioneScavo(); + List responsabiliLst = mapFields.get("Responsabile del documento"); - if(responsabiliLst!=null) { + if (responsabiliLst != null) { relazioneScavo.setResponsabili(responsabiliLst); } - + List autoreList = mapFields.get("Autore del documento"); - if(autoreList!=null) { - //TODO + if (autoreList != null) { + // TODO } - + List politicaDiAccessoLst = mapFields.get("Politica di accesso"); - if(politicaDiAccessoLst!=null && politicaDiAccessoLst.size()>0) { + if (politicaDiAccessoLst != null && politicaDiAccessoLst.size() > 0) { try { AccessPolicy ap = AccessPolicy.valueOf(politicaDiAccessoLst.get(0)); relazioneScavo.setPolicy(ap); - }catch (Exception e) { - LOG.warn("I cannot cast "+politicaDiAccessoLst.get(0) +" to "+AccessPolicy.values(),e); + } catch (Exception e) { + LOG.warn("I cannot cast " + politicaDiAccessoLst.get(0) + " to " + AccessPolicy.values(), e); } } - + List licenzaIdList = mapFields.get("ID Licenza"); - if(licenzaIdList!=null && licenzaIdList.size()>0) { + if (licenzaIdList != null && licenzaIdList.size() > 0) { relazioneScavo.setLicenseID(licenzaIdList.get(0)); } - - //TODO + + // TODO List periodoDiEmbargo = mapFields.get("Periodo di embargo"); - if(periodoDiEmbargo!=null && periodoDiEmbargo.size()>0) { + if (periodoDiEmbargo != null && periodoDiEmbargo.size() > 0) { String dateFromTo = periodoDiEmbargo.get(0); String[] dates = dateFromTo.split(","); String dateFrom = dates[0]; String dateTo = dates[0]; - //TODO MUST BE INTENGRATED IN THE SERVICE + // TODO MUST BE INTENGRATED IN THE SERVICE } - + return relazioneScavo; } - - public static RelazioneScavo toAbstractRelazioneScavo(GenericDatasetBean gdb) { - + + /** + * To abstract relazione scavo. + * + * @param gdb the gdb + * @return the abstract relazione + */ + public static AbstractRelazione toAbstractRelazioneScavo(GenericDatasetBean gdb) { + Map> mapFields = gdb.getFormDataEntryFields(); - - RelazioneScavo relazioneScavo = new RelazioneScavo(); - + + AbstractRelazione abstractrelazione = new AbstractRelazione(); + List abstractLstIta = mapFields.get("Abstract in Italiano"); - if(abstractLstIta!=null && abstractLstIta.size()>0) { - - /** - * TODO - */ - LOG.warn("TODO ABSTRACT RELAZIONE DI SCAVO IN ITA"); - //relazioneScavo.setAbstractIta(abstractLstIta.get(0)); + if (abstractLstIta != null && abstractLstIta.size() > 0) { + abstractrelazione.setAbstractIta(abstractLstIta.get(0)); } - + List abstractLstEng = mapFields.get("Abstract in Inglese"); - if(abstractLstEng!=null && abstractLstEng.size()>0) { - /** - * TODO - */ - LOG.warn("TODO ABSTRACT RELAZIONE DI SCAVO IN ENG"); - //relazioneScavo.setAbstractEng(abstractLstEng.get(0)); + if (abstractLstEng != null && abstractLstEng.size() > 0) { + abstractrelazione.setAbstractEng(abstractLstEng.get(0)); } - - //TODO IN FUTURE WHEN THE MODEL WILL BE UPDATED - /*List politicaDiAccessoLst = mapFields.get("Politica di accesso"); - if(politicaDiAccessoLst!=null && politicaDiAccessoLst.size()>0) { + + // THIS SHOULD BE ALWAYS "OPEN" + List politicaDiAccessoLst = mapFields.get("Politica di accesso"); + if (politicaDiAccessoLst != null && politicaDiAccessoLst.size() > 0) { try { AccessPolicy ap = AccessPolicy.valueOf(politicaDiAccessoLst.get(0)); - relazioneScavo.setPolicy(ap); - }catch (Exception e) { - LOG.warn("I cannot cast "+politicaDiAccessoLst.get(0) +" to "+AccessPolicy.values(),e); + abstractrelazione.setPolicy(ap); + } catch (Exception e) { + LOG.warn("I cannot cast " + politicaDiAccessoLst.get(0) + " to " + AccessPolicy.values(), e); } } - + + // THIS SHOULD BE ALWAYS "CC-BY-4.0" List licenzaIdList = mapFields.get("ID Licenza"); - if(licenzaIdList!=null && licenzaIdList.size()>0) { - relazioneScavo.setLicenseID(licenzaIdList.get(0)); - }*/ - - return relazioneScavo; + if (licenzaIdList != null && licenzaIdList.size() > 0) { + abstractrelazione.setLicenseID(licenzaIdList.get(0)); + } + + return abstractrelazione; } - + /** * To immagini rappresentative. * * @param gdb the gdb * @return the uploaded image */ - public static UploadedImage toImmaginiRappresentative(GenericDatasetBean gdb){ - + public static UploadedImage toImmaginiRappresentative(GenericDatasetBean gdb) { + Map> mapFields = gdb.getFormDataEntryFields(); - + UploadedImage uplaodedImage = new UploadedImage(); - + // List titoloLst = mapFields.get("Titolo"); // if(titoloLst!=null && titoloLst.size()>0) { // uplaodedImage.setTitolo(titoloLst.get(0)); // } // List didascaliaLst = mapFields.get("Didascalia"); - if(didascaliaLst!=null && didascaliaLst.size()>0) { + if (didascaliaLst != null && didascaliaLst.size() > 0) { uplaodedImage.setDidascalia(didascaliaLst.get(0)); } - - //NB here is setResponsabili but should be setAuthor + + // NB here is setResponsabili but should be setAuthor List responsabileLst = mapFields.get("Autore"); - if(responsabileLst!=null) { + if (responsabileLst != null) { uplaodedImage.setResponsabili(responsabileLst); - //uplaodedImage.setAuthor is missing!!! + // uplaodedImage.setAuthor is missing!!! } - + // List autoreList = mapFields.get("Autore"); // if(autoreList!=null) { // //TODO // } - + List politicaDiAccessoLst = mapFields.get("Politica di accesso"); - if(politicaDiAccessoLst!=null && politicaDiAccessoLst.size()>0) { + if (politicaDiAccessoLst != null && politicaDiAccessoLst.size() > 0) { try { AccessPolicy ap = AccessPolicy.valueOf(politicaDiAccessoLst.get(0)); uplaodedImage.setPolicy(ap); - }catch (Exception e) { - LOG.warn("I cannot cast "+politicaDiAccessoLst.get(0) +" to "+AccessPolicy.values(),e); + } catch (Exception e) { + LOG.warn("I cannot cast " + politicaDiAccessoLst.get(0) + " to " + AccessPolicy.values(), e); } } - + List licenzaIdList = mapFields.get("ID Licenza"); - if(licenzaIdList!=null && licenzaIdList.size()>0) { + if (licenzaIdList != null && licenzaIdList.size() > 0) { uplaodedImage.setLicenseID(licenzaIdList.get(0)); } - + return uplaodedImage; } - + /** * To layer concessione. * * @param gdb the gdb * @return the layer concessione */ - public static LayerConcessione toLayerConcessione(GenericDatasetBean gdb){ - + public static LayerConcessione toLayerConcessione(GenericDatasetBean gdb) { + Map> mapFields = gdb.getFormDataEntryFields(); - + LayerConcessione layerConcessione = new LayerConcessione(); - + List titoloLst = mapFields.get("Titolo"); - if(titoloLst!=null && titoloLst.size()>0) { + if (titoloLst != null && titoloLst.size() > 0) { layerConcessione.setTitolo(titoloLst.get(0)); } - + List politicaDiAccessoLst = mapFields.get("Politica di accesso"); - if(politicaDiAccessoLst!=null && politicaDiAccessoLst.size()>0) { + if (politicaDiAccessoLst != null && politicaDiAccessoLst.size() > 0) { try { AccessPolicy ap = AccessPolicy.valueOf(politicaDiAccessoLst.get(0)); layerConcessione.setPolicy(ap); - }catch (Exception e) { - LOG.warn("I cannot cast "+politicaDiAccessoLst.get(0) +" to "+AccessPolicy.values(),e); + } catch (Exception e) { + LOG.warn("I cannot cast " + politicaDiAccessoLst.get(0) + " to " + AccessPolicy.values(), e); } } List valutazioneQualitaLst = mapFields.get("Valutazione della qualità"); - if(valutazioneQualitaLst!=null && valutazioneQualitaLst.size()>0) { + if (valutazioneQualitaLst != null && valutazioneQualitaLst.size() > 0) { layerConcessione.setValutazioneQualita(valutazioneQualitaLst.get(0)); } List metodoRaccoltaDatiLst = mapFields.get("Metodo di raccolta dei dati"); - if(metodoRaccoltaDatiLst!=null && metodoRaccoltaDatiLst.size()>0) { + if (metodoRaccoltaDatiLst != null && metodoRaccoltaDatiLst.size() > 0) { layerConcessione.setMetodoRaccoltaDati(metodoRaccoltaDatiLst.get(0)); } - + List scalaAcquisizioneLst = mapFields.get("Scala di acquisizione dei dati"); - if(scalaAcquisizioneLst!=null && scalaAcquisizioneLst.size()>0) { + if (scalaAcquisizioneLst != null && scalaAcquisizioneLst.size() > 0) { layerConcessione.setScalaAcquisizione(scalaAcquisizioneLst.get(0)); } - - //TODO + + // TODO List periodoDiEmbargo = mapFields.get("Periodo di embargo"); - if(periodoDiEmbargo!=null && periodoDiEmbargo.size()>0) { + if (periodoDiEmbargo != null && periodoDiEmbargo.size() > 0) { String dateFromTo = periodoDiEmbargo.get(0); String[] dates = dateFromTo.split(","); String dateFrom = dates[0]; String dateTo = dates[0]; - //TODO MUST BE INTENGRATED IN THE SERVICE + // TODO MUST BE INTENGRATED IN THE SERVICE } - - + List licenzaIdList = mapFields.get("ID Licenza"); - if(licenzaIdList!=null && licenzaIdList.size()>0) { + if (licenzaIdList != null && licenzaIdList.size() > 0) { layerConcessione.setLicenseID(licenzaIdList.get(0)); } - + List autoreLst = mapFields.get("Autore"); - if(autoreLst!=null) { + if (autoreLst != null) { layerConcessione.setAuthors(autoreLst); } - + return layerConcessione; } - /** * To local date time. * @@ -416,20 +408,19 @@ public class ConvertToServiceModel { try { date = date.trim(); SimpleDateFormat dateFormat = null; - if(date.contains(HOURS_MINUTES_SEPARATOR)) { - dateFormat = new SimpleDateFormat(DATE_FORMAT+" "+TIME_FORMAT); - }else + if (date.contains(HOURS_MINUTES_SEPARATOR)) { + dateFormat = new SimpleDateFormat(DATE_FORMAT + " " + TIME_FORMAT); + } else dateFormat = new SimpleDateFormat(DATE_FORMAT); - + Date theDate = dateFormat.parse(date); theLocalDT = convertToLocalDateTimeViaInstant(theDate); } catch (ParseException e) { - LOG.error("No able to parse: "+date, e); + LOG.error("No able to parse: " + date, e); } return theLocalDT; } - - + /** * Convert to local date time via instant. * @@ -437,9 +428,7 @@ public class ConvertToServiceModel { * @return the local date time */ public static LocalDateTime convertToLocalDateTimeViaInstant(Date dateToConvert) { - return dateToConvert.toInstant() - .atZone(ZoneId.systemDefault()) - .toLocalDateTime(); + return dateToConvert.toInstant().atZone(ZoneId.systemDefault()).toLocalDateTime(); } - + } diff --git a/src/main/java/org/gcube/portlets/user/geoportaldataentry/server/GeoportalDataEntryServiceImpl.java b/src/main/java/org/gcube/portlets/user/geoportaldataentry/server/GeoportalDataEntryServiceImpl.java index 82172e8..6938db8 100644 --- a/src/main/java/org/gcube/portlets/user/geoportaldataentry/server/GeoportalDataEntryServiceImpl.java +++ b/src/main/java/org/gcube/portlets/user/geoportaldataentry/server/GeoportalDataEntryServiceImpl.java @@ -7,6 +7,7 @@ import java.util.HashMap; import java.util.List; import java.util.Map; +import org.gcube.application.geoportal.common.model.legacy.AbstractRelazione; import org.gcube.application.geoportal.common.model.legacy.Concessione; import org.gcube.application.geoportal.common.model.legacy.Concessione.Paths; import org.gcube.application.geoportal.common.model.legacy.LayerConcessione; @@ -121,7 +122,9 @@ public class GeoportalDataEntryServiceImpl extends RemoteServiceServlet implemen GenericDatasetBean ards = listGDB.get(0); LOG.debug("\n\n"); LOG.debug(ConcessioniFormCardTitle.ABSTRACT_RELAZIONE_DI_SCAVO + " building with client obj: " + ards); - RelazioneScavo abstractRelazioneDiScavo = ConvertToServiceModel.toAbstractRelazioneScavo(ards); + AbstractRelazione abstractRelazioneDiScavo = ConvertToServiceModel.toAbstractRelazioneScavo(ards); + // Setting Abstract Relazione + concessioneNew.setAbstractRelazione(abstractRelazioneDiScavo); // RELAZIONE_DI_SCAVO list = toMap.get(ConcessioniFormCardTitle.RELAZIONE_DI_SCAVO); @@ -133,7 +136,7 @@ public class GeoportalDataEntryServiceImpl extends RemoteServiceServlet implemen GenericDatasetBean rds = listGDB.get(0); LOG.debug("\n\n"); LOG.debug(ConcessioniFormCardTitle.RELAZIONE_DI_SCAVO + " building with client obj: " + rds); - RelazioneScavo relazioneDiScavo = ConvertToServiceModel.toRelazioneScavo(abstractRelazioneDiScavo, rds); + RelazioneScavo relazioneDiScavo = ConvertToServiceModel.toRelazioneScavo(rds); LOG.debug("Built " + ConcessioniFormCardTitle.RELAZIONE_DI_SCAVO + " as server obj: " + relazioneDiScavo); // Setting Relazione @@ -224,11 +227,11 @@ public class GeoportalDataEntryServiceImpl extends RemoteServiceServlet implemen // Building TempFile for ABSTRACT DELLA RELAZIONE DI SCAVO SessionUtil.getCurrentContext(this.getThreadLocalRequest(), true); List ardsFiles = serviceUtil.toTemFiles(ards.getFilesUploaded()); - // TODO DEVE ESSERE PASSATO CON PATH RELAZIONE OPPURE BISOGNA PREVEDERE UN - // ABSTRACT_RELAZIONE??? + // Building TempFile for ABSTRACT RELAZIONE if (ardsFiles != null && ardsFiles.size() > 0) { // saving into back-end - AddSectionToConcessioneRequest request = new AddSectionToConcessioneRequest(Paths.RELAZIONE, ardsFiles); + AddSectionToConcessioneRequest request = new AddSectionToConcessioneRequest(Paths.ABSTRACT_RELAZIONE, + ardsFiles); clientMongo.registerFileSet(mongoId, request); LOG.info("Registered the mongoId " + mongoId + " request " + Paths.RELAZIONE + "with " + ardsFiles.size() + " file/s"); @@ -322,24 +325,6 @@ public class GeoportalDataEntryServiceImpl extends RemoteServiceServlet implemen cRep.setMongoId(concessione.getMongo_id()); ValidationReportDV vr = ConvertToDataViewModel.toValidationReport(report); cRep.setValidationReportDV(vr); -// switch (report.getStatus()) { -// case PASSED: -// if (cRep.getMongoId() == null) { -// LOG.error("The mongo id is null!!!"); -// throw new Exception("Invalid identifier (mongoId is null) for the concessione"); -// } -// cRep.setMsg("OK"); -// break; -// case WARNING: -// cRep.setMsg(report.getWarningMessages().toString()); -// break; -// case ERROR: -// cRep.setMsg(report.getErrorMessages().toString()); -// break; -// -// default: -// break; -// } return cRep; } else @@ -632,7 +617,7 @@ public class GeoportalDataEntryServiceImpl extends RemoteServiceServlet implemen * @param itemId the item id * @param jsonUpdate the json update * @param recordType the record type - * @return the updated JSON string representing the itemId + * @return the updated JSON string representing the itemId * @throws Exception the exception */ @Override @@ -649,9 +634,9 @@ public class GeoportalDataEntryServiceImpl extends RemoteServiceServlet implemen MongoServiceUtil serviceUtil = new MongoServiceUtil(); MongoConcessioni clientMongo = serviceUtil.getInstanceMongoConcessioni(); Concessione concessione = clientMongo.update(itemId, jsonUpdate); - LOG.debug("Got Record updated: "+concessione); + LOG.debug("Got Record updated: " + concessione); ConcessioneDV concessionDV = ConvertToDataViewModel.toMetadataConcessione(concessione, true); - LOG.info("Record with id "+concessionDV.getItemId()+"updated correclty"); + LOG.info("Record with id " + concessionDV.getItemId() + "updated correclty"); return concessionDV; } @@ -669,7 +654,7 @@ public class GeoportalDataEntryServiceImpl extends RemoteServiceServlet implemen * * @param itemId the item id * @param recordType the record type - * @return the JSON record representing the itemId + * @return the JSON record representing the itemId * @throws Exception the exception */ @Override