Fixed typos and comments. Renamed class to make it more understandable.

Author: Nyholm

Collector/DebugPlugin.php

@@ -8,7 +8,7 @@
 use Psr\Http\Message\ResponseInterface;
 
 /**
- * A plugin used for log requests and responses passing through each plugin.
+ * A plugin used for log requests and responses. This plugin is executed between each normal plugin.
  *
  * @author Tobias Nyholm <[email protected]>
  */

Collector/DebugPluginCollector.php

@@ -11,6 +11,7 @@
 use Symfony\Component\HttpKernel\DataCollector\DataCollector;
 
 /**
+ * A data collector for the debug plugin. 
  * @author Tobias Nyholm <[email protected]>
  */
 class DebugPluginCollector extends DataCollector
@@ -121,11 +122,13 @@ public function getTotalRequests()
     }
 
     /**
-     * @return ClientDataProvider[]
+     * Return a RequestStackProvider for each client.
+     * 
+     * @return RequestStackProvider[]
      */
     public function getClients()
     {
-        return ClientDataProvider::createFromCollectedData($this->data);
+        return RequestStackProvider::createFromCollectedData($this->data);
     }
 
     /**

Collector/ClientDataProvider.php renamed to Collector/RequestStackProvider.php

@@ -3,14 +3,14 @@
 namespace Http\HttplugBundle\Collector;
 
 /**
- * An object to handle the collected data for a client. This is used to display data.
+ * An object that managed collected data for each client. This is used to display data.
  *
  * The Request object at $requests[0][2] is the state of the object between the third
  * and the fourth plugin. The response after that plugin is found in $responses[0][2].
  *
  * @author Tobias Nyholm <[email protected]>
  */
-class ClientDataProvider
+class RequestStackProvider
 {
     /**
      * Array that tell if a request errored or not. true = success, false = failure.
@@ -54,7 +54,7 @@ public function __construct(array $failure, array $requests, array $responses)
      *
      * @param array $data
      *
-     * @return ClientDataProvider[]
+     * @return RequestStackProvider[]
      */
     public static function createFromCollectedData(array $data)
     {
@@ -70,7 +70,7 @@ public static function createFromCollectedData(array $data)
      * @param array $messages is an array with keys 'failure', 'request' and 'response' which hold requests for each call to
      *                        sendRequest and for each depth.
      *
-     * @return ClientDataProvider
+     * @return RequestStackProvider
      */
     private static function createOne($messages)
     {
@@ -99,46 +99,6 @@ private static function createOne($messages)
         return new self($orderedFaulure, $orderedRequests, $orderedResponses);
     }
 
-    /**
-     * @return array
-     */
-    public function getRequests()
-    {
-        return $this->requests;
-    }
-
-    /**
-     * @param array $requests
-     *
-     * @return ClientDataProvider
-     */
-    public function setRequests($requests)
-    {
-        $this->requests = $requests;
-
-        return $this;
-    }
-
-    /**
-     * @return array
-     */
-    public function getResponses()
-    {
-        return $this->responses;
-    }
-
-    /**
-     * @param array $responses
-     *
-     * @return ClientDataProvider
-     */
-    public function setResponses($responses)
-    {
-        $this->responses = $responses;
-
-        return $this;
-    }
-
     /**
      * Get the index keys for the request and response stacks.
      *