ACL\Login move public var to private: login

the former public var $login is now private and if it is set can be
checked with loginActionSet (true if login_login was in _POST as login
action.

Some info update for phpUnit ACL\Login test file

4dev/tests/CoreLibsACLLoginTest.php

@@ -157,16 +157,17 @@ final class CoreLibsACLLoginTest extends TestCase
 	 */
 	public function loginProvider(): array
 	{
-		// 0: mock settings/override flag settings
+		// 0[mock]   : mock settings/override flag settings
-		// 2: get array IN
+		// 1[get]    : get array IN
-		// 1: post array IN
+		// 2[post]   : post array IN
-		//    login_login, login_username, login_password, login_logout
+		//             login_login, login_username, login_password, login_logout
-		//    change_password, pw_username, pw_old_password, pw_new_password,
+		//             change_password, pw_username, pw_old_password, pw_new_password,
-		//    pw_new_password_confirm
+		//             pw_new_password_confirm
-		// 2: override session set
+		// 3[session]: override session set
-		// 3: expected error code, 0 for all ok, 3000 for login page view
+		// 4[error]  : expected error code, 0 for all ok, 3000 for login page view
-		//    note that 1000 (no db), 2000 (no session) must be tested too
+		//             note that 1000 (no db), 2000 (no session) must be tested too
-		// 4: expected return array, eg login_error code, or other info data to match
+		// 5[return] : expected return array, eg login_error code,
+		//             or other info data to match
 		$tests = [
 			'load, no login' => [
 				// error code, only for exceptions
@@ -1546,6 +1547,14 @@ final class CoreLibsACLLoginTest extends TestCase
 			);
 		}
 
+		// if _POST login set check this is matching
+		if (!empty($post['login_login'])) {
+			$this->assertTrue(
+				$login_mock->loginActionRun(),
+				'Assert that post login_login was pressed'
+			);
+		}
+
 		// always check, even on error or not set
 		if (!$login_mock->loginGetLoginUserIdUnclean()) {
 			$this->assertEquals(

4dev/tests/CoreLibsDBExtendedArrayIOTest.php

@@ -36,9 +36,9 @@ final class CoreLibsDBExtendedArrayIOTest extends TestCase
 	 *
 	 * @return void
 	 */
-	public function testDBIO()
+	public function testArrayDBIO()
 	{
-		$this->assertTrue(true, 'DB Extended ArrayIO Tests not implemented');
+		// $this->assertTrue(true, 'DB Extended ArrayIO Tests not implemented');
 		$this->markTestIncomplete(
 			'DB\Extended\ArrayIO Tests have not yet been implemented'
 		);

4dev/tests/CoreLibsOutputFormElementsTest.php

@@ -22,7 +22,7 @@ final class CoreLibsOutputFormElementsTest extends TestCase
 	 */
 	public function testOutputFormElements()
 	{
-		$this->assertTrue(true, 'Output Form Elements Tests not implemented');
+		// $this->assertTrue(true, 'Output Form Elements Tests not implemented');
 		$this->markTestIncomplete(
 			'Output\Form\Elements Tests have not yet been implemented'
 		);

4dev/tests/CoreLibsOutputFormTokenTest.php

@@ -22,7 +22,7 @@ final class CoreLibsOutputFormTokenTest extends TestCase
 	 */
 	public function testOutputFormToken()
 	{
-		$this->assertTrue(true, 'Output Form Token Tests not implemented');
+		// $this->assertTrue(true, 'Output Form Token Tests not implemented');
 		$this->markTestIncomplete(
 			'Output\Form\Token Tests have not yet been implemented'
 		);

4dev/tests/CoreLibsOutputImageTest.php

@@ -22,7 +22,7 @@ final class CoreLibsOutputImageTest extends TestCase
 	 */
 	public function testOutputImage()
 	{
-		$this->assertTrue(true, 'Output Image Tests not implemented');
+		// $this->assertTrue(true, 'Output Image Tests not implemented');
 		$this->markTestIncomplete(
 			'Output\Image Tests have not yet been implemented'
 		);

4dev/tests/CoreLibsOutputProgressbarTest.php

@@ -22,10 +22,10 @@ final class CoreLibsOutputProgressbarTest extends TestCase
 	 */
 	public function testOutputProgressbar()
 	{
-		/* $this->assertTrue(true, 'Output Progressbar Tests not implemented');
+		// $this->assertTrue(true, 'Output Progressbar Tests not implemented');
-		$this->markTestIncomplete(
+		// $this->markTestIncomplete(
-			'Output\Progressbar Tests have not yet been implemented'
+		// 	'Output\Progressbar Tests have not yet been implemented'
-		); */
+		// );
 		$this->markTestSkipped('No implementation for Output\Progressbar at the moment');
 	}
 }

www/includes/admin_header.php

@@ -108,7 +108,7 @@ $data = [
 ];
 // log action
 // no log if login
-if (!$login->login) {
+if (!$login->loginActionRun()) {
 	$cms->adbEditLog('Submit', $data, 'BINARY');
 }
 //------------------------------ logging end

www/lib/CoreLibs/ACL/Login.php

@@ -79,12 +79,12 @@ class Login
 	/** @var string source, either _GET or _POST or empty */
 	private $login_user_id_source = '';
 	/** @var bool set to true if illegal characters where found in the login user id string */
-	private $login_unclear = false;
+	private $login_user_id_unclear = false;
 	// is set to one if login okay, or EUID is set and user is okay to access this page
 	/** @var bool */
 	private $permission_okay = false;
 	/** @var string pressed login */
-	public $login;
+	private $login = '';
 	/** @var string master action command */
 	private $action;
 	/** @var string login name */
@@ -1564,7 +1564,7 @@ EOM;
 			);
 			// flag unclean input data
 			if ($login_user_id_changed > 0) {
-				$this->login_unclear = true;
+				$this->login_user_id_unclear = true;
 				// error for invalid user id?
 				$this->log->debug('LOGIN USER ID', 'Invalid characters: '
 					. $login_user_id_changed . ' in loginUserId: '
@@ -2139,6 +2139,16 @@ EOM;
 		return false;
 	}
 
+	/**
+	 * Returns true if login button was pressed
+	 *
+	 * @return bool If login action was run, return true
+	 */
+	public function loginActionRun(): bool
+	{
+		return empty($this->login) ? false : true;
+	}
+
 	/**
 	 * Returns current set loginUserId or empty if unset
 	 *
@@ -2167,7 +2177,7 @@ EOM;
 	 */
 	public function loginGetLoginUserIdUnclean(): bool
 	{
-		return $this->login_unclear;
+		return $this->login_user_id_unclear;
 	}
 
 	/**