Puerari\Cwp\Autossl

Complexity

Total Complexity

4

Size/Duplication

Total Lines	59
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	createAutoSsl()	0	6	1
A	listAutoSsl()	0	6	1
A	deleteAutoSsl()	0	6	1
A	renewAutoSsl()	0	6	1

<?php

namespace Puerari\Cwp;

/**
 * @trait Autossl
 * @package Puerari\Cwp
 * @author Leandro Puerari <leandro@puerari.com.br>
 */
trait Autossl
{
    /**
     * @param string $user : Account username
     * @param string $name : Name of the domain or subdomain to generate the autossl
     * @return string|bool: false on failure, result on success (JSON / XML)
     * status -> OK
     * status -> Error, msj ->(array).
     */
    public function createAutoSsl(string $user, string $name)
    {
        $this->data = compact('user', 'name');

The property data does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->data['action'] = 'add';
        $this->cwpuri = 'autossl';

The property cwpuri does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        return $this->execCurl();

It seems like execCurl() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * @param string $user : Account user name
     * @param string $name : Name of the domain or subdomain to delete the autossl
     * @return string|bool: false on failure, result on success (JSON / XML)
     * status -> OK, msj -> Deleted Autossl
     * status -> Error, msj -> Error description
     */
    public function deleteAutoSsl(string $user, string $name)
    {
        $this->data = compact('user', 'name');

The property data does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->data['action'] = 'del';
        $this->cwpuri = 'autossl';

The property cwpuri does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        return $this->execCurl();
    }

    /**
     * @param string $user : Account username (If the user name is omitted, all AutoSSLs are listed)
     * @return string|bool: false on failure, result on success (JSON / XML)
     * status -> OK, msj -> Array with the different data of the certificate
     * status -> Error, msj -> Error description
     */
    public function listAutoSsl(string $user = null)
    {
        $this->data = compact('user');

The property data does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->data['action'] = 'list';
        $this->cwpuri = 'autossl';

The property cwpuri does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        return $this->execCurl();
    }

    /**
     * @param string $user : Account user name
     * @param string $name : Name of the domain or subdomain to renew
     * @return string|bool: false on failure, result on success (JSON / XML)
     * status -> OK
     * status -> Error, msj -> Error description
     */
    public function renewAutoSsl(string $user, string $name)
    {
        $this->data = compact('user', 'name');

The property data does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->data['action'] = 'del';
        $this->cwpuri = 'autossl';

The property cwpuri does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        return $this->execCurl();
    }
}