diff --git a/doc/release-notes/12218-allow-perm-mgmt-without-edit.md b/doc/release-notes/12218-allow-perm-mgmt-without-edit.md new file mode 100644 index 00000000000..838a30d0dc8 --- /dev/null +++ b/doc/release-notes/12218-allow-perm-mgmt-without-edit.md @@ -0,0 +1 @@ +Users without permission to edit a dataset or dataverse were not able to access the permission and group management pages via the dataset/dataverse page menus even if they had the relevant ManagePermissions permission. See #3981 and #12218 for details. \ No newline at end of file diff --git a/src/main/webapp/dataset.xhtml b/src/main/webapp/dataset.xhtml index e5caca9cb04..af2404a0264 100644 --- a/src/main/webapp/dataset.xhtml +++ b/src/main/webapp/dataset.xhtml @@ -15,21 +15,24 @@ - - - - + + + + - - - + + + + + @@ -53,13 +56,13 @@ - - + @@ -84,13 +88,13 @@ - + - + @@ -113,13 +117,13 @@ - + - + @@ -147,8 +151,8 @@ - - + + @@ -169,21 +173,21 @@

#{DatasetPage.displayTitle}

- - - - - + + + + + - + - + - - + +
@@ -315,17 +319,17 @@
  • -
  • +
  • - +
    • -
  • +
  • - +
    • @@ -379,26 +383,26 @@
      -
    • +
    • - #{bundle['dataset.publish.btn']}
      • -
    • +
    • - #{latestVersionInReview ? bundle['dataset.disabledSubmittedBtn'] : bundle['dataset.submitBtn']}
      • -
    • +
    • - #{bundle['dataset.rejectBtn']} @@ -410,112 +414,171 @@
      + and (canUpdateDataset or canManageDatasetPermissions or canManageFilesOnDataset)}">
      -
      +
      - + #{bundle['dataset.contactBtn']}
      - + @@ -633,7 +696,7 @@
      - + @@ -641,13 +704,13 @@
      - + - + - + ) @@ -657,7 +720,7 @@
      - +
      #{bundle['dataset.deaccession.reason']}
      -

      #{version.deaccessionNote}

      - +

      #{workingVersion.deaccessionNote}

      +

      #{bundle['dataset.beAccessedAt']} - #{version.deaccessionLink} - + #{workingVersion.deaccessionLink} +

      @@ -691,26 +754,26 @@
      - - + + - + - + @@ -790,7 +853,7 @@
      -

      #{DatasetPage.dataset.owner.name}

      +

      #{dataverse.name}

       @@ -861,11 +924,11 @@ - + - - - + + + @@ -891,7 +954,7 @@
      - + @@ -959,9 +1022,9 @@ - - - + + + @@ -981,16 +1044,16 @@ and (empty DatasetPage.editMode or DatasetPage.editMode == 'METADATA')}">
      + and canUpdateDataset)}"> #{bundle['file.dataFilesTab.metadata.addBtn']} -
      - + - - - - - + + + + + @@ -1026,7 +1089,7 @@ rendered="#{!deaccessioned and (empty DatasetPage.editMode or DatasetPage.editMode == 'LICENSE')}"> - + @@ -1035,7 +1098,7 @@ - + @@ -1067,7 +1130,7 @@ - + @@ -1107,12 +1170,12 @@

      -
      +
      #{bundle['metrics.citations.dialog.empty']}
      - + @@ -1126,7 +1189,7 @@

      #{bundle['dataset.noValidSelectedFilesForDownload']}

      - +

      #{DatasetPage.cantDownloadDueToEmbargoOrDVAccess ? bundle['dataset.requestAccessToRestrictedFilesWithEmbargo'] : bundle['dataset.requestAccessToRestrictedFiles']}
      @@ -1137,7 +1200,7 @@

      #{bundle['dataset.noValidSelectedFilesForTransfer']}

      - +

      #{DatasetPage.cantDownloadDueToEmbargoOrDVAccess ? bundle['dataset.requestAccessToRestrictedFilesWithEmbargo'] : bundle['dataset.requestAccessToRestrictedFiles']}
      @@ -1294,8 +1357,8 @@ action="#{DatasetPage.createPrivateUrl(true)}" update="privateUrlPanel,:messagePanel" rendered="#{anonymizedAccessEnabled and (empty(DatasetPage.privateUrl) or (!empty(DatasetPage.privateUrl) and !DatasetPage.anonymizedPrivateUrl))}" - disabled="#{DatasetPage.dataset.released or ((!empty(DatasetPage.privateUrl) and !DatasetPage.anonymizedPrivateUrl))}" - title="#{DatasetPage.dataset.released ? bundle['dataset.privateurl.createPrivateUrl.anonymized.unavailable']:'' }"/> + disabled="#{dataset.released or ((!empty(DatasetPage.privateUrl) and !DatasetPage.anonymizedPrivateUrl))}" + title="#{dataset.released ? bundle['dataset.privateurl.createPrivateUrl.anonymized.unavailable']:'' }"/>

      @@ -1365,7 +1428,7 @@

      #{bundle['file.deleteFileDialog.multiple.immediate']}

      - +

      #{bundle['file.deleteFileDialog.failed.tip']}
      @@ -1477,24 +1540,24 @@
      -
      #{DatasetPage.datasetVersionDifference.newVersion.title}
      +
      #{datasetVersionDifference.newVersion.title}
       
      - #{bundle['file.viewDiffDialog.version']}: #{DatasetPage.datasetVersionDifference.originalVersion.semanticVersion}
      - #{bundle['file.viewDiffDialog.lastUpdated']}: #{DatasetPage.datasetVersionDifference.originalVersion.localeLastUpdateTime}
      - #{bundle['file.viewDiffDialog.versionNote']}: #{DatasetPage.datasetVersionDifference.originalVersion.versionNote} + #{bundle['file.viewDiffDialog.version']}: #{datasetVersionDifference.originalVersion.semanticVersion}
      + #{bundle['file.viewDiffDialog.lastUpdated']}: #{datasetVersionDifference.originalVersion.localeLastUpdateTime}
      + #{bundle['file.viewDiffDialog.versionNote']}: #{datasetVersionDifference.originalVersion.versionNote}
      - #{bundle['file.viewDiffDialog.version']}: #{DatasetPage.datasetVersionDifference.newVersion.semanticVersion}
      - #{bundle['file.viewDiffDialog.lastUpdated']}: #{DatasetPage.datasetVersionDifference.newVersion.localeLastUpdateTime}
      - #{bundle['file.viewDiffDialog.versionNote']}: #{DatasetPage.datasetVersionDifference.newVersion.versionNote} + #{bundle['file.viewDiffDialog.version']}: #{datasetVersionDifference.newVersion.semanticVersion}
      + #{bundle['file.viewDiffDialog.lastUpdated']}: #{datasetVersionDifference.newVersion.localeLastUpdateTime}
      + #{bundle['file.viewDiffDialog.versionNote']}: #{datasetVersionDifference.newVersion.versionNote}
      - - + +
      @@ -1515,12 +1578,12 @@
      -
      -
      +
      +
      - +
      @@ -1552,8 +1615,8 @@   - + @@ -1587,11 +1650,11 @@
      -
      +
      - + @@ -1737,7 +1800,7 @@ - + @@ -1759,11 +1822,11 @@ + and !workingVersion.getDataset().getStorageIdentifier().startsWith('s3://')}"> - +
      - +

      #{bundle['dataset.selectVersionNumber']}

      @@ -2025,7 +2088,7 @@

      - +

      @@ -2074,8 +2137,8 @@ - - + + @@ -2087,18 +2150,18 @@
      - +

      - - + + - - + + @@ -2142,7 +2205,7 @@ - @@ -2169,7 +2232,7 @@ -

      @@ -2189,7 +2252,7 @@ type="text/javascript"> diff --git a/src/main/webapp/dataverse.xhtml b/src/main/webapp/dataverse.xhtml index f88ba1fd49a..ec903928820 100644 --- a/src/main/webapp/dataverse.xhtml +++ b/src/main/webapp/dataverse.xhtml @@ -13,11 +13,12 @@ - + + - + @@ -27,7 +28,7 @@ - + @@ -71,7 +72,7 @@
      -

      #{DataversePage.dataverse.getOwner().name}

      +

      #{dataverse.getOwner().name}
      @@ -126,7 +127,7 @@ data-toggle="tooltip" data-placement="auto right" data-original-title="#{bundle['dataverse.affiliation.title']}">
      - +
      @@ -143,7 +144,7 @@
      #{settingsWrapper.dataverseSiteUrl}/dataverse/
      @@ -156,7 +157,7 @@ data-toggle="tooltip" data-placement="auto right" data-original-title="#{bundle['dataverse.storage.title']}">
      - + @@ -169,7 +170,7 @@ data-toggle="tooltip" data-placement="auto right" data-original-title="#{bundle['dataverse.curationLabels.title']}">
      - + @@ -186,7 +187,7 @@ data-toggle="tooltip" data-placement="auto right" data-original-title="#{bundle['dataverse.category.title']}">
      - @@ -211,7 +212,7 @@ data-toggle="tooltip" data-placement="auto right" data-original-title="#{bundle['dataverse.email.title']}">
      - +
      @@ -223,13 +224,13 @@
      + actionListener="#{dataverse.addDataverseContact(valCount.index + 1)}"> + rendered="#{dataverse.dataverseContacts.size() > 1}" + actionListener="#{dataverse.removeDataverseContact(valCount.index)}"> @@ -246,7 +247,7 @@ data-toggle="tooltip" data-placement="auto right" data-original-title="#{bundle['dataverse.metadatalanguage.title']}">
      - + @@ -266,7 +267,7 @@

      - @@ -280,7 +281,7 @@ data-toggle="tooltip" data-placement="auto right" data-original-title="#{bundle['dataverse.guestbookentryatrequest.title']}">
      - + @@ -293,7 +294,7 @@ data-toggle="tooltip" data-placement="auto right" data-original-title="#{bundle['dataverse.pidProvider.title']}">
      - + @@ -329,12 +330,12 @@
      #{bundle['dataverse.metadataElements']}

      #{bundle['dataverse.metadataElements.tip']}

      - + - + #{bundle['dataverse.facetPickList.text']}

      #{bundle['dataverse.facetPickList.tip']}

      - + - + @@ -458,7 +459,7 @@ #{bundle['dataverse.selected']} - +
      @@ -466,11 +467,11 @@
      - + - - + +
      @@ -505,7 +506,7 @@ - + #{bundle['contact.contact']}
      + (permissionsWrapper.canIssueUpdateDataverseCommand(dataverse) + or permissionsWrapper.canIssuePublishDataverseCommand(dataverse))}"> - + @@ -537,7 +538,7 @@ #{bundle['link']}
        -
      • +
      • -
        +
        @@ -568,59 +571,76 @@
        -
        -
        #{DataversePage.dataverse.alias}
        +
        +
        #{dataverse.alias}
        • -
      • - - - -
        • -
      • - - - - -
        • - + +
      • + + + +
        • +
      • + + + + +
        • +         +
      • - +
        • -         -
      • - +
        • -
      • - - - - -
        • -
      • - - - - -
        • -
      • - - - -
        • - + +
      • + + + + +
        • +
      • + + + + +
        • +
      • + + + +
        • +         +
      • @@ -639,9 +659,9 @@
      -
      +
      - +
      @@ -746,7 +766,7 @@

      #{bundle['dataverse.share.dataverseShare.tip']}

      -
      +
      #{bundle['dataset.keywordDisplay.title']} + data-toggle="tooltip" data-placement="auto right" data-original-title="#{datasetVersionUI.keyword.datasetFieldType.localeDescription}"> #{DatasetPage.datasetVersionUI.keywordDisplay}#{datasetVersionUI.keywordDisplay}
      - #{DatasetPage.datasetVersionUI.datasetRelPublications.get(0).title} + #{datasetVersionUI.datasetRelPublications.get(0).title} + data-toggle="tooltip" data-placement="auto right" data-original-title="#{datasetVersionUI.datasetRelPublications.get(0).description}"> - + - #{empty DatasetPage.datasetVersionUI.relPublicationId ? DatasetPage.datasetVersionUI.relPublicationUrl:DatasetPage.datasetVersionUI.relPublicationId} - + #{empty datasetVersionUI.relPublicationId ? datasetVersionUI.relPublicationUrl:datasetVersionUI.relPublicationId} + @@ -762,8 +825,8 @@

      - - #{DatasetUtil:getLicenseName(version)} + + #{DatasetUtil:getLicenseName(workingVersion)}