diff --git a/src/FacetManager/DefaultFacetManager.php b/src/FacetManager/DefaultFacetManager.php index 728e957..c2dab74 100644 --- a/src/FacetManager/DefaultFacetManager.php +++ b/src/FacetManager/DefaultFacetManager.php @@ -15,7 +15,6 @@ use Drupal\facets\Processor\ProcessorInterface; use Drupal\facets\Processor\ProcessorPluginManager; use Drupal\facets\Processor\WidgetOrderProcessorInterface; use Drupal\facets\QueryType\QueryTypePluginManager; -use Drupal\facets\Result\Result; use Drupal\facets\Widget\WidgetPluginManager; /** @@ -297,13 +296,14 @@ class DefaultFacetManager { foreach ($facet->getProcessorsByStage(ProcessorInterface::STAGE_BUILD) as $processor) { /** @var \Drupal\facets\Processor\BuildProcessorInterface $build_processor */ $build_processor = $this->processorPluginManager->createInstance($processor->getPluginDefinition()['id'], ['facet' => $facet]); - if($build_processor instanceof WidgetOrderProcessorInterface){ + if ($build_processor instanceof WidgetOrderProcessorInterface) { // Sorting is handled last and together, to support nested sorts. // @todo For some reason getProcessorsByStage does not return the // correct configuration, thats why we use getProcessors() below. $processors = $facet->getProcessors(); $active_sorts[] = $processors[$build_processor->getPluginId()]; - }else{ + } + else { if (!$build_processor instanceof BuildProcessorInterface) { throw new InvalidProcessorException(new FormattableMarkup("The processor @processor has a build definition but doesn't implement the required BuildProcessorInterface interface", ['@processor' => $processor->getPluginDefinition()['id']])); } diff --git a/tests/src/Unit/Plugin/processor/CountWidgetOrderProcessorTest.php b/tests/src/Unit/Plugin/processor/CountWidgetOrderProcessorTest.php index 2ff8ab8..91427a1 100644 --- a/tests/src/Unit/Plugin/processor/CountWidgetOrderProcessorTest.php +++ b/tests/src/Unit/Plugin/processor/CountWidgetOrderProcessorTest.php @@ -5,7 +5,6 @@ namespace Drupal\Tests\facets\Unit\Plugin\processor; use Drupal\facets\Plugin\facets\processor\CountWidgetOrderProcessor; use Drupal\facets\Result\Result; use Drupal\Tests\UnitTestCase; -use Symfony\Component\DependencyInjection\ContainerBuilder; /** * Unit test for processor. diff --git a/tests/src/Unit/Plugin/processor/DisplayValueWidgetOrderProcessorTest.php b/tests/src/Unit/Plugin/processor/DisplayValueWidgetOrderProcessorTest.php index 9ef9f51..883ad13 100644 --- a/tests/src/Unit/Plugin/processor/DisplayValueWidgetOrderProcessorTest.php +++ b/tests/src/Unit/Plugin/processor/DisplayValueWidgetOrderProcessorTest.php @@ -84,7 +84,6 @@ class DisplayValueWidgetOrderProcessorTest extends UnitTestCase { $this->assertEquals(0, $result_count); } - /** * Tests that sorting uses the display value. */ diff --git a/tests/src/Unit/Plugin/processor/RawValueWidgetOrderProcessorTest.php b/tests/src/Unit/Plugin/processor/RawValueWidgetOrderProcessorTest.php index 95409f9..833d1c5 100644 --- a/tests/src/Unit/Plugin/processor/RawValueWidgetOrderProcessorTest.php +++ b/tests/src/Unit/Plugin/processor/RawValueWidgetOrderProcessorTest.php @@ -11,7 +11,6 @@ use Drupal\Tests\UnitTestCase; * * @group facets */ - class RawValueWidgetOrderProcessorTest extends UnitTestCase { /** @@ -80,4 +79,5 @@ class RawValueWidgetOrderProcessorTest extends UnitTestCase { $config = $this->processor->defaultConfiguration(); $this->assertEquals(['sort' => 'ASC'], $config); } + }