Skip to content

Commit

Permalink
merged branch egulias/master (PR #4658)
Browse files Browse the repository at this point in the history
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 4bfb6fd into 5a8e206).
  • Loading branch information
fabpot committed Jun 26, 2012
2 parents 5a8e206 + 4bfb6fd commit d131f9d
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ class ContainerDebugCommand extends ContainerAwareCommand
/**
* @var ContainerBuilder
*/
private $containerBuilder;
protected $containerBuilder;

/**
* @see Command
Expand Down Expand Up @@ -209,7 +209,7 @@ protected function getContainerBuilder()
*
* @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);
Expand Down

0 comments on commit d131f9d

Please sign in to comment.