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

Images of dental services cannot have white-spaces #160

Closed
MrDave1999 opened this issue Feb 9, 2023 · 1 comment · Fixed by #161
Closed

Images of dental services cannot have white-spaces #160

MrDave1999 opened this issue Feb 9, 2023 · 1 comment · Fixed by #161
Labels
invalid This doesn't seem right

Comments

@MrDave1999
Copy link
Member

On Windows there are no problems but on Linux distributions when a file name has whitespace, it is appended around single quotes. This leads to invalid behavior, because in the database the image name is stored without single quotes but in the file system it is (this should not be the case).

This would cause problems, for example, when you want to edit the image of a dental service, it will not be possible to delete the old image because the image name in the database will not match the image in the file system.

Sample:

File system:

'brackets_0023222.png'

database:

brackets_0023222.png
@MrDave1999 MrDave1999 added the invalid This doesn't seem right label Feb 9, 2023
@MrDave1999
Copy link
Member Author

Apparently the app still works on Windows 10 or Ubuntu even if the image name has whitespaces. Likewise, it would be nice if could replace the whitespaces with underscores to avoid possible problems with other operating systems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

1 participant