From 2ef487b2477303076356b3258b594dae0b3b9792 Mon Sep 17 00:00:00 2001
From: Ciprian Hossu <ciprian.hossu@sgalinski.de>
Date: Wed, 10 Jun 2020 12:17:04 +0300
Subject: [PATCH 1/8] [FEATURE] Add backend preview for the YouTube Videos
 plugin

---
 Classes/Hooks/PluginRenderer.php            | 177 ++++++++++++++++++++
 Resources/Private/Language/de.locallang.xlf |  10 +-
 Resources/Private/Language/locallang.xlf    |   8 +-
 ext_localconf.php                           |   3 +
 4 files changed, 196 insertions(+), 2 deletions(-)
 create mode 100644 Classes/Hooks/PluginRenderer.php

diff --git a/Classes/Hooks/PluginRenderer.php b/Classes/Hooks/PluginRenderer.php
new file mode 100644
index 0000000..f02fdf9
--- /dev/null
+++ b/Classes/Hooks/PluginRenderer.php
@@ -0,0 +1,177 @@
+<?php
+
+namespace SGalinski\SgYoutube\Hooks;
+
+/***************************************************************
+ *  Copyright notice
+ *  (c) sgalinski Internet Services (https://www.sgalinski.de)
+ *  All rights reserved
+ *  This script is part of the TYPO3 project. The TYPO3 project is
+ *  free software; you can redistribute it and/or modify
+ *  it under the terms of the GNU General Public License as published by
+ *  the Free Software Foundation; either version 3 of the License, or
+ *  (at your option) any later version.
+ *  The GNU General Public License can be found at
+ *  http://www.gnu.org/copyleft/gpl.html.
+ *  This script is distributed in the hope that it will be useful,
+ *  but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ *  GNU General Public License for more details.
+ *  This copyright notice MUST APPEAR in all copies of the script!
+ ***************************************************************/
+
+use TYPO3\CMS\Backend\Utility\BackendUtility;
+use TYPO3\CMS\Backend\View\PageLayoutView;
+use TYPO3\CMS\Backend\View\PageLayoutViewDrawItemHookInterface;
+use TYPO3\CMS\Core\Utility\GeneralUtility;
+use TYPO3\CMS\Form\Service\TranslationService;
+
+/**
+ * Class PluginRenderer
+ * Renders back-end preview for the SG YouTube Videos plugin
+ *
+ * @package SGalinski\SgYoutube\Hooks
+ */
+class PluginRenderer implements PageLayoutViewDrawItemHookInterface {
+	/**
+	 * Preprocesses the preview rendering of a content element of type "sg_youtube"
+	 *
+	 * @param PageLayoutView $parentObject Calling parent object
+	 * @param bool $drawItem Whether to draw the item using the default functionality
+	 * @param string $headerContent Header content
+	 * @param string $itemContent Item content
+	 * @param array $row Record row of tt_content
+	 * @return void
+	 */
+	public function preProcess(PageLayoutView &$parentObject, &$drawItem, &$headerContent, &$itemContent, array &$row) {
+		if ($row['list_type'] === 'sgyoutube_youtube') {
+
+			// Content element labels
+			$labelPluginName = BackendUtility::getLabelFromItemListMerged(
+				$row['pid'], 'tt_content', 'list_type', $row['list_type']
+			);
+			$labelHeader = BackendUtility::getItemLabel(
+				'tt_content', 'header'
+			);
+			$labelHeaderLink = BackendUtility::getItemLabel(
+				'tt_content', 'header_link'
+			);
+			$labelSubheader = BackendUtility::getItemLabel(
+				'tt_content', 'subheader'
+			);
+			$labelContentLink = BackendUtility::getItemLabel(
+				'tt_content', 'tx_sgcontentlink_contentlink'
+			);
+
+			// Make instance of TranslationService class
+			$translationServiceObject = GeneralUtility::makeInstance(TranslationService::class);
+
+			// Begin rendering of content element fields
+			// Plugin name
+			$headerContent = '<div style="padding: 0 4px 10px 0;border-bottom: 1px solid #CACACA;margin-bottom: 6px; font-weight:700;">' . $labelPluginName . '</div>';
+
+			// Header
+			if ($row['header']) {
+				if ($row['header_layout'] === '10') {
+					$headerContent .= '<p>' . $row['header'] . '</p>';
+				} elseif ($row['header_layout'] === '100') {
+					$headerContent .= '<div style="display:none;">' . $row['header'] . '</div>';
+				} else {
+					$headerContent .= '<h' . $row['header_layout'] . '>' . $row['header'] . '</h' . $row['header_layout'] . '>';
+				}
+			}
+
+			//Header link
+			if ($row['header_link']) {
+				$headerContent .= '<div><strong>' . $translationServiceObject->translate(
+						$labelHeaderLink
+					) . '&nbsp;(' . $translationServiceObject->translate(
+						$labelHeader
+					) . ')</strong>:&nbsp;' . $row['header_link'] . '</div>';
+			}
+
+			// Subheader
+			if ($row['subheader']) {
+				$headerContent .= '<div>' . '<strong>' . $translationServiceObject->translate(
+						$labelSubheader
+					) . '</strong>' . ':&nbsp;' . $row['subheader'] . '</div>';
+			}
+			// Content link
+			if ($row['tx_sgcontentlink_contentlink']) {
+				$headerContent .= '<div>' . '<strong>' . $translationServiceObject->translate(
+						$labelContentLink
+					) . '</strong>' . ':&nbsp;' . $row['tx_sgcontentlink_contentlink'] . '</div>';
+			}
+
+			// Set plugin options array
+			$pluginOptions = GeneralUtility::xml2array($row['pi_flexform'], 'T3:');
+
+			// Get the FlexForm configuration file
+			$xmlFilePath = $GLOBALS['TCA']['tt_content']['columns']['pi_flexform']['config']['ds']['sgyoutube_youtube,list'];
+			$xmlContent = GeneralUtility::getFileAbsFileName(
+				str_replace(
+					'FILE:EXT:', 'EXT:', $xmlFilePath
+				)
+			);
+
+			// Get contents of the FlexForm configuration file
+			$xmlString = file_get_contents($xmlContent);
+			$arrayJobs = GeneralUtility::xml2array($xmlString);
+
+			// Begin rendering of table
+			$itemContent .= '<br>' . '<table class="table table-striped">';
+
+			// Table header
+			$labelSettings = 'LLL:EXT:sg_youtube/Resources/Private/Language/locallang.xlf:settings';
+			$labelvalue = 'LLL:EXT:sg_youtube/Resources/Private/Language/locallang.xlf:values';
+			$itemContent .= '<row>' . '<th style="width: 50%">' . $translationServiceObject->translate(
+					$labelSettings
+				) . '</th>';
+			$itemContent .= '<th style="width: 50%">' . $translationServiceObject->translate(
+					$labelvalue
+				) . '</th>' . '</tr>';
+
+			// Table content rows
+			// Loop through plugin options
+			foreach ($pluginOptions['data']['sDEF']['lDEF'] as $key => $setting) {
+				// if plugin option exists within the Flexform configuration file
+				if (array_key_exists($key, $arrayJobs['sheets']['sDEF']['ROOT']['el'])) {
+					// Display label
+					$labelId = $arrayJobs['sheets']['sDEF']['ROOT']['el'][$key]['TCEforms']['label'];
+					$itemContent .= '<row>' . '<td><strong>' . $translationServiceObject->translate(
+							trim($labelId)
+						) . '</strong></td>';
+					// Display value
+					$itemContent .= '<td>' . $setting['vDEF'] . '</td>' . '</tr>';
+				}
+			}
+
+			// Record Storage
+			if ($row['pages'] !== '') {
+				// Record Storage Page label
+				$labelRecordStorage = 'LLL:EXT:frontend/Resources/Private/Language/locallang_ttc.xlf:pages.ALT.list_formlabel';
+
+				// Get storage page(s)
+				$backendUtilityObject = GeneralUtility::makeInstance(BackendUtility::class);
+				$storagePageIds = explode(',', $row['pages']);
+				$storagePages = [];
+
+				foreach ($storagePageIds as $i => $storagePageId) {
+					$storagePages[] = $backendUtilityObject->getRecord(
+							'pages', $storagePageId, 'title'
+						)['title'] . ' ' . '(' . $storagePageId . ')';
+				}
+
+				// Render 'Record Storage Page' row
+				$itemContent .= '<row>' . '<td><strong>' . $translationServiceObject->translate(
+						$labelRecordStorage
+					) . '</strong></td>';
+				$itemContent .= '<td>' . implode(', ', $storagePages) . '</td>' . '</tr>';
+			}
+
+			// End rendering of table
+			$itemContent .= '</table>';
+			$drawItem = FALSE;
+		}
+	}
+}
diff --git a/Resources/Private/Language/de.locallang.xlf b/Resources/Private/Language/de.locallang.xlf
index 2560a6e..09a1846 100644
--- a/Resources/Private/Language/de.locallang.xlf
+++ b/Resources/Private/Language/de.locallang.xlf
@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8" standalone="yes" ?>
 <xliff version="1.0">
