[BUGFIX] Always use self closing tags with a space while importing 82/55582/2
authorCybercraft <info@cybercraft.de>
Tue, 6 Feb 2018 18:09:28 +0000 (19:09 +0100)
committerJo Hasenau <info@cybercraft.de>
Tue, 6 Feb 2018 18:12:39 +0000 (19:12 +0100)
Replaced other appearances of <br> and <br/> too except those
needed for replacement.

Resolves: #83061
Releases: master, 8-0
Change-Id: I6883bebc31b7a8f6c2da38667b63f59c184994b5
Reviewed-on: https://review.typo3.org/55582
Reviewed-by: Jo Hasenau <info@cybercraft.de>
Tested-by: Jo Hasenau <info@cybercraft.de>
Classes/Controller/Cm2.php
Classes/Controller/Cm3.php
Classes/Controller/LocalizationManager.php
Classes/Controller/TranslationTasks.php
Classes/View/CatXmlView.php
Classes/View/L10nHtmlListView.php

index 318638b..b4085c2 100644 (file)
@@ -169,14 +169,14 @@ class Cm2 extends BaseScriptClass
                     $tRows[] = $this->makeTableRow($rec);
                 }
             }
-            $output .= 'Path: <i>' . $pathShown . '</i><br><table border="0" cellpadding="1" cellspacing="1">' . implode('',
+            $output .= 'Path: <i>' . $pathShown . '</i><br /><table border="0" cellpadding="1" cellspacing="1">' . implode('',
                     $tRows) . '</table>';
             // Updating index
             if ($this->getBackendUser()->isAdmin()) {
-                $output .= '<br><br>Functions for "' . $table . ':' . $uid . '":<br/>
-       <input type="submit" name="_updateIndex" value="Update Index" /><br>
-       <input type="submit" name="_" value="Flush Translations" onclick="' . htmlspecialchars('document.location="../cm3/index.php?table=' . htmlspecialchars($table) . '&id=' . (int)$uid . '&cmd=flushTranslations";return false;') . '"/><br>
-       <input type="submit" name="_" value="Create priority" onclick="' . htmlspecialchars('document.location="' . $GLOBALS['BACK_PATH'] . 'alt_doc.php?returnUrl=' . rawurlencode('db_list.php?id=0&table=tx_l10nmgr_priorities') . '&edit[tx_l10nmgr_priorities][0]=new&defVals[tx_l10nmgr_priorities][element]=' . rawurlencode($table . '_' . $uid) . '";return false;') . '"/><br>
+                $output .= '<br /><br />Functions for "' . $table . ':' . $uid . '":<br />
+       <input type="submit" name="_updateIndex" value="Update Index" /><br />
+       <input type="submit" name="_" value="Flush Translations" onclick="' . htmlspecialchars('document.location="../cm3/index.php?table=' . htmlspecialchars($table) . '&id=' . (int)$uid . '&cmd=flushTranslations";return false;') . '"/><br />
+       <input type="submit" name="_" value="Create priority" onclick="' . htmlspecialchars('document.location="' . $GLOBALS['BACK_PATH'] . 'alt_doc.php?returnUrl=' . rawurlencode('db_list.php?id=0&table=tx_l10nmgr_priorities') . '&edit[tx_l10nmgr_priorities][0]=new&defVals[tx_l10nmgr_priorities][element]=' . rawurlencode($table . '_' . $uid) . '";return false;') . '"/><br />
        ';
             }
         }
@@ -235,7 +235,7 @@ class Cm2 extends BaseScriptClass
        <td align="center"' . ($rec['flag_unknown'] ? ' bgcolor="#FEFF5A"' : '') . '>' . ($rec['flag_unknown'] ? $rec['flag_unknown'] : '') . '</td>
        <td align="center"' . ($rec['flag_update'] ? ' bgcolor="#FF7161"' : '') . '>' . ($rec['flag_update'] ? $rec['flag_update'] : '') . '</td>
        <td align="center"' . ($rec['flag_noChange'] ? ' bgcolor="#78FF82"' : '') . '>' . ($rec['flag_noChange'] ? $rec['flag_noChange'] : '') . '</td>
