vultr / whmcs-vultr

Vultr Module for WHMCS

Apache License 2.0

95 stars 52 forks source link

use count to use check return by ->get() #101

Open jazz7381 opened 3 years ago

jazz7381 commented 3 years ago

Description

Related Issues

92

88

Checklist:

[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Have you linted your code locally prior to submission?
[x] Have you successfully ran tests with your changes locally?

jazz7381 commented 3 years ago

i have fixed this issue before but i didn't make a PR. Can anyone test this code cause i don't have access to whmcs anymore

Businezz99 commented 3 years ago

@jazz7381 That works! You saved me a lot of headache! I was looking for a solution for 8.1.x version. Thanks!

SaneChoice commented 3 years ago

@jazz7381 - Thank you from all of us!

jazz7381 commented 3 years ago

glad to help

Businezz99 commented 3 years ago

@jazz7381 It doesn't work in 8.1.1 version now again :(

Businezz99 commented 3 years ago

@SaneChoice Did you update to WHMCS 8.1.1? Are you having issues?

tmunsch commented 3 years ago

I am Still on 8.0.3 and It doesn't work anymore when it did before and i haven't changed anything. I think they are phasing out the API v1 so maybe they are shutting it down. Has anyone started on the API v2 version. I am dying over here lol

Businezz99 commented 3 years ago

I am Still on 8.0.3 and It doesn't work anymore when it did before and i haven't changed anything. I think they are phasing out the API v1 so maybe they are shutting it down. Has anyone started on the API v2 version. I am dying over here lol

I just started using Linode module, i bought one, less headaches to be honest.

jazz7381 commented 3 years ago

@jazz7381 It doesn't work in 8.1.1 version now again :(

jazz7381 commented 3 years ago

@jazz7381 It doesn't work in 8.1.1 version now again :(

sorry but i don't have access to whmcs anymore