-	<file source-language="en" target-language="de" datatype="plaintext" original="messages" date="2019-12-07T19:09:30Z">
+	<file source-language="en" target-language="de" datatype="plaintext" original="messages" date="2020-06-10T09:14:47Z">
 		<header>
 			<generator>LFEditor</generator>
 		</header>
@@ -49,6 +49,14 @@
 				<source><![CDATA[Maximum Amount]]></source>
 				<target><![CDATA[Maximale Anzahl]]></target>
 			</trans-unit>
+			<trans-unit id="settings" approved="yes">
+				<source><![CDATA[Settings]]></source>
+				<target><![CDATA[Einstellungen]]></target>
+			</trans-unit>
+			<trans-unit id="values" approved="yes">
+				<source><![CDATA[Values]]></source>
+				<target><![CDATA[Werte]]></target>
+			</trans-unit>
 		</body>
 	</file>
 </xliff>
\ No newline at end of file
diff --git a/Resources/Private/Language/locallang.xlf b/Resources/Private/Language/locallang.xlf
index 69910c0..11714e9 100644
--- a/Resources/Private/Language/locallang.xlf
+++ b/Resources/Private/Language/locallang.xlf
@@ -38,6 +38,12 @@
 			<trans-unit id="maxResults">
 				<source><![CDATA[Maximum Amount]]></source>
 			</trans-unit>
+			<trans-unit id="settings">
+				<source><![CDATA[Settings]]></source>
+			</trans-unit>
+			<trans-unit id="values">
+				<source><![CDATA[Values]]></source>
+			</trans-unit>
 		</body>
 	</file>
-</xliff>
\ No newline at end of file
+</xliff>
diff --git a/ext_localconf.php b/ext_localconf.php
index 2fb5f10..ad9a2bd 100644
--- a/ext_localconf.php
+++ b/ext_localconf.php
@@ -40,3 +40,6 @@ if (!defined('TYPO3_MODE')) {
 
 	]
 );
+//Hooks
+$GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['cms/layout/class.tx_cms_layout.php']['tt_content_drawItem']['sg_youtube']
+	= \SGalinski\SgYoutube\Hooks\PluginRenderer::class;
-- 
GitLab


From 14cad4e86bf46e57155fb3c46506516a8cc2fcbe Mon Sep 17 00:00:00 2001
From: Ciprian Hossu <ciprian.hossu@sgalinski.de>
Date: Thu, 11 Jun 2020 07:19:25 +0300
Subject: [PATCH 2/8] [BUGFIX] Remove header and content links from plugin
 preview

---
 Classes/Hooks/PluginRenderer.php | 28 ++--------------------------
 1 file changed, 2 insertions(+), 26 deletions(-)

diff --git a/Classes/Hooks/PluginRenderer.php b/Classes/Hooks/PluginRenderer.php
index f02fdf9..d094afb 100644
--- a/Classes/Hooks/PluginRenderer.php
+++ b/Classes/Hooks/PluginRenderer.php
@@ -50,20 +50,11 @@ class PluginRenderer implements PageLayoutViewDrawItemHookInterface {
 			$labelPluginName = BackendUtility::getLabelFromItemListMerged(
 				$row['pid'], 'tt_content', 'list_type', $row['list_type']
 			);
-			$labelHeader = BackendUtility::getItemLabel(
-				'tt_content', 'header'
-			);
-			$labelHeaderLink = BackendUtility::getItemLabel(
-				'tt_content', 'header_link'
-			);
 			$labelSubheader = BackendUtility::getItemLabel(
 				'tt_content', 'subheader'
 			);
-			$labelContentLink = BackendUtility::getItemLabel(
-				'tt_content', 'tx_sgcontentlink_contentlink'
-			);
 
-			// Make instance of TranslationService class
+			// Instantiate TranslationService class
 			$translationServiceObject = GeneralUtility::makeInstance(TranslationService::class);
 
 			// Begin rendering of content element fields
@@ -81,27 +72,12 @@ class PluginRenderer implements PageLayoutViewDrawItemHookInterface {
 				}
 			}
 
-			//Header link
-			if ($row['header_link']) {
-				$headerContent .= '<div><strong>' . $translationServiceObject->translate(
-						$labelHeaderLink
-					) . '&nbsp;(' . $translationServiceObject->translate(
-						$labelHeader
-					) . ')</strong>:&nbsp;' . $row['header_link'] . '</div>';
-			}
-
 			// Subheader
 			if ($row['subheader']) {
 				$headerContent .= '<div>' . '<strong>' . $translationServiceObject->translate(
 						$labelSubheader
 					) . '</strong>' . ':&nbsp;' . $row['subheader'] . '</div>';
 			}
-			// Content link
-			if ($row['tx_sgcontentlink_contentlink']) {
-				$headerContent .= '<div>' . '<strong>' . $translationServiceObject->translate(
-						$labelContentLink
-					) . '</strong>' . ':&nbsp;' . $row['tx_sgcontentlink_contentlink'] . '</div>';
-			}
 
 			// Set plugin options array
 			$pluginOptions = GeneralUtility::xml2array($row['pi_flexform'], 'T3:');
