Cannot unset share expiration date if admin has set a default-but-not-enforced expiration date
Nobody has claimed this yet.
- Dominant language
- Swift
- Stars
- 2.5k
- Forks
- 1k
- Avg merge
- 2d 18h
- Merged PRs (30d)
- 13
Description
Steps to reproduce
- Let the admin configure a "default expiration date for shares" under
/settings/admin/sharing, e.g. to 7 days. DO NOT check the box to enforce the default expiration date. - Open the iOS Files app
- Share a file and save the share
- Click on ... > Details to edit the share
- Click on "Set expiration date"
- Click "clear" --> notice how the date is cleared
- Save the share
- Click ... > Details again to view the expiration date
Expected behaviour
The "Set expiration date" field should have no date set.
Actual behaviour
The "Set expiration date" field is set to the admin's default, e.g. 7 days in the future.
Screenshots
n/a
Logs
n/a
Reasoning or why should it be changed/implemented?
It's a bug. If the admin does not enforce it, the user should be able to set no expiration date. That's also how Android and web do it.
Environment data
iOS version: e.g. iPadOS 17.2
Nextcloud iOS app version: 4.9.6.1
Server operating system: Ubuntu
Web server: Apache
Database: MariaDB
PHP version: 8.2
Nextcloud version: 28.0.0
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reproducing the share-editing flow in the iOS Files app with a non-enforced server default expiration, then compare the behavior with Android and web. The fix is done when clearing the expiration remains unset after saving and reopening the share.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- ios, swift
- Domain
- mobile
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100