don't do an empty query

diff --git a/addon/facebook/facebook.php b/addon/facebook/facebook.php
index f6f830435fd304e022e78e8b45b1e1333212370b..8999561c85abcaeeca2216a1afaab2d00ece0e4e 100644 (file)
--- 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.