Fixed bug #17130: [Caching framework] Remove commented code from db backend
authorChristian Kuhn <lolli@schwarzbu.ch>
Tue, 18 Jan 2011 21:56:30 +0000 (21:56 +0000)
committerChristian Kuhn <lolli@schwarzbu.ch>
Tue, 18 Jan 2011 21:56:30 +0000 (21:56 +0000)
git-svn-id: https://svn.typo3.org/TYPO3v4/Core/trunk@10128 709f56b5-9817-0410-a4d7-c38de5d9e867

ChangeLog
t3lib/cache/backend/class.t3lib_cache_backend_dbbackend.php
tests/t3lib/cache/backend/t3lib_cache_backend_dbbackendTest.php

index 3e5ba6d..2178dfc 100755 (executable)
--- a/ChangeLog
+++ b/ChangeLog
@@ -12,6 +12,7 @@
 
 2011-01-18  Christian Kuhn  <lolli@schwarzbu.ch>
 
+       * Fixed bug #17130: [Caching framework] Remove commented code from db backend
        * Fixed bug #12232: [Performance] md5_file() to check if a file has been changed is very expensive (Thanks to André Stösel and Vladimir Podkovanov)
        * Fixed bug #16685: [Performance] Not using table index while deleting cached imagesize (Thanks to Vladimir Podkovanov)
        * Fixed bug #16462: Link field for images is too short to provide links for every image (Thanks to Jo Hasenau)
index 5c32c66..37138e1 100644 (file)
@@ -399,37 +399,6 @@ class t3lib_cache_backend_DbBackend extends t3lib_cache_backend_AbstractBackend
         * @author Ingo Renner <ingo@typo3.org>
         */
        public function setCacheTable($cacheTable) {
-               /*
-
-         TODO reenable this check or remove it before 4.3 final
-
-         This check causes mysql warnings when not being logged in and calling
-         typo3/backend.php or the install tool.
-         Reason: the caches in typo3/init.php get initialized before a DB connection
-         has been established.
-         Related Question: Why aren't there warnings in the FE as the caches get
-         initialized in tslib_fe's constructor which is also before a DB conection
-         exsits?
-         Assumption Ingo Renner: Is a custom error_reporting level causing that?
-
-         There's also an unit test for that check (also deactivated for now).
-
-                 $result = $GLOBALS['TYPO3_DB']->exec_SELECTgetRows(
-                         'id',
-                         $cacheTable,
-                         '',
-                         '',
-                         '',
-                         1
-                 );
-
-                 if (!is_array($result)) {
-                         throw new t3lib_cache_Exception(
-                                 'The table "' . $cacheTable . '" does not exist.',
-                                 1236516444
-                         );
-                 }
-  */
                $this->cacheTable = $cacheTable;
                $this->initializeCommonReferences();
        }
index 137985e..a6ae64a 100644 (file)
@@ -104,16 +104,6 @@ class t3lib_cache_backend_DbBackendTest extends tx_phpunit_testcase {
                );
        }
 
-       /**
-        * @test
-        * @expectedException t3lib_cache_Exception
-        * @author Ingo Renner <ingo@typo3.org>
-        */
-# deactivated as the according check in the DB backend causes trouble during TYPO3's initialization
-#      public function setCacheTableThrowsExceptionOnNonExistentTable() {
-#              $this->backend->setCacheTable('test_cache_non_existent_table');
-#      }
-
        /**
         * @test
         * @author Ingo Renner <ingo@typo3.org>