Merge pull request #36 from SynergyWholesale/develop

Develop -> Master (v2.1.9)

Author: Cameron Hall

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

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"