bubbstore/iugu-php-sdk

View on GitHub

Showing 22 of 22 total issues

Avoid variables with short names like $id. Configured minimum length is 3.
Open

    public function update($customerId, $id, array $params)

ShortVariable

Since: 0.2

Detects when a field, local, or parameter has a very short name.

Example

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

Source https://phpmd.org/rules/naming.html#shortvariable

Method __construct has 6 arguments (exceeds 4 allowed). Consider refactoring.
Open

        $apiKey,
        CustomerInterface $customer = null,
        PaymentMethodInterface $paymentMethod = null,
        ChargeInterface $charge = null,
        InvoiceInterface $invoice = null,
Severity: Minor
Found in src/bubbstore/Iugu/Iugu.php - About 45 mins to fix
    Severity
    Category
    Status
    Source
    Language