Merge pull request #125 from itsgoingd/fix-build-query

Explicitly set arg separator to "&" in http_build_query calls.

Author: Fosco Marotto

src/Facebook/FacebookRedirectLoginHelper.php

@@ -100,7 +100,7 @@ public function getLoginUrl($scope = array(), $version = null)
       'scope' => implode(',', $scope)
     );
     return 'https://www.facebook.com/' . $version . '/dialog/oauth?' .
-      http_build_query($params);
+      http_build_query($params, null, '&');
   }
 
   /**
@@ -118,7 +118,7 @@ public function getLogoutUrl(FacebookSession $session, $next)
       'next' => $next,
       'access_token' => $session->getToken()
     );
-    return 'https://www.facebook.com/logout.php?' . http_build_query($params);
+    return 'https://www.facebook.com/logout.php?' . http_build_query($params, null, '&');
   }
 
   /**

src/Facebook/FacebookRequest.php

@@ -298,7 +298,7 @@ public static function appendParamsToUrl($url, $params = array())
     }
 
     if (strpos($url, '?') === false) {
-      return $url . '?' . http_build_query($params);
+      return $url . '?' . http_build_query($params, null, '&');
     }
 
     list($path, $query_string) = explode('?', $url, 2);
@@ -307,7 +307,7 @@ public static function appendParamsToUrl($url, $params = array())
     // Favor params from the original URL over $params
     $params = array_merge($params, $query_array);
 
-    return $path . '?' . http_build_query($params);
+    return $path . '?' . http_build_query($params, null, '&');
   }
 
 }

src/Facebook/HttpClients/FacebookStreamHttpClient.php

@@ -112,7 +112,7 @@ public function send($url, $method = 'GET', $parameters = array())
     );
 
     if ($parameters) {
-      $options['http']['content'] = http_build_query($parameters);
+      $options['http']['content'] = http_build_query($parameters, null, '&');
 
       $this->addRequestHeader('Content-type', 'application/x-www-form-urlencoded');
     }