From c0f06234fa3b898c67c532661f7708ad7ab0c967 Mon Sep 17 00:00:00 2001
From: Jan <j.ackermann91@gmail.com>
Date: Tue, 1 Aug 2023 14:37:51 +0200
Subject: [PATCH 1/4] Fix set or remove expiration date on groups not possible

---
 .../Shares/Collaborators/EditDropdown.vue       | 17 +++++++++--------
 .../InviteCollaborator/ExpirationDatepicker.vue |  4 +---
 2 files changed, 10 insertions(+), 11 deletions(-)

diff --git a/packages/web-app-files/src/components/SideBar/Shares/Collaborators/EditDropdown.vue b/packages/web-app-files/src/components/SideBar/Shares/Collaborators/EditDropdown.vue
index f2fe4eab03c..08cad53e625 100644
--- a/packages/web-app-files/src/components/SideBar/Shares/Collaborators/EditDropdown.vue
+++ b/packages/web-app-files/src/components/SideBar/Shares/Collaborators/EditDropdown.vue
@@ -28,7 +28,11 @@
                 @click="togglePopover"
               >
                 <oc-icon name="calendar-event" fill-type="line" size="medium" variation="passive" />
-                <span v-if="isExpirationDateSet" v-text="$gettext('Edit expiration date')" />
+                <span
+                  class="oc-ml-s"
+                  v-if="isExpirationDateSet"
+                  v-text="$gettext('Edit expiration date')"
+                />
                 <span v-else v-text="$gettext('Set expiration date')" />
               </oc-button>
             </template>
