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

Launcher does not open files in system that are not in english #3457

Merged
merged 1 commit into from
May 21, 2020
Merged

Launcher does not open files in system that are not in english #3457

merged 1 commit into from
May 21, 2020

Conversation

ricardosantos9521
Copy link
Contributor

@ricardosantos9521 ricardosantos9521 commented May 21, 2020

Because of the way windows works in systems that are not in english, launcher was trying to open localized paths instead of the real path.s File explorer can translate them to the "real path" but Proccess.Start doesn't do that.

PR Checklist

  • [ x ] Applies to Launcher does not open files #3209
  • [ x ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

When Microsoft.Plugin.Indexer is querying it is selecting the System.ItemPathDisplay that returns the localized path of the file/folder. For some reason Proccess.Start does not like localized paths i supposed that is the same reason why powershell does not let me launch localized paths.

Detailed Description of the Pull Request:

I changed System.ItemPathDisplay to System.ItemUrl and remove the scheme from the url during the ExecuteQuery.

Validation Steps Performed

Launch Powertoys Run and search for a file or folder.
Files appear as they should with icons and all actions work.

@arjunbalgovind arjunbalgovind requested review from alekhyareddy28 and a team May 21, 2020 17:56
Copy link
Contributor

@alekhyareddy28 alekhyareddy28 left a comment

Choose a reason for hiding this comment

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

Awesome! LGTM.

@crutkas crutkas merged commit c7fb1f6 into microsoft:master May 21, 2020
@ricardosantos9521 ricardosantos9521 deleted the launcher-localized-path-bug branch June 12, 2020 17:20
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