comments addressed

CeON · Feb 4, 2025 · 3319c1a · 3319c1a
1 parent 19a25f1
commit 3319c1a
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 6 deletions.
diff --git a/dataverse-webapp/src/main/java/edu/harvard/iq/dataverse/PermissionsWrapper.java b/dataverse-webapp/src/main/java/edu/harvard/iq/dataverse/PermissionsWrapper.java
@@ -120,9 +120,7 @@ public boolean canEditDataverseTextMessagesAndBanners(Long dataverseId) {
     }
 
     public String authorizedToEditTextMessagesAndBannersOf(final Long dataverseId) {
-        final User user = this.dvRequestService.getDataverseRequest().getUser();
-        return this.permissionService
-                .isUserCanEditDataverseTextMessagesAndBanners(user, dataverseId)
+        return canEditDataverseTextMessagesAndBanners(dataverseId)
                         ? EMPTY
                         : notAuthorized();
     }

diff --git a/...bapp/src/main/java/edu/harvard/iq/dataverse/bannersandmessages/banners/NewBannerPage.java b/...bapp/src/main/java/edu/harvard/iq/dataverse/bannersandmessages/banners/NewBannerPage.java
@@ -173,7 +173,7 @@ private boolean isDataCorrect() {
                 result = false;
             }
         }
-        ;
+
         if (!this.banner.isFromTimePresent()) {
             this.uiMessages.addComponentErrorMessage(
                     "edit-text-messages-form:message-fromtime",

diff --git a/...p/src/main/java/edu/harvard/iq/dataverse/bannersandmessages/banners/dto/BannerMapper.java b/...p/src/main/java/edu/harvard/iq/dataverse/bannersandmessages/banners/dto/BannerMapper.java
@@ -116,8 +116,6 @@ private DataverseLocalizedBanner mapToLocalizedBanner(DataverseBanner banner, Da
     private List<DataverseLocalizedBannerDto> mapDefaultLocales() {
 
         Map<String, String> locales = settingsWrapper.getConfiguredLocales();
-
-        System.out.println("!=====: mapDefaultLocales: " + locales);
 
         return locales.entrySet().stream()
                 .map(e -> new DataverseLocalizedBannerDto(e.getKey()))