Fix WinUtil Crashing on Startup - Xaml Error from Generated Applications List #2361
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This will fix WinUtil refusing to startup issue, giving an Xaml Syntax error.
At first I thought it's from your new Easter Egg.. cogh.. but after literally staring at the screen for 10 minute 😆 (just looking for syntax errors in your changes).. I was stuck to no avail ...
but at last, and by using a similar technique I've used before in PR #1850 (going through commit history, one commit at a time).. I've found the root cause of this issue (in PR #2316), which's the App Name Entry having a dash/minus symbol.. I'm just astounded on how it came to be, and how it passed the testing pipeline, but it was simple fix nonetheless.
Issue related to PR
Additional Information
We should try avoiding this issue by updating the docs to tell people not to add specific symbols in Field Names, and make the testing pipeline handle such situations without ignoring them.
Checklist