From e64549fd2b6d8f4d0760e1c842cbc21ab4286143 Mon Sep 17 00:00:00 2001 From: "francesco.mangiacrapa" Date: Fri, 24 Sep 2021 11:46:06 +0200 Subject: [PATCH] #22040 Revisited the "Abstract Relazione di Scavo" --- CHANGELOG.md | 1 + .../ConvertToDataViewModel.java | 87 +++++++++++-------- .../model/AbstractRelazioneScavoDV.java | 67 ++++---------- 3 files changed, 71 insertions(+), 84 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 573b7dc..075cc6e 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -12,6 +12,7 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm [#20595] Porting common model [#21890] Passed to mongoID [#22002] Integrated with ValidationReport bean +[#22040] Revisited the "Abstract Relazione di Scavo" Passed to gcube-bom 2.0.1 diff --git a/src/main/java/org/gcube/application/geoportalcommon/ConvertToDataViewModel.java b/src/main/java/org/gcube/application/geoportalcommon/ConvertToDataViewModel.java index 285dada..eaa90c7 100644 --- a/src/main/java/org/gcube/application/geoportalcommon/ConvertToDataViewModel.java +++ b/src/main/java/org/gcube/application/geoportalcommon/ConvertToDataViewModel.java @@ -6,6 +6,7 @@ import java.time.format.DateTimeFormatter; import java.util.ArrayList; import java.util.List; +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.BBOX; import org.gcube.application.geoportal.common.model.legacy.Concessione; @@ -100,7 +101,7 @@ public class ConvertToDataViewModel { theConcessione.setRecordType(concessione.getRecordType().name()); - theConcessione.setAbstractRelazioneScavo(toAbstractRelazioneScavo(concessione.getRelazioneScavo())); + theConcessione.setAbstractRelazioneScavo(toAbstractRelazioneScavo(concessione.getAbstractRelazione())); theConcessione.setRelazioneScavo(toRelazioneScavo(concessione.getRelazioneScavo())); if (concessione.getImmaginiRappresentative() != null) { @@ -151,12 +152,13 @@ public class ConvertToDataViewModel { /** * To metadata concessione. * - * @param concessione the concessione + * @param concessione the concessione * @param includeValidationReport the include validation report * @return the concessione DV * @throws Exception the exception */ - public static ConcessioneDV toMetadataConcessione(Concessione concessione, boolean includeValidationReport) throws Exception { + public static ConcessioneDV toMetadataConcessione(Concessione concessione, boolean includeValidationReport) + throws Exception { LOG.debug("called toConcessione for: " + concessione); if (concessione == null) @@ -192,10 +194,10 @@ public class ConvertToDataViewModel { theConcessione.setTitolariCopyright(concessione.getTitolareCopyright()); theConcessione.setTitolariLicenza(concessione.getTitolareLicenza()); theConcessione.setTitolari(concessione.getTitolari()); - - if(includeValidationReport) { + + if (includeValidationReport) { theConcessione.setValidationReport(toValidationReport(concessione.getReport())); - if(theConcessione.getValidationReport()!=null) + if (theConcessione.getValidationReport() != null) theConcessione.setValidationStatus(theConcessione.getValidationReport().getStatus()); } @@ -208,6 +210,13 @@ public class ConvertToDataViewModel { } + /** + * To validation report. + * + * @param validationReport the validation report + * @return the validation report DV + * @throws Exception the exception + */ public static ValidationReportDV toValidationReport(ValidationReport validationReport) throws Exception { LOG.debug("called toValidationReport for: " + validationReport); @@ -341,29 +350,38 @@ public class ConvertToDataViewModel { /** * To abstract relazione scavo. * - * @param relazioneScavo the relazione scavo + * @param abstractRelazione the abstract relazione * @return the abstract relazione scavo DV */ - public static AbstractRelazioneScavoDV toAbstractRelazioneScavo(RelazioneScavo relazioneScavo) { + public static AbstractRelazioneScavoDV toAbstractRelazioneScavo(AbstractRelazione abstractRelazione) { - if (relazioneScavo == null) + if (abstractRelazione == null) return null; - AbstractRelazioneScavoDV theRelazioneDiScavo = new AbstractRelazioneScavoDV(); - theRelazioneDiScavo.setAbstractSectionIta(relazioneScavo.getAbstractIta()); - theRelazioneDiScavo.setAbstractSectionEng(relazioneScavo.getAbstractEng()); + AbstractRelazioneScavoDV theARelDiScavo = new AbstractRelazioneScavoDV(); + theARelDiScavo.setAbstractSectionIta(abstractRelazione.getAbstractIta()); + theARelDiScavo.setAbstractSectionEng(abstractRelazione.getAbstractEng()); - theRelazioneDiScavo.setCreationTime(DateUtils.asDate(relazioneScavo.getCreationTime())); - theRelazioneDiScavo.setId(relazioneScavo.getId()); - theRelazioneDiScavo.setLicenseID(relazioneScavo.getLicenseID()); - theRelazioneDiScavo.setPolicy(toPolicy(relazioneScavo.getPolicy())); - // theRelazioneDiScavo.setRecord(recordDV); - theRelazioneDiScavo.setResponsabili(relazioneScavo.getResponsabili()); - theRelazioneDiScavo.setSoggetto(relazioneScavo.getSoggetto()); - theRelazioneDiScavo.setTitolo(relazioneScavo.getTitolo()); + theARelDiScavo.setCreationTime(DateUtils.asDate(abstractRelazione.getCreationTime())); + theARelDiScavo.setId(abstractRelazione.getId()); + theARelDiScavo.setLicenseID(abstractRelazione.getLicenseID()); + theARelDiScavo.setPolicy(toPolicy(abstractRelazione.getPolicy())); + theARelDiScavo.setTitolo(abstractRelazione.getTitolo()); - LOG.debug("Returning: " + theRelazioneDiScavo); - return theRelazioneDiScavo; + List actContent = abstractRelazione.getActualContent(); + if (actContent != null && actContent.size() > 0) { + List listWsContent = new ArrayList(); + + for (PersistedContent content : actContent) { + if (content != null && content instanceof WorkspaceContent) { + listWsContent.add(toWorkspaceContent((WorkspaceContent) content)); + } + } + theARelDiScavo.setListWsContent(listWsContent); + } + + LOG.debug("Returning: " + theARelDiScavo); + return theARelDiScavo; } /** @@ -377,15 +395,15 @@ public class ConvertToDataViewModel { if (relazioneScavo == null) return null; - RelazioneScavoDV theRelazioneDiScavo = new RelazioneScavoDV(); - theRelazioneDiScavo.setCreationTime(DateUtils.asDate(relazioneScavo.getCreationTime())); - theRelazioneDiScavo.setId(relazioneScavo.getId()); - theRelazioneDiScavo.setLicenseID(relazioneScavo.getLicenseID()); - theRelazioneDiScavo.setPolicy(toPolicy(relazioneScavo.getPolicy())); + RelazioneScavoDV theRelDiScavo = new RelazioneScavoDV(); + theRelDiScavo.setCreationTime(DateUtils.asDate(relazioneScavo.getCreationTime())); + theRelDiScavo.setId(relazioneScavo.getId()); + theRelDiScavo.setLicenseID(relazioneScavo.getLicenseID()); + theRelDiScavo.setPolicy(toPolicy(relazioneScavo.getPolicy())); // theRelazioneDiScavo.setRecord(recordDV); - theRelazioneDiScavo.setResponsabili(relazioneScavo.getResponsabili()); - theRelazioneDiScavo.setSoggetto(relazioneScavo.getSoggetto()); - theRelazioneDiScavo.setTitolo(relazioneScavo.getTitolo()); + theRelDiScavo.setResponsabili(relazioneScavo.getResponsabili()); + theRelDiScavo.setSoggetto(relazioneScavo.getSoggetto()); + theRelDiScavo.setTitolo(relazioneScavo.getTitolo()); List actContent = relazioneScavo.getActualContent(); if (actContent != null && actContent.size() > 0) { @@ -396,11 +414,11 @@ public class ConvertToDataViewModel { listWsContent.add(toWorkspaceContent((WorkspaceContent) content)); } } - theRelazioneDiScavo.setListWsContent(listWsContent); + theRelDiScavo.setListWsContent(listWsContent); } - LOG.debug("Returning: " + theRelazioneDiScavo); - return theRelazioneDiScavo; + LOG.debug("Returning: " + theRelDiScavo); + return theRelDiScavo; } /** @@ -616,8 +634,7 @@ public class ConvertToDataViewModel { /** * To JSON. * - * @param the generic type - * @param report the report + * @param theObj the the obj * @return the string */ public static String toJSON(Object theObj) { diff --git a/src/main/java/org/gcube/application/geoportalcommon/shared/products/model/AbstractRelazioneScavoDV.java b/src/main/java/org/gcube/application/geoportalcommon/shared/products/model/AbstractRelazioneScavoDV.java index 2877896..a1b7695 100644 --- a/src/main/java/org/gcube/application/geoportalcommon/shared/products/model/AbstractRelazioneScavoDV.java +++ b/src/main/java/org/gcube/application/geoportalcommon/shared/products/model/AbstractRelazioneScavoDV.java @@ -4,18 +4,17 @@ import java.io.Serializable; import java.util.List; import org.gcube.application.geoportalcommon.shared.products.content.AssociatedContentDV; - +import org.gcube.application.geoportalcommon.shared.products.content.WorkspaceContentDV; /** * The Class AbstractRelazioneScavoDV. * - * @author Francesco Mangiacrapa at ISTI-CNR (francesco.mangiacrapa@isti.cnr.it) + * @author Francesco Mangiacrapa at ISTI-CNR francesco.mangiacrapa@isti.cnr.it * - * Dec 21, 2020 + * Sep 24, 2021 */ public class AbstractRelazioneScavoDV extends AssociatedContentDV implements Serializable { - /** * */ @@ -23,14 +22,10 @@ public class AbstractRelazioneScavoDV extends AssociatedContentDV implements Ser /** The abstract section. */ private String abstractSectionIta; - + private String abstractSectionEng; - /** The responsabili. */ - private List responsabili; - - /** The soggetto. */ - private List soggetto; + private List listWsContent; /** * Instantiates a new abstract relazione scavo. @@ -40,19 +35,15 @@ public class AbstractRelazioneScavoDV extends AssociatedContentDV implements Ser } /** - * Instantiates a new abstract relazione scavo. + * Instantiates a new abstract relazione scavo DV. * * @param abstractSectionIta the abstract section ita * @param abstractSectionEng the abstract section eng - * @param responsabili the responsabili - * @param soggetto the soggetto */ - public AbstractRelazioneScavoDV(String abstractSectionIta, String abstractSectionEng, List responsabili, List soggetto) { + public AbstractRelazioneScavoDV(String abstractSectionIta, String abstractSectionEng) { super(); this.abstractSectionIta = abstractSectionIta; this.abstractSectionEng = abstractSectionEng; - this.responsabili = responsabili; - this.soggetto = soggetto; } /** @@ -92,39 +83,21 @@ public class AbstractRelazioneScavoDV extends AssociatedContentDV implements Ser } /** - * Gets the responsabili. + * Gets the list ws content. * - * @return the responsabili + * @return the list ws content */ - public List getResponsabili() { - return responsabili; + public List getListWsContent() { + return listWsContent; } /** - * Sets the responsabili. + * Sets the list ws content. * - * @param responsabili the new responsabili + * @param listWsContent the new list ws content */ - public void setResponsabili(List responsabili) { - this.responsabili = responsabili; - } - - /** - * Gets the soggetto. - * - * @return the soggetto - */ - public List getSoggetto() { - return soggetto; - } - - /** - * Sets the soggetto. - * - * @param soggetto the new soggetto - */ - public void setSoggetto(List soggetto) { - this.soggetto = soggetto; + public void setListWsContent(List listWsContent) { + this.listWsContent = listWsContent; } /** @@ -135,18 +108,14 @@ public class AbstractRelazioneScavoDV extends AssociatedContentDV implements Ser @Override public String toString() { StringBuilder builder = new StringBuilder(); - builder.append("RelazioneScavoDV [abstractSectionIta="); + builder.append("AbstractRelazioneScavoDV [abstractSectionIta="); builder.append(abstractSectionIta); builder.append(", abstractSectionEng="); builder.append(abstractSectionEng); - builder.append(", responsabili="); - builder.append(responsabili); - builder.append(", soggetto="); - builder.append(soggetto); + builder.append(", listWsContent="); + builder.append(listWsContent); builder.append("]"); return builder.toString(); } - - }