formatted code

solariumphp · Jan 3, 2024 · 9df8626 · 9df8626
1 parent e5b9251
commit 9df8626
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 13 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,10 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
+### Added
+- Option `buildAll` for Suggesters
+- NoResponseRequest Plugin
+
 ### Fixed
 - PHP 8.2 deprecations for Solarium\QueryType\Server\Collections results
 

diff --git a/src/Component/ComponentTraits/SuggesterTrait.php b/src/Component/ComponentTraits/SuggesterTrait.php
@@ -145,7 +145,7 @@ public function getReload(): ?bool
      */
     public function setBuildAll(bool $buildAll): SuggesterInterface
     {
-      return $this->setOption('buildAll', $buildAll);
+        return $this->setOption('buildAll', $buildAll);
     }
 
     /**
@@ -155,6 +155,6 @@ public function setBuildAll(bool $buildAll): SuggesterInterface
      */
     public function getBuildAll(): ?bool
     {
-      return $this->getOption('buildAll');
+        return $this->getOption('buildAll');
     }
 }
diff --git a/src/Plugin/NoResponseRequest.php b/src/Plugin/NoResponseRequest.php
@@ -71,16 +71,16 @@ public function preExecuteRequest($event): self
      */
     public function postExecuteRequest($event): self
     {
-      if ($this->client->getAdapter() instanceof TimeoutAwareInterface) {
-          // Restore the previous timeout.
-          $this->client->getAdapter()->setTimeout($this->getTimeout());
-      }
+        if ($this->client->getAdapter() instanceof TimeoutAwareInterface) {
+            // Restore the previous timeout.
+            $this->client->getAdapter()->setTimeout($this->getTimeout());
+        }
 
-      if ($this->client->getAdapter() instanceof Curl) {
-        $this->client->getAdapter()->setOption('return_transfer', true);
-      }
+        if ($this->client->getAdapter() instanceof Curl) {
+            $this->client->getAdapter()->setOption('return_transfer', true);
+        }
 
-      return $this;
+        return $this;
     }
 
     /**

diff --git a/tests/Plugin/NoResponseRequestTest.php b/tests/Plugin/NoResponseRequestTest.php
@@ -12,7 +12,6 @@
 use Solarium\Core\Event\PostExecuteRequest;
 use Solarium\Core\Event\PreExecuteRequest as PreExecuteRequestEvent;
 use Solarium\Plugin\NoResponseRequest;
-use Solarium\Plugin\PostBigRequest;
 use Solarium\QueryType\Select\Query\Query;
 use Solarium\Tests\Integration\TestClientFactory;
 
@@ -98,13 +97,13 @@ public function testExecuteRequest()
         $this->assertSame($requestInput->getQueryString(), $requestOutput->getRawData());
         $this->assertSame('', $requestOutput->getQueryString());
         $this->assertSame(TimeoutAwareInterface::MINIMUM_TIMEOUT, $this->client->getAdapter()->getTimeout());
-        $this->assertSame(false, $this->client->getAdapter()->getOption('return_transfer'));
+        $this->assertFalse($this->client->getAdapter()->getOption('return_transfer'));
 
         $event = new PostExecuteRequest($requestOutput, $endpoint, new Response('response'));
         $this->plugin->preExecuteRequest($event);
 
         $this->assertSame(TimeoutAwareInterface::DEFAULT_TIMEOUT, $this->client->getAdapter()->getTimeout());
-        $this->assertSame(true, $this->client->getAdapter()->getOption('return_transfer'));
+        $this->assertTrue($this->client->getAdapter()->getOption('return_transfer'));
     }
 
     public function testPluginIntegration()