Drupal\mongodb_watchdog\Controller\RequestController

Complexity

Total Complexity

3

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	create()	0	5	1
A	track()	0	7	1

<?php

namespace Drupal\mongodb_watchdog\Controller;

use Drupal\Core\DependencyInjection\ContainerInjectionInterface;
use Drupal\mongodb_watchdog\Logger;
use Symfony\Component\DependencyInjection\ContainerInterface;

/**
 * Implements the controller for the request events page.
 */
class RequestController implements ContainerInjectionInterface {

  /**
   * The mongodb_watchdog logger, to access events.
   *
   * @var \Drupal\mongodb_watchdog\Logger
   */
  protected $watchdog;

  public function __construct(Logger $watchdog) {

Missing function doc comment

    $this->watchdog = $watchdog;
  }

  /**
   * {@inheritdoc}
   */
  public static function create(ContainerInterface $container) {
    /** @var \Drupal\mongodb_watchdog\Logger $watchdog */
    $watchdog = $container->get('mongodb.logger');
    return new static($watchdog);
  }

  /**
   * Controller.
   *
   * @param string $unique_id
   *   The unique request id from mod_unique_id.
   *
   * @return array

Consider making the return type a bit more specific; maybe use array<string,string>.

   *   A render array.
   */
  public function track($unique_id) {
    $events = $this->watchdog->requestEvents($unique_id);
    ksm($events);
    return [
      '#markup' => $unique_id,
    ];
  }

}

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