[TASK] Change duplicate icon identifiers to actions-add 89/52289/4
authorMarco Christian Krenn <krenn@webconsulting.at>
Fri, 31 Mar 2017 10:56:14 +0000 (12:56 +0200)
committerBenjamin Kott <benjamin.kott@outlook.com>
Fri, 31 Mar 2017 19:27:32 +0000 (21:27 +0200)
The following icons which were previously used in the Install tool
were all replaced by "actions-add" which acts as a synonym.

* actions-edit-add

Releases: master
Resolves: #80603
Change-Id: Ie35e600799f49d53e68a2308d8e9f007189cdbdd
Reviewed-on: https://review.typo3.org/52289
Reviewed-by: Frank Naegler <frank.naegler@typo3.org>
Tested-by: Frank Naegler <frank.naegler@typo3.org>
Reviewed-by: Benjamin Kott <benjamin.kott@outlook.com>
Tested-by: Benjamin Kott <benjamin.kott@outlook.com>
typo3/sysext/backend/Classes/RecordList/ElementBrowserRecordList.php
typo3/sysext/beuser/Resources/Private/Partials/BackendUser/IndexListRow.html
typo3/sysext/core/Classes/Imaging/IconRegistry.php
typo3/sysext/core/Documentation/Changelog/master/Deprecation-80603-ChangeDuplicateIconIdentifiersToActions-add.rst [new file with mode: 0644]
typo3/sysext/recordlist/Classes/Browser/DatabaseBrowser.php
typo3/sysext/recordlist/Classes/Browser/FileBrowser.php
typo3/sysext/recordlist/Classes/Browser/FolderBrowser.php
typo3/sysext/rtehtmlarea/Classes/ImageHandler/AddImageHandler.php

index fbdb8c4..9a27e75 100644 (file)
@@ -61,7 +61,7 @@ class ElementBrowserRecordList extends DatabaseRecordList
         $ATag_alt = '<a href="#" data-close="1" title="' . htmlspecialchars($this->getLanguageService()->getLL('addToList')) . '">';
         $ATag_e = '</a>';
         $out = '<span data-uid="' . htmlspecialchars($row['uid']) . '" data-table="' . htmlspecialchars($table) . '" data-title="' . htmlspecialchars($title) . '" data-icon="' . htmlspecialchars($ficon) . '">';
-        $out .= $ATag . $this->iconFactory->getIcon('actions-edit-add', Icon::SIZE_SMALL)->render() . $ATag_e . $ATag_alt . $code . $ATag_e;
+        $out .= $ATag . $this->iconFactory->getIcon('actions-add', Icon::SIZE_SMALL)->render() . $ATag_e . $ATag_alt . $code . $ATag_e;
         $out .= '</span>';
 
         return $out;
index c545a92..7d68e03 100644 (file)
@@ -51,7 +51,7 @@
                        <a class="btn btn-default" href="#" onclick="top.launchView('be_users', '{backendUser.uid}'); return false;"><core:icon identifier="actions-document-info" /></a>
                </div>
                <div class="btn-group" role="group">
-                       <f:link.action action="addToCompareList" arguments="{uid: backendUser.uid}" class="btn btn-default"><core:icon identifier="actions-edit-add" size="small"/> <f:translate key="compare" /></f:link.action>
+                       <f:link.action action="addToCompareList" arguments="{uid: backendUser.uid}" class="btn btn-default"><core:icon identifier="actions-add" size="small"/> <f:translate key="compare" /></f:link.action>
                        <bu:SwitchUser backendUser="{backendUser}" />
                </div>
        </td>
index 5136cc6..45e54ae 100644 (file)
@@ -3152,6 +3152,10 @@ class IconRegistry implements SingletonInterface
         'actions-document-close' => [
             'message' => '%s is deprecated since TYPO3 CMS 8, this icon will be removed in TYPO3 CMS 9',
             'replacement' => 'actions-close'
+        ],
+        'actions-edit-add' => [
+            'message' => '%s is deprecated since TYPO3 CMS 8, this icon will be removed in TYPO3 CMS 9',
+            'replacement' => 'actions-add'
         ]
     ];
 
