Disable expiration date for alias link (#10083)

* Disable expiration date for alias link

* Add changelog

* Remove unused input

* Use elseif

* Add unittests

changelog/unreleased/bugfix-disable-expiration-date-alias-link

@@ -0,0 +1,7 @@
+Bugfix: Disable expiration date for alias link (internal)
+
+We've disabled the option to set an expiration date on an internal link,
+since it could be misleading and doesn't serve any use.
+
+https://github.com/owncloud/web/pull/10083
+https://github.com/owncloud/web/issues/9923

packages/web-app-files/src/components/SideBar/Shares/Links/DetailsAndEdit.vue

@@ -326,7 +326,7 @@ export default defineComponent({
               })
           }
         })
-      } else {
+      } else if (!this.isAliasLink) {
         result.push({
           id: 'add-expiration',
           title: this.$gettext('Set expiration date'),

packages/web-app-files/tests/unit/components/SideBar/Shares/Links/DetailsAndEdit.spec.ts

@@ -49,6 +49,23 @@ describe('DetailsAndEdit component', () => {
       expect(setModalInputErrorMessageStub).toHaveBeenCalledWith(expect.anything())
     })
   })
+
+  describe('editOptions computed property', () => {
+    it('does not add "add-expiration" option if isAliasLink is true', () => {
+      const exampleLinkInternal = {
+        name: 'Example link',
+        url: 'https://some-url.com/abc',
+        permissions: 0
+      }
+      const { wrapper } = getShallowMountedWrapper(exampleLinkInternal, false, true)
+      expect(wrapper.vm.editOptions.some((option) => option.id === 'add-expiration')).toBe(false)
+    })
+
+    it('adds "add-expiration" option if isAliasLink is false', () => {
+      const { wrapper } = getShallowMountedWrapper(exampleLink, false, true)
+      expect(wrapper.vm.editOptions.some((option) => option.id === 'add-expiration')).toBe(true)
+    })
+  })
 })
 
 function getShallowMountedWrapper(link, expireDateEnforced = false, isModifiable = false) {