Change order of constructor parameters:

Simon Yousoufov · Simon Yousoufov · commit 89ad098dca61 · 2016-10-17T09:19:04.000-04:00
diff --git a/src/AccessDeniedHttpException.php b/src/AccessDeniedHttpException.php
@@ -27,7 +27,7 @@ class AccessDeniedHttpException extends HttpException
 	 * @param array      $headers
 	 * @param \Exception $previous
 	 */
-	public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)
+	public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)
 	{
 		parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);
 	}
diff --git a/src/BadRequestHttpException.php b/src/BadRequestHttpException.php
@@ -27,7 +27,7 @@ class BadRequestHttpException extends HttpException
 	 * @param array      $headers
 	 * @param \Exception $previous
 	 */
-	public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)
+	public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)
 	{
 		parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);
 	}
diff --git a/src/ConflictHttpException.php b/src/ConflictHttpException.php
@@ -27,7 +27,7 @@ class ConflictHttpException extends HttpException
 	 * @param array      $headers
 	 * @param \Exception $previous
 	 */
-	public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)
+	public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)
 	{
 		parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);
 	}
diff --git a/src/GoneHttpException.php b/src/GoneHttpException.php
@@ -27,7 +27,7 @@ class GoneHttpException extends HttpException
 	 * @param array      $headers
 	 * @param \Exception $previous
 	 */
-	public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)
+	public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)
 	{
 		parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);
 	}
diff --git a/src/InternalServerErrorHttpException.php b/src/InternalServerErrorHttpException.php
@@ -27,7 +27,7 @@ class InternalServerErrorHttpException extends HttpException
 	 * @param array      $headers
 	 * @param \Exception $previous
 	 */
-	public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)
+	public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)
 	{
 		parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);
 	}
diff --git a/src/LengthRequiredHttpException.php b/src/LengthRequiredHttpException.php
@@ -27,7 +27,7 @@ class LengthRequiredHttpException extends HttpException
 	 * @param array      $headers
 	 * @param \Exception $previous
 	 */
-	public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)
+	public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)
 	{
 		parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);
 	}
diff --git a/src/NotAcceptableHttpException.php b/src/NotAcceptableHttpException.php
@@ -27,7 +27,7 @@ class NotAcceptableHttpException extends HttpException
 	 * @param array      $headers
 	 * @param \Exception $previous
 	 */
-	public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)
+	public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)
 	{
 		parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);
 	}
diff --git a/src/NotFoundHttpException.php b/src/NotFoundHttpException.php
@@ -27,7 +27,7 @@ class NotFoundHttpException extends HttpException
 	 * @param array      $headers
 	 * @param \Exception $previous
 	 */
-	public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)
+	public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)
 	{
 		parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);
 	}
diff --git a/src/PreconditionFailedHttpException.php b/src/PreconditionFailedHttpException.php
@@ -27,7 +27,7 @@ class PreconditionFailedHttpException extends HttpException
 	 * @param array      $headers
 	 * @param \Exception $previous
 	 */
-	public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)
+	public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)
 	{
 		parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);
 	}
diff --git a/src/PreconditionRequiredHttpException.php b/src/PreconditionRequiredHttpException.php
@@ -27,7 +27,7 @@ class PreconditionRequiredHttpException extends HttpException
 	 * @param array      $headers
 	 * @param \Exception $previous
 	 */
-	public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)
+	public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)
 	{
 		parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);
 	}
diff --git a/src/UnauthorizedHttpException.php b/src/UnauthorizedHttpException.php
@@ -27,7 +27,7 @@ class UnauthorizedHttpException extends HttpException
 	 * @param array      $headers
 	 * @param \Exception $previous
 	 */
-	public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)
+	public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)
 	{
 		parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);
 	}
diff --git a/src/UnprocessableEntityHttpException.php b/src/UnprocessableEntityHttpException.php
@@ -27,7 +27,7 @@ class UnprocessableEntityHttpException extends HttpException
 	 * @param array      $headers
 	 * @param \Exception $previous
 	 */
-	public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)
+	public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)
 	{
 		parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);
 	}
diff --git a/src/UnsupportedMediaTypeHttpException.php b/src/UnsupportedMediaTypeHttpException.php
@@ -27,7 +27,7 @@ class UnsupportedMediaTypeHttpException extends HttpException
 	 * @param array      $headers
 	 * @param \Exception $previous
 	 */
-	public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)
+	public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)
 	{
 		parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);
 	}
