RebelCode/rcmod-wp-bookings-cqrs

View on GitHub

Showing 63 of 63 total issues

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

        $option = $c->get('wp_bookings_cqrs/migrations/db_version_option');
Severity: Minor
Found in src/Module/UpdateDbVersionHandler.php by phpmd

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 method setup() has 704 lines of code. Current threshold is set to 100. Avoid really long methods.
Open

    public function setup()
    {
        return $this->_setupContainer(
            $this->_loadPhpConfigFile(RC_WP_BOOKINGS_CQRS_MODULE_CONFIG_FILE),
            [
Severity: Minor
Found in src/Module/WpBookingsCqrsModule.php by phpmd

The method __construct has 10 parameters. Consider reducing the number of parameters to less than 10.
Open

    public function __construct(
        SelectCapableInterface $selectRm,
        InsertCapableInterface $insertRm,
        UpdateCapableInterface $updateRm,
        DeleteCapableInterface $deleteRm,
Severity: Minor
Found in src/ResourcesEntityManager.php by phpmd
Severity
Category
Status
Source
Language