Merge pull request #2860 from Instrye/patch-1

fix. getGetPost and getPostGet can't work in index empty

Author: lonnieezell

system/HTTP/IncomingRequest.php

@@ -427,7 +427,7 @@ public function getPostGet($index = null, $filter = null, $flags = null)
 		// Use $_POST directly here, since filter_has_var only
 		// checks the initial POST data, not anything that might
 		// have been added since.
-		return isset($_POST[$index]) ? $this->getPost($index, $filter, $flags) : $this->getGet($index, $filter, $flags);
+		return isset($_POST[$index]) ? $this->getPost($index, $filter, $flags) : (isset($_GET[$index]) ? $this->getGet($index, $filter, $flags) : $this->getPost());
 	}
 
 	//--------------------------------------------------------------------
@@ -446,7 +446,7 @@ public function getGetPost($index = null, $filter = null, $flags = null)
 		// Use $_GET directly here, since filter_has_var only
 		// checks the initial GET data, not anything that might
 		// have been added since.
-		return isset($_GET[$index]) ? $this->getGet($index, $filter, $flags) : $this->getPost($index, $filter, $flags);
+		return isset($_GET[$index]) ? $this->getGet($index, $filter, $flags) : (isset($_POST[$index]) ? $this->getPost($index, $filter, $flags) : $this->getGet());
 	}
 
 	//--------------------------------------------------------------------

tests/system/HTTP/IncomingRequestTest.php

@@ -423,4 +423,15 @@ public function testSpoofing()
 		$this->assertEquals('wink', $this->request->getMethod());
 	}
 
+	/**
+	 * @see https://github.com/codeigniter4/CodeIgniter4/issues/2839
+	 */
+	public function testGetPostEmpty()
+	{
+		$_POST['TEST'] = 5;
+		$_GET['TEST']  = 3;
+		$this->assertEquals($_POST, $this->request->getPostGet());
+		$this->assertEquals($_GET, $this->request->getGetPost());
+	}
+
 }