james/fembooru
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

These variables where never defined depending on the execution path.

Browse Source
This commit is contained in:
jgen 2014-04-25 17:40:14 -04:00
parent 6f685ca227
commit c85630198f
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
ext/danbooru_api/main.php
View File

@ -261,6 +261,8 @@ class DanbooruApi extends Extension {
if(($event->get_arg(1) == 'find_posts') || (($event->get_arg(1) == 'post') && ($event->get_arg(2) == 'index.xml'))) if(($event->get_arg(1) == 'find_posts') || (($event->get_arg(1) == 'post') && ($event->get_arg(2) == 'index.xml')))
{ {
$this->authenticate_user(); $this->authenticate_user();
$start = 0;
if(isset($_GET['md5'])) if(isset($_GET['md5']))
{ {
$md5list = explode(",",$_GET['md5']); $md5list = explode(",",$_GET['md5']);
@ -268,6 +270,7 @@ class DanbooruApi extends Extension {
{ {
$results[] = Image::by_hash($md5); $results[] = Image::by_hash($md5);
} }
$count = count($results);
} elseif(isset($_GET['id'])) } elseif(isset($_GET['id']))
{ {
$idlist = explode(",",$_GET['id']); $idlist = explode(",",$_GET['id']);
@ -275,6 +278,7 @@ class DanbooruApi extends Extension {
{ {
$results[] = Image::by_id($id); $results[] = Image::by_id($id);
} }
$count = count($results);
} else } else
{ {
$limit = isset($_GET['limit']) ? int_escape($_GET['limit']) : 100; $limit = isset($_GET['limit']) ? int_escape($_GET['limit']) : 100;
@ -294,7 +298,7 @@ class DanbooruApi extends Extension {
// Now we have the array $results filled with Image objects // Now we have the array $results filled with Image objects
// Let's display them // Let's display them
$xml = "<posts count=\"$count\" offset=\"$start\">\n"; $xml = "<posts count=\"{$count}\" offset=\"{$start}\">\n";
foreach($results as $img) foreach($results as $img)
{ {
// Sanity check to see if $img is really an image object // Sanity check to see if $img is really an image object