From e46cfcddbf2c80ac099bc423f6269a2edddf89c6 Mon Sep 17 00:00:00 2001
From: Angelo Vargas <angelo@nivler.com>
Date: Wed, 14 Apr 2021 19:50:23 -0500
Subject: [PATCH] Allow class overriding

I wanted to experiment with a subtle behaviour change and stumbled upon a limitation that I could easily fix by extending my own Router. In theory it should be fine, but some methods and instance variables are set to `private`, when they could be set to `protected` and allow overriding.
---
 src/Bramus/Router/Router.php | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/Bramus/Router/Router.php b/src/Bramus/Router/Router.php
index 94e37ad..dd6ac0e 100644
--- a/src/Bramus/Router/Router.php
+++ b/src/Bramus/Router/Router.php
@@ -15,12 +15,12 @@ class Router
     /**
      * @var array The route patterns and their handling functions
      */
-    private $afterRoutes = array();
+    protected $afterRoutes = array();
 
     /**
      * @var array The before middleware route patterns and their handling functions
      */
-    private $beforeRoutes = array();
+    protected $beforeRoutes = array();
 
     /**
      * @var object|callable The function to be executed when no route has been matched
@@ -30,22 +30,22 @@ class Router
     /**
      * @var string Current base route, used for (sub)route mounting
      */
-    private $baseRoute = '';
+    protected $baseRoute = '';
 
     /**
      * @var string The Request Method that needs to be handled
      */
-    private $requestedMethod = '';
+    protected $requestedMethod = '';
 
     /**
      * @var string The Server Base Path for Router Execution
      */
-    private $serverBasePath;
+    protected $serverBasePath;
 
     /**
      * @var string Default Controllers Namespace
      */
-    private $namespace = '';
+    protected $namespace = '';
 
     /**
      * Store a before middleware route and a handling function to be executed when accessed using one of the specified methods.
@@ -335,7 +335,7 @@ public function trigger404(){
      *
      * @return int The number of routes handled
      */
-    private function handle($routes, $quitAfterRun = false)
+    protected function handle($routes, $quitAfterRun = false)
     {
         // Counter to keep track of the number of routes we've handled
         $numHandled = 0;
@@ -382,7 +382,7 @@ private function handle($routes, $quitAfterRun = false)
         return $numHandled;
     }
 
-    private function invoke($fn, $params = array())
+    protected function invoke($fn, $params = array())
     {
         if (is_callable($fn)) {
             call_user_func_array($fn, $params);