CS fixes.

Author: paul-m

form_api_example/src/Plugin/Block/SimpleFormBlock.php

@@ -27,14 +27,14 @@ class SimpleFormBlock extends BlockBase implements ContainerFactoryPluginInterfa
    *
    * @var \Drupal\Core\Form\FormBuilderInterface
    */
-  protected $form_builder;
+  protected $formBuilder;
 
   /**
    * {@inheritdoc}
    */
   public function __construct(array $configuration, $plugin_id, $plugin_definition, FormBuilderInterface $form_builder) {
     parent::__construct($configuration, $plugin_id, $plugin_definition);
-    $this->form_builder = $form_builder;
+    $this->formBuilder = $form_builder;
   }
 
   /**
@@ -55,7 +55,7 @@ public static function create(ContainerInterface $container, array $configuratio
   public function build() {
     $output = [
       'description' => [
-        '#markup' => $this->t('Using form provided by ' . SimpleForm::class),
+        '#markup' => $this->t('Using form provided by @classname', ['@classname' => SimpleForm::class]),
       ],
     ];
 
@@ -66,7 +66,7 @@ public function build() {
     //
     // In this case the build() method of a block plugin is expected to return
     // a render array so we add the form to the existing output and return it.
-    $output['form'] = $this->form_builder->getForm(SimpleForm::class);
+    $output['form'] = $this->formBuilder->getForm(SimpleForm::class);
     return $output;
   }
 

form_api_example/tests/src/Functional/SimpleFormBlockTest.php

@@ -2,7 +2,6 @@
 
 namespace Drupal\Tests\form_api_example\Functional;
 
-use Drupal\Core\Url;
 use Drupal\Tests\BrowserTestBase;
 
 /**

session_example/src/Form/SessionExampleForm.php

@@ -2,11 +2,11 @@
 
 namespace Drupal\session_example\Form;
 
+use Drupal\Core\Cache\CacheTagsInvalidatorInterface;
 use Drupal\Core\Form\FormBase;
 use Drupal\Core\Form\FormStateInterface;
 use Drupal\Core\Link;
 use Symfony\Component\DependencyInjection\ContainerInterface;
-use Drupal\Core\Cache\CacheTagsInvalidatorInterface;
 use Symfony\Component\HttpFoundation\Session\SessionInterface;
 
 /**
@@ -37,17 +37,19 @@ class SessionExampleForm extends FormBase {
    *
    * @var \Drupal\Core\Cache\CacheTagsInvalidatorInterface
    */
-  protected $cache_tag_invalidator;
+  protected $cacheTagInvalidator;
 
   /**
    * Constructs a new SessionExampleForm object.
    *
-   * @param \Symfony\Component\HttpFoundation\Session\SessionInterface
+   * @param \Symfony\Component\HttpFoundation\Session\SessionInterface $session
    *   The session object.
+   * @param \Drupal\Core\Cache\CacheTagsInvalidatorInterface $invalidator
+   *   The cache tag invalidator service.
    */
   public function __construct(SessionInterface $session, CacheTagsInvalidatorInterface $invalidator) {
     $this->session = $session;
-    $this->cache_tag_invalidator = $invalidator;
+    $this->cacheTagInvalidator = $invalidator;
   }
 
   /**
@@ -155,7 +157,7 @@ public function submitForm(array &$form, FormStateInterface $form_state) {
     // Tell the user what happened here, and that they can look at another page
     // to see the result.
     $this->messenger()->addMessage($this->t('The session has been saved successfully. @link', [
-      '@link' => Link::createFromRoute('Check here.', 'session_example.view')->toString()
+      '@link' => Link::createFromRoute('Check here.', 'session_example.view')->toString(),
     ]));
     // Since we might have changed the session information, we will invalidate
     // the cache tag for this session.
@@ -188,7 +190,7 @@ public function submitClearSession(array &$form, FormStateInterface $form_state)
    * updates their information in the submit handlers.
    */
   protected function invalidateCacheTag() {
-    $this->cache_tag_invalidator->invalidateTags(['session_example:' . $this->session->getId()]);
+    $this->cacheTagInvalidator->invalidateTags(['session_example:' . $this->session->getId()]);
   }
 
 }

session_example/tests/src/Functional/SessionExampleTest.php

@@ -17,6 +17,9 @@ class SessionExampleTest extends BrowserTestBase {
 
   public static $modules = ['session_example', 'block'];
 
+  /**
+   * {@inheritdoc}
+   */
   protected function setUp() {
     parent::setUp();
     // Place our blocks.
@@ -119,6 +122,9 @@ public function testSessionExample() {
     $assert->pageTextContains('No color');
   }
 
+  /**
+   * Ensure the session data does not follow different users around.
+   */
   public function testUserIsolation() {
     $assert = $this->assertSession();
     // Our setUp() method has already logged in a user, so let's add some data.