[!!!][TASK] Remove deprecated code from ext:beuser 70/45370/6
authorWouter Wolters <typo3@wouterwolters.nl>
Sat, 19 Dec 2015 19:47:03 +0000 (20:47 +0100)
committerAndreas Fernandez <typo3@scripting-base.de>
Tue, 22 Dec 2015 10:40:29 +0000 (11:40 +0100)
Resolves: #72372
Releases: master
Change-Id: I1984ea732c9e21a99ed75fa8fc3a58faeb4b4275
Reviewed-on: https://review.typo3.org/45370
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
typo3/sysext/beuser/Classes/Controller/PermissionAjaxController.php [changed mode: 0644->0755]
typo3/sysext/core/Documentation/Changelog/master/Breaking-72372-RemovedDeprecatedCodeFromBeuser.rst [new file with mode: 0644]

old mode 100644 (file)
new mode 100755 (executable)
index 2c6d623..6b6b81b
@@ -258,40 +258,6 @@ class PermissionAjaxController
     }
 
     /**
-     * Print the string with the new owner of a page record
-     *
-     * @param int $page The TYPO3 page id
-     * @param int $ownerUid The new page user uid
-     * @param string $username The TYPO3 BE username (used to display in the element)
-     * @param bool $validUser Must be set to FALSE, if the user has no name or is deleted
-     * @return string The new group wrapped in HTML
-     * @deprecated since TYPO3 CMS 7, will be removed in TYPO3 CMS 8. This is now solved with fluid.
-     */
-    public static function renderOwnername($page, $ownerUid, $username, $validUser = true)
-    {
-        GeneralUtility::logDeprecatedFunction();
-        $elementId = 'o_' . $page;
-        return '<span id="' . $elementId . '"><a class="ug_selector changeowner" data-page="' . $page . '" data-owner="' . $ownerUid . '" data-username="' . htmlspecialchars($username) . '">' . ($validUser ? ($username == '' ? '<span class=not_set>[' . $GLOBALS['LANG']->getLL('notSet') . ']</span>' : htmlspecialchars(GeneralUtility::fixed_lgd_cs($username, 20))) : '<span class=not_set title="' . htmlspecialchars(GeneralUtility::fixed_lgd_cs($username, 20)) . '">[' . $GLOBALS['LANG']->getLL('deleted') . ']</span>') . '</a></span>';
-    }
-
-    /**
-     * Print the string with the new group of a page record
-     *
-     * @param int $page The TYPO3 page id
-     * @param int $groupUid The new page group uid
-     * @param string $groupname The TYPO3 BE groupname (used to display in the element)
-     * @param bool $validGroup Must be set to FALSE, if the group has no name or is deleted
-     * @return string The new group wrapped in HTML
-     * @deprecated since TYPO3 CMS 7, will be removed in TYPO3 CMS 8. This is now solved with fluid.
-     */
-    public static function renderGroupname($page, $groupUid, $groupname, $validGroup = true)
-    {
-        GeneralUtility::logDeprecatedFunction();
-        $elementId = 'g_' . $page;
-        return '<span id="' . $elementId . '"><a class="ug_selector changegroup" data-page="' . $page . '" data-group="' . $groupUid . '" data-groupname="' . htmlspecialchars($groupname) . '">' . ($validGroup ? ($groupname == '' ? '<span class=not_set>[' . $GLOBALS['LANG']->getLL('notSet') . ']</span>' : htmlspecialchars(GeneralUtility::fixed_lgd_cs($groupname, 20))) : '<span class=not_set title="' . htmlspecialchars(GeneralUtility::fixed_lgd_cs($groupname, 20)) . '">[' . $GLOBALS['LANG']->getLL('deleted') . ']</span>') . '</a></span>';
-    }
-
-    /**
      * Print the string with the new edit lock state of a page record
      *
      * @param int $page The TYPO3 page id
@@ -310,50 +276,6 @@ class PermissionAjaxController
     }
 
     /**
-     * Print a set of permissions. Also used in index.php
-     *
-     * @param int $int Permission integer (bits)
-     * @param int $pageId The TYPO3 page id
-     * @param string $who The scope (user, group or everybody)
-     * @return string HTML marked up x/* indications.
-     * @deprecated since TYPO3 CMS 7, will be removed in TYPO3 CMS 8. This is now solved with fluid.
-     */
-    public static function renderPermissions($int, $pageId = 0, $who = 'user')
-    {
-        GeneralUtility::logDeprecatedFunction();
-        $str = '';
-        $permissions = array(1, 16, 2, 4, 8);
-        /** @var IconFactory $iconFactory */
-        $iconFactory = GeneralUtility::makeInstance(IconFactory::class);
-        foreach ($permissions as $permission) {
-            if ($int & $permission) {
-                $str .= '<span title="' . $GLOBALS['LANG']->getLL($permission, true)
-                    . ' class="change-permission text-success"'
-                    . ' data-page="' . (int)$pageId . '"'
-                    . ' data-permissions="' . (int)$int . '"'
-                    . ' data-mode="delete"'
-                    . ' data-who="' . htmlspecialchars($who) . '"'
-                    . ' data-bits="' . $permission . '"'
-                    . ' style="cursor:pointer">'
-                    . $iconFactory->getIcon('status-status-permission-granted', Icon::SIZE_SMALL)->render()
-                    . '</span>';
-            } else {
-                $str .= '<span title="' . $GLOBALS['LANG']->getLL($permission, true) . '"'
-                    . ' class="change-permission text-danger"'
-                    . ' data-page="' . (int)$pageId . '"'
-                    . ' data-permissions="' . (int)$int . '"'
-                    . ' data-mode="add"'
-                    . ' data-who="' . htmlspecialchars($who) . '"'
-                    . ' data-bits="' . $permission . '"'
-                    . ' style="cursor:pointer">'
-                    . $iconFactory->getIcon('status-status-permission-denied', Icon::SIZE_SMALL)->render()
-                    . '</span>';
-            }
-        }
-        return '<span id="' . $pageId . '_' . $who . '">' . $str . '</span>';
-    }
-
-    /**
      * @return \TYPO3\CMS\Lang\LanguageService
      */
     protected function getLanguageService()
diff --git a/typo3/sysext/core/Documentation/Changelog/master/Breaking-72372-RemovedDeprecatedCodeFromBeuser.rst b/typo3/sysext/core/Documentation/Changelog/master/Breaking-72372-RemovedDeprecatedCodeFromBeuser.rst
new file mode 100644 (file)
index 0000000..c746908
--- /dev/null
@@ -0,0 +1,30 @@
+======================================================
+Breaking: #72372 - Removed deprecated code from beuser
+======================================================
+
+Description
+===========
+
+The following methods have been removed from `PermissionAjaxController`
+
+``renderOwnername``
+``renderPermissions``
+``renderGroupname``
+
+
+Impact
+======
+
+Using the methods above directly in any third party extension will result in a fatal error.
+
+
+Affected Installations
+======================
+
+Instances which use calls to the methods above.
+
+
+Migration
+=========
+
+Migrate your code that calls one of the methods to Fluid templates.