Http exception visibility (#37)

Greg Bowler · web-flow · commit 2d626b1603d9 · 2020-05-27T21:45:28.000+01:00
* Build up request URI in parts for greater compatibility

* Change visibility to public for all getStatusCode functions
diff --git a/src/ResponseStatusException/AbstractResponseStatusException.php b/src/ResponseStatusException/AbstractResponseStatusException.php
@@ -12,5 +12,5 @@ public function __construct(string $message = "") {
 		);
 	}
 
-	abstract protected function getHttpCode():int;
+	abstract public function getHttpCode():int;
 }
diff --git a/src/ResponseStatusException/HttpBadGateway.php b/src/ResponseStatusException/HttpBadGateway.php
@@ -9,7 +9,7 @@
  * @link https://httpstatuses.com/502
  */
 class HttpBadGateway extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::BAD_GATEWAY;
 	}
 }
diff --git a/src/ResponseStatusException/HttpBadRequest.php b/src/ResponseStatusException/HttpBadRequest.php
@@ -10,7 +10,7 @@
  * @link https://httpstatuses.com/400
  */
 class HttpBadRequest extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::BAD_REQUEST;
 	}
 }
diff --git a/src/ResponseStatusException/HttpConflict.php b/src/ResponseStatusException/HttpConflict.php
@@ -4,7 +4,7 @@
 use Gt\Http\StatusCode;
 
 class HttpConflict extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::CONFLICT;
 	}
 }
diff --git a/src/ResponseStatusException/HttpExpectationFailed.php b/src/ResponseStatusException/HttpExpectationFailed.php
@@ -4,7 +4,7 @@
 use Gt\Http\StatusCode;
 
 class HttpExpectationFailed extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::EXPECTATION_FAILED;
 	}
 }
diff --git a/src/ResponseStatusException/HttpFailedDependency.php b/src/ResponseStatusException/HttpFailedDependency.php
@@ -4,7 +4,7 @@
 use Gt\Http\StatusCode;
 
 class HttpFailedDependency extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::FAILED_DEPENDENCY;
 	}
 }
diff --git a/src/ResponseStatusException/HttpForbidden.php b/src/ResponseStatusException/HttpForbidden.php
@@ -14,7 +14,7 @@
  * @link https://httpstatuses.com/403
  */
 class HttpForbidden extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::FORBIDDEN;
 	}
 }
diff --git a/src/ResponseStatusException/HttpFound.php b/src/ResponseStatusException/HttpFound.php
@@ -15,7 +15,7 @@
  * @link https://httpstatuses.com/302
  */
 class HttpFound extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::FOUND;
 	}
 }
diff --git a/src/ResponseStatusException/HttpGatewayTimeout.php b/src/ResponseStatusException/HttpGatewayTimeout.php
@@ -9,7 +9,7 @@
  * @link https://httpstatuses.com/504
  */
 class HttpGatewayTimeout extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::GATEWAY_TIMEOUT;
 	}
 }
diff --git a/src/ResponseStatusException/HttpGone.php b/src/ResponseStatusException/HttpGone.php
@@ -4,7 +4,7 @@
 use Gt\Http\StatusCode;
 
 class HttpGone extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::GONE;
 	}
 }
diff --git a/src/ResponseStatusException/HttpImATeapot.php b/src/ResponseStatusException/HttpImATeapot.php
@@ -4,7 +4,7 @@
 use Gt\Http\StatusCode;
 
 class HttpImATeapot extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::IM_A_TEAPOT;
 	}
 }
diff --git a/src/ResponseStatusException/HttpInsufficientStorage.php b/src/ResponseStatusException/HttpInsufficientStorage.php
@@ -9,7 +9,7 @@
  * @link https://httpstatuses.com/507
  */
 class HttpInsufficientStorage extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::INSUFFICIENT_STORAGE;
 	}
 }
diff --git a/src/ResponseStatusException/HttpInternalServerError.php b/src/ResponseStatusException/HttpInternalServerError.php
@@ -9,7 +9,7 @@
  * @link https://httpstatuses.com/500
  */
 class HttpInternalServerError extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::INTERNAL_SERVER_ERROR;
 	}
 }
diff --git a/src/ResponseStatusException/HttpLengthRequired.php b/src/ResponseStatusException/HttpLengthRequired.php
@@ -4,7 +4,7 @@
 use Gt\Http\StatusCode;
 
 class HttpLengthRequired extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::LENGTH_REQUIRED;
 	}
 }