diff --git a/typo3/sysext/core/Documentation/Changelog/master/Deprecation-80603-ChangeDuplicateIconIdentifiersToActions-add.rst b/typo3/sysext/core/Documentation/Changelog/master/Deprecation-80603-ChangeDuplicateIconIdentifiersToActions-add.rst
new file mode 100644 (file)
index 0000000..b16f564
--- /dev/null
@@ -0,0 +1,33 @@
+.. include:: ../../Includes.txt
+
+======================================================================
+Deprecation: #80603 - Change duplicate icon identifiers to actions-add
+======================================================================
+
+See :issue:`80603`
+
+Description
+===========
+
+The icon ``actions-edit-add`` has been marked as deprecated.
+
+
+Impact
+======
+
+Using the icon identifier ``actions-edit-add`` will trigger a deprecation log entry.
+
+Affected Installations
+======================
+
+Any installation using the icon identifier.
+
+
+Migration
+=========
+
+Use the new icon identifier ``actions-add`` instead.
+
+
+
+.. index:: Backend
index f56c23d..6f409e3 100644 (file)
@@ -218,7 +218,7 @@ class DatabaseBrowser extends AbstractElementBrowser implements ElementBrowserIn
             if (in_array('pages', $tablesArr, true)) {
                 $out .= '<span data-uid="' . htmlspecialchars($mainPageRecord['uid']) . '" data-table="pages" data-title="' . htmlspecialchars($mainPageRecord['title']) . '" data-icon="">';
                 $out .= '<a href="#" data-close="0">'
-                    . $this->iconFactory->getIcon('actions-edit-add', Icon::SIZE_SMALL)->render()
+                    . $this->iconFactory->getIcon('actions-add', Icon::SIZE_SMALL)->render()
                     . '</a>'
                     . '<a href="#" data-close="1">'
                     . $pText
index e426cec..674318f 100644 (file)
@@ -302,7 +302,7 @@ class FileBrowser extends AbstractElementBrowser implements ElementBrowserInterf
             ];
             if ($this->fileIsSelectableInFileList($fileObject, $imgInfo)) {
                 $ATag = '<a href="#" class="btn btn-default" title="' . htmlspecialchars($fileObject->getName()) . '" data-file-index="' . htmlspecialchars($filesIndex) . '" data-close="0">';
-                $ATag .= '<span title="' . htmlspecialchars($lang->getLL('addToList')) . '">' . $this->iconFactory->getIcon('actions-edit-add', Icon::SIZE_SMALL)->render() . '</span>';
+                $ATag .= '<span title="' . htmlspecialchars($lang->getLL('addToList')) . '">' . $this->iconFactory->getIcon('actions-add', Icon::SIZE_SMALL)->render() . '</span>';
                 $ATag_alt = '<a href="#" title="' . htmlspecialchars($fileObject->getName()) . '" data-file-index="' . htmlspecialchars($filesIndex) . '" data-close="1">';
                 $ATag_e = '</a>';
                 $bulkCheckBox = '<label class="btn btn-default btn-checkbox"><input type="checkbox" class="typo3-bulk-item" name="file_' . $filesIndex . '" value="0" /><span class="t3-icon fa"></span></label>';
index b510618..32351e4 100644 (file)
@@ -176,7 +176,7 @@ class FolderBrowser extends AbstractElementBrowser implements ElementBrowserInte
             $lines[] = '
                                <tr>
                                        <td nowrap="nowrap">' . $foldernameAndIcon . '&nbsp;</td>
-                                       <td>' . $aTag . '<span title="' . htmlspecialchars($lang->getLL('addToList')) . '">' . $this->iconFactory->getIcon('actions-edit-add', Icon::SIZE_SMALL)->render() . '</span>' . $aTag_e . '</td>
+                                       <td>' . $aTag . '<span title="' . htmlspecialchars($lang->getLL('addToList')) . '">' . $this->iconFactory->getIcon('actions-add', Icon::SIZE_SMALL)->render() . '</span>' . $aTag_e . '</td>
                                        <td>&nbsp;</td>
                                </tr>';
             $lines[] = '
index 8e2ed44..a0b3217 100644 (file)
@@ -387,7 +387,7 @@ class AddImageHandler implements LinkParameterProviderInterface, LinkHandlerInte
                                        <tr class="file_list_normal">
                                                <td class="col-title" nowrap="nowrap">' . $filenameAndIcon . '&nbsp;</td>
                                                <td class="col-control">
-                                                       <div class="btn-group">' . $ATag . '<span title="' . htmlspecialchars($lang->getLL('addToList')) . '">' . $this->iconFactory->getIcon('actions-edit-add', Icon::SIZE_SMALL)->render() . '</span>' . $ATag_e . '
+                                                       <div class="btn-group">' . $ATag . '<span title="' . htmlspecialchars($lang->getLL('addToList')) . '">' . $this->iconFactory->getIcon('actions-add', Icon::SIZE_SMALL)->render() . '</span>' . $ATag_e . '
                                                        <a href="' . htmlspecialchars($Ahref) . '" class="btn btn-default" title="' . htmlspecialchars($lang->getLL('info')) . '">' . $this->iconFactory->getIcon('actions-document-info', Icon::SIZE_SMALL) . '</a>
                                                </td>
                                                <td class="col-clipboard" valign="top">' . $bulkCheckBox . '</td>