src/api.php from detain/myadmin-vps-module

src/api.php

Summary

Maintainability

7 hrs

Test Coverage

Method `api_buy_vps_admin` has 48 lines of code (exceeds 25 allowed). Consider refactoring.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php - About 1 hr to fix

Method `api_buy_vps` has 43 lines of code (exceeds 25 allowed). Consider refactoring.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php - About 1 hr to fix

Method `api_buy_vps_admin` has 11 arguments (exceeds 4 allowed). Consider refactoring.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)

Found in src/api.php - About 1 hr to fix

Method `api_buy_vps` has 10 arguments (exceeds 4 allowed). Consider refactoring.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)

Found in src/api.php - About 1 hr to fix

Method `api_validate_buy_vps` has 10 arguments (exceeds 4 allowed). Consider refactoring.
Open

function api_validate_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)

Found in src/api.php - About 1 hr to fix

The function api_validate_buy_vps has 10 parameters. Consider reducing the number of parameters to less than 10.
Open

function api_validate_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    //if ($GLOBALS['tf']->ima == 'admin')
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');

Found in src/api.php by phpmd

Exclude checks
- Disable engine
- Disable check

The function api_buy_vps has 10 parameters. Consider reducing the number of parameters to less than 10.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Exclude checks
- Disable engine
- Disable check

The function api_buy_vps_admin has 11 parameters. Consider reducing the number of parameters to less than 10.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Exclude checks
- Disable engine
- Disable check

Avoid unused local variables such as '$monthly_service_cost'.
Open

    $monthly_service_cost = $validation['monthly_service_cost'];

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

UnusedLocalVariable

Since: 0.2

Detects when a local variable is declared and/or assigned, but not used.

Example

class Foo {
    public function doSomething()
    {
        $i = 5; // Unused
    }
}

Source https://phpmd.org/rules/unusedcode.html#unusedlocalvariable

Avoid unused local variables such as '$repeat_slice_cost'.
Open

    $repeat_slice_cost = $validation['repeat_slice_cost'];

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

UnusedLocalVariable

Since: 0.2

Detects when a local variable is declared and/or assigned, but not used.

Example

class Foo {
    public function doSomething()
    {
        $i = 5; // Unused
    }
}

Source https://phpmd.org/rules/unusedcode.html#unusedlocalvariable

Avoid unused local variables such as '$monthly_service_cost'.
Open

    $monthly_service_cost = $validation['monthly_service_cost'];

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

UnusedLocalVariable

Since: 0.2

Detects when a local variable is declared and/or assigned, but not used.

Example

class Foo {
    public function doSomething()
    {
        $i = 5; // Unused
    }
}

Source https://phpmd.org/rules/unusedcode.html#unusedlocalvariable

Avoid unused local variables such as '$repeat_slice_cost'.
Open

    $repeat_slice_cost = $validation['repeat_slice_cost'];

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

UnusedLocalVariable

Since: 0.2

Detects when a local variable is declared and/or assigned, but not used.

Example

class Foo {
    public function doSomething()
    {
        $i = 5; // Unused
    }
}

Source https://phpmd.org/rules/unusedcode.html#unusedlocalvariable

The variable $service_type is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $order_response is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $repeat_slice_cost is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $coupon_code is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $original_cost is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $service_type is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $original_cost is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $repeat_slice_cost is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $total_cost is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $order_response is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $total_cost is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $coupon_code is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $repeat_service_cost is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $order_response is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $coupon_code is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $service_cost is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $original_slice_cost is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $order_response is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $service_type is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $monthly_service_cost is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $order_response is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $monthly_service_cost is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $service_cost is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $service_type is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $order_response is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $original_slice_cost is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $service_cost is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $slice_cost is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $total_cost is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $slice_cost is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $slice_cost is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $repeat_service_cost is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $service_cost is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $repeat_service_cost is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $service_cost is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $original_slice_cost is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $service_cost is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $original_cost is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $order_response is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $real_iids is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $repeat_service_cost is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $order_response is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $total_cost is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $real_iids is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $real_iids is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $slice_cost is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $original_slice_cost is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $original_cost is not named in camelCase.
Open