diff --git a/src/ResponseStatusException/HttpLocked.php b/src/ResponseStatusException/HttpLocked.php
@@ -4,7 +4,7 @@
 use Gt\Http\StatusCode;
 
 class HttpLocked extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::LOCKED;
 	}
 }
diff --git a/src/ResponseStatusException/HttpLoopDetected.php b/src/ResponseStatusException/HttpLoopDetected.php
@@ -9,7 +9,7 @@
  * @link https://httpstatuses.com/508
  */
 class HttpLoopDetected extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::LOOP_DETECTED;
 	}
 }
diff --git a/src/ResponseStatusException/HttpMethodNotAllowed.php b/src/ResponseStatusException/HttpMethodNotAllowed.php
@@ -10,7 +10,7 @@
  * @link https://httpstatuses.com/405
  */
 class HttpMethodNotAllowed extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::METHOD_NOT_ALLOWED;
 	}
 }
diff --git a/src/ResponseStatusException/HttpMisdirectedRequest.php b/src/ResponseStatusException/HttpMisdirectedRequest.php
@@ -4,7 +4,7 @@
 use Gt\Http\StatusCode;
 
 class HttpMisdirectedRequest extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::MISDIRECTED_REQUEST;
 	}
 }
diff --git a/src/ResponseStatusException/HttpMovedPermanently.php b/src/ResponseStatusException/HttpMovedPermanently.php
@@ -8,7 +8,7 @@
  * @link https://httpstatuses.com/301
  */
 class HttpMovedPermanently extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::MOVED_PERMANENTLY;
 	}
 }
diff --git a/src/ResponseStatusException/HttpMultipleChoices.php b/src/ResponseStatusException/HttpMultipleChoices.php
@@ -11,7 +11,7 @@
  * @link https://httpstatuses.com/300
  */
 class HttpMultipleChoices extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::MULTIPLE_CHOICES;
 	}
 }
diff --git a/src/ResponseStatusException/HttpNetworkAuthenticationRequired.php b/src/ResponseStatusException/HttpNetworkAuthenticationRequired.php
@@ -11,7 +11,7 @@
  * @link https://httpstatuses.com/511
  */
 class HttpNetworkAuthenticationRequired extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::NETWORK_AUTHENTICATION_REQUIRED;
 	}
 }
diff --git a/src/ResponseStatusException/HttpNotAcceptable.php b/src/ResponseStatusException/HttpNotAcceptable.php
@@ -9,7 +9,7 @@
  * @linkhttps://httpstatuses.com/406
  */
 class HttpNotAcceptable extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::NOT_ACCEPTABLE;
 	}
 }
diff --git a/src/ResponseStatusException/HttpNotExtended.php b/src/ResponseStatusException/HttpNotExtended.php
@@ -8,7 +8,7 @@
  * @link https://httpstatuses.com/510
  */
 class HttpNotExtended extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::NOT_EXTENDED;
 	}
 }
diff --git a/src/ResponseStatusException/HttpNotFound.php b/src/ResponseStatusException/HttpNotFound.php
@@ -9,7 +9,7 @@
  * @link https://httpstatuses.com/404
  */
 class HttpNotFound extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::NOT_FOUND;
 	}
 }
diff --git a/src/ResponseStatusException/HttpNotImplemented.php b/src/ResponseStatusException/HttpNotImplemented.php
@@ -10,7 +10,7 @@
  * @link https://httpstatuses.com/501
  */
 class HttpNotImplemented extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::NOT_IMPLEMENTED;
 	}
 }
diff --git a/src/ResponseStatusException/HttpNotModified.php b/src/ResponseStatusException/HttpNotModified.php
@@ -11,7 +11,7 @@
  * @link https://httpstatuses.com/304
  */
 class HttpNotModified extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::NOT_MODIFIED;
 	}
 }
diff --git a/src/ResponseStatusException/HttpPayloadTooLarge.php b/src/ResponseStatusException/HttpPayloadTooLarge.php
@@ -4,7 +4,7 @@
 use Gt\Http\StatusCode;
 
 class HttpPayloadTooLarge extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::PAYLOAD_TOO_LARGE;
 	}
 }
diff --git a/src/ResponseStatusException/HttpPaymentRequired.php b/src/ResponseStatusException/HttpPaymentRequired.php
@@ -16,7 +16,7 @@
  * @linkhttps://httpstatuses.com/402
  */
 class HttpPaymentRequired extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::PAYMENT_REQUIRED;
 	}
 }
diff --git a/src/ResponseStatusException/HttpPermanentRedirect.php b/src/ResponseStatusException/HttpPermanentRedirect.php
@@ -11,7 +11,7 @@
  * @link https://httpstatuses.com/308
  */
 class HttpPermanentRedirect extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::PERMANENT_REDIRECT;
 	}
 }
