Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix log spamming for shares without label #50243

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Dennis1993
Copy link
Contributor

Fixes #49682

  • Resolves: #

Summary

Database field is null and we should check it with empt() function to prevent error like this: #49682

Checklist

Copy link
Contributor

@susnux susnux left a comment

Choose a reason for hiding this comment

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

This also forbids the string '0'.
Moreover this only hides the problem, it should never be null in the first place.

So would be good to investigate why null is set instead of the empty string.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: 30.0.3 Log spamming Invalid label data providedInitialState by files_sharing
2 participants