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

SocketAPI: Do not strip trailing whitespace from paths #9452

Merged
merged 1 commit into from
Feb 18, 2022

Conversation

erikjv
Copy link
Collaborator

@erikjv erikjv commented Feb 16, 2022

When stripped, the wrong file name is queried in the DB, and the status
badges shown in Finder will not be there.

#9030

erikjv added a commit that referenced this pull request Feb 16, 2022
When stripped, the wrong file name is queried in the DB, and the status
badges shown in Finder will not be there.

#9030
#9452
@erikjv erikjv force-pushed the work/undo-strip-whitespace-in-SocketAPI branch from 1306a94 to 00b4468 Compare February 16, 2022 18:47
@TheOneRing
Copy link
Member

Ouch, that was me, thx for fixing.

erikjv added a commit that referenced this pull request Feb 17, 2022
When stripped, the wrong file name is queried in the DB, and the status
badges shown in Finder will not be there.

#9030
#9452
@erikjv erikjv force-pushed the work/undo-strip-whitespace-in-SocketAPI branch from 00b4468 to 0fa2acb Compare February 17, 2022 09:25
@TheOneRing
Copy link
Member

Changlog is wrong, needs to be Bugfix:

@TheOneRing
Copy link
Member

And could you uses the Fixed: #issue syntax so that the issue is closed once the change reaches master.

@erikjv erikjv force-pushed the work/undo-strip-whitespace-in-SocketAPI branch 2 times, most recently from 06253e1 to 55598bb Compare February 17, 2022 11:24
When stripped, the wrong file name is queried in the DB, and the status
badges shown in Finder will not be there.

Fixes: #9030
@erikjv erikjv force-pushed the work/undo-strip-whitespace-in-SocketAPI branch from 55598bb to 01a0eb0 Compare February 17, 2022 13:42
@sonarcloud
Copy link

sonarcloud bot commented Feb 17, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@TheOneRing TheOneRing merged commit fd361e8 into 2.10 Feb 18, 2022
@delete-merged-branch delete-merged-branch bot deleted the work/undo-strip-whitespace-in-SocketAPI branch February 18, 2022 10:55
@TheOneRing TheOneRing mentioned this pull request Mar 10, 2022
55 tasks
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.

None yet

2 participants