Fixed bug #15211: listQuery does not handle passed in int values correctly (follow...
authorErnesto Baschny <ernst@cron-it.de>
Sat, 24 Jul 2010 06:53:55 +0000 (06:53 +0000)
committerErnesto Baschny <ernst@cron-it.de>
Sat, 24 Jul 2010 06:53:55 +0000 (06:53 +0000)
git-svn-id: https://svn.typo3.org/TYPO3v4/Core/trunk@8259 709f56b5-9817-0410-a4d7-c38de5d9e867

ChangeLog
t3lib/class.t3lib_db.php
tests/t3lib/t3lib_dbTest.php [new file with mode: 0644]

index 88ee711..7ef91b3 100755 (executable)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2010-07-24  Ernesto Baschny  <ernst@cron-it.de>
+
+       * Fixed bug #15211: listQuery does not handle passed in int values correctly (follow-up to #14818) (Thanks to Bjoern Pedersen)
+
 2010-07-23  Steffen Kamper  <steffen@typo3.org>
 
        * Fixed bug #15195: TSanalyzer: Add link to breakpoints also in syntaxhighlight mode
index 2f6eeb2..f691898 100644 (file)
@@ -638,6 +638,7 @@ class t3lib_DB {
         * @return      string          WHERE clause for a query
         */
        public function listQuery($field, $value, $table) {
+               $value = (string)$value;
                if (strpos(',', $value) !== FALSE) {
                        throw new InvalidArgumentException('$value must not contain a comma (,) in $this->listQuery() !');
                }
diff --git a/tests/t3lib/t3lib_dbTest.php b/tests/t3lib/t3lib_dbTest.php
new file mode 100644 (file)
index 0000000..c49b718
--- /dev/null
@@ -0,0 +1,66 @@
+<?php
+/***************************************************************
+* Copyright notice
+*
+* (c) 2010 Ernesto Baschny (ernst@cron-it.de)
+* All rights reserved
+*
+* This script is part of the TYPO3 project. The TYPO3 project is
+* free software; you can redistribute it and/or modify
+* it under the terms of the GNU General Public License as published by
+* the Free Software Foundation; either version 2 of the License, or
+* (at your option) any later version.
+*
+* The GNU General Public License can be found at
+* http://www.gnu.org/copyleft/gpl.html.
+*
+* This script is distributed in the hope that it will be useful,
+* but WITHOUT ANY WARRANTY; without even the implied warranty of
+* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+* GNU General Public License for more details.
+*
+* This copyright notice MUST APPEAR in all copies of the script!
+***************************************************************/
+
+/**
+ * Testcase for the t3lib_cs class in the TYPO3 Core.
+ *
+ * @package TYPO3
+ * @subpackage t3lib
+ *
+ * @author Ernesto Baschny <ernst@cron-it.de>
+ */
+class t3lib_dbTest extends tx_phpunit_testcase {
+       /**
+        * @var t3lib_db
+        */
+       private $fixture = null;
+
+       public function setUp() {
+               $this->fixture = new t3lib_db();
+       }
+
+       public function tearDown() {
+               unset($this->fixture);
+       }
+
+
+       ////////////////////////////////
+       // Tests concerning listQuery
+       ////////////////////////////////
+
+       /**
+        * @test
+        *
+        * @see http://bugs.typo3.org/view.php?id=15211
+        */
+       public function listQueryWithIntegerCommaAsValue() {
+                       // Note: 44 = ord(',')
+               $this->assertEquals(
+                       $this->fixture->listQuery('dummy', 44, 'table'),
+                       $this->fixture->listQuery('dummy', '44', 'table')
+               );
+       }
+
+}
+?>
\ No newline at end of file