[TASK] Remove TYPO3_tables_script and handling
authorChristian Kuhn <lolli@schwarzbu.ch>
Fri, 15 Jun 2012 12:31:40 +0000 (14:31 +0200)
committerOliver Hader <oliver.hader@typo3.org>
Fri, 15 Jun 2012 13:58:18 +0000 (15:58 +0200)
The definition of constant TYPO3_tables_script in localconf.php allows to
set a different path to the basic tca definitions than the default
t3lib/stddb/tables.php. This was never recommended and there is a
preferred way to overload the default file with a "extTablesDef" script.

Since the TYPO3_tables_script was probably never used by anyone,
is hidden, underdocumentated and not very useful, the functionality is
now removed without substitution.

Change-Id: Ie780f56c6ae31f581c286a959168b719b1106079
Releases: 6.0
Resolves: #38081
Reviewed-on: http://review.typo3.org/12093
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
Reviewed-by: Wouter Wolters
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader
NEWS.txt
t3lib/config_default.php
t3lib/stddb/tables.php
typo3/cli_dispatch.phpsh
typo3/init.php
typo3/sysext/cms/tslib/class.tslib_fe.php
typo3/sysext/install/mod/class.tx_install.php
typo3/sysext/install/updates/class.tx_coreupdates_migrateworkspaces.php

index 6d452f9..d76be43 100644 (file)
--- a/NEWS.txt
+++ b/NEWS.txt
@@ -70,6 +70,12 @@ The ancient noBlur TypoScript setting for old Browsers (Internet Explorer <= 5.5
 was removed without alternative. The setting has no effect anymore and can be
 removed from custom TypoScript objects, especially MENU.
 
+* Removed TYPO3_tables_script constant
+
+It was possible to define a different base TCA definition file by defining the
+variable typo_db_tables_script in localconf.php. This functionality is removed
+without substitution.
+
 ===============================================================================
 Changes and Improvements
 ===============================================================================
index 3af5d94..3740cb0 100644 (file)
@@ -739,7 +739,6 @@ $typo_db = '';                                      // The database name
 $typo_db_username = '';                        // The database username
 $typo_db_password = '';                        // The database password
 $typo_db_host = '';                            // The database host
-$typo_db_tables_script = '';   // The filename of the tables.php script in typo3conf/ folder IF the default t3lib/stddb/tables.php should NOT be used for some reason. It's recommended to use the default and modify it through the extTableDef-script, see below.
 $typo_db_extTableDef_script = '';      // The filename of an additional script in typo3conf/-folder which is included after tables.php. Code in this script should modify the tables.php-configuration only, and this provides a good way to extend the standard-distributed tables.php file.
 
        // Include localconf.php. Use this file to configure TYPO3 for your needs and database
@@ -903,7 +902,6 @@ define('TYPO3_db', $typo_db);
 define('TYPO3_db_username', $typo_db_username);
 define('TYPO3_db_password', $typo_db_password);
 define('TYPO3_db_host', $typo_db_host);
-define('TYPO3_tables_script', $typo_db_tables_script);
 define('TYPO3_extTableDef_script', $typo_db_extTableDef_script);
 
        // Initialize the locales handled by TYPO3
@@ -914,7 +912,6 @@ unset($typo_db);
 unset($typo_db_username);
 unset($typo_db_password);
 unset($typo_db_host);
-unset($typo_db_tables_script);
 unset($typo_db_extTableDef_script);
 
        // Based on the configuration of the image processing some options may be forced:
index ffb0fea..ef9d783 100644 (file)
@@ -38,8 +38,7 @@
  * This file is included from "typo3/init.php" (backend) and "index_ts.php"
  * (frontend) as the first file of a three-fold inclusion session
  * (see references):
- * 1) First this script is included (unless the constant "TYPO3_tables_script"
- * instructs another filename to substitute it, see t3lib/config_default.php);
+ * 1) First this script is included.
  * This should initialize the variables shown above.
  * 2) Then either the "typo3conf/temp_CACHED_??????_ext_tables.php" cache file
  * OR "stddb/load_ext_tables.php" is included in order to let extensions
index 0a286ca..4789a39 100755 (executable)
@@ -59,12 +59,8 @@ Typo3_Bootstrap::initializeTypo3DbGlobal(TRUE);
 
 Typo3_Bootstrap_Cli::initializeCliKeyOrDie();
 
