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 the if check for invalid virtual LNK files #7781

Merged
merged 1 commit into from
Jan 22, 2025

Conversation

mgallien
Copy link
Collaborator

should ensure we fix the DB metadata of LNK windows shortcuts with virtual files type

should ensure we fix the DB metadata of LNK windows shortcuts with
virtual files type

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
@mgallien mgallien added this to the 3.16.0 milestone Jan 22, 2025
@mgallien
Copy link
Collaborator Author

follow up for #7779
something was missed before I merged it
sorry

@nextcloud-desktop-bot
Copy link

AppImage file: nextcloud-PR-7781-a45e96b793da55815a7f14044f8bf848b30985f0-x86_64.AppImage

To test this change/fix you can simply download above AppImage file and test it.

Please make sure to quit your existing Nextcloud app and backup your data.

Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
E Maintainability Rating on New Code (required ≥ A)
28 New Code Smells (required ≤ 0)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@mgallien mgallien merged commit c6b1a0d into master Jan 22, 2025
13 of 17 checks passed
@mgallien mgallien deleted the bugfix/readOnlyClassicalSyncLnkShortcutsWindows branch January 22, 2025 13:02
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.

3 participants