Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Nov 25, 2020. It is now read-only.

Twig extension as parameter in service file #108

Closed
wants to merge 1 commit into from

Conversation

rlavolee
Copy link

  • twig extension as parameter
  • allow easy class overriding

- allow easy class overriding
@richsage
Copy link
Contributor

@pablodip see this - thoughts going forward?

@pablodip
Copy link
Contributor

"When someone wants to change a service, just changing the class name is rarely enough: the new service probably have some different constructor arguments, ... Or put another way, overriding a service by just changing the class name of a service is a very rare use case."

Yeah, it makes no sense to override a service only by changing the class.

@pablodip pablodip closed this Mar 30, 2015
@rlavolee
Copy link
Author

According to the Symfony's doc, it's relevant

@pablodip
Copy link
Contributor

It's going to be changed: symfony/symfony#11881

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