Issue #1763964 by ParisLiakos, mrded: Use #type => link for theme_aggregator_block_item()

core/modules/aggregator/aggregator.module

@@ -66,10 +66,6 @@ function aggregator_theme() {
       'render element' => 'elements',
       'file' => 'aggregator.theme.inc',
     ),
-    'aggregator_block_item' => array(
-      'variables' => array('item' => NULL, 'feed' => 0),
-      'file' => 'aggregator.theme.inc',
-    ),
     'aggregator_item' => array(
       'render element' => 'elements',
       'file' => 'aggregator.theme.inc',

core/modules/aggregator/aggregator.theme.inc

@@ -48,20 +48,3 @@ function template_preprocess_aggregator_feed(&$variables) {
   $variables['full'] = $variables['elements']['#view_mode'] == 'full';
   $variables['title'] = String::checkPlain($feed->label());
 }
-
-/**
- * Prepares variables for individual feed item block templates.
- *
- * Default template: aggregator-block-item.html.twig.
- *
- * @param array $variables
- *   An associative array containing:
- *   - item: The item to be displayed.
- *   - feed: Not used.
- */
-function template_preprocess_aggregator_block_item(&$variables) {
-  // Display the external link to the item.
-  $item = $variables['item'];
-  $variables['url'] = check_url($item->getLink());
-  $variables['title'] = String::checkPlain($item->label());
-}

core/modules/aggregator/src/Plugin/Block/AggregatorFeedBlock.php

@@ -171,8 +171,9 @@ class AggregatorFeedBlock extends BlockBase implements ContainerFactoryPluginInt
       $rendered_items = array();
       foreach ($items as $item) {
         $aggregator_block_item = array(
-          '#theme' => 'aggregator_block_item',
-          '#item' => $item,
+          '#type' => 'link',
+          '#href' => $item->getLink(),
+          '#title' => $item->label(),
         );
         $rendered_items[] = drupal_render($aggregator_block_item);
       }

core/modules/aggregator/templates/aggregator-block-item.html.twig

@@ -1,15 +0,0 @@
-{#
-/**
- * @file
- * Default theme implementation for feed item for display in the block.
- *
- * Available variables:
- * - url: URL to the feed item.
- * - title: Title of the feed item.
- *
- * @see template_preprocess_aggregator_block_item()
- *
- * @ingroup themeable
- */
-#}
-<a href="{{ url }}">{{ title }}</a>

core/themes/classy/templates/aggregator/aggregator-block-item.html.twig

@@ -1,15 +0,0 @@
-{#
-/**
- * @file
- * Default theme implementation for feed item for display in the block.
- *
- * Available variables:
- * - url: URL to the feed item.
- * - title: Title of the feed item.
- *
- * @see template_preprocess_aggregator_block_item()
- *
- * @ingroup themeable
- */
-#}
-<a href="{{ url }}">{{ title }}</a>