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 crash when image path contains encoded URI #73

Merged
merged 4 commits into from
Aug 23, 2024
Merged

Fix crash when image path contains encoded URI #73

merged 4 commits into from
Aug 23, 2024

Conversation

ryonakano
Copy link
Contributor

Fixes #24

  • Correct the way to convert a URI to a path
  • Do not try to convert to a path if a variable is already a path
  • Do not name 'uri' if a variable contains a path

Gijs Goudzwaard and others added 4 commits August 21, 2024 19:08
- Correct the way to convert a URI to a path
- Do not try to convert to a path if a variable is already a path
- Do not name 'uri' if a variable contains a path
@GijsGoudzwaard GijsGoudzwaard force-pushed the master branch 2 times, most recently from c46772c to 6cfacdc Compare August 23, 2024 15:45
@GijsGoudzwaard
Copy link
Owner

Looks good, thank you for the continued help squashing those pesky bugs!

@GijsGoudzwaard GijsGoudzwaard merged commit c08f95f into GijsGoudzwaard:master Aug 23, 2024
@ryonakano ryonakano deleted the fix-handle-path branch August 23, 2024 15:59
GijsGoudzwaard added a commit that referenced this pull request Aug 23, 2024
Fix crash when image path contains encoded URI
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.

App crashes if directory name contains Chinese characters
2 participants