function api_buy_vps($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass)
{
    $custid = get_custid($GLOBALS['tf']->session->account_id, 'vps');
    function_requirements('validate_buy_vps');
    $validation = validate_buy_vps($custid, $os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass);

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $coupon_code is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $real_iids is not named in camelCase.
Open

function api_buy_vps_admin($os, $slices, $platform, $controlpanel, $period, $location, $version, $hostname, $coupon, $rootpass, $server = 0)
{
    if ($GLOBALS['tf']->ima != 'admin') {
        $server = 0;
    } else {

Found in src/api.php by phpmd

Read up
Read up
Exclude checks
- Disable engine
- Disable check

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

detain/myadmin-vps-module

Summary

Maintainability

Test Coverage

Method api_buy_vps_admin has 48 lines of code (exceeds 25 allowed). Consider refactoring. Open

Method api_buy_vps has 43 lines of code (exceeds 25 allowed). Consider refactoring. Open

Method api_buy_vps_admin has 11 arguments (exceeds 4 allowed). Consider refactoring. Open

Method api_buy_vps has 10 arguments (exceeds 4 allowed). Consider refactoring. Open

Method api_validate_buy_vps has 10 arguments (exceeds 4 allowed). Consider refactoring. Open

The function api_validate_buy_vps has 10 parameters. Consider reducing the number of parameters to less than 10. Open

The function api_buy_vps has 10 parameters. Consider reducing the number of parameters to less than 10. Open

The function api_buy_vps_admin has 11 parameters. Consider reducing the number of parameters to less than 10. Open

Avoid unused local variables such as '$monthly_service_cost'. Open

UnusedLocalVariable

Since: 0.2

Example

Source https://phpmd.org/rules/unusedcode.html#unusedlocalvariable

Avoid unused local variables such as '$repeat_slice_cost'. Open

UnusedLocalVariable

Since: 0.2

Example

Source https://phpmd.org/rules/unusedcode.html#unusedlocalvariable

Avoid unused local variables such as '$monthly_service_cost'. Open

UnusedLocalVariable

Since: 0.2

Example

Source https://phpmd.org/rules/unusedcode.html#unusedlocalvariable

Avoid unused local variables such as '$repeat_slice_cost'. Open

UnusedLocalVariable

Since: 0.2

Example

Source https://phpmd.org/rules/unusedcode.html#unusedlocalvariable

The variable $service_type is not named in camelCase. Open

CamelCaseVariableName

Since: 0.2

Example

Source

The variable $order_response is not named in camelCase. Open

CamelCaseVariableName

Since: 0.2

Example

Source

The variable $repeat_slice_cost is not named in camelCase. Open

CamelCaseVariableName

Since: 0.2

Example

Source

The variable $coupon_code is not named in camelCase. Open

CamelCaseVariableName

Since: 0.2

Example

Source

The variable $original_cost is not named in camelCase. Open

CamelCaseVariableName

Since: 0.2

Example

Source

The variable $service_type is not named in camelCase. Open

CamelCaseVariableName

Since: 0.2

Example

Source

The variable $original_cost is not named in camelCase. Open

CamelCaseVariableName

Since: 0.2

Example

Source

The variable $repeat_slice_cost is not named in camelCase. Open

CamelCaseVariableName

Since: 0.2

Example

Source

The variable $total_cost is not named in camelCase. Open

CamelCaseVariableName

Since: 0.2

Example

Source

The variable $order_response is not named in camelCase. Open

CamelCaseVariableName

Since: 0.2

Method `api_buy_vps_admin` has 48 lines of code (exceeds 25 allowed). Consider refactoring.
Open

Method `api_buy_vps` has 43 lines of code (exceeds 25 allowed). Consider refactoring.
Open

Method `api_buy_vps_admin` has 11 arguments (exceeds 4 allowed). Consider refactoring.
Open

Method `api_buy_vps` has 10 arguments (exceeds 4 allowed). Consider refactoring.
Open

Method `api_validate_buy_vps` has 10 arguments (exceeds 4 allowed). Consider refactoring.
Open

The function api_validate_buy_vps has 10 parameters. Consider reducing the number of parameters to less than 10.
Open

The function api_buy_vps has 10 parameters. Consider reducing the number of parameters to less than 10.
Open

The function api_buy_vps_admin has 11 parameters. Consider reducing the number of parameters to less than 10.
Open

Avoid unused local variables such as '$monthly_service_cost'.
Open

Avoid unused local variables such as '$repeat_slice_cost'.
Open

Avoid unused local variables such as '$monthly_service_cost'.
Open

Avoid unused local variables such as '$repeat_slice_cost'.
Open

The variable $service_type is not named in camelCase.
Open

The variable $order_response is not named in camelCase.
Open

The variable $repeat_slice_cost is not named in camelCase.
Open

The variable $coupon_code is not named in camelCase.
Open

The variable $original_cost is not named in camelCase.
Open

The variable $service_type is not named in camelCase.
Open

The variable $original_cost is not named in camelCase.
Open

The variable $repeat_slice_cost is not named in camelCase.
Open

The variable $total_cost is not named in camelCase.
Open

The variable $order_response is not named in camelCase.
Open

The variable $total_cost is not named in camelCase.
Open

The variable $coupon_code is not named in camelCase.
Open

The variable $repeat_service_cost is not named in camelCase.
Open

The variable $order_response is not named in camelCase.
Open

The variable $coupon_code is not named in camelCase.
Open

The variable $service_cost is not named in camelCase.
Open

The variable $original_slice_cost is not named in camelCase.
Open

The variable $order_response is not named in camelCase.
Open

The variable $service_type is not named in camelCase.
Open

The variable $monthly_service_cost is not named in camelCase.
Open

The variable $order_response is not named in camelCase.
Open

The variable $monthly_service_cost is not named in camelCase.
Open

The variable $service_cost is not named in camelCase.
Open

The variable $service_type is not named in camelCase.
Open