Drupal dependency injection on Controller/ItemListController

Nice change. I have a few remarks.

1. +++ b/src/Controller/ItemListController.php
   @@ -9,12 +9,37 @@ use Drupal\Core\Entity\Exception\UndefinedLinkTemplateException;
   +  protected $entityQuery;
   ...
   +  protected $dateFormatter;
   ...
   +   * {@inheritdoc}
   

   The contructor parameters and the class properties need to be documented.

2. +++ b/src/Controller/ItemListController.php
   @@ -56,7 +81,7 @@ class ItemListController extends ControllerBase {
   -      $ago = \Drupal::service('date.formatter')->formatInterval(REQUEST_TIME - $entity->get('feeds_item')->imported);
   +      $ago = $this->dateFormatter->formatInterval(REQUEST_TIME - $entity->get('feeds_item')->imported);
   

   Now that we are at it, we might as well replace the constant REQUEST_TIME which is deprecated since Drupal 8.3.
   See https://api.drupal.org/api/drupal/core%21includes%21bootstrap.inc/consta...

It looks like that patch is meant for the Google Analytics module. ;)

+++ b/src/Controller/ItemListController.php
@@ -9,12 +9,65 @@ use Drupal\Core\Entity\Exception\UndefinedLinkTemplateException;
+use Drupal\Core\Entity\Query\QueryFactory;
...
+  /**
+   * Creating entity query objects.
+   *
+   * @var \Drupal\Core\Entity\Query\QueryFactory
+   */
+  protected $entityQuery;
...
+   * @param \Drupal\Core\Entity\Query\QueryFactory $entity_query
+   *   Entity query objects.
...
+  public function __construct(QueryFactory $entity_query, Time $time, DateFormatter $date_formatter) {
+    $this->entityQuery = $entity_query;
...
+      $container->get('entity.query'),

Sorry, I forgot that the entity.query service is also deprecated. See https://www.drupal.org/node/2849874

I think that instead we need to pass the entity type manager here.

I was about to commit this, but I noticed that classes were referenced instead of interfaces. It is a best practice to require interfaces in methods instead of classes if available (a prerequisite is of course that the interface has the methods that you want to call on the object).

Also changed:

• Sorted the use statements on the top of the file.
• Removed a blank line in ::create().
• Swapped the lines in the constructor so $time and $date_formatter are everywhere in the same order.

Thanks for checking, @thalles! Committed #10.