Skip to content

[stable30] fix(share): Return empty string if no label is set - #48703

Merged
susnux merged 2 commits into
stable30from
backport/48673/stable30
Oct 17, 2024
Merged

[stable30] fix(share): Return empty string if no label is set#48703
susnux merged 2 commits into
stable30from
backport/48673/stable30

Conversation

@backportbot

Copy link
Copy Markdown

Backport of PR #48673

* Resolves: #48629
While the database supports NULL, the typing has always said it only returns *string*.
So to not break any apps that might trust the typings we should return `''` if the database is set to `NULL`.
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
This reverts commit 01c4fa3.
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@backportbotbackportbotBot added bug 3. to review Waiting for reviews php Pull requests that update Php code labels Oct 14, 2024
@backportbotbackportbotBot added this to the Nextcloud 30.0.1 milestone Oct 14, 2024
@blizzzblizzz mentioned this pull request Oct 17, 2024
@susnux
susnux enabled auto-merge October 17, 2024 12:59
@susnux
susnux merged commit 761f7e1 into stable30Oct 17, 2024
@susnux
susnux deleted the backport/48673/stable30 branch October 17, 2024 16:25
@AltahrimAltahrim mentioned this pull request Oct 30, 2024
2 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbugphpPull requests that update Php code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@susnux@provokateurin@blizzz