Fix unintended API change

`Client::$etag` and `Client::$last_modified` ought always to be strings per their docblocks, but the code sets them to null when not present in the response since commit c69c97a.
nicolus · Sep 9, 2020 · cb6af68 · cb6af68
1 parent 9c5828e
commit cb6af68
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/PicoFeed/Client/Client.php b/lib/PicoFeed/Client/Client.php
@@ -287,8 +287,8 @@ protected function handleNotModifiedResponse(ResponseInterface $response)
             $this->is_modified = false;
         } elseif ($response->getStatusCode() == 200) {
             $this->is_modified = $this->hasBeenModified($response, $this->etag, $this->last_modified);
-            $this->etag = $response->getHeader('ETag')[0] ?? null;
-            $this->last_modified = $response->getHeader('Last-Modified')[0] ?? null;
+            $this->etag = $response->getHeader('ETag')[0] ?? '';
+            $this->last_modified = $response->getHeader('Last-Modified')[0] ?? '';
         }
 
         if ($this->is_modified === false) {