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

[RouterDebugCommand] add link to Controllers #30520

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
4 participants
@nicoweb
Copy link
Contributor

nicoweb commented Mar 11, 2019

Q A
Branch? master for features
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets
License MIT
Doc PR

Adds a link to the controller method on your IDE from dev's terminal:

pr-debug-router

Configuration in your services.yaml:

parameters:
    debug.file_link_format: phpstorm://open?file=%%f&line=%%l

@nicolas-grekas nicolas-grekas force-pushed the nicoweb:feature/add-link-to-services branch from 1521372 to 6dd8ce5 Mar 12, 2019

@nicoweb nicoweb force-pushed the nicoweb:feature/add-link-to-services branch from 6dd8ce5 to 0238846 Mar 12, 2019

@nicoweb nicoweb force-pushed the nicoweb:feature/add-link-to-services branch from 0238846 to 847a50b Mar 12, 2019

return $anchorText;
}
$fileLink = $this->fileLinkFormatter->format($r->getFileName(), $r->getStartLine());

This comment has been minimized.

@chalasr

chalasr Mar 23, 2019

Member

$this->fileLinkFormatter can be null

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.