Whmcs version 8.3, updateclientdomain.php , not work

[persis]

Hi 

Whmcs version 8.3, updateclientdomain.php , not work

 

[steven99]

Provide more info you can ? 

[persis]

2 minutes ago, steven99 said:

Provide more info you can ? 

this code not work with updateclientdomain.php (v8.3) but work with updateclientdomain.php(8.1.2)

 

<?php

use WHMCS\Database\Capsule;

add_hook('AfterCronJob', 1, function($vars)
{
    $adminUsername = ''; // Optional for WHMCS 7.2 and later
    
    foreach (Capsule::table('tbldomains')->pluck('id') as $v)
    {
        localAPI('UpdateClientDomain', array('domainid' => $v, 'autorecalc' => true), $adminUsername);
    }

});

[steph.hope]

Do you mean v8.1.3? Or 8.2?  There's no v8.3 yet.

Edited July 12, 2021 by steph.hope

[persis]

Version:8.2.0 (8.2.0-release.1) not work

Version:8.1.3 (8.1.3-release.1) working

[persis]

please follow : 

 

 

[wsa]

On 7/11/2021 at 10:17 PM, steph.hope said:

There's no v8.3 yet.

Yes they is a 8.3 but in beta here

[pRieStaKos]

You should use a return to see where is the issue:

$tesult = localAPI('UpdateClientDomain', array('domainid' => $v, 'autorecalc' => true), $adminUsername);

var_dump($result);

There will be a “result” and “message” if there is an issue. Or you will get “result: success” and API function is OK.

[persis]

array(2) {
  ["result"]=>
  string(5) "error"
  ["message"]=>
  string(267) "SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'where `id` = ?' at line 1 (SQL: update `tbldomains` set  where `id` = xxx)"
}

 

 

[pRieStaKos]

2 minutes ago, persis said:

array(2) {
  ["result"]=>
  string(5) "error"
  ["message"]=>
  string(267) "SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'where `id` = ?' at line 1 (SQL: update `tbldomains` set  where `id` = xxx)"
}

 

 

Do the same for your query on Capsule. You are not providing id to the API.

[persis]

I manualy hide the ID when copying and pasting for you!

I think whmcs codes in this api cant work with mariadb 10.3! 

I am moving from mariadb to mysql for test...

[persis]

same result with mariadb 10.5

[pRieStaKos]

9 hours ago, persis said:

I manualy hide the ID when copying and pasting for you!

I think whmcs codes in this api cant work with mariadb 10.3! 

I am moving from mariadb to mysql for test...

I’m referring to the sql error. It’s not MariaDB

[WHMCS Alex]

Hi,

This issue has been resolved in version 8.3 of WHMCS: https://docs.whmcs.com/Changelog:WHMCS_V8.3.0_Beta_1

Quote

CORE-16847 - Prevent erroneous SQL error when there is no change regarding autocalc request to UpdateClientDomain

Once version 8.3 reaches stable release tier, updating to it will resolve this.

[persis]

Many Thanks