[!!!][TASK] Remove possibility for index.php?32 12/46812/3
authorBenni Mack <benni@typo3.org>
Mon, 22 Feb 2016 17:07:24 +0000 (18:07 +0100)
committerGeorg Ringer <georg.ringer@gmail.com>
Wed, 24 Feb 2016 08:22:29 +0000 (09:22 +0100)
A Frontend request could previously be called with
index.php?id=[id] or just index.php?[id].

The support for calling pages through the syntax of index.php?[id]
was removed, along with the method $TSFE->setIDfromArgV.

Resolves: #73602
Releases: master
Change-Id: Ic0f0131e15e72281ab8324785c42c3887deed5cb
Reviewed-on: https://review.typo3.org/46812
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/core/Documentation/Changelog/master/Breaking-73602-Short-URLWithoutIdIDRemoved.rst [new file with mode: 0644]
typo3/sysext/frontend/Classes/Controller/TypoScriptFrontendController.php

diff --git a/typo3/sysext/core/Documentation/Changelog/master/Breaking-73602-Short-URLWithoutIdIDRemoved.rst b/typo3/sysext/core/Documentation/Changelog/master/Breaking-73602-Short-URLWithoutIdIDRemoved.rst
new file mode 100644 (file)
index 0000000..c664cf6
--- /dev/null
@@ -0,0 +1,31 @@
+===================================================
+Breaking: #73602 - Short-URL without ?id=ID removed
+===================================================
+
+Description
+===========
+
+The support for resolving URLs using ``index.php?23`` instead of ``index.php?id=23`` with no real GET parameter given
+was removed.
+
+The method ``$TSFE->setIDfromArgV()`` was removed as well.
+
+
+Impact
+======
+
+Calling a frontend page with the short-handed URL will result in not detecting a page ID at all.
+
+Calling ``$TSFE->setIDfromArgV()`` directly within PHP will result in a fatal PHP error.
+
+
+Affected Installations
+======================
+
+Any TYPO3 installation with an extension using the pre-4.0 syntax.
+
+
+Migration
+=========
+
+Use the proper ``index.php?id=23`` when using URLs to be called in the frontend.
\ No newline at end of file
index 2eb867a..c46b9cd 100644 (file)
@@ -1226,8 +1226,6 @@ class TypoScriptFrontendController
                 GeneralUtility::callUserFunction($functionReference, $parameters, $this);
             }
         }
-        // Getting ARG-v values if some
-        $this->setIDfromArgV();
         // If there is a Backend login we are going to check for any preview settings:
         $this->getTimeTracker()->push('beUserLogin', '');
         $originalFrontendUser = null;
@@ -1848,23 +1846,6 @@ class TypoScriptFrontendController
     }
 
     /**
-     * This checks if there are ARGV-parameters in the QUERY_STRING and if so, those are used for the id
-     * $this->id must be 'FALSE' in order for any processing to happen in here
-     * If an id/alias value is extracted from the QUERY_STRING it is set in $this->id
-     *
-     * @return void
-     * @access private
-     */
-    public function setIDfromArgV()
-    {
-        if (!$this->id) {
-            list($theAlias) = explode('&', GeneralUtility::getIndpEnv('QUERY_STRING'));
-            $theAlias = trim($theAlias);
-            $this->id = $theAlias != '' && strpos($theAlias, '=') === false ? $theAlias : 0;
-        }
-    }
-
-    /**
      * Gets ->page and ->rootline information based on ->id. ->id may change during this operation.
      * If not inside domain, then default to first page in domain.
      *