Codiad / Codiad

Web Based, Cloud IDE
http://codiad.com
MIT License
2.85k stars 698 forks source link

recommendation: windows path issue #1032

Open goofballtech opened 7 years ago

goofballtech commented 7 years ago

windows/nginx install.

I was having issue gettng things rolling where it would tell me incorrect username/password. On the recommendation of a friend i double checked the autogenerated config.php file.

i found define("BASE_PATH", "C:\nginx\html/codiad");

corrected this to define("BASE_PATH", "C:/nginx/html/codiad"); and all was well.

recommend a str_replace() in the config.php generation script to help windows users not see this issues.

Thanks very much. looking forward to using the app!

daeks commented 7 years ago

just for reference, windows pathing is with \ instead of /

So basically spoken, you should use C:\nginx\html\codiad instead of C:/nginx/html/codiad

goofballtech commented 7 years ago

When speaking of windows paths in general you are correct. In PHP the backslash has to be escaped and doesn't work as expected. Your path would not give the apropriate result (I know because I tried and have had similar issues in the past) It will resolve the / fine and work.