diogo/symfony
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

merged branch egulias/master (PR #4658)

Browse Source 
Commits
-------

4bfb6fd [FrameworkBundle][Command] ContainerDebugCommand - Changed visibility to allow re use in childs

Discussion
----------

[FrameworkBundle][Command] ContainerDebugCommand

Bug fix: no
Feature addition: no
Backwards compatibility break: no
Symfony2 tests pass: yes
Fixes the following tickets: -
Todo: -
License of the code: MIT
Documentation PR: -

I'm currently working on a command to debug (service) listeners. I wanted to extend the ContainerDebugCommand from the FrameworkBundle but I found that the method resolveServiceDefinition($serviceId) was private so it avoids its usage from children.

This PR changes the visibility so it allows the re-use of this method when extending.

---------------------------------------------------------------------------

by travisbot at 2012-06-26T07:11:31Z

This pull request [passes](http://travis-ci.org/symfony/symfony/builds/1708139) (merged 4bfb6fdd into 5a8e2064).
This commit is contained in:
Fabien Potencier 2012-06-26 09:48:29 +02:00
commit d131f9d32f
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/Symfony/Bundle/FrameworkBundle/Command/ContainerDebugCommand.php
View File

@ -32,7 +32,7 @@ class ContainerDebugCommand extends ContainerAwareCommand
/**
* @var ContainerBuilder
*/
private $containerBuilder;
protected $containerBuilder;
/**
* @see Command
@ -209,7 +209,7 @@ EOF
*
* @return \Symfony\Component\DependencyInjection\Definition|\Symfony\Component\DependencyInjection\Alias
*/
private function resolveServiceDefinition($serviceId)
protected function resolveServiceDefinition($serviceId)
{
if ($this->containerBuilder->hasDefinition($serviceId)) {
return $this->containerBuilder->getDefinition($serviceId);