@@ -154,7 +130,7 @@ class PluginRenderer implements PageLayoutViewDrawItemHookInterface {
 				// Get storage page(s)
 				$backendUtilityObject = GeneralUtility::makeInstance(BackendUtility::class);
 				$storagePageIds = explode(',', $row['pages']);
-				$storagePages = [];
+				$storagePages = array();
 
 				foreach ($storagePageIds as $i => $storagePageId) {
 					$storagePages[] = $backendUtilityObject->getRecord(
-- 
GitLab


From 698beec5cc8d0602fdabc346d1828eae4e05f772 Mon Sep 17 00:00:00 2001
From: Ciprian Hossu <ciprian.hossu@sgalinski.de>
Date: Thu, 11 Jun 2020 14:13:47 +0300
Subject: [PATCH 3/8] [BUGFIX] Use icons for displaying the value of the Show
 API result field

---
 Classes/Hooks/PluginRenderer.php | 26 +++++++++++++++++++++++---
 1 file changed, 23 insertions(+), 3 deletions(-)

diff --git a/Classes/Hooks/PluginRenderer.php b/Classes/Hooks/PluginRenderer.php
index d094afb..1f3bf83 100644
--- a/Classes/Hooks/PluginRenderer.php
+++ b/Classes/Hooks/PluginRenderer.php
@@ -23,6 +23,8 @@ namespace SGalinski\SgYoutube\Hooks;
 use TYPO3\CMS\Backend\Utility\BackendUtility;
 use TYPO3\CMS\Backend\View\PageLayoutView;
 use TYPO3\CMS\Backend\View\PageLayoutViewDrawItemHookInterface;
+use TYPO3\CMS\Core\Imaging\Icon;
+use TYPO3\CMS\Core\Imaging\IconFactory;
 use TYPO3\CMS\Core\Utility\GeneralUtility;
 use TYPO3\CMS\Form\Service\TranslationService;
 
@@ -107,18 +109,36 @@ class PluginRenderer implements PageLayoutViewDrawItemHookInterface {
 					$labelvalue
 				) . '</th>' . '</tr>';
 
-			// Table content rows
-			// Loop through plugin options
+			// Loop through plugin options and render rows
 			foreach ($pluginOptions['data']['sDEF']['lDEF'] as $key => $setting) {
+
 				// if plugin option exists within the Flexform configuration file
 				if (array_key_exists($key, $arrayJobs['sheets']['sDEF']['ROOT']['el'])) {
+
 					// Display label
 					$labelId = $arrayJobs['sheets']['sDEF']['ROOT']['el'][$key]['TCEforms']['label'];
 					$itemContent .= '<row>' . '<td><strong>' . $translationServiceObject->translate(
 							trim($labelId)
 						) . '</strong></td>';
+
 					// Display value
-					$itemContent .= '<td>' . $setting['vDEF'] . '</td>' . '</tr>';
+					// Show different icon for the 'Show API Result' field, depending on checkbox value
+					if ($key === 'settings.showApiResult') {
+						if ($setting['vDEF'] === '0') {
+							$itemContent .= '<td>' . GeneralUtility::makeInstance(IconFactory::class)->getIcon(
+									'actions-close', Icon::SIZE_SMALL
+								)
+									->render('inline') . '</td>' . '</tr>';
+						} else {
+							$itemContent .= '<td>' . GeneralUtility::makeInstance(IconFactory::class)->getIcon(
+									'actions-check', Icon::SIZE_SMALL
+								)
+									->render('inline') . '</td>' . '</tr>';
+						}
+					// Remaining values
+					} else {
+						$itemContent .= '<td>' . $setting['vDEF'] . '</td>' . '</tr>';
+					}
 				}
 			}
 
-- 
GitLab


From 880fbabf9d190a06bda5efca24c7f3493176661f Mon Sep 17 00:00:00 2001
From: Ciprian Hossu <ciprian.hossu@sgalinski.de>
Date: Fri, 12 Jun 2020 14:52:45 +0300
Subject: [PATCH 4/8] [BUGFIX] Change configuration array variable name.

---
 Classes/Hooks/PluginRenderer.php | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/Classes/Hooks/PluginRenderer.php b/Classes/Hooks/PluginRenderer.php
index 1f3bf83..1607890 100644
--- a/Classes/Hooks/PluginRenderer.php
+++ b/Classes/Hooks/PluginRenderer.php
@@ -94,7 +94,7 @@ class PluginRenderer implements PageLayoutViewDrawItemHookInterface {
 
 			// Get contents of the FlexForm configuration file
 			$xmlString = file_get_contents($xmlContent);
-			$arrayJobs = GeneralUtility::xml2array($xmlString);
+			$configurationArray = GeneralUtility::xml2array($xmlString);
 
 			// Begin rendering of table
 			$itemContent .= '<br>' . '<table class="table table-striped">';
@@ -113,10 +113,10 @@ class PluginRenderer implements PageLayoutViewDrawItemHookInterface {
 			foreach ($pluginOptions['data']['sDEF']['lDEF'] as $key => $setting) {
 
 				// if plugin option exists within the Flexform configuration file
-				if (array_key_exists($key, $arrayJobs['sheets']['sDEF']['ROOT']['el'])) {
+				if (array_key_exists($key, $configurationArray['sheets']['sDEF']['ROOT']['el'])) {
 
 					// Display label
-					$labelId = $arrayJobs['sheets']['sDEF']['ROOT']['el'][$key]['TCEforms']['label'];
+					$labelId = $configurationArray['sheets']['sDEF']['ROOT']['el'][$key]['TCEforms']['label'];
 					$itemContent .= '<row>' . '<td><strong>' . $translationServiceObject->translate(
 							trim($labelId)
 						) . '</strong></td>';
-- 
GitLab


From 72479e3cfe22da99ddb62d3dbde3d8e225c73fb8 Mon Sep 17 00:00:00 2001
From: Ciprian Hossu <ciprian.hossu@sgalinski.de>
Date: Mon, 15 Jun 2020 14:03:37 +0300
Subject: [PATCH 5/8] [BUGFIX] Add condition for flexform configuration file.
 Add function for returning records array.

---
 Classes/Hooks/PluginRenderer.php | 158 +++++++++++++++++--------------
 1 file changed, 86 insertions(+), 72 deletions(-)

diff --git a/Classes/Hooks/PluginRenderer.php b/Classes/Hooks/PluginRenderer.php
index 1607890..448c812 100644
--- a/Classes/Hooks/PluginRenderer.php
+++ b/Classes/Hooks/PluginRenderer.php
@@ -35,6 +35,25 @@ use TYPO3\CMS\Form\Service\TranslationService;
  * @package SGalinski\SgYoutube\Hooks
  */
 class PluginRenderer implements PageLayoutViewDrawItemHookInterface {
+	/**
+	 * Return an array of pages or categories, with their respective ids
+	 *
+	 * @param $recodIds
+	 * @return array $records
+	 */
+	public function buildRecordArray($table, $recodIds) {
+		$backendUtilityObject = GeneralUtility::makeInstance(BackendUtility::class);
+		$recordsArray = explode(',', $recodIds);
+		$records = [];
+
+		foreach ($recordsArray as $i => $recordId) {
+			$records[] = $backendUtilityObject->getRecord(
+					$table, $recordId, 'title'
+				)['title'] . ' ' . '(' . $recordId . ')';
+		}
+		return $records;
+	}
+
 	/**
 	 * Preprocesses the preview rendering of a content element of type "sg_youtube"
 	 *
@@ -48,6 +67,8 @@ class PluginRenderer implements PageLayoutViewDrawItemHookInterface {
 	public function preProcess(PageLayoutView &$parentObject, &$drawItem, &$headerContent, &$itemContent, array &$row) {
 		if ($row['list_type'] === 'sgyoutube_youtube') {
 
+			$drawItem = FALSE;
+
 			// Content element labels
 			$labelPluginName = BackendUtility::getLabelFromItemListMerged(
 				$row['pid'], 'tt_content', 'list_type', $row['list_type']
@@ -86,88 +107,81 @@ class PluginRenderer implements PageLayoutViewDrawItemHookInterface {
 
 			// Get the FlexForm configuration file
 			$xmlFilePath = $GLOBALS['TCA']['tt_content']['columns']['pi_flexform']['config']['ds']['sgyoutube_youtube,list'];
-			$xmlContent = GeneralUtility::getFileAbsFileName(
-				str_replace(
-					'FILE:EXT:', 'EXT:', $xmlFilePath
-				)
-			);
-
-			// Get contents of the FlexForm configuration file
-			$xmlString = file_get_contents($xmlContent);
-			$configurationArray = GeneralUtility::xml2array($xmlString);
-
-			// Begin rendering of table
-			$itemContent .= '<br>' . '<table class="table table-striped">';
-
-			// Table header
-			$labelSettings = 'LLL:EXT:sg_youtube/Resources/Private/Language/locallang.xlf:settings';
-			$labelvalue = 'LLL:EXT:sg_youtube/Resources/Private/Language/locallang.xlf:values';
-			$itemContent .= '<row>' . '<th style="width: 50%">' . $translationServiceObject->translate(
-					$labelSettings
-				) . '</th>';
-			$itemContent .= '<th style="width: 50%">' . $translationServiceObject->translate(
-					$labelvalue
-				) . '</th>' . '</tr>';
 
-			// Loop through plugin options and render rows
-			foreach ($pluginOptions['data']['sDEF']['lDEF'] as $key => $setting) {
-
-				// if plugin option exists within the Flexform configuration file
-				if (array_key_exists($key, $configurationArray['sheets']['sDEF']['ROOT']['el'])) {
-
-					// Display label
-					$labelId = $configurationArray['sheets']['sDEF']['ROOT']['el'][$key]['TCEforms']['label'];
-					$itemContent .= '<row>' . '<td><strong>' . $translationServiceObject->translate(
-							trim($labelId)
-						) . '</strong></td>';
-
-					// Display value
-					// Show different icon for the 'Show API Result' field, depending on checkbox value
-					if ($key === 'settings.showApiResult') {
-						if ($setting['vDEF'] === '0') {
-							$itemContent .= '<td>' . GeneralUtility::makeInstance(IconFactory::class)->getIcon(
-									'actions-close', Icon::SIZE_SMALL
-								)
-									->render('inline') . '</td>' . '</tr>';
+			if ($xmlFilePath !== '') {
+				$xmlContent = GeneralUtility::getFileAbsFileName(
+					str_replace(
+						'FILE:EXT:', 'EXT:', $xmlFilePath
+					)
+				);
+
+				// Get contents of the FlexForm configuration file
+				$xmlString = file_get_contents($xmlContent);
+				$configurationArray = GeneralUtility::xml2array($xmlString);
+
+				// Begin rendering of table
+				$itemContent .= '<br>' . '<table class="table table-striped">';
+
+				// Table header
+				$labelSettings = 'LLL:EXT:sg_youtube/Resources/Private/Language/locallang.xlf:settings';
+				$labelvalue = 'LLL:EXT:sg_youtube/Resources/Private/Language/locallang.xlf:values';
+				$itemContent .= '<row>' . '<th style="width: 50%">' . $translationServiceObject->translate(
+						$labelSettings
+					) . '</th>';
+				$itemContent .= '<th style="width: 50%">' . $translationServiceObject->translate(
+						$labelvalue
+					) . '</th>' . '</tr>';
+
+				// Loop through plugin options and render rows
+				foreach ($pluginOptions['data']['sDEF']['lDEF'] as $key => $setting) {
+
+					// if plugin option exists within the Flexform configuration file
+					if (array_key_exists($key, $configurationArray['sheets']['sDEF']['ROOT']['el'])) {
+
+						// Display label
+						$labelId = $configurationArray['sheets']['sDEF']['ROOT']['el'][$key]['TCEforms']['label'];
+						$itemContent .= '<row>' . '<td><strong>' . $translationServiceObject->translate(
+								trim($labelId)
+							) . '</strong></td>';
+
+						// Display value
+						// Show different icon for the 'Show API Result' field, depending on checkbox value
+						if ($key === 'settings.showApiResult') {
+							if ($setting['vDEF'] === '0') {
+								$itemContent .= '<td>' . GeneralUtility::makeInstance(IconFactory::class)->getIcon(
+										'actions-close', Icon::SIZE_SMALL
+									)
+										->render('inline') . '</td>' . '</tr>';
+							} else {
+								$itemContent .= '<td>' . GeneralUtility::makeInstance(IconFactory::class)->getIcon(
+										'actions-check', Icon::SIZE_SMALL
+									)
+										->render('inline') . '</td>' . '</tr>';
+							}
+							// Remaining values
 						} else {
-							$itemContent .= '<td>' . GeneralUtility::makeInstance(IconFactory::class)->getIcon(
-									'actions-check', Icon::SIZE_SMALL
-								)
-									->render('inline') . '</td>' . '</tr>';
+							$itemContent .= '<td>' . $setting['vDEF'] . '</td>' . '</tr>';
 						}
-					// Remaining values
-					} else {
-						$itemContent .= '<td>' . $setting['vDEF'] . '</td>' . '</tr>';
 					}
 				}
-			}
 
-			// Record Storage
-			if ($row['pages'] !== '') {
-				// Record Storage Page label
-				$labelRecordStorage = 'LLL:EXT:frontend/Resources/Private/Language/locallang_ttc.xlf:pages.ALT.list_formlabel';
+				// Record Storage
+				if ($row['pages'] !== '') {
+					// Record Storage Page label
+					$labelRecordStorage = 'LLL:EXT:frontend/Resources/Private/Language/locallang_ttc.xlf:pages.ALT.list_formlabel';
 
-				// Get storage page(s)
-				$backendUtilityObject = GeneralUtility::makeInstance(BackendUtility::class);
-				$storagePageIds = explode(',', $row['pages']);
-				$storagePages = array();
-
-				foreach ($storagePageIds as $i => $storagePageId) {
-					$storagePages[] = $backendUtilityObject->getRecord(
-							'pages', $storagePageId, 'title'
-						)['title'] . ' ' . '(' . $storagePageId . ')';
+					// Render 'Record Storage Page' row
+					$itemContent .= '<row>' . '<td><strong>' . $translationServiceObject->translate(
+							$labelRecordStorage
+						) . '</strong></td>';
+					$itemContent .= '<td>' . implode(
+							', ', $this->buildRecordArray('pages', $row['pages'])
+						) . '</td>' . '</tr>';
 				}
 
-				// Render 'Record Storage Page' row
-				$itemContent .= '<row>' . '<td><strong>' . $translationServiceObject->translate(
-						$labelRecordStorage
-					) . '</strong></td>';
-				$itemContent .= '<td>' . implode(', ', $storagePages) . '</td>' . '</tr>';
+				// End rendering of table
+				$itemContent .= '</table>';
 			}
-
-			// End rendering of table
-			$itemContent .= '</table>';
-			$drawItem = FALSE;
 		}
 	}
 }
-- 
GitLab


From 286cacb3e0204c99530e7890041c7d5df3937e5d Mon Sep 17 00:00:00 2001
From: Tim Wagner <tim.wagner@sgalinski.de>
Date: Wed, 24 Jun 2020 20:23:45 +0200
Subject: [PATCH 6/8] [TASK] Render plugin backend preview with Fluid template

---
 .../Hooks/PageLayoutView/PluginRenderer.php   | 108 ++++++++++
 Classes/Hooks/PluginRenderer.php              | 187 ------------------
 .../FlexForms/flexform_sgyoutube_youtube.xml  |   4 +-
 Resources/Private/Language/de.locallang.xlf   |  42 ++--
 Resources/Private/Language/locallang.xlf      |  34 ++--
 .../Private/Templates/Youtube/Backend.html    |  31 +++
 ext_localconf.php                             |   2 +-
 7 files changed, 187 insertions(+), 221 deletions(-)
 create mode 100644 Classes/Hooks/PageLayoutView/PluginRenderer.php
 delete mode 100644 Classes/Hooks/PluginRenderer.php
 create mode 100644 Resources/Private/Templates/Youtube/Backend.html

diff --git a/Classes/Hooks/PageLayoutView/PluginRenderer.php b/Classes/Hooks/PageLayoutView/PluginRenderer.php
new file mode 100644
index 0000000..ebd6630
--- /dev/null
+++ b/Classes/Hooks/PageLayoutView/PluginRenderer.php
@@ -0,0 +1,108 @@
+<?php
+
+namespace SGalinski\SgYoutube\Hooks\PageLayoutView;
+
+/***************************************************************
+ *  Copyright notice
+ *  (c) sgalinski Internet Services (https://www.sgalinski.de)
+ *  All rights reserved
+ *  This script is part of the TYPO3 project. The TYPO3 project is
+ *  free software; you can redistribute it and/or modify
+ *  it under the terms of the GNU General Public License as published by
+ *  the Free Software Foundation; either version 3 of the License, or
+ *  (at your option) any later version.
+ *  The GNU General Public License can be found at
+ *  http://www.gnu.org/copyleft/gpl.html.
+ *  This script is distributed in the hope that it will be useful,
+ *  but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ *  GNU General Public License for more details.
+ *  This copyright notice MUST APPEAR in all copies of the script!
+ ***************************************************************/
+
+use TYPO3\CMS\Backend\Utility\BackendUtility;
+use TYPO3\CMS\Backend\View\PageLayoutView;
+use TYPO3\CMS\Backend\View\PageLayoutViewDrawItemHookInterface;
+use TYPO3\CMS\Core\Utility\GeneralUtility;
+use TYPO3\CMS\Fluid\View\StandaloneView;
+use \TYPO3\CMS\Core\Localization\LanguageService;
+
+/**
+ * Class PluginRenderer
+ * Renders back-end preview for the SG YouTube Videos plugin
+ *
+ * @package SGalinski\SgYoutube\Hooks
+ */
+class PluginRenderer implements PageLayoutViewDrawItemHookInterface {
+	/**
+	 * Return an array of pages or categories, with their respective ids
+	 *
+	 * @param $recodIds
+	 * @return array $records
+	 */
+	public function buildRecordArray($table, $recodIds) {
+		$backendUtilityObject = GeneralUtility::makeInstance(BackendUtility::class);
+		$recordsArray = explode(',', $recodIds);
+		$records = [];
+
+		foreach ($recordsArray as $i => $recordId) {
+			$records[] = $backendUtilityObject::getRecord(
+					$table, $recordId, 'title'
+				)['title'] . ' ' . '(' . $recordId . ')';
+		}
+		return $records;
+	}
+
+	/**
+	 * Preprocesses the preview rendering of a content element of type "sg_youtube"
+	 *
+	 * @param PageLayoutView $parentObject Calling parent object
+	 * @param bool $drawItem Whether to draw the item using the default functionality
+	 * @param string $headerContent Header content
+	 * @param string $itemContent Item content
+	 * @param array $row Record row of tt_content
+	 * @return void
+	 * @noinspection ReferencingObjectsInspection
+	 */
+	public function preProcess(
+		PageLayoutView &$parentObject, &$drawItem, &$headerContent, &$itemContent, array &$row
+	): void {
+		if ($row['list_type'] === 'sgyoutube_youtube') {
+			$drawItem = FALSE;
+
+			$view = GeneralUtility::makeInstance(StandaloneView::class);
+			$view->setTemplateRootPaths(['EXT:sg_youtube/Resources/Private/Templates/Youtube']);
+			$view->setTemplate('Backend.html');
+			$view->assign('uid', $row['uid']);
+
+			// Set plugin options array
+			$pluginOptions = GeneralUtility::xml2array($row['pi_flexform'], 'T3:');
+			$templateOptions = [];
+			foreach ($pluginOptions['data']['sDEF']['lDEF'] as $option => $value) {
+				if ($option !== 'settings.showApiResult') {
+					// $option has the format 'settings.<key>', but in the template we need the language label which
+					// has the format 'flexform.<key>'.
+					$templateOptions[str_replace('settings.', 'flexform.', $option)] = $value['vDEF'];
+				}
+			}
+
+			$languageService = GeneralUtility::makeInstance(LanguageService::class);
+			$templateOptions['flexform.thumbnailType'] = $languageService->sL(
+				'LLL:EXT:sg_youtube/Resources/Private/Language/locallang.xlf:flexform.thumbnailType.' . $templateOptions['flexform.thumbnailType']
+			);
+
+			$view->assignMultiple([
+				'pluginOptions' => $templateOptions,
+				'showApiResult' => $pluginOptions['data']['sDEF']['lDEF']['settings.showApiResult']['vDEF']
+			]);
+
+			// Content element labels
+			$pluginName = BackendUtility::getLabelFromItemListMerged(
+				$row['pid'], 'tt_content', 'list_type', $row['list_type']
+			);
+			$headerContent = '<strong>[' . $pluginName . '] </strong>' . $headerContent;
+
+			$itemContent .= $view->render();
+		}
+	}
+}
diff --git a/Classes/Hooks/PluginRenderer.php b/Classes/Hooks/PluginRenderer.php
deleted file mode 100644
index 448c812..0000000
--- a/Classes/Hooks/PluginRenderer.php
+++ /dev/null
@@ -1,187 +0,0 @@
-<?php
-
-namespace SGalinski\SgYoutube\Hooks;
-
-/***************************************************************
- *  Copyright notice
- *  (c) sgalinski Internet Services (https://www.sgalinski.de)
- *  All rights reserved
- *  This script is part of the TYPO3 project. The TYPO3 project is
- *  free software; you can redistribute it and/or modify
- *  it under the terms of the GNU General Public License as published by
- *  the Free Software Foundation; either version 3 of the License, or
- *  (at your option) any later version.
- *  The GNU General Public License can be found at
- *  http://www.gnu.org/copyleft/gpl.html.
- *  This script is distributed in the hope that it will be useful,
- *  but WITHOUT ANY WARRANTY; without even the implied warranty of
- *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- *  GNU General Public License for more details.
- *  This copyright notice MUST APPEAR in all copies of the script!
- ***************************************************************/
-
-use TYPO3\CMS\Backend\Utility\BackendUtility;
-use TYPO3\CMS\Backend\View\PageLayoutView;
-use TYPO3\CMS\Backend\View\PageLayoutViewDrawItemHookInterface;
-use TYPO3\CMS\Core\Imaging\Icon;
-use TYPO3\CMS\Core\Imaging\IconFactory;
-use TYPO3\CMS\Core\Utility\GeneralUtility;
-use TYPO3\CMS\Form\Service\TranslationService;
-
-/**
- * Class PluginRenderer
- * Renders back-end preview for the SG YouTube Videos plugin
- *
- * @package SGalinski\SgYoutube\Hooks
- */
-class PluginRenderer implements PageLayoutViewDrawItemHookInterface {
-	/**
-	 * Return an array of pages or categories, with their respective ids
-	 *
-	 * @param $recodIds
-	 * @return array $records
-	 */
-	public function buildRecordArray($table, $recodIds) {
-		$backendUtilityObject = GeneralUtility::makeInstance(BackendUtility::class);
-		$recordsArray = explode(',', $recodIds);
-		$records = [];
-
-		foreach ($recordsArray as $i => $recordId) {
-			$records[] = $backendUtilityObject->getRecord(
-					$table, $recordId, 'title'
-				)['title'] . ' ' . '(' . $recordId . ')';
-		}
-		return $records;
-	}
-
-	/**
-	 * Preprocesses the preview rendering of a content element of type "sg_youtube"
-	 *
-	 * @param PageLayoutView $parentObject Calling parent object
-	 * @param bool $drawItem Whether to draw the item using the default functionality
-	 * @param string $headerContent Header content
-	 * @param string $itemContent Item content
-	 * @param array $row Record row of tt_content
-	 * @return void
-	 */
-	public function preProcess(PageLayoutView &$parentObject, &$drawItem, &$headerContent, &$itemContent, array &$row) {
-		if ($row['list_type'] === 'sgyoutube_youtube') {
-
-			$drawItem = FALSE;
-
-			// Content element labels
-			$labelPluginName = BackendUtility::getLabelFromItemListMerged(
-				$row['pid'], 'tt_content', 'list_type', $row['list_type']
-			);
-			$labelSubheader = BackendUtility::getItemLabel(
-				'tt_content', 'subheader'
-			);
-
-			// Instantiate TranslationService class
-			$translationServiceObject = GeneralUtility::makeInstance(TranslationService::class);
-
-			// Begin rendering of content element fields
-			// Plugin name
-			$headerContent = '<div style="padding: 0 4px 10px 0;border-bottom: 1px solid #CACACA;margin-bottom: 6px; font-weight:700;">' . $labelPluginName . '</div>';
-
-			// Header
-			if ($row['header']) {
-				if ($row['header_layout'] === '10') {
-					$headerContent .= '<p>' . $row['header'] . '</p>';
-				} elseif ($row['header_layout'] === '100') {
-					$headerContent .= '<div style="display:none;">' . $row['header'] . '</div>';
-				} else {
-					$headerContent .= '<h' . $row['header_layout'] . '>' . $row['header'] . '</h' . $row['header_layout'] . '>';
-				}
-			}
-
-			// Subheader
-			if ($row['subheader']) {
-				$headerContent .= '<div>' . '<strong>' . $translationServiceObject->translate(
-						$labelSubheader
-					) . '</strong>' . ':&nbsp;' . $row['subheader'] . '</div>';
-			}
-
-			// Set plugin options array
-			$pluginOptions = GeneralUtility::xml2array($row['pi_flexform'], 'T3:');
-
-			// Get the FlexForm configuration file
-			$xmlFilePath = $GLOBALS['TCA']['tt_content']['columns']['pi_flexform']['config']['ds']['sgyoutube_youtube,list'];
-
-			if ($xmlFilePath !== '') {
-				$xmlContent = GeneralUtility::getFileAbsFileName(
-					str_replace(
-						'FILE:EXT:', 'EXT:', $xmlFilePath
-					)
-				);
-
-				// Get contents of the FlexForm configuration file
-				$xmlString = file_get_contents($xmlContent);
-				$configurationArray = GeneralUtility::xml2array($xmlString);
-
-				// Begin rendering of table
-				$itemContent .= '<br>' . '<table class="table table-striped">';
-
-				// Table header
-				$labelSettings = 'LLL:EXT:sg_youtube/Resources/Private/Language/locallang.xlf:settings';
-				$labelvalue = 'LLL:EXT:sg_youtube/Resources/Private/Language/locallang.xlf:values';
-				$itemContent .= '<row>' . '<th style="width: 50%">' . $translationServiceObject->translate(
-						$labelSettings
-					) . '</th>';
-				$itemContent .= '<th style="width: 50%">' . $translationServiceObject->translate(
-						$labelvalue
-					) . '</th>' . '</tr>';
-
-				// Loop through plugin options and render rows
-				foreach ($pluginOptions['data']['sDEF']['lDEF'] as $key => $setting) {
-
-					// if plugin option exists within the Flexform configuration file
-					if (array_key_exists($key, $configurationArray['sheets']['sDEF']['ROOT']['el'])) {
-
-						// Display label
-						$labelId = $configurationArray['sheets']['sDEF']['ROOT']['el'][$key]['TCEforms']['label'];
-						$itemContent .= '<row>' . '<td><strong>' . $translationServiceObject->translate(
-								trim($labelId)
-							) . '</strong></td>';
-
-						// Display value
-						// Show different icon for the 'Show API Result' field, depending on checkbox value
-						if ($key === 'settings.showApiResult') {
-							if ($setting['vDEF'] === '0') {
-								$itemContent .= '<td>' . GeneralUtility::makeInstance(IconFactory::class)->getIcon(
-										'actions-close', Icon::SIZE_SMALL
-									)
-										->render('inline') . '</td>' . '</tr>';
-							} else {
-								$itemContent .= '<td>' . GeneralUtility::makeInstance(IconFactory::class)->getIcon(
-										'actions-check', Icon::SIZE_SMALL
-									)
-										->render('inline') . '</td>' . '</tr>';
-							}
-							// Remaining values
-						} else {
-							$itemContent .= '<td>' . $setting['vDEF'] . '</td>' . '</tr>';
-						}
-					}
-				}
-
-				// Record Storage
-				if ($row['pages'] !== '') {
-					// Record Storage Page label
-					$labelRecordStorage = 'LLL:EXT:frontend/Resources/Private/Language/locallang_ttc.xlf:pages.ALT.list_formlabel';
-
-					// Render 'Record Storage Page' row
-					$itemContent .= '<row>' . '<td><strong>' . $translationServiceObject->translate(
-							$labelRecordStorage
-						) . '</strong></td>';
-					$itemContent .= '<td>' . implode(
-							', ', $this->buildRecordArray('pages', $row['pages'])
-						) . '</td>' . '</tr>';
-				}
-
-				// End rendering of table
-				$itemContent .= '</table>';
-			}
-		}
-	}
-}
diff --git a/Configuration/FlexForms/flexform_sgyoutube_youtube.xml b/Configuration/FlexForms/flexform_sgyoutube_youtube.xml
index e2d3945..5522d30 100644
--- a/Configuration/FlexForms/flexform_sgyoutube_youtube.xml
+++ b/Configuration/FlexForms/flexform_sgyoutube_youtube.xml
@@ -16,7 +16,7 @@
 					<settings.id>
 						<TCEforms>
 							<exclude>0</exclude>
-							<label>LLL:EXT:sg_youtube/Resources/Private/Language/locallang.xlf:id
+							<label>LLL:EXT:sg_youtube/Resources/Private/Language/locallang.xlf:flexform.id
 							</label>
 							<config>
 								<type>input</type>
@@ -28,7 +28,7 @@
 					<settings.maxResults>
 						<TCEforms>
 							<exclude>0</exclude>
-							<label>LLL:EXT:sg_youtube/Resources/Private/Language/locallang.xlf:maxResults
+							<label>LLL:EXT:sg_youtube/Resources/Private/Language/locallang.xlf:flexform.maxResults
 							</label>
 							<config>
 								<type>input</type>
diff --git a/Resources/Private/Language/de.locallang.xlf b/Resources/Private/Language/de.locallang.xlf
index 09a1846..948ea40 100644
--- a/Resources/Private/Language/de.locallang.xlf
+++ b/Resources/Private/Language/de.locallang.xlf
@@ -1,14 +1,38 @@
 <?xml version="1.0" encoding="utf-8" standalone="yes" ?>
 <xliff version="1.0">
-	<file source-language="en" target-language="de" datatype="plaintext" original="messages" date="2020-06-10T09:14:47Z">
+	<file source-language="en" target-language="de" datatype="plaintext" original="messages" date="2020-06-24T18:20:58Z">
 		<header>
 			<generator>LFEditor</generator>
 		</header>
 		<body>
+			<trans-unit id="backendPreview.apiDebugWarning.header" approved="yes">
+				<source><![CDATA[API Debugging Information is Shown in the Frontend]]></source>
+				<target><![CDATA[API-Debugging-Details werden im Frontend gerendert]]></target>
+			</trans-unit>
+			<trans-unit id="backendPreview.apiDebugWarning.message" approved="yes">
+				<source><![CDATA[Uncheck the debugging option in the plugin options to not show this information.]]></source>
+				<target><![CDATA[Deaktivieren Sie die Debugging-Option in den Plugin-Einstellungen, um diese Informationen nicht anzuzeigen.]]></target>
+			</trans-unit>
+			<trans-unit id="backendPreview.settings" approved="yes">
+				<source><![CDATA[Settings]]></source>
+				<target><![CDATA[Einstellungen]]></target>
+			</trans-unit>
+			<trans-unit id="backendPreview.values" approved="yes">
+				<source><![CDATA[Values]]></source>
+				<target><![CDATA[Werte]]></target>
+			</trans-unit>
 			<trans-unit id="flexform.aspectRatio" approved="yes">
 				<source><![CDATA[Aspect Ratio (only used if thumbnail type is set to check that)]]></source>
 				<target><![CDATA[Aspect Ratio (nur genutzt, wenn der Thumbnail-Typ dies prüfen soll)]]></target>
 			</trans-unit>
+			<trans-unit id="flexform.id" approved="yes">
+				<source><![CDATA[ID of Channel (UC), Playlist (PL) or Single Video (youtube.com/watch?v=%ID%)]]></source>
+				<target><![CDATA[ID eines Channels (UC), einer Playlist (PL) oder eines einzelnen Videos (youtube.com/watch?v=%ID%)]]></target>
+			</trans-unit>
+			<trans-unit id="flexform.maxResults" approved="yes">
+				<source><![CDATA[Maximum Amount]]></source>
+				<target><![CDATA[Maximale Anzahl]]></target>
+			</trans-unit>
 			<trans-unit id="flexform.showApiResult" approved="yes">
 				<source><![CDATA[Show API Result (Debug Output!)]]></source>
 				<target><![CDATA[Zeige API-Ergebnis an (Debug-Informationen!)]]></target>
@@ -41,22 +65,6 @@
 				<source><![CDATA[Standard Resolution]]></source>
 				<target><![CDATA[Standard-Auflösung]]></target>
 			</trans-unit>
-			<trans-unit id="id" approved="yes">
-				<source><![CDATA[ID of Channel (UC), Playlist (PL) or Single Video (youtube.com/watch?v=%ID%)]]></source>
-				<target><![CDATA[ID eines Channels (UC), einer Playlist (PL) oder eines einzelnen Videos (youtube.com/watch?v=%ID%)]]></target>
-			</trans-unit>
-			<trans-unit id="maxResults" approved="yes">
-				<source><![CDATA[Maximum Amount]]></source>
-				<target><![CDATA[Maximale Anzahl]]></target>
-			</trans-unit>
-			<trans-unit id="settings" approved="yes">
-				<source><![CDATA[Settings]]></source>
-				<target><![CDATA[Einstellungen]]></target>
-			</trans-unit>
-			<trans-unit id="values" approved="yes">
-				<source><![CDATA[Values]]></source>
-				<target><![CDATA[Werte]]></target>
-			</trans-unit>
 		</body>
 	</file>
 </xliff>
\ No newline at end of file
diff --git a/Resources/Private/Language/locallang.xlf b/Resources/Private/Language/locallang.xlf
index 11714e9..137e03d 100644
--- a/Resources/Private/Language/locallang.xlf
+++ b/Resources/Private/Language/locallang.xlf
@@ -1,13 +1,31 @@
 <?xml version="1.0" encoding="utf-8" standalone="yes" ?>
 <xliff version="1.0">
-	<file source-language="en" datatype="plaintext" original="messages" date="2019-12-07T19:09:30Z">
+	<file source-language="en" datatype="plaintext" original="messages" date="2020-06-24T18:20:58Z">
 		<header>
 			<generator>LFEditor</generator>
 		</header>
 		<body>
+			<trans-unit id="backendPreview.apiDebugWarning.header">
+				<source><![CDATA[API Debugging Information is Shown in the Frontend]]></source>
+			</trans-unit>
+			<trans-unit id="backendPreview.apiDebugWarning.message">
+				<source><![CDATA[Uncheck the debugging option in the plugin options to not show this information.]]></source>
+			</trans-unit>
+			<trans-unit id="backendPreview.settings">
+				<source><![CDATA[Settings]]></source>
+			</trans-unit>
+			<trans-unit id="backendPreview.values">
+				<source><![CDATA[Values]]></source>
+			</trans-unit>
 			<trans-unit id="flexform.aspectRatio">
 				<source><![CDATA[Aspect Ratio (only used if thumbnail type is set to check that)]]></source>
 			</trans-unit>
+			<trans-unit id="flexform.id">
+				<source><![CDATA[ID of Channel (UC), Playlist (PL) or Single Video (youtube.com/watch?v=%ID%)]]></source>
+			</trans-unit>
+			<trans-unit id="flexform.maxResults">
+				<source><![CDATA[Maximum Amount]]></source>
+			</trans-unit>
 			<trans-unit id="flexform.showApiResult">
 				<source><![CDATA[Show API Result (Debug Output!)]]></source>
 			</trans-unit>
@@ -32,18 +50,6 @@
 			<trans-unit id="flexform.thumbnailType.standard">
 				<source><![CDATA[Standard Resolution]]></source>
 			</trans-unit>
-			<trans-unit id="id">
-				<source><![CDATA[ID of Channel (UC), Playlist (PL) or Single Video (youtube.com/watch?v=%ID%)]]></source>
-			</trans-unit>
-			<trans-unit id="maxResults">
-				<source><![CDATA[Maximum Amount]]></source>
-			</trans-unit>
-			<trans-unit id="settings">
-				<source><![CDATA[Settings]]></source>
-			</trans-unit>
-			<trans-unit id="values">
-				<source><![CDATA[Values]]></source>
-			</trans-unit>
 		</body>
 	</file>
-</xliff>
+</xliff>
\ No newline at end of file
diff --git a/Resources/Private/Templates/Youtube/Backend.html b/Resources/Private/Templates/Youtube/Backend.html
new file mode 100644
index 0000000..042ec99
--- /dev/null
+++ b/Resources/Private/Templates/Youtube/Backend.html
@@ -0,0 +1,31 @@
+{namespace be=TYPO3\CMS\Backend\ViewHelpers}
+
+<be:link.editRecord uid="{uid}" table="tt_content">
+	<br/>
+	<f:if condition="{showApiResult} == 1">
+		<div class="alert alert-warning">
+			<h4 class="alert-title"><f:translate key="backendPreview.apiDebugWarning.header" extensionName="sg_youtube"/></h4>
+			<p class="alert-message"><f:translate key="backendPreview.apiDebugWarning.message" extensionName="sg_youtube"/></p>
+		</div>
+	</f:if>
+	<table class="table table-striped">
+		<tr>
+			<th style="width: 50%">
+				<f:translate key="backendPreview.settings" extensionName="sg_youtube"/>
+			</th>
+			<th style="width: 50%">
+				<f:translate key="backendPreview.values" extensionName="sg_youtube"/>
+			</th>
+		</tr>
+		<f:for each="{pluginOptions}" as="option" key="labelKey">
+			<tr>
+				<th>
+					<f:translate key="{labelKey}" extensionName="sg_youtube"/>
+				</th>
+				<td>
+					{option}
+				</td>
+			</tr>
+		</f:for>
+	</table>
+</be:link.editRecord>
diff --git a/ext_localconf.php b/ext_localconf.php
index ad9a2bd..aead3e8 100644
--- a/ext_localconf.php
+++ b/ext_localconf.php
@@ -42,4 +42,4 @@ if (!defined('TYPO3_MODE')) {
 );
 //Hooks
 $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['cms/layout/class.tx_cms_layout.php']['tt_content_drawItem']['sg_youtube']
-	= \SGalinski\SgYoutube\Hooks\PluginRenderer::class;
+	= \SGalinski\SgYoutube\Hooks\PageLayoutView\PluginRenderer::class;
-- 
GitLab


From 91f92a0516a3e7dee29024c6087a7f96725841ce Mon Sep 17 00:00:00 2001
From: Tim Wagner <tim.wagner@sgalinski.de>
Date: Wed, 24 Jun 2020 23:14:10 +0200
Subject: [PATCH 7/8] [TASK] Clean up PluginRenderer and improve Fluid for
 backend preview

---
 .../Hooks/PageLayoutView/PluginRenderer.php   | 23 +---------
 .../Private/Templates/Youtube/Backend.html    | 42 +++++++++++--------
 2 files changed, 27 insertions(+), 38 deletions(-)

diff --git a/Classes/Hooks/PageLayoutView/PluginRenderer.php b/Classes/Hooks/PageLayoutView/PluginRenderer.php
index ebd6630..d0a8de4 100644
--- a/Classes/Hooks/PageLayoutView/PluginRenderer.php
+++ b/Classes/Hooks/PageLayoutView/PluginRenderer.php
@@ -34,25 +34,6 @@ use \TYPO3\CMS\Core\Localization\LanguageService;
  * @package SGalinski\SgYoutube\Hooks
  */
 class PluginRenderer implements PageLayoutViewDrawItemHookInterface {
-	/**
-	 * Return an array of pages or categories, with their respective ids
-	 *
-	 * @param $recodIds
-	 * @return array $records
-	 */
-	public function buildRecordArray($table, $recodIds) {
-		$backendUtilityObject = GeneralUtility::makeInstance(BackendUtility::class);
-		$recordsArray = explode(',', $recodIds);
-		$records = [];
-
-		foreach ($recordsArray as $i => $recordId) {
-			$records[] = $backendUtilityObject::getRecord(
-					$table, $recordId, 'title'
-				)['title'] . ' ' . '(' . $recordId . ')';
-		}
-		return $records;
-	}
-
 	/**
 	 * Preprocesses the preview rendering of a content element of type "sg_youtube"
 	 *
@@ -80,9 +61,9 @@ class PluginRenderer implements PageLayoutViewDrawItemHookInterface {
 			$templateOptions = [];
 			foreach ($pluginOptions['data']['sDEF']['lDEF'] as $option => $value) {
 				if ($option !== 'settings.showApiResult') {
-					// $option has the format 'settings.<key>', but in the template we need the language label which
+					// $option has the format 'settings.<key>', but in the template we want the language label which
 					// has the format 'flexform.<key>'.
-					$templateOptions[str_replace('settings.', 'flexform.', $option)] = $value['vDEF'];
+					$templateOptions[str_replace('settings.', '', $option)] = $value['vDEF'];
 				}
 			}
 
diff --git a/Resources/Private/Templates/Youtube/Backend.html b/Resources/Private/Templates/Youtube/Backend.html
index 042ec99..a22eabb 100644
--- a/Resources/Private/Templates/Youtube/Backend.html
+++ b/Resources/Private/Templates/Youtube/Backend.html
@@ -4,28 +4,36 @@
 	<br/>
 	<f:if condition="{showApiResult} == 1">
 		<div class="alert alert-warning">
-			<h4 class="alert-title"><f:translate key="backendPreview.apiDebugWarning.header" extensionName="sg_youtube"/></h4>
-			<p class="alert-message"><f:translate key="backendPreview.apiDebugWarning.message" extensionName="sg_youtube"/></p>
+			<h4 class="alert-title">
+				<f:translate key="backendPreview.apiDebugWarning.header" extensionName="SgYoutube"/>
+			</h4>
+			<p class="alert-message">
+				<f:translate key="backendPreview.apiDebugWarning.message" extensionName="SgYoutube"/>
+			</p>
 		</div>
 	</f:if>
 	<table class="table table-striped">
-		<tr>
-			<th style="width: 50%">
-				<f:translate key="backendPreview.settings" extensionName="sg_youtube"/>
-			</th>
-			<th style="width: 50%">
-				<f:translate key="backendPreview.values" extensionName="sg_youtube"/>
-			</th>
-		</tr>
-		<f:for each="{pluginOptions}" as="option" key="labelKey">
+		<thead>
 			<tr>
-				<th>
-					<f:translate key="{labelKey}" extensionName="sg_youtube"/>
+				<th scope="col" style="width: 50%">
+					<f:translate key="backendPreview.settings" extensionName="SgYoutube"/>
+				</th>
+				<th scope="col" style="width: 50%">
+					<f:translate key="backendPreview.values" extensionName="SgYoutube"/>
 				</th>
-				<td>
-					{option}
-				</td>
 			</tr>
-		</f:for>
+		</thead>
+		<tbody>
+			<f:for each="{pluginOptions}" as="option" key="labelKey">
+				<tr>
+					<th scope="row">
+						<f:translate key="flexform.{labelKey}" extensionName="SgYoutube"/>
+					</th>
+					<td>
+						{option}
+					</td>
+				</tr>
+			</f:for>
+		</tbody>
 	</table>
 </be:link.editRecord>
-- 
GitLab


From a2a1c5fac2c32676514c42e71fc88f2ceae0c70f Mon Sep 17 00:00:00 2001
From: Kevin Ditscheid <kevin.ditscheid@sgalinski.de>
Date: Fri, 7 Aug 2020 12:08:04 +0200
Subject: [PATCH 8/8] [BUGFIX] Fix broken flexform configuration field

---
 .../Hooks/PageLayoutView/PluginRenderer.php   | 30 ++++++++++---------
 .../FlexForms/flexform_sgyoutube_youtube.xml  |  2 ++
 2 files changed, 18 insertions(+), 14 deletions(-)

diff --git a/Classes/Hooks/PageLayoutView/PluginRenderer.php b/Classes/Hooks/PageLayoutView/PluginRenderer.php
index d0a8de4..4096148 100644
--- a/Classes/Hooks/PageLayoutView/PluginRenderer.php
+++ b/Classes/Hooks/PageLayoutView/PluginRenderer.php
@@ -59,23 +59,25 @@ class PluginRenderer implements PageLayoutViewDrawItemHookInterface {
 			// Set plugin options array
 			$pluginOptions = GeneralUtility::xml2array($row['pi_flexform'], 'T3:');
 			$templateOptions = [];
-			foreach ($pluginOptions['data']['sDEF']['lDEF'] as $option => $value) {
-				if ($option !== 'settings.showApiResult') {
-					// $option has the format 'settings.<key>', but in the template we want the language label which
-					// has the format 'flexform.<key>'.
-					$templateOptions[str_replace('settings.', '', $option)] = $value['vDEF'];
+			if (is_array($pluginOptions)) {
+				foreach ($pluginOptions['data']['sDEF']['lDEF'] as $option => $value) {
+					if ($option !== 'settings.showApiResult') {
+						// $option has the format 'settings.<key>', but in the template we want the language label which
+						// has the format 'flexform.<key>'.
+						$templateOptions[str_replace('settings.', '', $option)] = $value['vDEF'];
+					}
 				}
-			}
 
-			$languageService = GeneralUtility::makeInstance(LanguageService::class);
-			$templateOptions['flexform.thumbnailType'] = $languageService->sL(
-				'LLL:EXT:sg_youtube/Resources/Private/Language/locallang.xlf:flexform.thumbnailType.' . $templateOptions['flexform.thumbnailType']
-			);
+				$languageService = GeneralUtility::makeInstance(LanguageService::class);
+				$templateOptions['flexform.thumbnailType'] = $languageService->sL(
+					'LLL:EXT:sg_youtube/Resources/Private/Language/locallang.xlf:flexform.thumbnailType.' . $templateOptions['flexform.thumbnailType']
+				);
 
-			$view->assignMultiple([
-				'pluginOptions' => $templateOptions,
-				'showApiResult' => $pluginOptions['data']['sDEF']['lDEF']['settings.showApiResult']['vDEF']
-			]);
+				$view->assignMultiple([
+					'pluginOptions' => $templateOptions,
+					'showApiResult' => $pluginOptions['data']['sDEF']['lDEF']['settings.showApiResult']['vDEF']
+				]);
+			}
 
 			// Content element labels
 			$pluginName = BackendUtility::getLabelFromItemListMerged(
diff --git a/Configuration/FlexForms/flexform_sgyoutube_youtube.xml b/Configuration/FlexForms/flexform_sgyoutube_youtube.xml
index 5522d30..167177c 100644
--- a/Configuration/FlexForms/flexform_sgyoutube_youtube.xml
+++ b/Configuration/FlexForms/flexform_sgyoutube_youtube.xml
@@ -58,6 +58,7 @@
 							<label>LLL:EXT:sg_youtube/Resources/Private/Language/locallang.xlf:flexform.aspectRatio</label>
 							<config>
 								<type>select</type>
+								<renderType>selectSingle</renderType>
 								<items type="array">
 									<numIndex index="0" type="array">
 										<numIndex index="0">16:9</numIndex>
@@ -76,6 +77,7 @@
 							<label>LLL:EXT:sg_youtube/Resources/Private/Language/locallang.xlf:flexform.thumbnailType</label>
 							<config>
 								<type>select</type>
+								<renderType>selectSingle</renderType>
 								<items type="array">
 									<numIndex index="0" type="array">
 										<numIndex index="0">LLL:EXT:sg_youtube/Resources/Private/Language/locallang.xlf:flexform.thumbnailType.byAspectRatio</numIndex>
-- 
GitLab