Issue #2945501 by borisson_: Fix coding standards

facets.module

@@ -8,9 +8,9 @@
 use Drupal\Component\Utility\Html;
 use Drupal\Core\Breadcrumb\Breadcrumb;
 use Drupal\Core\Form\FormStateInterface;
-use Drupal\Core\Language\LanguageInterface;
 use Drupal\Core\Link;
 use Drupal\Core\Routing\RouteMatchInterface;
+use Drupal\Core\Url;
 use Drupal\facets\Entity\Facet;
 use Drupal\facets\Entity\FacetSource;
 use Drupal\facets\FacetInterface;
@@ -355,9 +355,9 @@ function facets_system_breadcrumb_alter(Breadcrumb &$breadcrumb, RouteMatchInter
 }
 
 /**
- * Implements hook_language_switch_links_alter
+ * Implements hook_language_switch_links_alter().
  */
-function facets_language_switch_links_alter(array &$links, $type, \Drupal\Core\Url $url) {
+function facets_language_switch_links_alter(array &$links, $type, Url $url) {
   /** @var \Drupal\Core\Language\LanguageManagerInterface $languageManager */
   $languageManager = \Drupal::languageManager();
   $language_interface = $languageManager->getCurrentLanguage();

modules/facets_range_widget/config/schema/facets_range_widget.schema.yml

@@ -51,4 +51,3 @@ facet.widget.config.range_slider:
     step:
       type: float
       label: 'Step'
-

modules/facets_rest/tests/src/Functional/RestIntegrationTest.php

@@ -3,7 +3,6 @@
 namespace Drupal\Tests\facets_rest\Functional;
 
 use Drupal\Tests\facets\Functional\FacetsTestBase;
-use Drupal\views\Entity\View;
 use Symfony\Component\DependencyInjection\ContainerInterface;
 
 /**

modules/facets_summary/src/Annotation/SummaryProcessor.php

@@ -58,6 +58,8 @@ class SummaryProcessor extends Plugin {
    *
    * @var bool
    */
+  // @codingStandardsIgnoreStart
   public $default_enabled;
+  // @codingStandardsIgnoreEnd
 
 }

src/Annotation/FacetsFacetSource.php

@@ -46,6 +46,8 @@ class FacetsFacetSource extends Plugin {
    *
    * @var string
    */
+  // @codingStandardsIgnoreStart
   public $display_id;
+  // @codingStandardsIgnoreEnd
 
 }

src/Annotation/FacetsProcessor.php

@@ -65,6 +65,8 @@ class FacetsProcessor extends Plugin {
    *
    * @var bool
    */
+  // @codingStandardsIgnoreStart
   public $default_enabled;
+  // @codingStandardsIgnoreEnd
 
 }

src/FacetManager/DefaultFacetManager.php

@@ -326,7 +326,9 @@ class DefaultFacetManager {
               'class' => 'facet-empty',
             ],
             'empty_text' => [
+              // @codingStandardsIgnoreStart
               '#markup' => $this->t($empty_behavior['text']),
+              // @codingStandardsIgnoreEnd
             ],
           ],
         ];

tests/facets_custom_widget/src/Plugin/facets/processor/InvalidQT.php

@@ -33,4 +33,5 @@ class InvalidQT extends ProcessorPluginBase implements PreQueryProcessorInterfac
   public function preQuery(FacetInterface $facet) {
     // This can be empty for this test implementation.
   }
+
 }

tests/src/Unit/Plugin/processor/ShowOnlyDeepestLevelItemsProcessorTest.php

@@ -16,6 +16,8 @@ use Drupal\Tests\UnitTestCase;
 class ShowOnlyDeepestLevelItemsProcessorTest extends UnitTestCase {
 
   /**
+   * The processor under test.
+   *
    * @var \Drupal\facets\Plugin\facets\processor\ShowOnlyDeepestLevelItemsProcessor
    */
   protected $processor;

tests/src/Unit/Plugin/url_processor/QueryStringTest.php

@@ -418,6 +418,8 @@ class QueryStringTest extends UnitTestCase {
 namespace Drupal\facets\Plugin\facets\url_processor;
 
 /**
+ * Mocks the usage of drupal static.
+ *
  * @see \drupal_static
  */
 function &drupal_static($name, $default_value = NULL, $reset = FALSE) {