<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" /><style type="text/css"><!--
#msg dl { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; }
#msg dl a { font-weight: bold}
#msg dl a:link { color:#fc3; }
#msg dl a:active { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fc0 solid; padding: 6px; }
#msg ul, pre { overflow: auto; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<title>[14050] trunk/wp-includes/class-http.php:
Clean up the HTTP inline docs to reflect the times.</title>
</head>
<body>
<div id="msg">
<dl>
<dt>Revision</dt> <dd><a href="http://trac.wordpress.org/changeset/14050">14050</a></dd>
<dt>Author</dt> <dd>nacin</dd>
<dt>Date</dt> <dd>2010-04-10 11:23:21 +0000 (Sat, 10 Apr 2010)</dd>
</dl>
<h3>Log Message</h3>
<pre>Clean up the HTTP inline docs to reflect the times. props jacobsantos, fixes <a href="http://trac.wordpress.org/ticket/12882">#12882</a>.</pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkwpincludesclasshttpphp">trunk/wp-includes/class-http.php</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkwpincludesclasshttpphp"></a>
<div class="modfile"><h4>Modified: trunk/wp-includes/class-http.php (14049 => 14050)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/wp-includes/class-http.php 2010-04-10 10:48:30 UTC (rev 14049)
+++ trunk/wp-includes/class-http.php 2010-04-10 11:23:21 UTC (rev 14050)
</span><span class="lines">@@ -2,7 +2,8 @@
</span><span class="cx"> /**
</span><span class="cx"> * Simple and uniform HTTP request API.
</span><span class="cx"> *
</span><del>- * Will eventually replace and standardize the WordPress HTTP requests made.
</del><ins>+ * Standardizes the HTTP requests for WordPress. Handles cookies, gzip encoding and decoding, chunk
+ * decoding, if HTTP 1.1 and various other difficult HTTP protocol implementations.
</ins><span class="cx"> *
</span><span class="cx"> * @link http://trac.wordpress.org/ticket/4779 HTTP API Proposal
</span><span class="cx"> *
</span><span class="lines">@@ -14,13 +15,10 @@
</span><span class="cx"> /**
</span><span class="cx"> * WordPress HTTP Class for managing HTTP Transports and making HTTP requests.
</span><span class="cx"> *
</span><del>- * This class is called for the functionality of making HTTP requests and should replace Snoopy
- * functionality, eventually. There is no available functionality to add HTTP transport
- * implementations, since most of the HTTP transports are added and available for use.
</del><ins>+ * This class is called for the functionality of making HTTP requests and replaces Snoopy
+ * functionality. There is no available functionality to add HTTP transport implementations, since
+ * most of the HTTP transports are added and available for use.
</ins><span class="cx"> *
</span><del>- * The exception is that cURL is not available as a transport and lacking an implementation. It will
- * be added later and should be a patch on the WordPress Trac.
- *
</del><span class="cx"> * There are no properties, because none are needed and for performance reasons. Some of the
</span><span class="cx"> * functions are static and while they do have some overhead over functions in PHP4, the purpose is
</span><span class="cx"> * maintainability. When PHP5 is finally the requirement, it will be easy to add the static keyword
</span><span class="lines">@@ -52,12 +50,11 @@
</span><span class="cx"> /**
</span><span class="cx"> * PHP5 style Constructor - Set up available transport if not available.
</span><span class="cx"> *
</span><del>- * PHP4 does not have the 'self' keyword and since WordPress supports PHP4,
- * the class needs to be used for the static call.
</del><ins>+ * PHP4 does not have the 'self' keyword and since WordPress supports PHP4, the class needs to
+ * be used for the static call. The transport are set up to save time and will only be created
+ * once. This class can be created many times without having to go through the step of finding
+ * which transports are available.
</ins><span class="cx"> *
</span><del>- * The transport are set up to save time. This should only be called once, so
- * the overhead should be fine.
- *
</del><span class="cx"> * @since 2.7.0
</span><span class="cx"> * @return WP_Http
</span><span class="cx"> */
</span><span class="lines">@@ -69,20 +66,18 @@
</span><span class="cx"> /**
</span><span class="cx"> * Tests the WordPress HTTP objects for an object to use and returns it.
</span><span class="cx"> *
</span><del>- * Tests all of the objects and returns the object that passes. Also caches
- * that object to be used later.
</del><ins>+ * Tests all of the objects and returns the object that passes. Also caches that object to be
+ * used later.
</ins><span class="cx"> *
</span><del>- * The order for the GET/HEAD requests are HTTP Extension, cURL, Streams, Fopen,
- * and finally Fsockopen. fsockopen() is used last, because it has the most
- * overhead in its implementation. There isn't any real way around it, since
- * redirects have to be supported, much the same way the other transports
- * also handle redirects.
</del><ins>+ * The order for the GET/HEAD requests are HTTP Extension, cURL, Streams, Fopen, and finally
+ * Fsockopen. fsockopen() is used last, because it has the most overhead in its implementation.
+ * There isn't any real way around it, since redirects have to be supported, much the same way
+ * the other transports also handle redirects.
</ins><span class="cx"> *
</span><del>- * There are currently issues with "localhost" not resolving correctly with
- * DNS. This may cause an error "failed to open stream: A connection attempt
- * failed because the connected party did not properly respond after a
- * period of time, or established connection failed because connected host
- * has failed to respond."
</del><ins>+ * There are currently issues with "localhost" not resolving correctly with DNS. This may cause
+ * an error "failed to open stream: A connection attempt failed because the connected party did
+ * not properly respond after a period of time, or established connection failed because [the]
+ * connected host has failed to respond."
</ins><span class="cx"> *
</span><span class="cx"> * @since 2.7.0
</span><span class="cx"> * @access private
</span></span></pre>
</div>
</div>
</body>
</html>