Merge pull request #36 from SynergyWholesale/develop

Develop -> Master (v2.1.9)
SynergyWholesale · Sep 11, 2020 · 1f5fb61 · 1f5fb61
2 parents f5b621a + c6ddeba
commit 1f5fb61
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -15,6 +15,10 @@ Synergy Wholesale WHMCS Domains Module
 ### Removed
 -
 
+## 2.1.9 [Updated 25/08/2020]
+### Fixed
+- Fixed internal transfers not being renewed upon transfer-in. Fixes [#33](https://github.com/SynergyWholesale/WHMCS-Domains-Module/issues/33)
+
 
 ## 2.1.8 [Updated 27/07/2020]
 ### Added

diff --git a/modules/registrars/synergywholesaledomains/synergywholesaledomains.php b/modules/registrars/synergywholesaledomains/synergywholesaledomains.php
@@ -638,10 +638,13 @@ function synergywholesaledomains_TransferDomain(array $params)
 
     $request = [
         'authInfo' => $params['transfersecret'],
+        'doRenewal' => 1,
     ];
 
-    $canRenew = synergywholesaledomains_apiRequest('domainRenewRequired', $params, $request, false);
-    $request['doRenewal'] = (int) ('on' === $params['doRenewal'] && 'OK_RENEW' === $canRenew['status']);
+    if (preg_match('/\.au$/', $params['sld'])) {
+        $canRenew = synergywholesaledomains_apiRequest('domainRenewRequired', $params, $request, false);
+        $request['doRenewal'] = (int) ('on' === $params['doRenewal'] && 'OK_RENEWAL' === $canRenew['status']);
+    }
 
     /**
      * We don't want to send the idProtect flag with the "can renew"