From cda6b9aba464099b022b479d1a4af392e659d824 Mon Sep 17 00:00:00 2001
From: Jannik Stehle <jannik.stehle@gmail.com>
Date: Wed, 27 Mar 2024 13:42:56 +0100
Subject: [PATCH] fix: capitalization of banned password error message

---
 .../src/components/Modals/SetLinkPasswordModal.vue          | 4 +++-
 packages/web-pkg/src/components/CreateLinkModal.vue         | 5 ++++-
 tests/e2e/cucumber/features/shares/link.feature             | 6 ++----
 3 files changed, 9 insertions(+), 6 deletions(-)

diff --git a/packages/web-app-files/src/components/Modals/SetLinkPasswordModal.vue b/packages/web-app-files/src/components/Modals/SetLinkPasswordModal.vue
index ef0cb7a0354..f931ed3f248 100644
--- a/packages/web-app-files/src/components/Modals/SetLinkPasswordModal.vue
+++ b/packages/web-app-files/src/components/Modals/SetLinkPasswordModal.vue
@@ -19,6 +19,7 @@
 <script lang="ts">
 import { defineComponent, ref, unref, PropType } from 'vue'
 import { useGettext } from 'vue3-gettext'
+import { upperFirst } from 'lodash-es'
 import {
   Modal,
   useClientService,
@@ -65,7 +66,8 @@ export default defineComponent({
       } catch (e) {
         // Human-readable error message is provided, for example when password is on banned list
         if (e.response?.status === 400) {
-          errorMessage.value = $gettext(e.response.data.error.message)
+          const errorMsg = e.response.data.error.message
+          errorMessage.value = $gettext(upperFirst(errorMsg))
           return Promise.reject()
         }
 
diff --git a/packages/web-pkg/src/components/CreateLinkModal.vue b/packages/web-pkg/src/components/CreateLinkModal.vue
index 624ad8f3156..77f6152fc6c 100644
--- a/packages/web-pkg/src/components/CreateLinkModal.vue
+++ b/packages/web-pkg/src/components/CreateLinkModal.vue
@@ -141,6 +141,7 @@
 <script lang="ts">
 import { DateTime } from 'luxon'
 import { v4 as uuidV4 } from 'uuid'
+import { upperFirst } from 'lodash-es'
 import { useGettext } from 'vue3-gettext'
 import { computed, defineComponent, PropType, ref, reactive, unref, onMounted } from 'vue'
 import {
@@ -280,7 +281,9 @@ export default defineComponent({
             console.error(e)
             // Human-readable error message is provided, for example when password is on banned list
             if (e.response?.status === 400) {
-              userFacingErrors.push(e.response.data.error)
+              const error = e.response.data.error
+              error.message = upperFirst(error.message)
+              userFacingErrors.push(error)
             }
           })
       }
diff --git a/tests/e2e/cucumber/features/shares/link.feature b/tests/e2e/cucumber/features/shares/link.feature
index 3c10f45e6d3..ae3bd605b35 100644
--- a/tests/e2e/cucumber/features/shares/link.feature
+++ b/tests/e2e/cucumber/features/shares/link.feature
@@ -234,17 +234,15 @@ Feature: link
       | resource  | password |
       | lorem.txt | %public% |
     When "Alice" tries to sets a new password "ownCloud-1" of the public link named "Link" of resource "lorem.txt"
-    # https://github.com/owncloud/ocis/issues/8624
     Then "Alice" should see an error message
       """
-      unfortunately, your password is commonly used. please pick a harder-to-guess password for your safety
+      Unfortunately, your password is commonly used. please pick a harder-to-guess password for your safety
       """
     And "Alice" closes the public link password dialog box
     When "Alice" tries to sets a new password "ownCloud-1" of the public link named "Link" of resource "lorem.txt"
-    # https://github.com/owncloud/ocis/issues/8624
     Then "Alice" should see an error message
       """
-      unfortunately, your password is commonly used. please pick a harder-to-guess password for your safety
+      Unfortunately, your password is commonly used. please pick a harder-to-guess password for your safety
       """
     And "Alice" reveals the password of the public link
     And "Alice" hides the password of the public link