Skip to content

Commit

Permalink
minor #3770 Update factories.rst (AlaaAttya)
Browse files Browse the repository at this point in the history
This PR was submitted for the 2.4 branch but it was merged into the 2.3 branch instead (closes #3770).

Discussion
----------

Update factories.rst

your factory method should be static to work!

Commits
-------

97ed5ce Update factories.rst
aefc36c Update factories.rst
a9cfdb2 Update factories.rst
  • Loading branch information
weaverryan committed May 2, 2014
2 parents 14b9f14 + c818065 commit 127beed
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions components/dependency_injection/factories.rst
Original file line number Diff line number Diff line change
Expand Up @@ -79,8 +79,8 @@ class:
When you specify the class to use for the factory (via ``factory_class``)
the method will be called statically. If the factory itself should be instantiated
and the resulting object's method called (as in this example), configure the
factory itself as a service:
and the resulting object's method called, configure the factory itself as a service.
In this case, the method (e.g. get) should be changed to be non-static:

.. configuration-block::

Expand Down

0 comments on commit 127beed

Please sign in to comment.