-       // Include standard tables.php or own file
-if (TYPO3_tables_script) {
-       include(PATH_typo3conf . TYPO3_tables_script);
-} else {
-       include(PATH_t3lib . 'stddb/tables.php');
-}
+       // Include standard tables.php file
+include(PATH_t3lib . 'stddb/tables.php');
        // Load temp_CACHED file of ext_tables or each ext_tables.php of loaded extensions
 if ($TYPO3_LOADED_EXT['_CACHEFILE']) {
        include(PATH_typo3conf . $TYPO3_LOADED_EXT['_CACHEFILE'] . '_ext_tables.php');
index e3732c1..255f9cc 100644 (file)
@@ -91,12 +91,8 @@ if (!$CLIENT['BROWSER']) {
        throw new RuntimeException('Browser Error: Your browser version looks incompatible with this TYPO3 version!', 1294587023);
 }
 
-       // Include standard tables.php or own file
-if (TYPO3_tables_script) {
-       include(PATH_typo3conf . TYPO3_tables_script);
-} else {
-       include(PATH_t3lib . 'stddb/tables.php');
-}
+       // Include standard tables.php file
+include(PATH_t3lib . 'stddb/tables.php');
        // Load temp_CACHED file of ext_tables or each ext_tables.php of loaded extensions
 if ($TYPO3_LOADED_EXT['_CACHEFILE']) {
        include(PATH_typo3conf . $TYPO3_LOADED_EXT['_CACHEFILE'] . '_ext_tables.php');
index 6a4c776..d87a811 100644 (file)
@@ -2072,7 +2072,7 @@ class tslib_fe {
                global $TCA;
                if (!$this->TCAloaded) {
                        $GLOBALS['TCA'] = array();
-                       include (TYPO3_tables_script ? PATH_typo3conf.TYPO3_tables_script : PATH_t3lib.'stddb/tables.php');
+                       include(PATH_t3lib . 'stddb/tables.php');
                                // Extension additions
                        if ($GLOBALS['TYPO3_LOADED_EXT']['_CACHEFILE'] && file_exists(PATH_typo3conf . $GLOBALS['TYPO3_LOADED_EXT']['_CACHEFILE'] . '_ext_tables.php')) {
                                include(PATH_typo3conf.$GLOBALS['TYPO3_LOADED_EXT']['_CACHEFILE'].'_ext_tables.php');
@@ -4751,4 +4751,4 @@ if (version == "n3") {
                return $result;
        }
 }
-?>
\ No newline at end of file
+?>
index 48b88cb..3fc6e63 100644 (file)
@@ -7498,7 +7498,7 @@ $out="
                        // this line hast to stay, as included files use $TCA in global scope
                global $TCA;
 
-               include (TYPO3_tables_script ? PATH_typo3conf.TYPO3_tables_script : PATH_t3lib.'stddb/tables.php');
+               include (PATH_t3lib . 'stddb/tables.php');
 
                        // Extension additions
                if ($GLOBALS['TYPO3_LOADED_EXT']['_CACHEFILE']) {
index 232623b..1359399 100644 (file)
@@ -379,7 +379,7 @@ class tx_coreupdates_migrateworkspaces extends tx_coreupdates_installsysexts {
        protected function includeTCA() {
                global $TCA; // this is relevant because it's used within the included ext_tables.php files - do NOT remove it
 
-               include_once(TYPO3_tables_script ? PATH_typo3conf . TYPO3_tables_script : PATH_t3lib . 'stddb/tables.php');
+               include_once(PATH_t3lib . 'stddb/tables.php');
                        // Extension additions
                if ($GLOBALS['TYPO3_LOADED_EXT']['_CACHEFILE']) {
                        include_once(PATH_typo3conf . $GLOBALS['TYPO3_LOADED_EXT']['_CACHEFILE'] . '_ext_tables.php');
@@ -425,4 +425,4 @@ class tx_coreupdates_migrateworkspaces extends tx_coreupdates_installsysexts {
                return $GLOBALS['TYPO3_DB']->exec_SELECTgetRows('*', 'sys_workspace', $where);
        }
 }
-?>
\ No newline at end of file
+?>