diff --git a/src/main/java/edu/harvard/iq/dataverse/ControlledVocabularyValueConverter.java b/src/main/java/edu/harvard/iq/dataverse/ControlledVocabularyValueConverter.java index 6be05d1024c..1d530e136ba 100644 --- a/src/main/java/edu/harvard/iq/dataverse/ControlledVocabularyValueConverter.java +++ b/src/main/java/edu/harvard/iq/dataverse/ControlledVocabularyValueConverter.java @@ -6,6 +6,7 @@ package edu.harvard.iq.dataverse; import javax.ejb.EJB; +import javax.enterprise.inject.spi.CDI; import javax.faces.component.UIComponent; import javax.faces.context.FacesContext; @@ -19,8 +20,8 @@ @FacesConverter("controlledVocabularyValueConverter") public class ControlledVocabularyValueConverter implements Converter { - @EJB - DatasetFieldServiceBean datasetFieldService; + //@EJB + DatasetFieldServiceBean datasetFieldService = CDI.current().select(DatasetFieldServiceBean.class).get(); public Object getAsObject(FacesContext facesContext, UIComponent component, String submittedValue) { if (submittedValue == null || submittedValue.equals("")) { diff --git a/src/main/java/edu/harvard/iq/dataverse/DataFileConverter.java b/src/main/java/edu/harvard/iq/dataverse/DataFileConverter.java index feb4fa62651..18531f5203d 100644 --- a/src/main/java/edu/harvard/iq/dataverse/DataFileConverter.java +++ b/src/main/java/edu/harvard/iq/dataverse/DataFileConverter.java @@ -2,6 +2,7 @@ import java.util.logging.Logger; import javax.ejb.EJB; +import javax.enterprise.inject.spi.CDI; import javax.faces.component.UIComponent; import javax.faces.context.FacesContext; @@ -13,8 +14,8 @@ public class DataFileConverter implements Converter { private static final Logger logger = Logger.getLogger(DataFileConverter.class.getCanonicalName()); - @EJB - DataFileServiceBean dataFileService; + //@EJB + DataFileServiceBean dataFileService = CDI.current().select(DataFileServiceBean.class).get(); @Override public Object getAsObject(FacesContext facesContext, UIComponent component, String submittedValue) { diff --git a/src/main/java/edu/harvard/iq/dataverse/DatasetConverter.java b/src/main/java/edu/harvard/iq/dataverse/DatasetConverter.java index 71d1a48ace3..2d19cf5fe06 100644 --- a/src/main/java/edu/harvard/iq/dataverse/DatasetConverter.java +++ b/src/main/java/edu/harvard/iq/dataverse/DatasetConverter.java @@ -7,6 +7,7 @@ package edu.harvard.iq.dataverse; import javax.ejb.EJB; +import javax.enterprise.inject.spi.CDI; import javax.faces.component.UIComponent; import javax.faces.context.FacesContext; import javax.faces.convert.Converter; @@ -15,8 +16,8 @@ @FacesConverter("datasetConverter") public class DatasetConverter implements Converter { - @EJB - DatasetServiceBean datasetService; + //@EJB + DatasetServiceBean datasetService = CDI.current().select(DatasetServiceBean.class).get(); @Override public Object getAsObject(FacesContext facesContext, UIComponent component, String submittedValue) { diff --git a/src/main/java/edu/harvard/iq/dataverse/DatasetPage.java b/src/main/java/edu/harvard/iq/dataverse/DatasetPage.java index 508a561f149..80feb0b847f 100644 --- a/src/main/java/edu/harvard/iq/dataverse/DatasetPage.java +++ b/src/main/java/edu/harvard/iq/dataverse/DatasetPage.java @@ -242,6 +242,8 @@ public enum DisplayMode { @Inject DataverseHeaderFragment dataverseHeaderFragment; private Dataset dataset = new Dataset(); + + private Long id = null; private EditMode editMode; private boolean bulkFileDeleteInProgress = false; @@ -1446,6 +1448,9 @@ public void setDataset(Dataset dataset) { public DatasetVersion getWorkingVersion() { return workingVersion; } + + public Long getId() { return this.id; } + public void setId(Long id) { this.id = id; } public EditMode getEditMode() { return editMode; @@ -1799,9 +1804,9 @@ private String init(boolean initFull) { this.workingVersion = retrieveDatasetVersionResponse.getDatasetVersion(); logger.fine("retrieved version: id: " + workingVersion.getId() + ", state: " + this.workingVersion.getVersionState()); - } else if (dataset.getId() != null) { + } else if (this.getId() != null) { // Set Working Version and Dataset by Datasaet Id and Version - dataset = datasetService.find(dataset.getId()); + dataset = datasetService.find(this.getId()); if (dataset == null) { logger.warning("No such dataset: "+dataset); return permissionsWrapper.notFound(); diff --git a/src/main/java/edu/harvard/iq/dataverse/DatasetVersionConverter.java b/src/main/java/edu/harvard/iq/dataverse/DatasetVersionConverter.java index 79a801fbb55..98f0d707bdc 100644 --- a/src/main/java/edu/harvard/iq/dataverse/DatasetVersionConverter.java +++ b/src/main/java/edu/harvard/iq/dataverse/DatasetVersionConverter.java @@ -6,6 +6,7 @@ package edu.harvard.iq.dataverse; import javax.ejb.EJB; +import javax.enterprise.inject.spi.CDI; import javax.faces.component.UIComponent; import javax.faces.context.FacesContext; import javax.faces.convert.Converter; @@ -18,15 +19,15 @@ @FacesConverter("datasetVersionConverter") public class DatasetVersionConverter implements Converter { - @EJB - DatasetVersionServiceBean datasetVersionService; + //@EJB + DatasetVersionServiceBean datasetVersionService = CDI.current().select(DatasetVersionServiceBean.class).get(); @Override public Object getAsObject(FacesContext context, UIComponent component, String value) { if (value == null || value.equals("")) { return ""; - } else { + } else { return datasetVersionService.find(new Long(value)); } } diff --git a/src/main/java/edu/harvard/iq/dataverse/DataverseConverter.java b/src/main/java/edu/harvard/iq/dataverse/DataverseConverter.java index 685ec8132b3..7d09c300dde 100644 --- a/src/main/java/edu/harvard/iq/dataverse/DataverseConverter.java +++ b/src/main/java/edu/harvard/iq/dataverse/DataverseConverter.java @@ -7,6 +7,7 @@ package edu.harvard.iq.dataverse; import javax.ejb.EJB; +import javax.enterprise.inject.spi.CDI; import javax.faces.component.UIComponent; import javax.faces.context.FacesContext; import javax.faces.convert.Converter; @@ -20,8 +21,8 @@ public class DataverseConverter implements Converter { - @EJB - DataverseServiceBean dataverseService; + //@EJB + DataverseServiceBean dataverseService = CDI.current().select(DataverseServiceBean.class).get(); @Override public Object getAsObject(FacesContext facesContext, UIComponent component, String submittedValue) { diff --git a/src/main/java/edu/harvard/iq/dataverse/DataversePage.java b/src/main/java/edu/harvard/iq/dataverse/DataversePage.java index 6dc792aefed..75e3cd8af9c 100644 --- a/src/main/java/edu/harvard/iq/dataverse/DataversePage.java +++ b/src/main/java/edu/harvard/iq/dataverse/DataversePage.java @@ -110,11 +110,17 @@ public enum LinkMode { @Inject PermissionsWrapper permissionsWrapper; @Inject DataverseHeaderFragment dataverseHeaderFragment; - private Dataverse dataverse = new Dataverse(); + private Dataverse dataverse = new Dataverse(); + + /** + * View parameters + */ + private Long id = null; + private String alias = null; + private Long ownerId = null; private EditMode editMode; private LinkMode linkMode; - private Long ownerId; private DualListModel facets = new DualListModel<>(new ArrayList<>(), new ArrayList<>()); private DualListModel featuredDataverses = new DualListModel<>(new ArrayList<>(), new ArrayList<>()); private List dataversesForLinking; @@ -266,6 +272,12 @@ public Dataverse getDataverse() { public void setDataverse(Dataverse dataverse) { this.dataverse = dataverse; } + + public Long getId() { return this.id; } + public void setId(Long id) { this.id = id; } + + public String getAlias() { return this.alias; } + public void setAlias(String alias) { this.alias = alias; } public EditMode getEditMode() { return editMode; @@ -298,11 +310,11 @@ public void updateOwnerDataverse() { public String init() { //System.out.println("_YE_OLDE_QUERY_COUNTER_"); // for debug purposes - if (dataverse.getAlias() != null || dataverse.getId() != null || ownerId == null) {// view mode for a dataverse - if (dataverse.getAlias() != null) { - dataverse = dataverseService.findByAlias(dataverse.getAlias()); - } else if (dataverse.getId() != null) { - dataverse = dataverseService.find(dataverse.getId()); + if (this.getAlias() != null || this.getId() != null || this.getOwnerId() == null) {// view mode for a dataverse + if (this.getAlias() != null) { + dataverse = dataverseService.findByAlias(this.getAlias()); + } else if (this.getId() != null) { + dataverse = dataverseService.find(this.getId()); } else { try { dataverse = dataverseService.findRootDataverse(); @@ -323,7 +335,7 @@ public String init() { ownerId = dataverse.getOwner() != null ? dataverse.getOwner().getId() : null; } else { // ownerId != null; create mode for a new child dataverse editMode = EditMode.CREATE; - dataverse.setOwner(dataverseService.find(ownerId)); + dataverse.setOwner(dataverseService.find( this.getOwnerId())); if (dataverse.getOwner() == null) { return permissionsWrapper.notFound(); } else if (!permissionService.on(dataverse.getOwner()).has(Permission.AddDataverse)) { diff --git a/src/main/java/edu/harvard/iq/dataverse/FacetConverter.java b/src/main/java/edu/harvard/iq/dataverse/FacetConverter.java index 478bc2dee53..75ef62200bf 100644 --- a/src/main/java/edu/harvard/iq/dataverse/FacetConverter.java +++ b/src/main/java/edu/harvard/iq/dataverse/FacetConverter.java @@ -6,6 +6,7 @@ package edu.harvard.iq.dataverse; import javax.ejb.EJB; +import javax.enterprise.inject.spi.CDI; import javax.faces.component.UIComponent; import javax.faces.context.FacesContext; @@ -19,8 +20,8 @@ @FacesConverter("facetConverter") public class FacetConverter implements Converter { - @EJB - DatasetFieldServiceBean datasetFieldService; + //@EJB + DatasetFieldServiceBean datasetFieldService = CDI.current().select(DatasetFieldServiceBean.class).get(); public Object getAsObject(FacesContext facesContext, UIComponent component, String submittedValue) { return datasetFieldService.find(new Long(submittedValue)); diff --git a/src/main/java/edu/harvard/iq/dataverse/MetadataBlockConverter.java b/src/main/java/edu/harvard/iq/dataverse/MetadataBlockConverter.java index 92cbef6dda7..c5bd48ae785 100644 --- a/src/main/java/edu/harvard/iq/dataverse/MetadataBlockConverter.java +++ b/src/main/java/edu/harvard/iq/dataverse/MetadataBlockConverter.java @@ -6,6 +6,7 @@ package edu.harvard.iq.dataverse; import javax.ejb.EJB; +import javax.enterprise.inject.spi.CDI; import javax.faces.component.UIComponent; import javax.faces.context.FacesContext; @@ -19,8 +20,8 @@ @FacesConverter("metadataBlockConverter") public class MetadataBlockConverter implements Converter { - @EJB - DataverseServiceBean dataverseService; + //@EJB + DataverseServiceBean dataverseService = CDI.current().select(DataverseServiceBean.class).get(); public Object getAsObject(FacesContext facesContext, UIComponent component, String submittedValue) { MetadataBlock mdb = dataverseService.findMDB(new Long(submittedValue)); diff --git a/src/main/java/edu/harvard/iq/dataverse/RoleAssigneeConverter.java b/src/main/java/edu/harvard/iq/dataverse/RoleAssigneeConverter.java index b6b4c43fa1f..a5e4cebbd95 100644 --- a/src/main/java/edu/harvard/iq/dataverse/RoleAssigneeConverter.java +++ b/src/main/java/edu/harvard/iq/dataverse/RoleAssigneeConverter.java @@ -7,9 +7,8 @@ package edu.harvard.iq.dataverse; import edu.harvard.iq.dataverse.authorization.RoleAssignee; -import java.util.logging.Level; -import java.util.logging.Logger; import javax.ejb.EJB; +import javax.enterprise.inject.spi.CDI; import javax.faces.component.UIComponent; import javax.faces.context.FacesContext; @@ -23,8 +22,8 @@ @FacesConverter("roleAssigneeConverter") public class RoleAssigneeConverter implements Converter { - @EJB - RoleAssigneeServiceBean roleAssigneeService; + //@EJB + RoleAssigneeServiceBean roleAssigneeService = CDI.current().select(RoleAssigneeServiceBean.class).get(); public Object getAsObject(FacesContext facesContext, UIComponent component, String submittedValue) { RoleAssignee mdb = roleAssigneeService.getRoleAssignee(submittedValue); diff --git a/src/main/java/edu/harvard/iq/dataverse/TemplateConverter.java b/src/main/java/edu/harvard/iq/dataverse/TemplateConverter.java index 505e6c94789..98b24f84801 100644 --- a/src/main/java/edu/harvard/iq/dataverse/TemplateConverter.java +++ b/src/main/java/edu/harvard/iq/dataverse/TemplateConverter.java @@ -7,6 +7,7 @@ package edu.harvard.iq.dataverse; import javax.ejb.EJB; +import javax.enterprise.inject.spi.CDI; import javax.faces.component.UIComponent; import javax.faces.context.FacesContext; import javax.faces.convert.Converter; @@ -19,8 +20,8 @@ @FacesConverter("templateConverter") public class TemplateConverter implements Converter { - @EJB - TemplateServiceBean templateService; + //@EJB + TemplateServiceBean templateService = CDI.current().select(TemplateServiceBean.class).get(); @Override public Object getAsObject(FacesContext facesContext, UIComponent component, String submittedValue) { diff --git a/src/main/java/edu/harvard/iq/dataverse/provenance/ProvEntityFileDataConverter.java b/src/main/java/edu/harvard/iq/dataverse/provenance/ProvEntityFileDataConverter.java index cd3fb42ab84..65b97b80eea 100644 --- a/src/main/java/edu/harvard/iq/dataverse/provenance/ProvEntityFileDataConverter.java +++ b/src/main/java/edu/harvard/iq/dataverse/provenance/ProvEntityFileDataConverter.java @@ -1,5 +1,6 @@ package edu.harvard.iq.dataverse.provenance; +import javax.enterprise.inject.spi.CDI; import javax.faces.component.UIComponent; import javax.faces.context.FacesContext; import javax.faces.convert.Converter; @@ -14,8 +15,8 @@ @FacesConverter("provEntityFileDataConverter") public class ProvEntityFileDataConverter implements Converter{ - @Inject - ProvPopupFragmentBean provBean; + //@Inject + ProvPopupFragmentBean provBean = CDI.current().select(ProvPopupFragmentBean.class).get(); @Override public Object getAsObject(FacesContext context, UIComponent component, String value) { diff --git a/src/main/webapp/dataset.xhtml b/src/main/webapp/dataset.xhtml index 999efc38161..db7620bab71 100644 --- a/src/main/webapp/dataset.xhtml +++ b/src/main/webapp/dataset.xhtml @@ -9,8 +9,7 @@ xmlns:jsf="http://xmlns.jcp.org/jsf" xmlns:pt="http://java.sun.com/jsf/passthrough" xmlns:cc="http://java.sun.com/jsf/composite" - xmlns:o="http://omnifaces.org/ui" - xmlns:iqbs="http://xmlns.jcp.org/jsf/composite/iqbs"> + xmlns:o="http://omnifaces.org/ui"> @@ -58,8 +57,8 @@ - - + + @@ -631,6 +630,7 @@