LiquidWeb-WHMCS-Plugin icon indicating copy to clipboard operation
LiquidWeb-WHMCS-Plugin copied to clipboard

Published 20 hours ago • verified publisher icon liquidweb

Code Upgrades?

Open idevwebs opened this issue 8 years ago • 3 comments

Hi there. Pretty soon, support for mysql_* functions will be officially dropped. Are there any plans to upgrade the code in this plugin anytime soon?

idevwebs avatar Feb 01 '17 23:02 idevwebs

Thank you for reporting this concern, I will raise this issue internally and will see how this could be addressed.

mallardduck avatar May 21 '17 00:05 mallardduck

@mike7892 from what I can see this issue is still active and in the code.

As WHMCS has provided a builtin database connection library since 6.0 and on, I believe we need to begin targeting a project to rewrite code based on that functionality. I think this should be tagged as a project for a Minor release point, although depending on the rewrite size I could see it requiring a major version bump.

The builtin DBAL is called Capsule and also provides Laravels ORM called Eloquent. Between both we should easily be able to replace any directly used mysql_* functions in favor of one of these abstractions. This will provide a more cohesive WHMCS compatibility with out plugin and should help ensure our plugins compatibility.

As I'm quite familiar with Laravel if needed I'd be happy to help with this task - time permitting.

mallardduck avatar Jul 25 '17 14:07 mallardduck

Are there any plans to rewrite the module to use builtin WHMcs calls?

jmzolezzi avatar May 23 '19 18:05 jmzolezzi