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

I see no difference in PHPStorm after clicking "Configure PHPStorm" #1

Open
slim16165 opened this issue Jan 17, 2019 · 3 comments
Open

Comments

@slim16165
Copy link

There is no log, no way to check how it should work. How do I understand if it's working?

@Luc45
Copy link

Luc45 commented Jan 17, 2019

Yeah, nothing is happening to me as well. I'm trying to debug this line but I'm having a hard time:

    console.log(publicCWD.exists('./.idea/php.xml'));
    console.log(publicCWD.exists('./.idea/runConfigurations/Local-By-Flywheel.xml'));

It's not writing to the files.

@Luc45
Copy link

Luc45 commented Jan 17, 2019

I got it. We have to normalize the slashes in the dir path. It is trying to find ~/ and replace it with the home dir, but in my case it is ~\, so the replacement never occurs and the file doesn't write.

How to fix:

  1. Open FlyWheel
  2. Go to Addons -> Installed -> Xdebug +PhpStorm -> Show Folder
  3. Navigate to local-addon-xdebug-phpstorm/lib
  4. Edit renderer.js
  5. Replace line 21~:
const sitePath = site.path.replace('~/', userHome + '/').replace(/\/+$/, '') + '/';

With:

const sitePath = site.path.replace('\~\\', '~/').replace('~/', userHome + '/').replace(/\/+$/, '') + '/';
  1. Reload Flywheel, either from Develop -> Refresh Window or by re-starting it.
  2. Re-start PHPStorm

@paulotruta
Copy link

This one works like a charm for Windows environments.
@Luc45 Would be lovely if you could try to send this upstream in a way that works for both Windows and Mac!

Thanks for the tip!

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

No branches or pull requests

3 participants