-       <td>' . implode('<br/>', unserialize($rec['serializedDiff'])) . '</td>
+       <td>' . implode('<br />', unserialize($rec['serializedDiff'])) . '</td>
        </tr>';
     }
 
index 850cc6f..37e6947 100644 (file)
@@ -114,7 +114,7 @@ class Cm3 extends BaseScriptClass
                         $res = $this->l10nMgrTools->flushTranslations($table, $uid,
                             GeneralUtility::_POST('_flush') ? true : false);
                         if (!GeneralUtility::_POST('_flush')) {
-                            $output .= 'To flush the translations shown below, press the "Flush" button below:<br/><input type="submit" name="_flush" value="FLUSH" /><br/><br/>';
+                            $output .= 'To flush the translations shown below, press the "Flush" button below:<br /><input type="submit" name="_flush" value="FLUSH" /><br /><br />';
                         } else {
                             $output .= 'Translations below were flushed!';
                         }
index 05a8450..62c964b 100644 (file)
@@ -498,7 +498,7 @@ return false;
             $translationData->setPreviewLanguage($this->previewLanguage);
             GeneralUtility::unlink_tempfile($uploadedTempFile);
             $service->saveTranslation($l10ncfgObj, $translationData);
-            $info .= '<br/><br/>' . $this->moduleTemplate->icons(1) . $this->getLanguageService()->getLL('import.success.message') . '<br/><br/>';
+            $info .= '<br /><br />' . $this->moduleTemplate->icons(1) . $this->getLanguageService()->getLL('import.success.message') . '<br /><br />';
         }
         // If export of XML is asked for, do that (this will exit and push a file for download)
         if (GeneralUtility::_POST('export_excel')) {
@@ -657,24 +657,24 @@ return false;
                 $translationData->setLanguage($this->sysLanguage);
                 $translationData->setPreviewLanguage($this->previewLanguage);
                 $service->saveTranslation($l10ncfgObj, $translationData);
-                $actionInfo .= '<br/><br/>' . $this->moduleTemplate->icons(1) . 'Import done<br/><br/>(Command count:' . $service->lastTCEMAINCommandsCount . ')';
+                $actionInfo .= '<br /><br />' . $this->moduleTemplate->icons(1) . 'Import done<br /><br />(Command count:' . $service->lastTCEMAINCommandsCount . ')';
             } else {
                 // Relevant processing of XML Import with the help of the Importmanager
                 /** @var CatXmlImportManager $importManager */
                 $importManager = GeneralUtility::makeInstance(CatXmlImportManager::class, $uploadedTempFile,
                     $this->sysLanguage, $xmlString = "");
                 if ($importManager->parseAndCheckXMLFile() === false) {
-                    $actionInfo .= '<br/><br/>' . $this->moduleTemplate->header($this->getLanguageService()->getLL('import.error.title')) . $importManager->getErrorMessages();
+                    $actionInfo .= '<br /><br />' . $this->moduleTemplate->header($this->getLanguageService()->getLL('import.error.title')) . $importManager->getErrorMessages();
                 } else {
                     if (GeneralUtility::_POST('import_delL10N') == '1') {
-                        $actionInfo .= $this->getLanguageService()->getLL('import.xml.delL10N.message') . '<br/>';
+                        $actionInfo .= $this->getLanguageService()->getLL('import.xml.delL10N.message') . '<br />';
                         $delCount = $importManager->delL10N($importManager->getDelL10NDataFromCATXMLNodes($importManager->getXMLNodes()));
                         $actionInfo .= sprintf($this->getLanguageService()->getLL('import.xml.delL10N.count.message'),
-                                $delCount) . '<br/><br/>';
+                                $delCount) . '<br /><br />';
                     }
                     if (GeneralUtility::_POST('make_preview_link') == '1') {
                         $pageIds = $importManager->getPidsFromCATXMLNodes($importManager->getXmlNodes());
-                        $actionInfo .= '<b>' . $this->getLanguageService()->getLL('import.xml.preview_links.title') . '</b><br/>';
+                        $actionInfo .= '<b>' . $this->getLanguageService()->getLL('import.xml.preview_links.title') . '</b><br />';
                         /** @var MkPreviewLinkService $mkPreviewLinks */
                         $mkPreviewLinks = GeneralUtility::makeInstance(MkPreviewLinkService::class,
                             $t3_workspaceId = $importManager->headerData['t3_workspaceId'],
@@ -690,7 +690,7 @@ return false;
                     //$actionInfo.="<pre>".var_export($GLOBALS['BE_USER'],true)."</pre>";
                     unset($importManager);
                     $service->saveTranslation($l10ncfgObj, $translationData);
-                    $actionInfo .= '<br/>' . $this->moduleTemplate->icons(-1) . $this->getLanguageService()->getLL('import.xml.done.message') . '<br/><br/>(Command count:' . $service->lastTCEMAINCommandsCount . ')';
+                    $actionInfo .= '<br />' . $this->moduleTemplate->icons(-1) . $this->getLanguageService()->getLL('import.xml.done.message') . '<br /><br />(Command count:' . $service->lastTCEMAINCommandsCount . ')';
                 }
             }
             GeneralUtility::unlink_tempfile($uploadedTempFile);
index 94d6678..4741015 100644 (file)
@@ -118,7 +118,7 @@ class TranslationTasks extends BaseScriptClass
                 $c++;
                 $tRows[] = '
        <tr>
-       <td class="bgColor5"><strong>#' . ($c) . ': ' . htmlspecialchars($priorityRecord['title']) . '</strong><br>' . htmlspecialchars($priorityRecord['description']) . '</td>
+       <td class="bgColor5"><strong>#' . ($c) . ': ' . htmlspecialchars($priorityRecord['title']) . '</strong><br />' . htmlspecialchars($priorityRecord['description']) . '</td>
        </tr>
        <tr>
        <td>' . $lTable . '</td>
index 6d793b8..b2ac53b 100644 (file)
@@ -124,8 +124,8 @@ class CatXmlView extends AbstractExportView
                                             //Substitute > and < in non-RTE fields
                                             $dataForTranslation = str_replace(' < ', ' &lt; ', $dataForTranslation);
                                             $dataForTranslation = str_replace(' > ', ' &gt; ', $dataForTranslation);
-                                            $dataForTranslation = str_replace('<br>', '<br/>', $dataForTranslation);
-                                            $dataForTranslation = str_replace('<hr>', '<hr/>', $dataForTranslation);
+                                            $dataForTranslation = str_replace('<br>', '<br />', $dataForTranslation);
+                                            $dataForTranslation = str_replace('<hr>', '<hr />', $dataForTranslation);
                                             $params = $this->getBackendUser()->getModuleData('l10nmgr/cm1/prefs',
                                                 'prefs');
                                             if ($params['utf8'] == '1') {
index 69d2d59..1bed32f 100644 (file)
@@ -145,7 +145,7 @@ class L10nHtmlListView extends AbstractExportView
                                     }
                                     if (!$this->modeOnlyChanged || !$noChangeFlag) {
                                         $fieldCells = array();
-                                        $fieldCells[] = '<b>' . htmlspecialchars($fieldName) . '</b>' . ($tData['msg'] ? '<br/><em>' . htmlspecialchars($tData['msg']) . '</em>' : '');
+                                        $fieldCells[] = '<b>' . htmlspecialchars($fieldName) . '</b>' . ($tData['msg'] ? '<br /><em>' . htmlspecialchars($tData['msg']) . '</em>' : '');
                                         $fieldCells[] = nl2br(htmlspecialchars($tData['defaultValue']));
                                         $fieldCells[] = $edit && $this->modeWithInlineEdit ? ($tData['fieldType'] === 'text' ? '<textarea name="' . htmlspecialchars('translation[' . $table . '][' . $elementUid . '][' . $key . ']') . '" cols="60" rows="5">' . LF . htmlspecialchars($tData['translationValue']) . '</textarea>' : '<input name="' . htmlspecialchars('translation[' . $table . '][' . $elementUid . '][' . $key . ']') . '" value="' . htmlspecialchars($tData['translationValue']) . '" size="60" />') : nl2br(htmlspecialchars($tData['translationValue']));
                                         $fieldCells[] = $diff;