Apply suggestions/improvements from code review.

Co-Authored-By: ricknox <[email protected]>

Author: jmsche

best_practices/templates.rst

@@ -126,7 +126,7 @@ class in the constructor of the Twig extension::
                 new TwigFilter(
                     'md2html',
                     [$this, 'markdownToHtml'],
-                    ['is_safe' => ['html'], 'pre_escape' => 'html')
+                    ['is_safe' => ['html'], 'pre_escape' => 'html']
                 ),
             );
         }

components/cache/cache_invalidation.rst

@@ -36,7 +36,7 @@ cache items, as returned by cache adapters::
     // ...
     // add one or more tags
     $item->tag('tag_1');
-    $item->tag(['tag_2', 'tag_3'[);
+    $item->tag(['tag_2', 'tag_3']);
     $cache->save($item);
 
 If ``$cache`` implements :class:`Symfony\\Component\\Cache\\Adapter\\TagAwareAdapterInterface`,

components/ldap.rst

@@ -65,7 +65,7 @@ Or you could directly specify a connection string::
 
     use Symfony\Component\Ldap\Ldap;
 
-    $ldap = Ldap::create('ext_ldap', ['connection_string' => 'ldaps://my-server:636'));
+    $ldap = Ldap::create('ext_ldap', ['connection_string' => 'ldaps://my-server:636']);
 
 The :method:`Symfony\\Component\\Ldap\\Ldap::bind` method
 authenticates a previously configured connection using both the

components/property_access.rst

@@ -178,7 +178,7 @@ The ``getValue()`` method can also use the magic ``__get()`` method::
     {
         private $children = [
             'Wouter' => [...],
-        );
+        ];
 
         public function __get($id)
         {

components/translation/usage.rst

@@ -198,7 +198,7 @@ recommended format. These files are parsed by one of the loader classes.
                 'user' => [
                     'login' => 'Login',
                 ],
-            );
+            ];
 
     The multiple levels are flattened into single id/translation pairs by
     adding a dot (``.``) between every level, therefore the above examples are
@@ -431,7 +431,7 @@ The ``$messages`` variable will have the following structure::
             'Value should not be empty' => 'Valeur ne doit pas être vide',
             'Value is too long' => 'Valeur est trop long',
         ],
-    );
+    ];
 
 Adding Notes to Translation Contents
 ------------------------------------

contributing/community/review-comments.rst

@@ -113,7 +113,7 @@ If a piece of code is in fact wrong, explain why:
 
     * I profiled this change and it hurts performance significantly (if you don't profile, it's an opinion, so we can ignore)
 
-    * Code doesn't match Symfony's CS rules (e.g. use ``[]`` instead of ``[]``)
+    * Code doesn't match Symfony's CS rules (e.g. use ``[]`` instead of ``array()``)
 
     * We only provide integration with very popular projects (e.g. we integrate Bootstrap but not your own CSS framework)
 

create_framework/http_kernel_controller_resolver.rst

@@ -75,7 +75,7 @@ two colons and a method name as a valid callback, like 'class::method'::
     $routes->add('leap_year', new Routing\Route('/is_leap_year/{year}', [
         'year' => null,
         '_controller' => 'LeapYearController::indexAction',
-    ])));
+    ]));
 
 To make this code work, modify the framework code to use the controller
 resolver from HttpKernel::

doctrine/mongodb_session_storage.rst

@@ -94,7 +94,7 @@ need to change/add some parameters in the main configuration file:
                 ['mongodb://%mongodb_username%:%mongodb_password%@%mongodb_host%:27017'],
                 // if not using a username and password
                 ['mongodb://%mongodb_host%:27017'],
-            ));
+            ]);
 
         $container->register('session.handler.mongo', MongoDbSessionHandler::class)
             ->setArguments([