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

Psb: sanitize ImageMetadata.Name to prevent invalid file name error #104

Closed
wants to merge 1 commit into from
Closed

Conversation

pkuislm
Copy link

@pkuislm pkuislm commented Mar 26, 2023

some .mtn files use strings like src/normal/通常|キャラ|通常来海 (結合) as its images' name, which causes an error while we are trying to export them.
I add a check to GetFriendlyName to replace these invalid characters with _.

@UlyssesWu
Copy link
Owner

UlyssesWu commented Mar 26, 2023

Looks good.

Nice catch, but the solution is kind of wrong. Name should not be changed in GetFriendlyName.

Would you like to share a sample .mtn file by the way?

@pkuislm
Copy link
Author

pkuislm commented Mar 26, 2023

Sure.
title_bg.zip

@UlyssesWu
Copy link
Owner

Name should not be changed in GetFriendlyName.

I think this should be fixed. You'd like to try, or leave it to me?

@pkuislm
Copy link
Author

pkuislm commented Mar 26, 2023

Name should not be changed in GetFriendlyName.

I think this should be fixed. You'd like to try, or leave it to me?

Thank you for the invitation. However, I don't have a thorough understanding of this project, and I don't want to cause any unnecessary damage or confusion. So, I would appreciate it if you could handle the issue. Thank you again for your consideration.

@pkuislm pkuislm closed this Mar 26, 2023
@UlyssesWu
Copy link
Owner

Thanks anyway, I will note your name in next release.

UlyssesWu added a commit to Dual-Vector-Foil/FreeMote.Samples that referenced this pull request Mar 26, 2023
@UlyssesWu UlyssesWu added the bug label Mar 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants