Skip to content

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

Merged
susnux merged 1 commit into
stable28from
backport/48673/stable28
Oct 17, 2024
Merged

[stable28] fix(share): Return empty string if no label is set#48739
susnux merged 1 commit into
stable28from
backport/48673/stable28

Conversation

@backportbot

@backportbotbackportbotBot commented Oct 16, 2024

Copy link
Copy Markdown

Backport of #48673

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Remove all the empty commits

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbotbackportbotBot added bug 3. to review Waiting for reviews php Pull requests that update Php code labels Oct 16, 2024
@backportbotbackportbotBot added this to the Nextcloud 28.0.12 milestone Oct 16, 2024
@susnux
susnuxforce-pushed the backport/48673/stable28 branch from 3af3365 to f26d04fCompareOctober 16, 2024 18:43
@susnux
susnux marked this pull request as ready for review October 16, 2024 18:43
@susnux
susnux enabled auto-merge October 16, 2024 18:43
@susnux
susnuxforce-pushed the backport/48673/stable28 branch 2 times, most recently from 2bd160c to bfde2a0CompareOctober 17, 2024 16:27
* 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>
@susnux
susnuxforce-pushed the backport/48673/stable28 branch from bfde2a0 to 788233cCompareOctober 17, 2024 16:48
@susnux
susnux merged commit 86ed7aa into stable28Oct 17, 2024
@susnux
susnux deleted the backport/48673/stable28 branch October 17, 2024 17:49
@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.

2 participants

@susnux@provokateurin