@@ -186,10 +190,7 @@ export default defineComponent({
     },
 
     isExpirationSupported() {
-      return (
-        (this.editingUser && this.userExpirationDate) ||
-        (this.editingGroup && this.groupExpirationDate)
-      )
+      return this.editingUser || this.editingGroup
     },
 
     isExpirationDateSet() {
@@ -248,14 +249,14 @@ export default defineComponent({
 
     isExpirationDateEnforced() {
       if (this.editingUser) {
-        return this.userExpirationDate.enforced
+        return this.userExpirationDate?.enforced
       }
 
       if (this.editingGroup) {
-        return this.groupExpirationDate.enforced
+        return this.groupExpirationDate?.enforced
       }
 
-      return this.userExpirationDate.enforced || this.groupExpirationDate.enforced
+      return this.userExpirationDate?.enforced || this.groupExpirationDate?.enforced
     },
 
     maxExpirationDate() {
diff --git a/packages/web-app-files/src/components/SideBar/Shares/Collaborators/InviteCollaborator/ExpirationDatepicker.vue b/packages/web-app-files/src/components/SideBar/Shares/Collaborators/InviteCollaborator/ExpirationDatepicker.vue
index e1f94bda6dd..38c67771d14 100644
--- a/packages/web-app-files/src/components/SideBar/Shares/Collaborators/InviteCollaborator/ExpirationDatepicker.vue
+++ b/packages/web-app-files/src/components/SideBar/Shares/Collaborators/InviteCollaborator/ExpirationDatepicker.vue
@@ -1,5 +1,5 @@
 <template>
-  <div v-if="available" class="oc-flex oc-flex-middle oc-flex-nowrap">
+  <div class="oc-flex oc-flex-middle oc-flex-nowrap">
     <oc-datepicker
       v-model="dateCurrent"
       :min-date="dateMin"
@@ -72,7 +72,6 @@ export default defineComponent({
     const capabilities = computed(() => store.getters.capabilities)
     const optionsUser = computed(() => capabilities.value.files_sharing.user?.expire_date)
     const optionsGroup = computed(() => capabilities.value.files_sharing.group?.expire_date)
-    const available = computed(() => optionsUser.value || optionsGroup.value)
     const enforced = computed(() => optionsUser.value?.enforced || optionsGroup.value?.enforced)
     const dateMin = DateTime.now().setLocale(language.current).toJSDate()
     const dateDefault = computed(() => {
@@ -151,7 +150,6 @@ export default defineComponent({
     return {
       language,
       enforced,
-      available,
       dateCurrent,
       dateMin,
       dateMax,

From 4e13491c75f9f78a089a271494c1fa30d46a31f4 Mon Sep 17 00:00:00 2001
From: Jan <j.ackermann91@gmail.com>
Date: Tue, 1 Aug 2023 14:44:09 +0200
Subject: [PATCH 2/4] Add changelog item

---
 ...et-or-remove-expiration-date-on-group-share-not-possible | 6 ++++++
 1 file changed, 6 insertions(+)
 create mode 100644 changelog/unreleased/bugfix-set-or-remove-expiration-date-on-group-share-not-possible

diff --git a/changelog/unreleased/bugfix-set-or-remove-expiration-date-on-group-share-not-possible b/changelog/unreleased/bugfix-set-or-remove-expiration-date-on-group-share-not-possible
new file mode 100644
index 00000000000..9c419ea30c7
--- /dev/null
+++ b/changelog/unreleased/bugfix-set-or-remove-expiration-date-on-group-share-not-possible
@@ -0,0 +1,6 @@
+Bugfix: Set or remove expiration date on group share not possible
+
+We've fixed a bug where setting or remove a expiration on a group share wasn't possible.
+
+https://github.com/owncloud/web/pull/9513
+https://github.com/owncloud/web/issues/8419

From f5f06918d9b10ae0b2df2c4b500b02347aeda783 Mon Sep 17 00:00:00 2001
From: Jan <j.ackermann91@gmail.com>
Date: Tue, 1 Aug 2023 14:57:10 +0200
Subject: [PATCH 3/4] Rm unused snapshot

---
 .../InviteCollaborator/ExpirationDatepicker.spec.ts          | 5 -----
 .../__snapshots__/ExpirationDatepicker.spec.ts.snap          | 3 ---
 2 files changed, 8 deletions(-)
 delete mode 100644 packages/web-app-files/tests/unit/components/SideBar/Shares/Collaborators/InviteCollaborator/__snapshots__/ExpirationDatepicker.spec.ts.snap

diff --git a/packages/web-app-files/tests/unit/components/SideBar/Shares/Collaborators/InviteCollaborator/ExpirationDatepicker.spec.ts b/packages/web-app-files/tests/unit/components/SideBar/Shares/Collaborators/InviteCollaborator/ExpirationDatepicker.spec.ts
index a98a9dab50a..92b10857397 100644
--- a/packages/web-app-files/tests/unit/components/SideBar/Shares/Collaborators/InviteCollaborator/ExpirationDatepicker.spec.ts
+++ b/packages/web-app-files/tests/unit/components/SideBar/Shares/Collaborators/InviteCollaborator/ExpirationDatepicker.spec.ts
@@ -43,11 +43,6 @@ const enforcedCapabilities = merge({}, enabledCapabilities, {
 })
 
 describe('InviteCollaborator ExpirationDatepicker', () => {
-  it('only gets displayed if share expiration is supported', () => {
-    const { wrapper } = createWrapper()
-    expect(wrapper.html()).toMatchSnapshot()
-  })
-
   it('renders a button to open the datepicker and set an expiration date', () => {
     const { wrapper } = createWrapper({ capabilities: bareCapabilities })
     expect(wrapper.find('[data-testid="recipient-datepicker-btn"]').exists()).toBe(true)
diff --git a/packages/web-app-files/tests/unit/components/SideBar/Shares/Collaborators/InviteCollaborator/__snapshots__/ExpirationDatepicker.spec.ts.snap b/packages/web-app-files/tests/unit/components/SideBar/Shares/Collaborators/InviteCollaborator/__snapshots__/ExpirationDatepicker.spec.ts.snap
deleted file mode 100644
index 028ef0a523b..00000000000
--- a/packages/web-app-files/tests/unit/components/SideBar/Shares/Collaborators/InviteCollaborator/__snapshots__/ExpirationDatepicker.spec.ts.snap
+++ /dev/null
@@ -1,3 +0,0 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
-
-exports[`InviteCollaborator ExpirationDatepicker only gets displayed if share expiration is supported 1`] = `<!--v-if-->`;

From 20a7a3abcc5f484429750a8ac3e3b6055a06482f Mon Sep 17 00:00:00 2001
From: Jan <j.ackermann91@gmail.com>
Date: Thu, 3 Aug 2023 09:54:23 +0200
Subject: [PATCH 4/4] Update
 changelog/unreleased/bugfix-set-or-remove-expiration-date-on-group-share-not-possible

---
 ...ix-set-or-remove-expiration-date-on-group-share-not-possible | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/changelog/unreleased/bugfix-set-or-remove-expiration-date-on-group-share-not-possible b/changelog/unreleased/bugfix-set-or-remove-expiration-date-on-group-share-not-possible
index 9c419ea30c7..6498ac95cec 100644
--- a/changelog/unreleased/bugfix-set-or-remove-expiration-date-on-group-share-not-possible
+++ b/changelog/unreleased/bugfix-set-or-remove-expiration-date-on-group-share-not-possible
@@ -1,6 +1,6 @@
 Bugfix: Set or remove expiration date on group share not possible
 
-We've fixed a bug where setting or remove a expiration on a group share wasn't possible.
+We've fixed a bug where setting or removing an expiration on a group share wasn't possible.
 
 https://github.com/owncloud/web/pull/9513
 https://github.com/owncloud/web/issues/8419