Bug #16322
closedWarning: mysql_fetch_assoc(): supplied argument is not a valid MySQL result resource in ...
Added by edwin over 18 years ago. Updated almost 13 years ago.
0%
Description
Because there is nothing in the database which can be selected througth the select statement, made by exec_SELECTquery(). Than there is nothing to sql_fetch_assoc() so you get an error. Problem can be sloved by useing the @-sign. I report this because i've seen more functions where the same problem can be found.
I've found the problem in t3lib\class.t3lib_page.php getHash($hash,$expTime=0)
$res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('content', 'cache_hash', 'hash='.$GLOBALS['TYPO3_DB']->fullQuoteStr($hash, 'cache_hash').$whereAdd);
if ($row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res)) {
$GLOBALS['TYPO3_DB']->sql_free_result($res);
return $row['content'];
}
i've also another question, what is happening if $row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res) is false? Than nothing is returned.
(issue imported from #M3794)
Files
3794-1.diff (482 Bytes) 3794-1.diff | Administrator Admin, 2009-02-06 11:00 |
Updated by H Hahn over 18 years ago
I had the same problem in the following situation. As my customer had a temporary technical problem with his ISP, I started developing his website in a subdirectory of my own domain. My ISP offers MySQL version 5.0.22 (and PHP verrsion 4.4.2). Everything runs fine.
After moving to my customer's domain, this problem ("not a valid MySQL result") keeps occurring. Customer's ISP (one of the biggest over here... ;-) uses MySQL version 4.1.18-log (whatever that "log"may mean.. - elsewhere they say that MySQL Client is version 4.1.14). Their PHP is version 4.4.0.
Please note that the problem occurs with the OLDER MySQL version. So it is unlikely to be a matter of a newer version being less tolerant.
I dislike to "mask" error messages with the "@" as long as I do not exaclty know what they are about. Has anyone a clue?
Updated by Martin Kutschker about 18 years ago
Real protection and FALSE on miss':
$res = $GLOBALS['TYPO3_DB']->exec_SELECTquery( 'content', 'cache_hash', 'hash='.$GLOBALS['TYPO3_DB']->fullQuoteStr($hash, 'cache_hash'). $whereAdd ); if ($res!==FALSE && $row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res)) { $GLOBALS['TYPO3_DB']->sql_free_result($res); return is_array($row) ? $row['content'] : FALSE; }
Updated by Dmitry Dulepov about 18 years ago
If nothing is selected, mysql does not return false, it returns empty recordset, see docs! It returns false only if query has errors.
Updated by Ingo Renner over 17 years ago
mysql_fetch_assoc also returns false if there're no more rows: http://php.net/mysql_fetch_assoc
closed as it doesn't seem to be a bug
Updated by Martin Kutschker over 17 years ago
Reopened because the point was that the Core doesn't use @ to avid warnings in some places (and should check the result of SQL queries better).
Ingo, please don't close bugs. We're resolving them, so the bug submitter may reopen them. Old resolved bugs are closed in mass closings when a new version of TYPO3 is released.
Updated by David Bruchmann over 17 years ago
Had the same problem (shown at top of the BE-Extension-Manager) after upgrading the source of a packet (Yaml).
The concerned functions are sql_fetch_assoc($res) and sql_free_result($res).
Inside the functions I wrapped "if ($res) { ... }" around the function-content and could solve the problem at least skin deep.
In general the Warning can occure in nearly every function of t3lib, so a general solution is necessary. I tried out uninstalling all extensions and remarked that the functions are called also from inside the core without having a $res. I didn't try to find the caller because I think that t3lib should tolerate empty parameters without unnecessary warnings.
A debug-Mode which runs without my "if ($res) { ... }" but with a debug_backtrace-function perhaps could be included too -- But for that case the Output sometimes will get to long, so the debug function has to be case-dependent.
endorsement:
A call of a function with an empty parameter ($res) calls an error but in most cases is no fault in sense of any function. So the conclusion of Ingo is missing that case which is responsable for most warnings.
Updated by Urra over 17 years ago
Problem also occurs with 4.1.1 and 4.1.2 of typo3. I am seeing this warning message also on the frontend (and in the backend):
Warning: mysql_fetch_assoc(): supplied argument is not a valid MySQL result resource in
/srv/www/vhosts/domainname.de/httpdocs/typo3_src-4.1.2/t3lib/class.t3lib_db.php on line 796
After reloading the page the warning disappears.
PHP Version: 4.4.0
MySQL: 4.1.13
Line 796 in class.t3lib_db.php:
function sql_fetch_assoc($res) {
$this->debug_check_recordset($res);
return mysql_fetch_assoc($res); // Line 796
}
Updated by Jürgen Kendzorra over 17 years ago
Had the same problem with one installation after upgrade - for me dropping cache_pages and recreating with
CREATE TABLE cache_pages (
id int(11) unsigned NOT NULL auto_increment,
hash varchar(32) NOT NULL default '',
page_id int(11) unsigned NOT NULL default '0',
reg1 int(11) unsigned NOT NULL default '0',
HTML mediumblob NOT NULL,
temp_content int(1) NOT NULL default '0',
tstamp int(11) unsigned NOT NULL default '0',
expires int(10) unsigned NOT NULL default '0',
cache_data mediumblob NOT NULL,
KEY page_id (page_id),
KEY sel (hash,page_id),
PRIMARY KEY (id)
) TYPE=InnoDB;
worked it out.
Updated by M. Ecker almost 17 years ago
The method debug_check_recordset in class.t3lib_db.php could solve that problem, I think - if applied correctly.
This method returns false if $res is invalid and true if it is valid. It's called by nearly all sql_xxxxx methods, but the return value is not evaluated!
For example:
function sql_fetch_assoc($res) {
$this->debug_check_recordset($res);
return mysql_fetch_assoc($res);
}
could be rewritten as:
function sql_fetch_assoc($res) {
if ($this->debug_check_recordset($res))
return mysql_fetch_assoc($res);
}
(or such).
Updated by Bas v.d. Wiel over 16 years ago
I'm seeing similar errors in our apache log files and was a bit surprised to find how debug_check_recordset is implemented. Would it be safe to do something like:
function sql_fetch_assoc($res) {
if ($this->debug_check_recordset($res)) {
return mysql_fetch_assoc($res);
}
else {return($this->sql_error()); }
}
And let the caller handle any errors from there?
Updated by Sven Weiss over 16 years ago
I have the same problem:
Warning: mysql_fetch_assoc(): supplied argument is not a valid MySQL result resource in /htdocs/typo3/typo3_src-4.2.0/t3lib/class.t3lib_db.php on line 1047
Warning: mysql_fetch_assoc(): supplied argument is not a valid MySQL result resource in /htdocs/typo3/typo3_src-4.2.0/t3lib/class.t3lib_db.php on line 1047
Warning: mysql_fetch_assoc(): supplied argument is not a valid MySQL result resource in /htdocs/typo3/typo3_src-4.2.0/t3lib/class.t3lib_db.php on line 1047
Warning: Cannot modify header information - headers already sent by (output started at /htdocs/typo3/typo3_src-4.2.0/t3lib/class.t3lib_db.php:1047) in /htdocs/typo3/typo3_src-4.2.0/typo3/template.php on line 632
If I am in the Extension Manager and want to change settings of a Extension or if I want to install a Extension, it is very hard to do because I have to use the Tab-Key to navigate through it because I can't click anything because of the errors.
Perhaps it is a bug of an extension?
Updated by Alex Widschwendter over 16 years ago
confirmation of the problem for T3 version 4.2.0 on: 4.0.24_Debian-10sarge3-log
no prob on: 5.0.32-Debian_7etch5-log
hth alex
Updated by Mo over 16 years ago
Same Problem after upgrading from T3 4.0 to 4.2.1 on a machine running MySQL Version 4.0.
Command 'SHOW CHARACTER SET' is only supported from 4.1 upwards. Version Requirement for TYPO3 4.2 should be officially raised to mysql 4.1 so people are warned.
Updated by Franz Holzinger almost 16 years ago
See the extension debug_mysql_db for a solution to this.
An additional patch is needed however.
Updated by Dmitry Dulepov almost 13 years ago
- Status changed from New to Closed
- Priority changed from Should have to Won't have this time
- Target version deleted (
0)
Ingo was right. We should not mask errors anywhere with @ but have a bug report about each particular case that needs a change.
Closing again.