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

[Windows] Shortcuts are not created properly if the game name has invalid characters #760

Closed
Arron17 opened this issue Dec 2, 2021 · 11 comments
Labels
bug:confirmed Something isn't working. good first issue Good for newcomers.

Comments

@Arron17
Copy link

Arron17 commented Dec 2, 2021

IMPORTANT! Do not erase this template!

Is this a game issue? If YES please use our Discord to get help.

Open here only issues with Heroic like: Heroic doesn't open, some button is out of place, Heroic closes unexpectedly, installs got interrupted, etc.

Try to get help on our Wiki first: https://github.com/Heroic-Games-Launcher/HeroicGamesLauncher/wiki/Troubleshooting

Describe the bug
If you try to create a shortcut for a game that uses characters that are invalid in Windows File names, then the creation of the shortcuts do not complete correctly.

Add Logs
No Logs are produced when clicking on Add Shortcut

To Reproduce
Steps to reproduce the behavior:

  1. Go to a game that uses special characters that Windows can't use in a file name, for example "while True: learn()"
  2. Click on Add shortcut
  3. Message will appear to say a shortcut has been created on the Desktop and Start Menu
  4. Shortcut on the Desktop is created, however it is called while True, has no icon and is a zero byte file, not a shortcut.
  5. No shortcut is made in the start menu at all

Expected behavior
Needs to have some logic on Windows that stops it from trying to use Invalid characters in the shortcut filenames, added example screenshot:
image

Desktop (please complete the following information):

  • OS: Windows 11 21H2
  • Heroic Version 1.11.0 Alpha
  • Game: while True:learn()
  • Wine/Proton version: N/A
  • Hardware specs (RAM, CPU, GPU): N/A
@Arron17
Copy link
Author

Arron17 commented Dec 2, 2021

Would be linked to #740

@flavioislima
Copy link
Member

I see. I'll try to fix this on next version!

@flavioislima flavioislima added bug:confirmed Something isn't working. good first issue Good for newcomers. labels Dec 2, 2021
@ghost
Copy link

ghost commented Dec 4, 2021

Please assign it to me.

@flavioislima flavioislima assigned ghost Dec 4, 2021
@flavioislima
Copy link
Member

Please assign it to me.

Done!

@ghost
Copy link

ghost commented Dec 4, 2021

Can you download this build here and see if it works now? @Arron17

@Arron17
Copy link
Author

Arron17 commented Dec 4, 2021

@dragonDScript Same problem unfortunately.

image

@ghost
Copy link

ghost commented Dec 4, 2021

@dragonDScript Same problem unfortunately.

image

Have you deleted and created the shortcut again? It should work.

@Arron17
Copy link
Author

Arron17 commented Dec 4, 2021

@dragonDScript Deleted shortcut and tried it again, uninstalled the game and tried it again, changed the location of the game install and tried again. Same result each time.

@ghost
Copy link

ghost commented Dec 4, 2021

OK I will test this myself.

@Dark-Matter7232
Copy link

Screenshot 2022-06-09 153421

Issue still exists.

@flavioislima flavioislima unassigned ghost Nov 23, 2022
@flavioislima
Copy link
Member

Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug:confirmed Something isn't working. good first issue Good for newcomers.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants