Merge pull request #17 from ndeet/btcpay-store-selection

Make sure the user needs to select store.
btcpayserver · Jun 22, 2023 · 86b24c0 · 86b24c0
2 parents d9b8a5a + 4c340ee
commit 86b24c0
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/Helper/Data.php b/Helper/Data.php
@@ -291,8 +291,11 @@ public function getGenerateApiKeyUrl(int $magentoStoreId)
 
         $applicationIdentifier = 'magento2';
         $baseUrl = $this->btcPayService->getBtcPayServerBaseUrl();
+        $authorizeUrl = null;
 
-        $authorizeUrl = \BTCPayServer\Client\ApiKey::getAuthorizeUrl($baseUrl, \Storefront\BTCPay\Helper\Data::REQUIRED_API_PERMISSIONS, 'Magento 2 @ ' . $magentoRootDomain, true, false, $redirectToUrlAfterCreation, $applicationIdentifier);
+        if ($baseUrl) {
+            $authorizeUrl = \BTCPayServer\Client\ApiKey::getAuthorizeUrl($baseUrl, \Storefront\BTCPay\Helper\Data::REQUIRED_API_PERMISSIONS, 'Magento 2 @ ' . $magentoRootDomain, true, true, $redirectToUrlAfterCreation, $applicationIdentifier);
+        }
 
         return $authorizeUrl;
     }

diff --git a/Model/Config/ApiKeyComment.php b/Model/Config/ApiKeyComment.php
@@ -68,10 +68,10 @@ public function getCommentText($elementValue)
 
             $baseUrl = $this->btcPayService->getBtcPayServerBaseUrl();
             if ($baseUrl) {
-                $authorizeUrl = \BTCPayServer\Client\ApiKey::getAuthorizeUrl($baseUrl, \Storefront\BTCPay\Helper\Data::REQUIRED_API_PERMISSIONS, 'Magento 2 @ ' . $magentoRootDomain, true, false, $redirectToUrlAfterCreation, $applicationIdentifier);
+                $authorizeUrl = \BTCPayServer\Client\ApiKey::getAuthorizeUrl($baseUrl, \Storefront\BTCPay\Helper\Data::REQUIRED_API_PERMISSIONS, 'Magento 2 @ ' . $magentoRootDomain, true, true, $redirectToUrlAfterCreation, $applicationIdentifier);
                 $r = '<a target="_blank" href="' . $authorizeUrl . '">Generate API key</a>, but be sure to save any changes first.';
             } else {
-                $r = 'Make sure you configure the <strong>VTCPay Base Url</strong> above';
+                $r = 'Make sure you configure the <strong>BTCPay Base Url</strong> above';
             }
         }
         return $r;