From d61ff273b4c21ac06f20865fada4d33bfc2aae0f Mon Sep 17 00:00:00 2001 From: Richard Perez Date: Tue, 8 Dec 2015 08:45:34 -0500 Subject: [PATCH 1/2] Replace filters by formatters According to the context the correct word to use is "formatters" instead of "filters". --- components/dependency_injection/configurators.rst | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/components/dependency_injection/configurators.rst b/components/dependency_injection/configurators.rst index 02ccf800af4..5991fdfd608 100644 --- a/components/dependency_injection/configurators.rst +++ b/components/dependency_injection/configurators.rst @@ -114,9 +114,9 @@ to create a configurator class to configure these instances:: // ... } -The ``EmailConfigurator``'s job is to inject the enabled filters into ``NewsletterManager`` +The ``EmailConfigurator``'s job is to inject the enabled formatters into ``NewsletterManager`` and ``GreetingCardManager`` because they are not aware of where the enabled -filters come from. In the other hand, the ``EmailFormatterManager`` holds +formatters come from. In the other hand, the ``EmailFormatterManager`` holds the knowledge about the enabled formatters and how to load them, keeping the single responsibility principle. From ed92882529b1c8e7b27618048b27d557ec6f552d Mon Sep 17 00:00:00 2001 From: Richard Perez Date: Tue, 8 Dec 2015 08:56:16 -0500 Subject: [PATCH 2/2] Typo in a phrase In this context the correct phrase is "On the other hand" instead of "In the other hand", so just replace "I" by "O". --- components/dependency_injection/configurators.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/components/dependency_injection/configurators.rst b/components/dependency_injection/configurators.rst index 5991fdfd608..8e308eaa6f1 100644 --- a/components/dependency_injection/configurators.rst +++ b/components/dependency_injection/configurators.rst @@ -116,7 +116,7 @@ to create a configurator class to configure these instances:: The ``EmailConfigurator``'s job is to inject the enabled formatters into ``NewsletterManager`` and ``GreetingCardManager`` because they are not aware of where the enabled -formatters come from. In the other hand, the ``EmailFormatterManager`` holds +formatters come from. On the other hand, the ``EmailFormatterManager`` holds the knowledge about the enabled formatters and how to load them, keeping the single responsibility principle.