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 creating and updating plugin with dot in folder name #83329

Conversation

detomon
Copy link
Contributor

@detomon detomon commented Oct 14, 2023

This fixes a problem when creating a plugin with a dot in the name from the project settings or updating an existing plugin with a dot in the folder name will fail.

Test project: plugin_config_dialog.zip


At the moment, when creating a plugin with a dot in the name and leaving "Script Name" empty, creating the initial plugin script fails as it doesn't add the correct extension.

create-with-dot

Also, when a plugin folder is added via file system which contains a dot, information for that plugin cannot be be updated in the project settings as it won’t use the correct plugin folder name. Updating the author, for example, will silently fail.

update-with-dot

@AThousandShips

This comment was marked as outdated.

@detomon

This comment was marked as resolved.

@detomon

This comment was marked as resolved.

@YuriSizov YuriSizov modified the milestones: 4.2, 4.3 Nov 13, 2023
@YuriSizov YuriSizov added the cherrypick:4.2 Considered for cherry-picking into a future 4.2.x release label Nov 13, 2023
@YuriSizov YuriSizov requested a review from KoBeWi December 4, 2023 15:50
@akien-mga akien-mga merged commit 481d046 into godotengine:master Dec 13, 2023
15 checks passed
@akien-mga
Copy link
Member

Thanks!

@YuriSizov YuriSizov removed the cherrypick:4.2 Considered for cherry-picking into a future 4.2.x release label Jan 25, 2024
@YuriSizov
Copy link
Contributor

Cherry-picked for 4.2.2.

@detomon detomon deleted the fix-creating-and-updating-plugin-with-dot-in-folder-name branch July 14, 2024 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Creating a plugin with a dot in the name or updating a plugin with a dot in the folder name fails
5 participants