CS/QA: fix structure order within classes

src/Cookie.php

@@ -90,6 +90,15 @@ public function __construct($name, $value, $attributes = array(), $flags = array
 		$this->normalize();
 	}
 
+	/**
+	 * Get the cookie value
+	 *
+	 * Attributes and other data can be accessed via methods.
+	 */
+	public function __toString() {
+		return $this->value;
+	}
+
 	/**
 	 * Check if a cookie is expired.
 	 *
@@ -349,15 +358,6 @@ public function format_for_set_cookie() {
 		return $header_value;
 	}
 
-	/**
-	 * Get the cookie value
-	 *
-	 * Attributes and other data can be accessed via methods.
-	 */
-	public function __toString() {
-		return $this->value;
-	}
-
 	/**
 	 * Parse a cookie string into a cookie object
 	 *

src/Requests.php

@@ -104,20 +104,20 @@ class Requests {
 	const VERSION = '1.8.1';
 
 	/**
-	 * Registered transport classes
+	 * Selected transport name
+	 *
+	 * Use {@see get_transport()} instead
 	 *
 	 * @var array
 	 */
-	protected static $transports = array();
+	public static $transport = array();
 
 	/**
-	 * Selected transport name
-	 *
-	 * Use {@see get_transport()} instead
+	 * Registered transport classes
 	 *
 	 * @var array
 	 */
-	public static $transport = array();
+	protected static $transports = array();
 
 	/**
 	 * Default certificate path.

src/Response.php

@@ -20,13 +20,6 @@
  * @package Requests
  */
 class Response {
-	/**
-	 * Constructor
-	 */
-	public function __construct() {
-		$this->headers = new Headers();
-		$this->cookies = new Jar();
-	}
 
 	/**
 	 * Response body
@@ -98,6 +91,14 @@ public function __construct() {
 	 */
 	public $cookies = array();
 
+	/**
+	 * Constructor
+	 */
+	public function __construct() {
+		$this->headers = new Headers();
+		$this->cookies = new Jar();
+	}
+
 	/**
 	 * Is the response a redirect?
 	 *

src/Utility/FilteredIterator.php

@@ -37,6 +37,19 @@ public function __construct($data, $callback) {
 		$this->callback = $callback;
 	}
 
+	/**
+	 * @inheritdoc
+	 *
+	 * @phpcs:disable PHPCompatibility.FunctionNameRestrictions.NewMagicMethods.__unserializeFound
+	 */
+	#[ReturnTypeWillChange]
+	public function __unserialize($serialized) {}
+	// phpcs:enable
+
+	public function __wakeup() {
+		unset($this->callback);
+	}
+
 	/**
 	 * Get the current item's value after filtering
 	 *
@@ -58,16 +71,4 @@ public function current() {
 	 */
 	#[ReturnTypeWillChange]
 	public function unserialize($serialized) {}
-
-	/**
-	 * @inheritdoc
-	 *
-	 * @phpcs:disable PHPCompatibility.FunctionNameRestrictions.NewMagicMethods.__unserializeFound
-	 */
-	#[ReturnTypeWillChange]
-	public function __unserialize($serialized) {}
-
-	public function __wakeup() {
-		unset($this->callback);
-	}
 }

tests/Transport/BaseTestCase.php

@@ -13,6 +13,9 @@
 use WpOrg\Requests\Tests\TestCase;
 
 abstract class BaseTestCase extends TestCase {
+
+	protected $skip_https = false;
+
 	public function set_up() {
 		$callback  = array($this->transport, 'test');
 		$supported = call_user_func($callback);
@@ -27,7 +30,6 @@ public function set_up() {
 			$this->skip_https = true;
 		}
 	}
-	protected $skip_https = false;
 
 	protected function getOptions($other = array()) {
 		$options = array(