Actions
Bug #39854
closedt3lib_db->fullQuoteString unused method argument
Status:
Rejected
Priority:
Should have
Assignee:
Category:
Database API (Doctrine DBAL)
Target version:
-
Start date:
2012-08-16
Due date:
% Done:
0%
Estimated time:
TYPO3 Version:
4.7
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:
Description
Hello. It's not exactly a bug, but rather anoying thing, which could also resolve in error behaviour.
method t3lib_db->fullQuoteString has second argument $table, which is unused, but does not have set default value. So if you forget to set this argument, php raise warning, which is captured with some typo3 error handler silently, but script continues without neccesary string.
So I'm proposing to set default $table value to null (in all quote string methods), remove it completelly :
file ./t3lib/class.t3lib_db.php
function fullQuoteStr($str, $table = null)
function fullQuoteArray($arr, $table = null, $noQuote = FALSE)
function quoteStr($str, $table = null)
Actions