Drupal\mongodb_watchdog\Form\ConfigForm

Complexity

Total Complexity

9

Size/Duplication

Total Lines	91
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	1
Bugs	0	Features	1

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	create()	0	6	1
A	buildForm()	0	21	3
A	submitForm()	0	8	2
A	getEditableConfigNames()	0	3	1
A	getFormId()	0	3	1

<?php

namespace Drupal\mongodb_watchdog\Form;

use Drupal\Component\Utility\Unicode;
use Drupal\Core\Config\ConfigFactoryInterface;
use Drupal\Core\Form\ConfigFormBase;
use Drupal\Core\Form\FormStateInterface;
use Symfony\Component\DependencyInjection\ContainerInterface;

/**
 * Class ConfigForm provides configuration for the MongoDB watchdog module.
 */
class ConfigForm extends ConfigFormBase {

  /**
   * Name of the config being edited.
   */
  const CONFIG_NAME = 'mongodb_watchdog.settings';

  protected $typed;

  /**
   * ConfigForm constructor.
   *
   * @param \Drupal\Core\Config\ConfigFactoryInterface $config_factory
   *   The core config.factory service.
   * @param array $typed
   *   The type config for the module.
   */
  public function __construct(ConfigFactoryInterface $config_factory, array $typed) {
    parent::__construct($config_factory);
    $this->typed = $typed;
  }

  /**
   * {@inheritdoc}
   */
  public static function create(ContainerInterface $container) {
    return new static(
      $container->get('config.factory'),
      $container->get('config.typed')->getDefinition('mongodb_watchdog.settings')
    );
  }

  /**
   * {@inheritdoc}
   */
  public function buildForm(array $form, FormStateInterface $form_state) {

The return type could not be reliably inferred; please add a @return annotation.

    $config = $this->config(static::CONFIG_NAME);
    foreach ($config->getRawData() as $key => $default_value) {
      if (Unicode::substr($key, 0, 1) === '_') {
        continue;
      }
      $schema = $this->typed['mapping'][$key];
      list($title, $description) = explode(': ', $schema['label']);
      $form[$key] = [
        '#title' => $title,
        '#description' => $description,
        '#type' => 'number',
        '#min' => $schema['min'] ?? 0,

Expected 1 space after "?"; 0 found

        '#max' => $schema['max'] ?? PHP_INT_MAX,

Expected 1 space after "?"; 0 found

        '#default_value' => $default_value,
      ];
    }

    $form = parent::buildForm($form, $form_state);
    return $form;
  }

  /**
   * {@inheritdoc}
   */
  public function submitForm(array &$form, FormStateInterface $form_state) {
    $config = $this->config(static::CONFIG_NAME);
    foreach (array_keys($config->getRawData()) as $key) {
      $config->set($key, intval($form_state->getValue($key)));
    }
    $config->save();
    drupal_set_message($this->t('The configuration options have been saved.'));
  }

  /**
   * Gets the configuration names that will be editable.
   *
   * @return array

Consider making the return type a bit more specific; maybe use string[].

   *   An array of configuration object names that are editable if called in
   *   conjunction with the trait's config() method.
   */
  protected function getEditableConfigNames() {
    return ['mongodb_watchdog.settings'];
  }

  /**
   * Returns a unique string identifying the form.
   *
   * @return string
   *   The unique string identifying the form.
   */
  public function getFormId() {
    return 'mongodb_watchdog_config';
  }

}

As per coding style, files should not end with a newline character.