#178 Error when specified env vars are missing

Author: nfourtythree

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Release Notes for Shopify
 
+## Unreleased
+
+- Fixed a PHP error that could occur with missing environment variables. ([#178](https://github.com/craftcms/shopify/issues/178))
+
 ## 6.0.5 - 2025-09-09
 
 - Fixed a bug where Shopify data could be overwritten when syncing products. ([#177](https://github.com/craftcms/shopify/issues/177))

src/models/Settings.php

@@ -140,7 +140,7 @@ public function setApiVersion(string $apiVersion): void
      */
     public function getApiVersion(bool $parse = true): string
     {
-        return $parse ? App::parseEnv($this->_apiVersion) : $this->_apiVersion;
+        return ($parse ? App::parseEnv($this->_apiVersion) : $this->_apiVersion) ?? '';
     }
 
     /**
@@ -160,7 +160,7 @@ public function setApiKey(string $apiKey): void
      */
     public function getApiKey(bool $parse = true): string
     {
-        return $parse ? App::parseEnv($this->_apiKey) : $this->_apiKey;
+        return ($parse ? App::parseEnv($this->_apiKey) : $this->_apiKey) ?? '';
     }
 
     /**
@@ -180,7 +180,7 @@ public function setApiSecretKey(string $apiSecretKey): void
      */
     public function getApiSecretKey(bool $parse = true): string
     {
-        return $parse ? App::parseEnv($this->_apiSecretKey) : $this->_apiSecretKey;
+        return ($parse ? App::parseEnv($this->_apiSecretKey) : $this->_apiSecretKey) ?? '';
     }
 
     /**
@@ -200,7 +200,7 @@ public function setHostName(string $hostName): void
      */
     public function getHostName(bool $parse = true): string
     {
-        return $parse ? App::parseEnv($this->_hostName) : $this->_hostName;
+        return ($parse ? App::parseEnv($this->_hostName) : $this->_hostName) ?? '';
     }
 
     /**
@@ -220,7 +220,7 @@ public function setAccessToken(string $accessToken): void
      */
     public function getAccessToken(bool $parse = true): string
     {
-        return $parse ? App::parseEnv($this->_accessToken) : $this->_accessToken;
+        return ($parse ? App::parseEnv($this->_accessToken) : $this->_accessToken) ?? '';
     }
 
     /**