Skip to content

Fix bug where file extension had an extra '.' under some circumstances#15893

Merged
AUTOMATIC1111 merged 1 commit intoAUTOMATIC1111:devfrom
alcacode:dev
Jun 8, 2024
Merged

Fix bug where file extension had an extra '.' under some circumstances#15893
AUTOMATIC1111 merged 1 commit intoAUTOMATIC1111:devfrom
alcacode:dev

Conversation

@alcacode
Copy link
Contributor

Fix bug where under some circumstances an extra . was inserted between the file base name and the file extension.

The bug is triggered when the extension argument is one of "jpg", "jpeg", or "webp", and the image exceeds the format in question's width or height limit. Then the extension variable is set to ".png", resulting in the fullfn variable to evaluate to a string ending with ..png.

Fix bug where under some circumstances an extra "." was inserted between the file base name and the file extension.
The bug is triggered when the extension argument is one of "jpg", "jpeg", or "webp", and the image exceeds the format's dimension limit. Then the extension variable is set to ".png", resulting in the fullfn variable to evaluate to a string ending with "..png".
@alcacode alcacode requested a review from AUTOMATIC1111 as a code owner May 26, 2024 13:48
Copy link
Collaborator

@w-e-w w-e-w left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks
that was my bug

@AUTOMATIC1111 AUTOMATIC1111 merged commit 00f37ad into AUTOMATIC1111:dev Jun 8, 2024
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