SU-HKKU/cardinal_service_profile

Method submitForm has 32 lines of code (exceeds 25 allowed). Consider refactoring.

Wontfix

  public function submitForm(array &$form, FormStateInterface $form_state) {
    $before_count = $this->getNodeCount();
    // Invalidate the migrations so that we can alter the plugin after setting
    // the state for the file path.
    Cache::invalidateTags(['migration_plugins']);

Method validateForm has 29 lines of code (exceeds 25 allowed). Consider refactoring.

Wontfix

  public function validateForm(array &$form, FormStateInterface $form_state) {
    parent::validateForm($form, $form_state);
    if ($form_state->getTriggeringElement()['#name'] !== 'import') {
      return;
    }

Avoid using undefined variables such as '$help' which will lead to PHP notices.

Open

    $help[] = [

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$help' which will lead to PHP notices.

Open

    $help[] = [

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$help' which will lead to PHP notices.

Open

    return $help;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

No space found before comment text; expected "// @codeCoverageIgnoreEnd" but found "//@codeCoverageIgnoreEnd"

Wontfix

    //@codeCoverageIgnoreEnd

• Exclude checks
  • Disable engine
  • Disable check

There must be no blank line following an inline comment

Wontfix

    // @codeCoverageIgnoreEnd

• Exclude checks
  • Disable engine
  • Disable check