[Bug]: "Set expiration date" can be disabled but it doesn't look disabled

[thgoebel]

⚠️ This issue respects the following points: ⚠️

• This is a bug, not a question or a configuration/webserver/proxy issue.
• This issue is not already reported on Github OR Nextcloud Community Forum (I've searched it).
• Nextcloud Server is up to date. See Maintenance and Release Schedule for supported versions.
• I agree to follow Nextcloud's Code of Conduct.

Bug description

The "Set expiration date" checkbox cannot be visually unchecked if a non-enforced default expiration date is configured by the admin.

This seems to have been introduced in #40629 in an attempt to fix #40612.

Steps to reproduce

1. Admin: Enable "Set default expiration date for shares" and set a data but do NOT enable "Enforce expiration date" (under /settings/admin/sharing)
2. User: create a new share. Notice how since Updates to new sharing flow #40629 the default expiration date is correctly set
3. Click "Set expiration date" in an attempt to un-check it

Actual behaviour

The "Set expiration date" box remains checked. The date values are reset, and under the hood the expiration date is correctly unset. Just the UI is wrong.

Expected behavior

The checkbox should become unchecked.

Installation method

Community Docker image

Nextcloud Server version

27.1.3

Operating system

Debian/Ubuntu

PHP engine version

PHP 8.2

Web server

None

Database engine version

None

Is this bug present after an update or on a fresh install?

None

Are you using the Nextcloud Server Encryption module?

Encryption is Disabled

What user-backends are you using?

• Default user-backend (database)
• LDAP/ Active Directory
• SSO - SAML
• Other

Configuration report

n/a, see https://github.com/nextcloud/server/issues/40612

List of activated Apps

n/a, see https://github.com/nextcloud/server/issues/40612

Nextcloud Signing status

No errors have been found.

Nextcloud Logs

n/a

Additional info

Screenshot

[szaimen]

Cc @fenn-cs