Skip to content

[stable31] fix(files_sharing): use newPassword always for the unsaved password - #55073

Merged
susnux merged 2 commits into
stable31from
fix/files-sharing-password
Sep 12, 2025
Merged

[stable31] fix(files_sharing): use newPassword always for the unsaved password#55073
susnux merged 2 commits into
stable31from
fix/files-sharing-password

Conversation

@susnux

Copy link
Copy Markdown
Contributor

Summary

newPassword is the unsaved password, while share.password is the current saved password.

Checklist

@susnuxsusnux added this to the Nextcloud 31.0.10 milestone Sep 12, 2025
@susnux
susnux requested review from a team, Antreesy, nfebe, salmart-dev, sorbaugh and szaimen and removed request for a teamSeptember 12, 2025 10:04
@susnux

Copy link
Copy Markdown
ContributorAuthor

/backport to stable32

@susnux

Copy link
Copy Markdown
ContributorAuthor

/backport to master

@szaimenszaimen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🐘

@AntreesyAntreesy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested, looks good code-wise
Image

`newPassword` is the unsaved password, while `share.password` is the
current saved password.
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux
susnuxforce-pushed the fix/files-sharing-password branch from e95fadb to ee97330CompareSeptember 12, 2025 10:13
@susnux

Copy link
Copy Markdown
ContributorAuthor

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@mpivchev

Copy link
Copy Markdown

Is this patchable? Since it's vue and frontend? I am not sure

@susnux

Copy link
Copy Markdown
ContributorAuthor

@mpivchev not really, best to wait for the release on October 9, 2025

@skjnldsv

Copy link
Copy Markdown
Member

Regression fix in #56278

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@susnux@mpivchev@skjnldsv@szaimen@Antreesy@nextcloud-command