From 5ff6ca61549fec3680d02531bed7e564965cf54d Mon Sep 17 00:00:00 2001
From: Georg Ringer <georg.ringer@gmail.com>
Date: Fri, 16 Feb 2018 19:53:36 +0100
Subject: [PATCH] [TASK] Remove ErrorIconViewHelper and move logic into model
 and template

Simplify generation of the error icon by removing the ViewHelper and
generate the icon identifier in the model instead.

As the ViewHelper is no public API it can be removed without a RST file.

Resolves: #83945
Releases: master
Change-Id: I7515ff69a729ccb66288437f5236a1ce8fa591c1
Reviewed-on: https://review.typo3.org/55763
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Andreas Fernandez <a.fernandez@scripting-base.de>
Tested-by: Andreas Fernandez <a.fernandez@scripting-base.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
---
 .../belog/Classes/Domain/Model/LogEntry.php   | 18 +++++
 .../ViewHelpers/ErrorIconViewHelper.php       | 72 -------------------
 .../Private/Partials/Content/LogEntries.html  |  4 +-
 3 files changed, 21 insertions(+), 73 deletions(-)
 delete mode 100644 typo3/sysext/belog/Classes/ViewHelpers/ErrorIconViewHelper.php

diff --git a/typo3/sysext/belog/Classes/Domain/Model/LogEntry.php b/typo3/sysext/belog/Classes/Domain/Model/LogEntry.php
index ab1ea1e2988f..a71818a278ba 100644
--- a/typo3/sysext/belog/Classes/Domain/Model/LogEntry.php
+++ b/typo3/sysext/belog/Classes/Domain/Model/LogEntry.php
@@ -278,6 +278,24 @@ public function getError()
         return (int)$this->error;
     }
 
+    /**
+     * Get class name for the error code
+     *
+     * @return string
+     */
+    public function getErrorIconClass(): string
+    {
+        switch ($this->getError()) {
+            case 1:
+                return 'status-dialog-warning';
+            case 2:
+            case 3:
+                return 'status-dialog-error';
+            default:
+                return 'empty-empty';
+        }
+    }
+
     /**
      * Set details
      *
diff --git a/typo3/sysext/belog/Classes/ViewHelpers/ErrorIconViewHelper.php b/typo3/sysext/belog/Classes/ViewHelpers/ErrorIconViewHelper.php
deleted file mode 100644
index db58d08407f3..000000000000
--- a/typo3/sysext/belog/Classes/ViewHelpers/ErrorIconViewHelper.php
+++ /dev/null
@@ -1,72 +0,0 @@
-<?php
-namespace TYPO3\CMS\Belog\ViewHelpers;
-
-/*
- * This file is part of the TYPO3 CMS project.
- *
- * It is free software; you can redistribute it and/or modify it under
- * the terms of the GNU General Public License, either version 2
- * of the License, or any later version.
- *
- * For the full copyright and license information, please read the
- * LICENSE.txt file that was distributed with this source code.
- *
- * The TYPO3 project - inspiring people to share!
- */
-
-use TYPO3\CMS\Core\Imaging\Icon;
-use TYPO3\CMS\Core\Imaging\IconFactory;
-use TYPO3\CMS\Core\Utility\GeneralUtility;
-use TYPO3\CMS\Fluid\ViewHelpers\Be\AbstractBackendViewHelper;
-use TYPO3Fluid\Fluid\Core\Rendering\RenderingContextInterface;
-use TYPO3Fluid\Fluid\Core\ViewHelper\Traits\CompileWithRenderStatic;
-
-/**
- * Display error icon from error integer value
- * @internal
- */
-class ErrorIconViewHelper extends AbstractBackendViewHelper
-{
-    use CompileWithRenderStatic;
-
-    /**
-     * As this ViewHelper renders HTML, the output must not be escaped.
-     *
-     * @var bool
-     */
-    protected $escapeOutput = false;
-
-    /**
-     * Initializes the arguments
-     */
-    public function initializeArguments()
-    {
-        parent::initializeArguments();
-        $this->registerArgument('errorNumber', 'int', 'The error number (0 ... 3)', false, 0);
-    }
-
-    /**
-     * Renders an error icon link as known from the TYPO3 backend.
-     * Error codes 2 and three are mapped to "error" and 1 is mapped to "warning".
-     *
-     * @param array $arguments
-     * @param \Closure $renderChildrenClosure
-     * @param RenderingContextInterface $renderingContext
-     *
-     * @return string
-     */
-    public static function renderStatic(array $arguments, \Closure $renderChildrenClosure, RenderingContextInterface $renderingContext)
-    {
-        $errorSymbols = [
-            '0' => '',
-            '1' => 'status-dialog-warning',
-            '2' => 'status-dialog-error',
-            '3' => 'status-dialog-error'
-        ];
-        if ($errorSymbols[$arguments['errorNumber']]) {
-            $iconFactory = GeneralUtility::makeInstance(IconFactory::class);
-            return $iconFactory->getIcon($errorSymbols[$arguments['errorNumber']], Icon::SIZE_SMALL)->render();
-        }
-        return '';
-    }
-}
diff --git a/typo3/sysext/belog/Resources/Private/Partials/Content/LogEntries.html b/typo3/sysext/belog/Resources/Private/Partials/Content/LogEntries.html
index 5c832fc3bea6..d9b051fa0978 100644
--- a/typo3/sysext/belog/Resources/Private/Partials/Content/LogEntries.html
+++ b/typo3/sysext/belog/Resources/Private/Partials/Content/LogEntries.html
@@ -139,7 +139,9 @@ <h3>
 						<f:for each="{day}" as="logItem">
 							<tr>
 								<td>
-									<belog:ErrorIcon errorNumber="{logItem.error}"/>
+									<f:if condition="{logItem.errorIconClass}">
+										<core:icon identifier="{logItem.errorIconClass}" />
+									</f:if>
 								</td>
 								<td>
 									<f:format.date format="H:i:s">@{logItem.tstamp}</f:format.date>