diff --git a/src/ResponseStatusException/HttpPreconditionFailed.php b/src/ResponseStatusException/HttpPreconditionFailed.php
@@ -4,7 +4,7 @@
 use Gt\Http\StatusCode;
 
 class HttpPreconditionFailed extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::PRECONDITION_FAILED;
 	}
 }
diff --git a/src/ResponseStatusException/HttpPreconditionRequired.php b/src/ResponseStatusException/HttpPreconditionRequired.php
@@ -4,7 +4,7 @@
 use Gt\Http\StatusCode;
 
 class HttpPreconditionRequired extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::PRECONDITION_REQUIRED;
 	}
 }
diff --git a/src/ResponseStatusException/HttpProxyAuthenticationRequired.php b/src/ResponseStatusException/HttpProxyAuthenticationRequired.php
@@ -8,7 +8,7 @@
  * @linkhttps://httpstatuses.com/407
  */
 class HttpProxyAuthenticationRequired extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::PROXY_AUTHENTICATION_REQUIRED;
 	}
 }
diff --git a/src/ResponseStatusException/HttpRangeNotSatisfiable.php b/src/ResponseStatusException/HttpRangeNotSatisfiable.php
@@ -4,7 +4,7 @@
 use Gt\Http\StatusCode;
 
 class HttpRangeNotSatisfiable extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::REQUESTED_RANGE_NOT_SATISFIABLE;
 	}
 }
diff --git a/src/ResponseStatusException/HttpRequestHeaderFieldsTooLarge.php b/src/ResponseStatusException/HttpRequestHeaderFieldsTooLarge.php
@@ -4,7 +4,7 @@
 use Gt\Http\StatusCode;
 
 class HttpRequestHeaderFieldsTooLarge extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::REQUEST_HEADER_FIELDS_TOO_LARGE;
 	}
 }
diff --git a/src/ResponseStatusException/HttpRequestTimeout.php b/src/ResponseStatusException/HttpRequestTimeout.php
@@ -11,7 +11,7 @@
  * @linkhttps://httpstatuses.com/408
  */
 class HttpRequestTimeout extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::REQUEST_TIME_OUT;
 	}
 }
diff --git a/src/ResponseStatusException/HttpSeeOther.php b/src/ResponseStatusException/HttpSeeOther.php
@@ -11,7 +11,7 @@
  * @link https://httpstatuses.com/303
  */
 class HttpSeeOther extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::SEE_OTHER;
 	}
 }
diff --git a/src/ResponseStatusException/HttpTemporaryRedirect.php b/src/ResponseStatusException/HttpTemporaryRedirect.php
@@ -12,7 +12,7 @@
  * @link https://httpstatuses.com/307
  */
 class HttpTemporaryRedirect extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::TEMPORARY_REDIRECT;
 	}
 }
diff --git a/src/ResponseStatusException/HttpTooEarly.php b/src/ResponseStatusException/HttpTooEarly.php
@@ -9,7 +9,7 @@
  * @link https://tools.ietf.org/html/rfc8470
  */
 class HttpTooEarly extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::TOO_EARLY;
 	}
 }
diff --git a/src/ResponseStatusException/HttpTooManyRequests.php b/src/ResponseStatusException/HttpTooManyRequests.php
@@ -9,7 +9,7 @@
  * @link https://httpstatuses.com/429
  */
 class HttpTooManyRequests extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::TOO_MANY_REQUESTS;
 	}
 }
