Changeset 8090

Timestamp:

Dec 12, 2010, 11:16:12 AM (13 years ago)

Author:

patdenice

Message:

merge r8089 from branch 2.1 to trunk
feature 2057: use http_build_query instead of add_url_params for GET data.

Location:

trunk

Files:

• . (modified) (1 prop)
• admin/include/functions.php (modified) (4 diffs)

trunk/admin/include/functions.php

@@ -1726 +1726 @@
   $method  = empty($post_data) ? 'GET' : 'POST';
   $request = empty($post_data) ? '' : http_build_query($post_data, '', '&');
-  $src     = add_url_params($src, $get_data, '&');
+  if (!empty($get_data))
+  {
+    $src .= strpos($src, '?') === false ? '?' : '&';
+    $src .= http_build_query($get_data, '', '&');
+  }
 
   // Initialize $dest
@@ -1742 +1746 @@
     {
       @curl_setopt($ch, CURLOPT_POST, 1);
-      @curl_setopt($ch, CURLOPT_POSTFIELDS, $post_data);
+      @curl_setopt($ch, CURLOPT_POSTFIELDS, $request);
     }
     $content = @curl_exec($ch);
@@ -1766 +1770 @@
       'http' => array(
         'method' => $method,
-        'content' => $request,
         'user_agent' => $user_agent,
       )
     );
+    if ($method == 'POST')
+    {
+      $opts['http']['content'] = $request;
+    }
     $context = @stream_context_create($opts);
     $content = @file_get_contents($src, false, $context);
@@ -1792 +1799 @@
   $http_request  = $method." ".$path." HTTP/1.0\r\n";
   $http_request .= "Host: ".$host."\r\n";
-  $http_request .= "Content-Type: application/x-www-form-urlencoded;\r\n";
-  $http_request .= "Content-Length: ".strlen($request)."\r\n";
+  if ($method == 'POST')
+  {
+    $http_request .= "Content-Type: application/x-www-form-urlencoded;\r\n";
+    $http_request .= "Content-Length: ".strlen($request)."\r\n";
+  }
   $http_request .= "User-Agent: ".$user_agent."\r\n";
   $http_request .= "Accept: */*\r\n";