diff --git a/tests/HttpExceptionTest.php b/tests/HttpExceptionTest.php
@@ -34,31 +34,31 @@ public function testItCanGetErrorName()
 
 	public function testItCanGetStatusCode()
 	{
-		$exception = new StubException('message', 'formatted', ['foo' => 'bar'], ['header' => 'value'], new \Exception);
+		$exception = new StubException('message', ['foo' => 'bar'], 'formatted', ['header' => 'value'], new \Exception);
 		$this->assertEquals(404, $exception->getStatusCode());
 	}
 
 	public function testItCanGetErrorMessage()
 	{
-		$exception = new StubException('message', 'formatted', ['foo' => 'bar'], ['header' => 'value'], new \Exception);
+		$exception = new StubException('message', ['foo' => 'bar'], 'formatted', ['header' => 'value'], new \Exception);
 		$this->assertEquals('message', $exception->getMessage());
 	}
 
 	public function testItCanGetErrorFormatted()
 	{
-		$exception = new StubException('message', 'formatted', ['foo' => 'bar'], ['header' => 'value'], new \Exception);
+		$exception = new StubException('message', ['foo' => 'bar'], 'formatted', ['header' => 'value'], new \Exception);
 		$this->assertEquals('formatted', $exception->getErrorFormatted());
 	}
 
 	public function testItCanGetErrorData()
 	{
-		$exception = new StubException('message', 'formatted', ['foo' => 'bar'], ['header' => 'value'], new \Exception);
+		$exception = new StubException('message', ['foo' => 'bar'], 'formatted', ['header' => 'value'], new \Exception);
 		$this->assertEquals(['foo' => 'bar'], $exception->getErrorData());
 	}
 
 	public function testItCanGetErrorHeaders()
 	{
-		$exception = new StubException('message', 'formatted', ['foo' => 'bar'], ['header' => 'value'], new \Exception);
+		$exception = new StubException('message', ['foo' => 'bar'], 'formatted', ['header' => 'value'], new \Exception);
 		$this->assertEquals(['header' => 'value'], $exception->getHeaders());
 	}
 
@@ -192,7 +192,7 @@ class StubException extends HttpException
 
 	protected $error_name = 'stub_exception';
 
-	public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)
+	public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)
 	{
 		parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);
 	}

Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ class AccessDeniedHttpException extends HttpException`
`27`	`27`	`* @param array $headers`
`28`	`28`	`* @param \Exception $previous`
`29`	`29`	`*/`
`30`		`- public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)`
	`30`	`+ public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)`
`31`	`31`	`{`
`32`	`32`	`parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);`
`33`	`33`	`}`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ class BadRequestHttpException extends HttpException`
`27`	`27`	`* @param array $headers`
`28`	`28`	`* @param \Exception $previous`
`29`	`29`	`*/`
`30`		`- public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)`
	`30`	`+ public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)`
`31`	`31`	`{`
`32`	`32`	`parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);`
`33`	`33`	`}`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ class ConflictHttpException extends HttpException`
`27`	`27`	`* @param array $headers`
`28`	`28`	`* @param \Exception $previous`
`29`	`29`	`*/`
`30`		`- public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)`
	`30`	`+ public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)`
`31`	`31`	`{`
`32`	`32`	`parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);`
`33`	`33`	`}`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ class GoneHttpException extends HttpException`
`27`	`27`	`* @param array $headers`
`28`	`28`	`* @param \Exception $previous`
`29`	`29`	`*/`
`30`		`- public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)`
	`30`	`+ public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)`
`31`	`31`	`{`
`32`	`32`	`parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);`
`33`	`33`	`}`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ class InternalServerErrorHttpException extends HttpException`
`27`	`27`	`* @param array $headers`
`28`	`28`	`* @param \Exception $previous`
`29`	`29`	`*/`
`30`		`- public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)`
	`30`	`+ public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)`
`31`	`31`	`{`
`32`	`32`	`parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);`
`33`	`33`	`}`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ class LengthRequiredHttpException extends HttpException`
`27`	`27`	`* @param array $headers`
`28`	`28`	`* @param \Exception $previous`
`29`	`29`	`*/`
`30`		`- public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)`
	`30`	`+ public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)`
`31`	`31`	`{`
`32`	`32`	`parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);`
`33`	`33`	`}`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ class NotAcceptableHttpException extends HttpException`
`27`	`27`	`* @param array $headers`
`28`	`28`	`* @param \Exception $previous`
`29`	`29`	`*/`
`30`		`- public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)`
	`30`	`+ public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)`
`31`	`31`	`{`
`32`	`32`	`parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);`
`33`	`33`	`}`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ class NotFoundHttpException extends HttpException`
`27`	`27`	`* @param array $headers`
`28`	`28`	`* @param \Exception $previous`
`29`	`29`	`*/`
`30`		`- public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)`
	`30`	`+ public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)`
`31`	`31`	`{`
`32`	`32`	`parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);`
`33`	`33`	`}`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ class PreconditionFailedHttpException extends HttpException`
`27`	`27`	`* @param array $headers`
`28`	`28`	`* @param \Exception $previous`
`29`	`29`	`*/`
`30`		`- public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)`
	`30`	`+ public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)`
`31`	`31`	`{`
`32`	`32`	`parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);`
`33`	`33`	`}`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ class PreconditionRequiredHttpException extends HttpException`
`27`	`27`	`* @param array $headers`
`28`	`28`	`* @param \Exception $previous`
`29`	`29`	`*/`
`30`		`- public function __construct($message = null, $error_formatted = null, array $error_data = [], array $headers = [], \Exception $previous = null)`
	`30`	`+ public function __construct($message = null, array $error_data = [], $error_formatted = null, array $headers = [], \Exception $previous = null)`
`31`	`31`	`{`
`32`	`32`	`parent::__construct($message, $error_formatted, $error_data, $headers, self::HTTP_CODE, $previous);`
`33`	`33`	`}`