Merge branch 'release/0.8.14'

Author: ljonesfl

.version.json

@@ -2,6 +2,6 @@
   "strategy": "semver",
   "major": 0,
   "minor": 8,
-  "patch": 13,
+  "patch": 14,
   "build": 0
 }

src/Mvc/Application.php

@@ -265,9 +265,11 @@ public function executeController( array $parameters, string $requestName = '' )
 			throw new MissingMethod( "Method '$method'' not found." );
 		}
 
-		$request = null;
-
-		if( !empty( $requestName ) )
+		if( empty( $requestName ) )
+		{
+			$request = new Request();
+		}
+		else
 		{
 			$request = $this->getRequest( $requestName );
 
@@ -281,6 +283,8 @@ public function executeController( array $parameters, string $requestName = '' )
 			}
 		}
 
+
+
 		return $controller->$method(
 			$parameters,
 			$request

src/Mvc/Controllers/Base.php

@@ -4,9 +4,6 @@
 
 use League\CommonMark\Exception\CommonMarkException;
 use Neuron\Core\Exceptions\BadRequestMethod;
-use Neuron\Data\Filter\Get;
-use Neuron\Data\Filter\Post;
-use Neuron\Data\Filter\Server;
 use Neuron\Data\Setting\SettingManager;
 use Neuron\Mvc\Application;
 use Neuron\Mvc\Cache\CacheConfig;
@@ -18,7 +15,6 @@
 use Neuron\Mvc\Views\Html;
 use Neuron\Mvc\Views\Json;
 use Neuron\Mvc\Views\Markdown;
-use Neuron\Mvc\Views\NotFound;
 use Neuron\Mvc\Views\Xml;
 use Neuron\Routing\Router;
 
@@ -885,46 +881,4 @@ public function __call( string $method, array $arguments ): ?string
 		// If no URL helper pattern matches, throw an exception
 		throw new \BadMethodCallException( "Method '$method' not found in " . static::class );
 	}
-
-	/**
-	 * Filter and retrieve a GET parameter.
-	 *
-	 * @param string $paramName The name of the GET parameter
-	 * @param mixed $default The default value if parameter is not present
-	 * @return mixed The filtered GET parameter value or default
-	 */
-	public function filterGet( string $paramName, mixed $default = null ) : mixed
-	{
-		$get = new Get();
-
-		return $get->filterScalar( $paramName ) ?? $default;
-	}
-
-	/**
-	 * Filter and retrieve a POST parameter.
-	 *
-	 * @param string $paramName The name of the POST parameter
-	 * @param mixed $default The default value if parameter is not present
-	 * @return mixed The filtered POST parameter value or default
-	 */
-	public function filterPost( string $paramName, mixed $default = null ) : mixed
-	{
-		$post = new Post();
-
-		return $post->filterScalar( $paramName ) ?? $default;
-	}
-
-	/**
-	 * Filter and retrieve a SERVER parameter.
-	 *
-	 * @param string $paramName The name of the SERVER parameter
-	 * @param mixed $default The default value if parameter is not present
-	 * @return mixed The filtered SERVER parameter value or default
-	 */
-	public function filterServer( string $paramName, mixed $default = null ): mixed
-	{
-		$server = new Server();
-
-		return $server->filterScalar( $paramName ) ?? $default;
-	}
 }

src/Mvc/Requests/Request.php

@@ -3,7 +3,12 @@
 namespace Neuron\Mvc\Requests;
 
 use Neuron\Core\Exceptions\Validation;
+use Neuron\Data\Filter\Get;
+use Neuron\Data\Filter\Post;
+use Neuron\Data\Filter\Server;
+use Neuron\Data\Filter\Session;
 use Neuron\Log\Log;
+use Neuron\Routing\DefaultIpResolver;
 use Neuron\Routing\RequestMethod;
 use Symfony\Component\Yaml\Yaml;
 
@@ -17,14 +22,79 @@ class Request
 	private array $_headers = [];
 	private array $_parameters;
 	private array $_errors = [];
+	private DefaultIpResolver $_ipResolver;
+	private Get $_get;
+	private Post $_post;
+	private Server $_server;
+	private Session $_session;
 
 	/**
 	 * Request constructor.
 	 */
 	public function __construct()
 	{
+		$this->_ipResolver = new DefaultIpResolver();
+		$this->_get = new Get();
+		$this->_post = new Post();
+		$this->_server = new Server();
+		$this->_session = new Session();
 	}
 
+	/**
+	 * @return string
+	 */
+	public function getClientIp(): string
+	{
+		return $this->_ipResolver->resolve( $_SERVER );
+	}
+
+	/**
+	 * Filtered GET parameter
+	 * @param string $key
+	 * @param mixed|null $default
+	 * @return mixed
+	 */
+	public function get( string $key, mixed $default = null ): mixed
+	{
+		return $this->_get->filterScalar( $key, $default );
+	}
+
+	/**
+	 * Filtered POST parameter
+	 * @param string $key
+	 * @param mixed|null $default
+	 * @return mixed
+	 */
+	public function post( string $key, mixed $default = null ): mixed
+	{
+		return $this->_post->filterScalar( $key, $default );
+	}
+
+	/**
+	 * Filtered SERVER parameter
+	 * @param string $key
+	 * @param mixed|null $default
+	 * @return mixed
+	 */
+	public function server( string $key, mixed $default = null ): mixed
+	{
+		return $this->_server->filterScalar( $key, $default );
+	}
+
+	/**
+	 *
+	 * @param string $key
+	 * @param mixed|null $default
+	 * @return mixed
+	 */
+	public function session( string $key, mixed $default = null ): mixed
+	{
+		return $this->_session->filterScalar( $key, $default );
+	}
+
+	/**
+	 * @return array
+	 */
 	public function getErrors(): array
 	{
 		return $this->_errors;

tests/Mvc/TestController.php

@@ -37,7 +37,7 @@ public function partial( array $Parameters, ?Request $Request ) : string
 	public function no_request( array $Parameters, ?Request $Request )
 	{
 		global $ControllerState;
-		$ControllerState = $Request === null;
+		$ControllerState = true;
 	}
 
 

versionlog.md

@@ -1,3 +1,6 @@
+## 0.8.14 2025-11-13
+* Dispatched controller methods now always receive request objects, even if empty.
+
 ## 0.8.13 2025-11-13
 * Added get/post/server filter wrappers to controller base.