Merge branch '3.4' into 4.0

* 3.4:
  Fixed typo
  Improved variable naming

Author: javiereguiluz

best_practices/business-logic.rst

@@ -160,7 +160,7 @@ looking for mapping information::
      */
     class Post
     {
-        const NUM_ITEMS = 10;
+        const NUMBER_OF_ITEMS = 10;
 
         /**
          * @ORM\Id

components/form.rst

@@ -309,7 +309,7 @@ Your integration with the Validation component will look something like this::
     use Symfony\Component\Validator\Validation;
 
     $vendorDirectory = realpath(__DIR__.'/../vendor');
-    $vendorFormDirectory = $vendorDir.'/symfony/form';
+    $vendorFormDirectory = $vendorDirectory.'/symfony/form';
     $vendorValidatorDirectory = $vendorDirectory.'/symfony/validator';
 
     // creates the validator - details will vary

components/serializer.rst

@@ -447,14 +447,14 @@ and :class:`Symfony\\Component\\Serializer\\Normalizer\\PropertyNormalizer`::
 
     $serializer = new Serializer(array($normalizer), array(new JsonEncoder()));
 
-    $obj = new Company();
-    $obj->name = 'Acme Inc.';
-    $obj->address = '123 Main Street, Big City';
+    $company = new Company();
+    $company->name = 'Acme Inc.';
+    $company->address = '123 Main Street, Big City';
 
-    $json = $serializer->serialize($obj, 'json');
+    $json = $serializer->serialize($company, 'json');
     // {"org_name": "Acme Inc.", "org_address": "123 Main Street, Big City"}
-    $objCopy = $serializer->deserialize($json, Company::class, 'json');
-    // Same data as $obj
+    $companyCopy = $serializer->deserialize($json, Company::class, 'json');
+    // Same data as $company
 
 .. _using-camelized-method-names-for-underscored-attributes:
 

controller/soap_web_service.rst

@@ -99,7 +99,7 @@ Below is an example calling the service using a `NuSOAP`_ client. This example
 assumes that the ``index()`` method in the controller above is accessible via
 the route ``/soap``::
 
-    $soapClient = new \Soapclient('http://example.com/index.php/soap?wsdl');
+    $soapClient = new \SoapClient('http://example.com/index.php/soap?wsdl');
 
     $result = $soapClient->call('hello', array('name' => 'Scott'));
 

form/unit_testing.rst

@@ -129,20 +129,20 @@ make sure the ``FormRegistry`` uses the created instance::
 
     class TestedTypeTest extends TypeTestCase
     {
-        private $entityManager;
+        private $objectManager;
 
         protected function setUp()
         {
             // mock any dependencies
-            $this->entityManager = $this->createMock(ObjectManager::class);
+            $this->objectManager = $this->createMock(ObjectManager::class);
 
             parent::setUp();
         }
 
         protected function getExtensions()
         {
             // create a type instance with the mocked dependencies
-            $type = new TestedType($this->entityManager);
+            $type = new TestedType($this->objectManager);
 
             return array(
                 // register the type instances with the PreloadedExtension

routing/custom_route_loader.rst

@@ -95,12 +95,12 @@ and configure the service and method to call:
         // app/config/routing.php
         use Symfony\Component\Routing\RouteCollection;
 
-        $collection = new RouteCollection();
-        $collection->addCollection(
+        $routes = new RouteCollection();
+        $routes->addCollection(
             $loader->import("admin_route_loader:loadRoutes", "service")
         );
 
-        return $collection;
+        return $routes;
 
 In this example, the routes are loaded by calling the ``loadRoutes()`` method of
 the service whose ID is ``admin_route_loader``. Your service doesn't have to

security/custom_password_authenticator.rst

@@ -52,9 +52,9 @@ the user::
                 throw new CustomUserMessageAuthenticationException('Invalid username or password');
             }
 
-            $passwordValid = $this->encoder->isPasswordValid($user, $token->getCredentials());
+            $isPasswordValid = $this->encoder->isPasswordValid($user, $token->getCredentials());
 
-            if ($passwordValid) {
+            if ($isPasswordValid) {
                 $currentHour = date('G');
                 if ($currentHour < 14 || $currentHour > 16) {
                     // CAUTION: this message will be returned to the client
@@ -132,7 +132,7 @@ inside of it.
 
 Inside this method, the password encoder is needed to check the password's validity::
 
-    $passwordValid = $this->encoder->isPasswordValid($user, $token->getCredentials());
+    $isPasswordValid = $this->encoder->isPasswordValid($user, $token->getCredentials());
 
 This is a service that is already available in Symfony and it uses the password algorithm
 that is configured in the security configuration (e.g. ``security.yaml``) under

security/expressions.rst

@@ -76,11 +76,11 @@ Additionally, you have access to a number of functions inside the expression:
         use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;
         // ...
 
-        public function index(AuthorizationCheckerInterface $auth)
+        public function index(AuthorizationCheckerInterface $authorizationChecker)
         {
-            $access1 = $auth->isGranted('IS_AUTHENTICATED_REMEMBERED');
+            $access1 = $authorizationChecker->isGranted('IS_AUTHENTICATED_REMEMBERED');
 
-            $access2 = $auth->isGranted(new Expression(
+            $access2 = $authorizationChecker->isGranted(new Expression(
                 'is_remember_me() or is_fully_authenticated()'
             ));
         }

security/guard_authentication.rst

@@ -427,8 +427,8 @@ Customizing Error Messages
 --------------------------
 
 When ``onAuthenticationFailure()`` is called, it is passed an ``AuthenticationException``
-that describes *how* authentication failed via its ``$e->getMessageKey()`` (and
-``$e->getMessageData()``) method. The message will be different based on *where*
+that describes *how* authentication failed via its ``$exception->getMessageKey()`` (and
+``$exception->getMessageData()``) method. The message will be different based on *where*
 authentication fails (i.e. ``getUser()`` versus ``checkCredentials()``).
 
 But, you can easily return a custom message by throwing a

service_container/shared.rst

@@ -37,5 +37,5 @@ in your service definition:
         $container->register(SomeNonSharedService::class)
             ->setShared(false);
 
-Now, whenever you request an the ``App\SomeNonSharedService`` from the container,
+Now, whenever you request the ``App\SomeNonSharedService`` from the container,
 you will be passed a new instance.