wol-soft/php-json-schema-model-generator

View on GitHub
src/SchemaProcessor/Hook/SchemaHookResolver.php

Summary

Maintainability
A
0 mins
Test Coverage
A
100%

The method resolveSetterBeforeValidationHook has a boolean flag argument $batchUpdate, which is a certain sign of a Single Responsibility Principle violation.
Open

    public function resolveSetterBeforeValidationHook(PropertyInterface $property, bool $batchUpdate = false): string

BooleanArgumentFlag

Since: 1.4.0

A boolean flag argument is a reliable indicator for a violation of the Single Responsibility Principle (SRP). You can fix this problem by extracting the logic in the boolean flag into its own class or method.

Example

class Foo {
    public function bar($flag = true) {
    }
}

Source https://phpmd.org/rules/cleancode.html#booleanargumentflag

The method resolveSetterAfterValidationHook has a boolean flag argument $batchUpdate, which is a certain sign of a Single Responsibility Principle violation.
Open

    public function resolveSetterAfterValidationHook(PropertyInterface $property, bool $batchUpdate = false): string

BooleanArgumentFlag

Since: 1.4.0

A boolean flag argument is a reliable indicator for a violation of the Single Responsibility Principle (SRP). You can fix this problem by extracting the logic in the boolean flag into its own class or method.

Example

class Foo {
    public function bar($flag = true) {
    }
}

Source https://phpmd.org/rules/cleancode.html#booleanargumentflag

syntax error, unexpected ')'
Open

        );

Line indented incorrectly; expected 8 spaces, found 12
Open

            static function (SchemaHookInterface $hook) use ($filterHook): bool {

Line indented incorrectly; expected 8 spaces, found 12
Open

            },

There are no issues that match your filters.

Category
Status