blallo/tt-rss
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

apparently it's a bad idea to do is_resource() on a mysqli result

Browse source
This commit is contained in:
Andrew Dolgov 2015-07-12 14:23:32 +03:00
parent 83ce77a2e8
commit ec57104d6e
2 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
classes/api.php
View file

@ -700,7 +700,7 @@ class API extends Handler {
'id' => $feed_id,
'is_cat' => $is_cat);
if (is_resource($result)) {
if (!is_numeric($result)) {
while ($line = db_fetch_assoc($result)) {
$line["content_preview"] = truncate_string(strip_tags($line["content"]), $excerpt_length);
foreach (PluginHost::getInstance()->get_hooks(PluginHost::HOOK_QUERY_HEADLINES) as $p) {

4
classes/feeds.php
View file

@ -295,7 +295,7 @@ class Feeds extends Handler_Protected {
}
}
if (is_resource($result) && $this->dbh->num_rows($result) > 0) {
if (!is_numeric($result) && $this->dbh->num_rows($result) > 0) {
$lnum = $offset;
@ -753,7 +753,7 @@ class Feeds extends Handler_Protected {
if ($_REQUEST["debug"]) $timing_info = print_checkpoint("PE", $timing_info);
} else if (is_resource($result)) {
} else if (!is_numeric($result)) {
$message = "";
switch ($view_mode) {