-
Notifications
You must be signed in to change notification settings - Fork 2
Related project #1
Comments
Also I've noticed that urlencoded file urls aren't supported, because they are not decoded before given to PhpStorm. But that doesn't matter now I guess. |
Great minders think alike. I saw your solution, nice catch! I prefer to stick to KISS and keep the app ready for use immediately after downloading. As of May 16 there are built-in idea:// and phpstorm:// protocols support in PhpStorm 8 EAP 138.190+ according to this comment from phpstorm team member. As to urlencoding - who cares? I don't think that someone have files names with ampersands and equal sign to intersect with url schema. Thanks for picking me up, I even forgot to track that bug on jetbrains site! |
Ha, then you don't encode urls properly in your apps. Since in The solution via LinCastor (mentioned on my repo) deals with encoded urls just fine by the way. |
Can you post some meta-example to show the problem? I guess |
Sure, it's bug report in my repo, that's why I know: aik099/PhpStormProtocol#6 |
I don't see there any link example and in addition it seems to be that the problem is windows-specific there. Maybe it is not a right way to deal with such kind of potential issues, but (at least my app) works even without any url encoding/decoding. |
Of course. The more users use the app, the more bug reports comes in. I also thought that I got all cases covered when released it 😄 |
It appears that I need to have this protocol handler 2 months before you and created this project: https://github.com/aik099/PhpStormProtocol
It also has instructions on how to setup this in Linux & Windows.
At the end after PhpStorm 8 release both our repos wouldn't be useful due native url handler support by PhpStorm itself.
The text was updated successfully, but these errors were encountered: