-
Notifications
You must be signed in to change notification settings - Fork 800
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 SVG rendering error in SvgImageProvider #5666
Conversation
SonarCloud Quality Gate failed. |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #5666 +/- ##
==========================================
- Coverage 60.08% 60.06% -0.03%
==========================================
Files 145 145
Lines 18756 18746 -10
==========================================
- Hits 11270 11260 -10
Misses 7486 7486 |
/backport to stable-3.9 |
276f959
to
258d09e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@claucambra Please squash before merging
7edfe21
to
bc514ca
Compare
Signed-off-by: Claudio Cambra <[email protected]>
Signed-off-by: Claudio Cambra <[email protected]>
Signed-off-by: Claudio Cambra <[email protected]>
bc514ca
to
ed05436
Compare
SonarCloud Quality Gate failed. 0 Bugs 76.0% Coverage Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
AppImage file: nextcloud-PR-5666-ed05436ea572a4e17f0d43d6913a345c60b19935-x86_64.AppImage |
As part of
IconUtils::createSvgImageWithCustomColor
we try to see if there is a pre-existing image with the desired colour combination string at a given path. However, we do not check if the file exists and instead try directly to create the image; this creates an unnecessary error when we could just check for the existence of the file at the path.2023-05-09 14:49:16:215 [ warning qt.svg unknown:0 ]: Cannot open file ':/client/theme/black/share.svg', because: No such file or directory
This PR adds this file check first, avoiding the error.
Additionally, this PR includes some clean up of the relevant code.