[TASK] Deprecate uc->classicPageEditMode and editRegularContentFromId 74/39474/2
authorBenjamin Mack <benni@typo3.org>
Tue, 12 May 2015 04:28:12 +0000 (12:28 +0800)
committerGeorg Ringer <georg.ringer@gmail.com>
Thu, 21 May 2015 08:53:53 +0000 (10:53 +0200)
There is some leftover, never-called code in a regular TYPO3 instance
regarding the old uc->classicPageEditMode option and its companion
functionality in EditDocumentController->editRegularContentFromId
which has been marked for deprecation.

Resolves: #66905
Releases: master
Change-Id: I0a428e411b9d7a0a646c7356d773a369c9397dde
Reviewed-on: http://review.typo3.org/39474
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
typo3/sysext/backend/Classes/ClickMenu/ClickMenu.php
typo3/sysext/backend/Classes/Controller/EditDocumentController.php
typo3/sysext/core/Documentation/Changelog/master/Deprecation-66905-ClassicPageModuleAndEditRegularContentFromId.rst [new file with mode: 0644]
typo3/sysext/t3editor/res/jslib/parse_typoscript/tokenizetyposcript.js

index db7af65..2f9c7e4 100644 (file)
@@ -689,9 +689,7 @@ class ClickMenu {
                // If another module was specified, replace the default Page module with the new one
                $newPageModule = trim($this->backendUser->getTSConfigVal('options.overridePageModule'));
                $pageModule = BackendUtility::isModuleSetInTBE_MODULES($newPageModule) ? $newPageModule : 'web_layout';
-               $editOnClick = '';
                $loc = 'top.content.list_frame';
-               $addParam = '';
                $theIcon = 'actions-document-open';
 
                $link = BackendUtility::getModuleUrl('record_edit', array(
@@ -700,15 +698,8 @@ class ClickMenu {
 
                if ($this->iParts[0] === 'pages' && $this->iParts[1] && $this->backendUser->check('modules', $pageModule)) {
                        $this->editPageIconSet = TRUE;
-                       if ($this->backendUser->uc['classicPageEditMode']) {
-                               $addParam = '&editRegularContentFromId=' . (int)$this->iParts[1];
-                       } else {
-                               $editOnClick = 'if(' . $loc . '){' . $loc . '.location.href=' . GeneralUtility::quoteJSvalue($link . '&returnUrl=') . '+top.rawurlencode(' . $this->frameLocation(($loc . '.document')) . '.pathname+' . $this->frameLocation(($loc . '.document')) . '.search)+' . GeneralUtility::quoteJSvalue($addParam) . ';}';
-                       }
-               }
-               if (!$editOnClick) {
-                       $editOnClick = 'if(' . $loc . '){' . $loc . '.location.href=' . GeneralUtility::quoteJSvalue($link . '&returnUrl=') . '+top.rawurlencode(' . $this->frameLocation(($loc . '.document')) . '.pathname+' . $this->frameLocation(($loc . '.document')) . '.search)+' . GeneralUtility::quoteJSvalue($addParam) . ';}';
                }
+               $editOnClick = 'if(' . $loc . '){' . $loc . '.location.href=' . GeneralUtility::quoteJSvalue($link . '&returnUrl=') . '+top.rawurlencode(' . $this->frameLocation(($loc . '.document')) . '.pathname+' . $this->frameLocation(($loc . '.document')) . '.search);}';
                return $this->linkItem($this->label('edit'), IconUtility::getSpriteIcon($theIcon), $editOnClick . ';');
        }
 
index 834a9e3..0377374 100644 (file)
@@ -164,6 +164,7 @@ class EditDocumentController {
         * (NORMAL column/default language) from that page into the form!
         *
         * @var int
+        * @deprecated since TYPO3 CMS 7, will be removed with TYPO3 CMS 8
         */
        public $editRegularContentFromId;
 
@@ -1513,8 +1514,10 @@ class EditDocumentController {
         * Function, which populates the internal editconf array with editing commands for all tt_content elements from the normal column in normal language from the page pointed to by $this->editRegularContentFromId
         *
         * @return void
+        * @deprecated since TYPO3 CMS 7, will be removed with TYPO3 CMS 8
         */
        public function editRegularContentFromId() {
+               GeneralUtility::logDeprecatedFunction();
                $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('uid', 'tt_content', 'pid=' . (int)$this->editRegularContentFromId . BackendUtility::deleteClause('tt_content') . BackendUtility::versioningPlaceholderClause('tt_content') . ' AND colPos=0 AND sys_language_uid=0', '', 'sorting');
                if ($GLOBALS['TYPO3_DB']->sql_num_rows($res)) {
                        $ecUids = array();
diff --git a/typo3/sysext/core/Documentation/Changelog/master/Deprecation-66905-ClassicPageModuleAndEditRegularContentFromId.rst b/typo3/sysext/core/Documentation/Changelog/master/Deprecation-66905-ClassicPageModuleAndEditRegularContentFromId.rst
new file mode 100644 (file)
index 0000000..00a63fa
--- /dev/null
@@ -0,0 +1,29 @@
+===========================================================================================
+Deprecation: #66905 - Deprecate uc->classicPageEditMode and editRegularContentFromId option
+===========================================================================================
+
+Description
+===========
+
+The BE-User uc option "classicPageEditMode" which was used prior to TYPO3 CMS 4.0 has been removed some time ago.
+The functionality ``editRegularContentFromId`` which was then triggered in EditDocumentController has been marked
+for deprecation.
+
+
+Impact
+======
+
+Any direct calls using ``editRegularContentFromId`` via GET parameter or calling ``editRegularContentFromId()``
+directly from a third-party extension will output a deprecation message.
+
+
+Affected Installations
+======================
+
+Any installation using third-party code to restore the old behaviour.
+
+
+Migration
+=========
+
+Remove calls to the functionality.
index 1bbeda7..efc6cfb 100644 (file)
@@ -223,7 +223,6 @@ var typoscriptWords = {
        'classesCharacter': 'reserved',
        'classesImage': 'reserved',
        'classesParagraph': 'reserved',
-       'classicPageEditMode': 'reserved',
        'clear': 'reserved',
        'clearCache': 'reserved',
        'clearCache_disable': 'reserved',
@@ -1545,7 +1544,6 @@ var typoscriptWords = {
        'classesCharacter': 'reserved',
        'classesImage': 'reserved',
        'classesParagraph': 'reserved',
-       'classicPageEditMode': 'reserved',
        'clear': 'reserved',
        'clearCache': 'reserved',
        'clearCache_disable': 'reserved',