Applying reviews

Author: mtarld

service_container/service_decoration.rst

@@ -313,12 +313,12 @@ The generated code will be the following::
 
     $this->services[Foo::class] = new Baz(new Bar(new Foo()));
 
-Control the behavior when the decorated service does not exist
+Control the Behavior When the Decorated Service Does Not Exist
 --------------------------------------------------------------
 
-Let's imagine you have a decorator that decorates a service defined in another bundle.
+Let's imagine you have a decorator that decorates a service defined by another bundle.
 If for some reasons, that bundle is missing, the ``decoration_on_invalid``
-option will allow you to handle the lack of decorated service.
+option will allow you to handle the lack of the decorated service.
 
 .. configuration-block::
 
@@ -375,18 +375,18 @@ option will allow you to handle the lack of decorated service.
 
 Three different behaviors are available:
 
-* ``exception``: The default behavior. When used, a ``ServiceNotFoundException``
-  will be thrown telling that decorator's dependency is missing.
+* ``exception``: When used, a ``ServiceNotFoundException``
+  will be thrown telling that decorator's dependency is missing. (default)
 * ``ignore``: Using that behavior, the container will remove either decorator
   and decorated services.
 * ``null``: When used, the container will keep the decorator
-  service and will set the decorated one to `null`.
+  service and will set the decorated one to ``null``.
 
 .. tip::
 
     When using ``null``, you may update the decorator's constructor in
     order to make decorated dependency nullable. In this way, you will be able to
-    check the presence of decorated service in other methods.
+    check the presence of the decorated service in other methods.
 
     .. code-block:: yaml
 
@@ -395,7 +395,7 @@ Three different behaviors are available:
             decoration_on_invalid: null
             arguments: ['@App\Service\DecoratorService.inner']
 
-    ::
+    .. code-block: php
 
         namespace App\Service;
 
@@ -405,17 +405,16 @@ Three different behaviors are available:
         {
             private $decorated;
 
-            - public function __construct(OptionalService $decorated)
-            + public function __construct(?OptionalService $decorated)
+            public function __construct(?OptionalService $decorated)
             {
                 $this->decorated = $decorated;
             }
 
             public function tellInterestingStuff(): string
             {
-                + if (!$this->decorated) {
-                +     return 'Just one interesting thing';
-                + }
+                if (!$this->decorated) {
+                    return 'Just one interesting thing';
+                }
 
                 return $this->decorated->tellInterestingStuff().' + one more interesting thing';
             }
@@ -426,6 +425,6 @@ Three different behaviors are available:
     Sometimes, you may want to add a compiler pass that creates service
     definitions on the fly. If you want to decorate such a service,
     be sure that your compiler pass is registered with ``PassConfig::TYPE_BEFORE_OPTIMIZATION``
-    type so that the decoration pass will be able to find created services.
+    type so that the decoration pass will be able to find the created services.
 
 .. _`Decorator pattern`: https://en.wikipedia.org/wiki/Decorator_pattern