Use ResponseStatus constants everywhere it is needed

Author: na2axl

src/WaterPipe/HTTP/Response/Response.php

@@ -143,7 +143,7 @@ public function sendHtml(string $body, int $status = ResponseStatus::OkCode)
      *
      * @throws \Exception
      */
-    public function sendJsonString(string $body, int $status = 200)
+    public function sendJsonString(string $body, int $status = ResponseStatus::OkCode)
     {
         $config = WaterPipeConfig::get();
 
@@ -162,7 +162,7 @@ public function sendJsonString(string $body, int $status = 200)
      *
      * @throws \Exception
      */
-    public function sendJson(array $json, int $status = 200)
+    public function sendJson(array $json, int $status = ResponseStatus::OkCode)
     {
         $this->sendJsonString(json_encode($json), $status);
     }
@@ -175,7 +175,7 @@ public function sendJson(array $json, int $status = 200)
      *
      * @throws \Exception
      */
-    public function sendText(string $body, int $status = 200)
+    public function sendText(string $body, int $status = ResponseStatus::OkCode)
     {
         $config = WaterPipeConfig::get();
 
@@ -196,7 +196,7 @@ public function sendText(string $body, int $status = 200)
      * @throws \Exception
      * @throws FileNotFoundException When the file was not found at the given path.
      */
-    public function sendFile(string $path, int $status = 200, string $mime = null)
+    public function sendFile(string $path, int $status = ResponseStatus::OkCode, string $mime = null)
     {
         $config = WaterPipeConfig::get();
 

src/WaterPipe/WaterPipe.php

@@ -38,6 +38,7 @@
 use ElementaryFramework\WaterPipe\HTTP\Request\RequestMethod;
 use ElementaryFramework\WaterPipe\HTTP\Request\RequestUri;
 use ElementaryFramework\WaterPipe\HTTP\Response\Response;
+use ElementaryFramework\WaterPipe\HTTP\Response\ResponseStatus;
 use ElementaryFramework\WaterPipe\Routing\Middleware\Middleware;
 use ElementaryFramework\WaterPipe\Routing\Middleware\IRouteMiddleware;
 use ElementaryFramework\WaterPipe\Routing\Middleware\MiddlewareWrapper;
@@ -547,8 +548,8 @@ private function _executeRequest()
             }
 
             if ($runner === null) {
-                if (isset($this->_errorsRegistry[404]))
-                    return $this->_executeAction($this->_errorsRegistry[404]);
+                if (isset($this->_errorsRegistry[ResponseStatus::NotFoundCode]))
+                    return $this->_executeAction($this->_errorsRegistry[ResponseStatus::NotFoundCode]);
                 else throw new NotFoundErrorException();
             }
 
@@ -565,8 +566,8 @@ private function _executeRequest()
         } catch (\Exception $e) {
             $this->_runningException = $e;
 
-            if (isset($this->_errorsRegistry[500])) {
-                $this->_executeAction($this->_errorsRegistry[500]);
+            if (isset($this->_errorsRegistry[ResponseStatus::InternalServerErrorCode])) {
+                $this->_executeAction($this->_errorsRegistry[ResponseStatus::InternalServerErrorCode]);
             } else {
                 $config = WaterPipeConfig::get();
                 if ($config->useStderr()) {