Improve description about "framework.ide" config option

[aik099]

According to code in https://github.com/symfony/symfony/blob/master/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php#L179 when both xdebug.file_link_format (in php.ini) and framework.ide (in app/config/config.yml) are defined, then xdebug.file_link_format wins, but documentetion says the opposite.

P.S.

• PR is targeting the oldest supported documentation branch, where fixed text fragment is present.
• It should be merged to 3.3 and 3.4 branches as well.
• The documentation is changed and not mentioned code, because changing code would introduce a BC break.

[ro0NL]

symfony/symfony#24129 might need clarification as well 👍

[aik099]

@ro0NL , I've left a note in that issue. Hope that helps.

[ro0NL]

I think the part as of When running your app in a container or in a virtual machine... should be updated to a config context, we are in fact describing the ide config setting.

Thus instead of

// /path/to/guest/.../file will be opened
// as /path/to/host/.../file on the host
// and /foo/.../file as /bar/.../file also
'myide://%f:%l&/path/to/guest/>/path/to/host/&/foo/>/bar/&...'

I'd say

framework:
    ide: 'myide://%%f:%%l&/path/to/guest/>/path/to/host/&/foo/>/bar/&...'

Thus escaped.

[aik099]

#8405 (comment)

@ro0NL , is this related to changes I've made in this PR?

If so, the I don't understand what you want me to change. If you have push access to this repo, then you already can add more commits to my PR branch.

[ro0NL]

Not really related. I thought it was an issue (not a PR) so i quickly comment as well :) related to the same subject though.

Ill have a look in different PR soonish otherwise. Issue solved, feel free to ignore :)

[wouterj]

👍 Seems correct to me. Thanks for your detailed PR description & patch!

status: reviewed

[xabbuh]

Thank you @aik099.