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

phpstorm-debugging-guide #384

Merged
merged 1 commit into from
Nov 18, 2016
Merged

Conversation

LarryEitel
Copy link
Contributor

Guide to remote debugging Laravel in PHPStorm.

@LarryEitel
Copy link
Contributor Author

This is an initial crack at providing a guide to debugging Laravel in PHPStorm. Although it will be useful to anyone interested, it is not considered a release version. I felt it important to get SOMETHING on the table sooner than later. :) It would be great if other users of PHPStorm could follow this guide to confirm whether this guide is complete and accurate.

@Mahmoudz
Copy link
Member

@LarryEitel the documentation looks very nice.

Just make sure to disable xdebug by default. And mention that xdebug needs to be enabled first..

Best,

@LarryEitel
Copy link
Contributor Author

@Mahmoudz I think it is pretty clean now. Don't hesitate to share any suggestions/corrections.

One thing that could be done would be to squash commits. :)

One more thing that I find useful is to have a branch off of my fork which is all configured/wired up for debugging. Then all I have to do is merge from master as needed. Here is an example LarryEitel/laradock/tree/phpstorm-debug for anyone interested.

@Exadra37
Copy link

@LarryEitel no need to use a separated branch to debug if we go as per #424.

@LarryEitel
Copy link
Contributor Author

I agree with the idea to set/override all config values through .env. As for this debugging guide, I made a branch although it only consists of a guide. I made a branch on MY fork to set appropriate values to verify debugging works. This would not be necessary if these values could be read from .env.

I DO wonder if docker-compose will behave well if it sees no change in the actual .yml file.

@Exadra37
Copy link

Good point about docker compose detecting the changes... at least when I testing it by run docker-compose config it was showing the changes to the .env file.

I will test later with docker-compose up.

@philtrep
Copy link
Member

@LarryEitel This looks really good. What do you think is missing for this to be considered a release version (aside from squashing commits)?

Even if the guide isn't 100% accurate, merging this in would at least give some tips to users and would increase the chances of others contributing.

@LarryEitel
Copy link
Contributor Author

Actually, I verified the debug guide works with current LaraDock. It doesn't affect any functionality. It only provides guidance to get debugging working. If/when anyone has a further question, I will try to update/adjust accordingly.

@philtrep
Copy link
Member

@LarryEitel Awesome, let me know when you're done squashing and I'll merge this in! 😃

-	modified:   README.md
-	new file:   _guides/photos/KiTTY/Connection.png
-	new file:   _guides/photos/KiTTY/ConnectionData.png
-	new file:   _guides/photos/KiTTY/ConnectionSSH.png
-	new file:   _guides/photos/KiTTY/ConnectionSSHAuth.png
-	new file:   _guides/photos/KiTTY/Session.png
-	new file:   _guides/photos/KiTTY/Terminal.png
-	new file:   _guides/photos/KiTTY/TerminalKeyboard.png
-	new file:   _guides/photos/KiTTY/TerminalShell.png
-	new file:   _guides/photos/KiTTY/Window.png
-	new file:   _guides/photos/KiTTY/WindowAppearance.png
-	new file:   _guides/photos/PHPStorm/DebugRemoteOn.png
-	new file:   _guides/photos/PHPStorm/RemoteDebuggingSuccess.png
-	new file:   _guides/photos/PHPStorm/RemoteHost.png
-	new file:   _guides/photos/PHPStorm/RemoteTestDebuggingSuccess.png
-	new file:   _guides/photos/PHPStorm/RemoteWebDebuggingSuccess.png
-	new file:   _guides/photos/PHPStorm/Settings/BuildDeploymentConnection.png
-	new file:   _guides/photos/PHPStorm/Settings/BuildDeploymentConnectionMappings.png
-	new file:   _guides/photos/PHPStorm/Settings/BuildDeploymentDebugger.png
-	new file:   _guides/photos/PHPStorm/Settings/EditRunConfigurationRemoteExampleTestDebug.png
-	new file:   _guides/photos/PHPStorm/Settings/EditRunConfigurationRemoteWebDebug.png
-	new file:   _guides/photos/PHPStorm/Settings/LangsPHPDebug.png
-	new file:   _guides/photos/PHPStorm/Settings/LangsPHPInterpreters.png
-	new file:   _guides/photos/PHPStorm/Settings/LangsPHPPHPUnit.png
-	new file:   _guides/photos/PHPStorm/Settings/LangsPHPServers.png
-	new file:   _guides/photos/PHPStorm/Settings/WindowsFirewallAllowedApps.png
-	new file:   _guides/photos/PHPStorm/Settings/WindowsHyperVManager.png
-	new file:   _guides/photos/PHPStorm/Settings/hosts.png
-	new file:   _guides/photos/SimpleHostsEditor/AddHost_laravel.png
-	new file:   _guides/phpstorm.md
@LarryEitel
Copy link
Contributor Author

@philtrep OK, good to go. :)

@philtrep philtrep merged commit 225fb8c into laradock:master Nov 18, 2016
@philtrep
Copy link
Member

@LarryEitel All done. Thanks again for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants