rugk/xenforo-threema-gateway

View on GitHub
src/library/ThreemaGateway/DataWriter/ActionThrottle.php

Summary

Maintainability
A
0 mins
Test Coverage

Avoid unused parameters such as '$data'.
Open

    protected function _getExistingData($data)

UnusedFormalParameter

Since: 0.2

Avoid passing parameters to methods or constructors and then not using those parameters.

Example

class Foo
{
    private function bar($howdy)
    {
        // $howdy is not used
    }
}

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

Avoid unused parameters such as '$tableName'.
Open

    protected function _getUpdateCondition($tableName)

UnusedFormalParameter

Since: 0.2

Avoid passing parameters to methods or constructors and then not using those parameters.

Example

class Foo
{
    private function bar($howdy)
    {
        // $howdy is not used
    }
}

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

The method _getUpdateCondition is not named in camelCase.
Open

    protected function _getUpdateCondition($tableName)
    {
        return '';
    }

CamelCaseMethodName

Since: 0.2

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

Example

class ClassName {
    public function get_name() {
    }
}

Source

The method _getFields is not named in camelCase.
Open

    protected function _getFields()
    {
        return [
            ThreemaGateway_Model_ActionThrottle::DB_TABLE => [
                'action_id' => [

CamelCaseMethodName

Since: 0.2

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

Example

class ClassName {
    public function get_name() {
    }
}

Source

The method _getActionThrottleModel is not named in camelCase.
Open

    protected function _getActionThrottleModel()
    {
        return $this->getModelFromCache('ThreemaGateway_Model_ActionThrottle');
    }

CamelCaseMethodName

Since: 0.2

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

Example

class ClassName {
    public function get_name() {
    }
}

Source

The method _getExistingData is not named in camelCase.
Open

    protected function _getExistingData($data)
    {
        return [];
    }

CamelCaseMethodName

Since: 0.2

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

Example

class ClassName {
    public function get_name() {
    }
}

Source

There are no issues that match your filters.

Category
Status