From 0db25abcefd871d5e87f02cbd896f71a7795a8e6 Mon Sep 17 00:00:00 2001 From: Friendika Date: Sat, 30 Apr 2011 17:28:57 -0700 Subject: don't do an empty query --- addon/facebook/facebook.php | 25 +++++++++++++++---------- 1 file changed, 15 insertions(+), 10 deletions(-) (limited to 'addon') diff --git a/addon/facebook/facebook.php b/addon/facebook/facebook.php index f6f830435..8999561c8 100644 --- a/addon/facebook/facebook.php +++ b/addon/facebook/facebook.php @@ -379,18 +379,23 @@ function facebook_post_hook(&$a,&$b) { $deny = array_unique(array_merge($deny_people,$deny_groups)); $allow_str = dbesc(implode(', ',$recipients)); - $r = q("SELECT `notify` FROM `contact` WHERE `id` IN ( $allow_str ) AND `network` = 'face'"); - $allow_arr = array(); - if(count($r)) - foreach($r as $rr) - $allow_arr[] = $rr['notify']; + if($allow_str) { + $r = q("SELECT `notify` FROM `contact` WHERE `id` IN ( $allow_str ) AND `network` = 'face'"); + $allow_arr = array(); + if(count($r)) + foreach($r as $rr) + $allow_arr[] = $rr['notify']; + } $deny_str = dbesc(implode(', ',$deny)); - $r = q("SELECT `notify` FROM `contact` WHERE `id` IN ( $deny_str ) AND `network` = 'face'"); - $deny_arr = array(); - if(count($r)) - foreach($r as $rr) - $deny_arr[] = $rr['notify']; + if($deny_str) { + $r = q("SELECT `notify` FROM `contact` WHERE `id` IN ( $deny_str ) AND `network` = 'face'"); + $deny_arr = array(); + if(count($r)) + foreach($r as $rr) + $deny_arr[] = $rr['notify']; + } + if(count($deny_arr) && (! count($allow_arr))) { // One or more FB folks were denied access but nobody on FB was specifically allowed access. -- cgit v1.2.3