8000 Added the Symfony default ide file link web view by jeremyFreeAgent · Pull Request #99 · silexphp/Silex-WebProfiler · GitHub
[go: up one dir, main page]

Skip to content
This repository was archived by the owner on Sep 14, 2018. It is now read-only.

Added the Symfony default ide file link web view #99

Merged
merged 1 commit into from
Oct 27, 2016

Conversation

jeremyFreeAgent
Copy link
Contributor

This add the feature added by symfony/symfony#19973

@SpacePossum
Copy link
Contributor

We only know if this works if your SF PR is merged, as the class FileLinkFormatter doesn't exists yet :)
(maybe add a utest for this change?)

@fabpot
Copy link
Member
fabpot commented Oct 9, 2016

I supposed @jeremyFreeAgent tested it manually, right?

@jeremyFreeAgent
Copy link
Contributor Author

Yes I needed to test it manually since the dependencies are not merged yet. If the FileLinkFormatter is not here, it fallbacks on the previous one.

@jeremyFreeAgent jeremyFreeAgent force-pushed the debug_file_link_format branch 2 times, most recently from 02df5f4 to 49739e2 Compare October 25, 2016 13:51
@jeremyFreeAgent
Copy link
Contributor Author

I've pushed the changes to reflect the ones made in the Symfony PR.

}
$baseDir[] = $rootDir[$i];
}
$baseDir = implode(DIRECTORY_SEPARATOR, $baseDir);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you move this into a private method?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done!

@fabpot
Copy link
Member
fabpot commented Oct 27, 2016

Thank you @jeremyFreeAgent.

@fabpot fabpot merged commit 7e7f062 into silexphp:master Oct 27, 2016
fabpot added a commit that referenced this pull request Oct 27, 2016
…reeAgent)

This PR was merged into the 2.0.x-dev branch.

Discussion
----------

Added the Symfony default ide file link web view

This add the feature added by symfony/symfony#19973

Commits
-------

7e7f062 Added the Symfony default ide file link web view
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0