Refactor code to use hasHeader & getHeader methods

jorenvh · jorenvh · commit 721f1948816a · 2021-10-08T09:54:27.000+02:00
diff --git a/src/Api/ProductMediaFileApi.php b/src/Api/ProductMediaFileApi.php
@@ -130,14 +130,12 @@ public function download(string $code): ResponseInterface
      */
     protected function extractCodeFromCreationResponse(ResponseInterface $response)
     {
-        $headers = $response->getHeaders();
-
-        if (isset($headers['Location'][0])) {
-            $locationHeader = $headers['Location'][0];
+        if ($response->hasHeader('Location')) {
+            $locationHeader = $response->getHeader('Location')[0];
         }
 
-        if (isset($headers['location'][0])) {
-            $locationHeader = $headers['location'][0];
+        if ($response->hasHeader('location')) {
+            $locationHeader = $response->getHeader('location')[0];
         }
 
         if (!isset($locationHeader)) {

Original file line number	Diff line number	Diff line change
`@@ -130,14 +130,12 @@ public function download(string $code): ResponseInterface`
`130`	`130`	`*/`
`131`	`131`	`protected function extractCodeFromCreationResponse(ResponseInterface $response)`
`132`	`132`	`{`
`133`		`- $headers = $response->getHeaders();`
`134`		`-`
`135`		`- if (isset($headers['Location'][0])) {`
`136`		`- $locationHeader = $headers['Location'][0];`
	`133`	`+ if ($response->hasHeader('Location')) {`
	`134`	`+ $locationHeader = $response->getHeader('Location')[0];`
`137`	`135`	`}`
`138`	`136`
`139`		`- if (isset($headers['location'][0])) {`
`140`		`- $locationHeader = $headers['location'][0];`
	`137`	`+ if ($response->hasHeader('location')) {`
	`138`	`+ $locationHeader = $response->getHeader('location')[0];`
`141`	`139`	`}`
`142`	`140`
`143`	`141`	`if (!isset($locationHeader)) {`