diff --git a/src/ResponseStatusException/HttpUnauthorized.php b/src/ResponseStatusException/HttpUnauthorized.php
@@ -16,7 +16,7 @@
  * @link https://httpstatuses.com/401
  */
 class HttpUnauthorized extends AbstractResponseStatusException {
-	protected function getHttpCode():int {
+	public function getHttpCode():int {
 		return StatusCode::UNAUTHORIZED;
 	}
 }
diff --git a/src/ResponseStatusException/HttpUnavailableForLegalReasons.php b/src/ResponseStatusException/HttpUnavailableForLegalReasons.php
diff --git a/src/ResponseStatusException/HttpUnprocessableEntity.php b/src/ResponseStatusException/HttpUnprocessableEntity.php
diff --git a/src/ResponseStatusException/HttpUnsupportedMediaType.php b/src/ResponseStatusException/HttpUnsupportedMediaType.php
diff --git a/src/ResponseStatusException/HttpUpgradeRequired.php b/src/ResponseStatusException/HttpUpgradeRequired.php
diff --git a/src/ResponseStatusException/HttpUriTooLong.php b/src/ResponseStatusException/HttpUriTooLong.php
diff --git a/src/ResponseStatusException/HttpVariantAlsoNegotiates.php b/src/ResponseStatusException/HttpVariantAlsoNegotiates.php
diff --git a/src/ResponseStatusException/HttpVersionNotSupported.php b/src/ResponseStatusException/HttpVersionNotSupported.php
diff --git a/src/ResponseStatusException/ServiceUnavailable.php b/src/ResponseStatusException/ServiceUnavailable.php

Original file line number	Diff line number	Diff line change
`@@ -12,5 +12,5 @@ public function __construct(string $message = "") {`
`12`	`12`	`);`
`13`	`13`	`}`
`14`	`14`
`15`		`- abstract protected function getHttpCode():int;`
	`15`	`+ abstract public function getHttpCode():int;`
`16`	`16`	`}`
Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@`
`9`	`9`	`* @link https://httpstatuses.com/502`
`10`	`10`	`*/`
`11`	`11`	`class HttpBadGateway extends AbstractResponseStatusException {`
`12`		`- protected function getHttpCode():int {`
	`12`	`+ public function getHttpCode():int {`
`13`	`13`	`return StatusCode::BAD_GATEWAY;`
`14`	`14`	`}`
`15`	`15`	`}`
Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@`
`10`	`10`	`* @link https://httpstatuses.com/400`
`11`	`11`	`*/`
`12`	`12`	`class HttpBadRequest extends AbstractResponseStatusException {`
`13`		`- protected function getHttpCode():int {`
	`13`	`+ public function getHttpCode():int {`
`14`	`14`	`return StatusCode::BAD_REQUEST;`
`15`	`15`	`}`
`16`	`16`	`}`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`	`use Gt\Http\StatusCode;`
`5`	`5`
`6`	`6`	`class HttpConflict extends AbstractResponseStatusException {`
`7`		`- protected function getHttpCode():int {`
	`7`	`+ public function getHttpCode():int {`
`8`	`8`	`return StatusCode::CONFLICT;`
`9`	`9`	`}`
`10`	`10`	`}`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`	`use Gt\Http\StatusCode;`
`5`	`5`
`6`	`6`	`class HttpExpectationFailed extends AbstractResponseStatusException {`
`7`		`- protected function getHttpCode():int {`
	`7`	`+ public function getHttpCode():int {`
`8`	`8`	`return StatusCode::EXPECTATION_FAILED;`
`9`	`9`	`}`
`10`	`10`	`}`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`	`use Gt\Http\StatusCode;`
`5`	`5`
`6`	`6`	`class HttpFailedDependency extends AbstractResponseStatusException {`
`7`		`- protected function getHttpCode():int {`
	`7`	`+ public function getHttpCode():int {`
`8`	`8`	`return StatusCode::FAILED_DEPENDENCY;`
`9`	`9`	`}`
`10`	`10`	`}`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@`
`14`	`14`	`* @link https://httpstatuses.com/403`
`15`	`15`	`*/`
`16`	`16`	`class HttpForbidden extends AbstractResponseStatusException {`
`17`		`- protected function getHttpCode():int {`
	`17`	`+ public function getHttpCode():int {`
`18`	`18`	`return StatusCode::FORBIDDEN;`
`19`	`19`	`}`
`20`	`20`	`}`
Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@`
`15`	`15`	`* @link https://httpstatuses.com/302`
`16`	`16`	`*/`
`17`	`17`	`class HttpFound extends AbstractResponseStatusException {`
`18`		`- protected function getHttpCode():int {`
	`18`	`+ public function getHttpCode():int {`
`19`	`19`	`return StatusCode::FOUND;`
`20`	`20`	`}`
`21`	`21`	`}`
Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@`
`9`	`9`	`* @link https://httpstatuses.com/504`
`10`	`10`	`*/`
`11`	`11`	`class HttpGatewayTimeout extends AbstractResponseStatusException {`
`12`		`- protected function getHttpCode():int {`
	`12`	`+ public function getHttpCode():int {`
`13`	`13`	`return StatusCode::GATEWAY_TIMEOUT;`
`14`	`14`	`}`
`15`	`15`	`}`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`	`use Gt\Http\StatusCode;`
`5`	`5`
`6`	`6`	`class HttpGone extends AbstractResponseStatusException {`
`7`		`- protected function getHttpCode():int {`
	`7`	`+ public function getHttpCode():int {`
`8`	`8`	`return StatusCode::GONE;`
`9`	`9`	`}`
`10`	`10`	`}`