ErrorMessage add notice level

publish/last.published

@@ -1 +1 @@
-9.7.2
+9.7.7

src/Admin/EditBase.php

@@ -35,6 +35,8 @@ class EditBase
 	private \CoreLibs\Output\Form\Generate $form;
 	/** @var \CoreLibs\Logging\Logging */
 	public \CoreLibs\Logging\Logging $log;
+	/** @var \CoreLibs\Language\L10n */
+	public \CoreLibs\Language\L10n $l;
 	/** @var \CoreLibs\ACL\Login */
 	public \CoreLibs\ACL\Login $login;
 
@@ -57,6 +59,7 @@ class EditBase
 	) {
 		$this->log = $log;
 		$this->login = $login;
+		$this->l = $l10n;
 		// smarty template engine (extended Translation version)
 		$this->smarty = new \CoreLibs\Template\SmartyExtend(
 			$l10n,
@@ -77,7 +80,7 @@ class EditBase
 			echo "I am sorry, but this page cannot be viewed by a mobile phone";
 			exit;
 		}
-		// $this->form->log->debug('POST', $this->form->log->prAr($_POST));
+		// $this->log->debug('POST', $this->log->prAr($_POST));
 	}
 
 	/**
@@ -179,7 +182,7 @@ class EditBase
 		} // while read data ...
 
 		// html title
-		$this->HEADER['HTML_TITLE'] = $this->form->l->__('Edit Order');
+		$this->HEADER['HTML_TITLE'] = $this->l->__('Edit Order');
 
 		$messages = [];
 		$error = $_POST['error'] ?? 0;
@@ -632,7 +635,7 @@ class EditBase
 			'editAdmin_' . $this->smarty->lang
 		);
 
-		$this->form->log->debug('DEBUGEND', '==================================== [Form END]');
+		$this->log->debug('DEBUGEND', '==================================== [Form END]');
 	}
 }
 

src/DB/Extended/ArrayIO.php

@@ -198,7 +198,7 @@ class ArrayIO extends \CoreLibs\DB\IO
 	public function dbCheckPkSet(): bool
 	{
 		// if pk_id is set, overrule ...
-		if ($this->pk_id) {
+		if (!empty($this->pk_id)) {
 			$this->table_array[$this->pk_name]['value'] = $this->pk_id;
 		}
 		// if not set ... produce error

src/DB/IO.php

@@ -3232,7 +3232,7 @@ class IO
 	 *
 	 * @param  array<mixed> $write_array     list of elements to write
 	 * @param  array<mixed> $not_write_array list of elements not to write
-	 * @param  int          $primary_key     id key to decide if we write insert or update
+	 * @param  int|null     $primary_key     id key to decide if we write insert or update
 	 * @param  string       $table           name for the target table
 	 * @param  array<mixed> $data            data array to override _POST data
 	 * @return int|false                     primary key
@@ -3240,7 +3240,7 @@ class IO
 	public function dbWriteData(
 		array $write_array,
 		array $not_write_array,
-		int $primary_key,
+		?int $primary_key,
 		string $table,
 		array $data = []
 	): int|false {
@@ -3260,19 +3260,19 @@ class IO
 	 * PARAM INFO: $primary key
 	 * this can be a plain string/int and will be internal transformed into the array form
 	 * or it takes the array form of array [row => column, value => pk value]
-	 * @param  array<mixed>            $write_array     list of elements to write
+	 * @param  array<mixed>                 $write_array     list of elements to write
-	 * @param  int|string|array<mixed> $primary_key     primary key string or array set
+	 * @param  null|int|string|array<mixed> $primary_key     primary key string or array set
-	 * @param  string                  $table           name for the target table
+	 * @param  string                       $table           name for the target table
-	 * @param  array<mixed>            $not_write_array list of elements not to write (optional)
+	 * @param  array<mixed>                 $not_write_array list of elements not to write (optional)
-	 * @param  array<mixed>            $not_write_update_array list of elements not
+	 * @param  array<mixed>                 $not_write_update_array list of elements not
-	 *                                                         to write during update (optional)
+	 *                                                       to write during update (optional)
-	 * @param  array<mixed>            $data            optional array with data
+	 * @param  array<mixed>                 $data            optional array with data
-	 *                                                  if not _POST vars are used
+	 *                                                       if not _POST vars are used
-	 * @return int|false                                primary key
+	 * @return int|false                                     primary key
 	 */
 	public function dbWriteDataExt(
 		array $write_array,
-		int|string|array $primary_key,
+		null|int|string|array $primary_key,
 		string $table,
 		array $not_write_array = [],
 		array $not_write_update_array = [],

src/Logging/ErrorMessage.php

@@ -15,7 +15,7 @@ use CoreLibs\Logging\Logger\MessageLevel;
 
 class ErrorMessage
 {
-	/** @var array<int,array{id:string,level:string,str:string,target:string,highlight:string[]}> */
+	/** @var array<int,array{id:string,level:string,str:string,target:string,target_style:string,highlight:string[]}> */
 	private array $error_str = [];
 	/** @var \CoreLibs\Logging\Logging $log */
 	public \CoreLibs\Logging\Logging $log;
@@ -42,6 +42,7 @@ class ErrorMessage
 	 * error_id: internal Error ID (should be unique)
 	 * level: error level, can only be ok, info, warn, error, abort, crash
 	 *        ok and info are positive response: success
+	 *        notice: a debug message for information only
 	 *        warn: success, but there might be some things that are not 100% ok
 	 *        error: input error or error in executing request
 	 *        abort: an internal error happened as mandatory information that normally is
@@ -52,23 +53,28 @@ class ErrorMessage
 	 *        not set: unkown, will be logged as "emergency"
 	 * target/highlight: id target name for frontend where to attach this message
 	 *                   highlight is a list of other target points to highlight
+	 *                   for highlight targets css names are $level without a prefix and should be
+	 *                   nested in the target element "input .error { ... }"
+	 * target_style: if not set uses 'error-' $level as css style. applies to targets or main only
 	 *
-	 * @param  string        $error_id  Any internal error ID for this error
+	 * @param  string        $error_id     Any internal error ID for this error
-	 * @param  string        $level     Error level in ok/info/warn/error
+	 * @param  string        $level        Error level in ok/info/warn/error
-	 * @param  string        $str       Error message (out)
+	 * @param  string        $str          Error message (out)
-	 * @param  string        $target    alternate attachment point for this error message
+	 * @param  string        $target       alternate attachment point for this error message
-	 * @param  array<string> $highlight Any additional error data as error OR
+	 * @param  string        $target_style Alternate color style for the error message
-	 *                                  highlight points for field highlights
+	 * @param  array<string> $highlight    Any additional error data as error OR
-	 * @param  string|null   $message   If abort/crash, non localized $str
+	 *                                     highlight points for field highlights
-	 * @param  array<mixed>  $context   Additionl info for abort/crash messages
+	 * @param  string|null   $message      If abort/crash, non localized $str
-	 * @param  bool|null     $log_error [=null] log level 'error' to error, if null use global,
+	 * @param  array<mixed>  $context      Additionl info for abort/crash messages
-	 *                                  else set for this call only
+	 * @param  bool|null     $log_error    [=null] log level 'error' to error, if null use global,
+	 *                                     else set for this call only
 	 */
 	public function setErrorMsg(
 		string $error_id,
 		string $level,
 		string $str,
 		string $target = '',
+		string $target_style = '',
 		array $highlight = [],
 		?string $message = null,
 		array $context = [],
@@ -88,10 +94,17 @@ class ErrorMessage
 			'level' => $level,
 			'str' => $str,
 			'target' => $target,
+			'target_style' => $target_style,
 			'highlight' => $highlight,
 		];
 		// write to log for abort/crash
 		switch ($level) {
+			case 'notice':
+				$this->log->notice($message ?? $str, array_merge([
+					'id' => $error_id,
+					'level' => $original_level,
+				], $context));
+				break;
 			case 'error':
 				if ($log_error) {
 					$this->log->error($message ?? $str, array_merge([
@@ -126,28 +139,40 @@ class ErrorMessage
 	 * Note, the parameter order is different and does not need an error id
 	 * This is for backend alerts
 	 *
-	 * @param  string        $level     error level (ok/warn/info/error)
+	 * @param  string        $level        error level (ok/warn/info/error)
-	 * @param  string        $str       error string
+	 * @param  string        $str          error string
-	 * @param  string|null   $error_id  optional error id for precise error lookup
+	 * @param  string|null   $error_id     optional error id for precise error lookup
-	 * @param  string        $target    Alternate id name for output target on frontend
+	 * @param  string        $target       Alternate id name for output target on frontend
-	 * @param  array<string> $highlight Any additional error data as error OR
+	 * @param  string        $target_style Alternate color style for the error message
-	 *                                  highlight points for field highlights
+	 * @param  array<string> $highlight    Any additional error data as error OR
-	 * @param  string|null   $message   If abort/crash, non localized $str
+	 *                                     highlight points for field highlights
-	 * @param  array<mixed>  $context   Additionl info for abort/crash messages
+	 * @param  string|null   $message      If abort/crash, non localized $str
-	 * @param  bool|null     $log_error [=null] log level 'error' to error, if null use global,
+	 * @param  array<mixed>  $context      Additionl info for abort/crash messages
-	 *                                  else set for this call only
+	 * @param  bool|null     $log_error    [=null] log level 'error' to error, if null use global,
+	 *                                     else set for this call only
 	 */
 	public function setMessage(
 		string $level,
 		string $str,
 		?string $error_id = null,
 		string $target = '',
+		string $target_style = '',
 		array $highlight = [],
 		?string $message = null,
 		array $context = [],
 		?bool $log_error = null,
 	): void {
-		$this->setErrorMsg($error_id ?? '', $level, $str, $target, $highlight, $message, $context, $log_error);
+		$this->setErrorMsg(
+			$error_id ?? '',
+			$level,
+			$str,
+			$target,
+			$target_style,
+			$highlight,
+			$message,
+			$context,
+			$log_error
+		);
 	}
 
 	// *********************************************************************
@@ -178,7 +203,7 @@ class ErrorMessage
 	 * Gets the LAST entry in the array list.
 	 * If nothing found returns empty array set
 	 *
-	 * @return array{id:string,level:string,str:string,target:string,highlight:string[]} Error block
+	 * @return array{id:string,level:string,str:string,target:string,target:string,highlight:string[]} Error block
 	 */
 	public function getLastErrorMsg(): array
 	{
@@ -187,6 +212,7 @@ class ErrorMessage
 			'str' => '',
 			'id' => '',
 			'target' => '',
+			'target_string' => '',
 			'highlight' => [],
 		];
 	}

src/Logging/Logger/MessageLevel.php

@@ -15,6 +15,7 @@ enum MessageLevel: int
 {
 	case ok = 100;
 	case info = 200;
+	case notice = 250;
 	case warn = 300;
 	case error = 400;
 	case abort = 500;
@@ -30,6 +31,7 @@ enum MessageLevel: int
 		return match (strtolower($name)) {
 			'ok' => self::ok,
 			'info' => self::info,
+			'notice' => self::notice,
 			'warn', 'warning' => self::warn,
 			'error' => self::error,
 			'abort' => self::abort,

src/Output/Form/Generate.php

@@ -300,7 +300,8 @@ class Generate extends \CoreLibs\DB\Extended\ArrayIO
 	// log
 	/** @var \CoreLibs\Logging\Logging */
 	public \CoreLibs\Logging\Logging $log;
-
+	/** @var \CoreLibs\DB\Extended\ArrayIO */
+	public \CoreLibs\DB\Extended\ArrayIO $dba;
 	// now some default error msgs (english)
 	/** @var array<mixed> */
 	public array $language_array = [];
@@ -382,6 +383,15 @@ class Generate extends \CoreLibs\DB\Extended\ArrayIO
 			$this->base_acl_level,
 			$this->acl_admin
 		);
+		$this->dba = new \CoreLibs\DB\Extended\ArrayIO(
+			$db_config,
+			$config_array['table_array'],
+			$config_array['table_name'],
+			$this->log,
+			// set the ACL
+			$this->base_acl_level,
+			$this->acl_admin
+		);
 		// here should be a check if the config_array is correct ...
 		if (isset($config_array['show_fields']) && is_array($config_array['show_fields'])) {
 			$this->field_array = $config_array['show_fields'];

src/Output/Form/TableArrays/EditOrder.php

@@ -0,0 +1,40 @@
+<?php
+
+declare(strict_types=1);
+
+namespace CoreLibs\Output\Form\TableArrays;
+
+class EditOrder implements Interface\TableArraysInterface
+{
+	/** @var \CoreLibs\Output\Form\Generate */
+	private \CoreLibs\Output\Form\Generate $form;
+
+	/**
+	 * constructor
+	 * @param \CoreLibs\Output\Form\Generate $form base form class
+	 */
+	public function __construct(\CoreLibs\Output\Form\Generate $form)
+	{
+		$this->form = $form;
+		$this->form->log->debug('CLASS LOAD', __NAMESPACE__ . __CLASS__);
+	}
+
+	/**
+	 * NOTE: this is a dummy array to just init the Form\Generate class and is not used for anything else
+	 *
+	 * @return array<mixed>
+	 */
+	public function setTableArray(): array
+	{
+		return [
+			'table_array' => [
+				'-'
+			],
+			'table_name' => '-',
+			'load_query' => '',
+			'show_fields' => [],
+		];
+	}
+}
+
+// __END__

test/phpunit/Logging/CoreLibsLoggingErrorMessagesTest.php

@@ -44,6 +44,11 @@ final class CoreLibsLoggingErrorMessagesTest extends TestCase
 				'str' => 'INFO',
 				'expected' => 'info',
 			],
+			'notice' => [
+				'level' => 'notice',
+				'str' => 'NOTICE',
+				'expected' => 'notice',
+			],
 			'warn' => [
 				'level' => 'warn',
 				'str' => 'WARN',
@@ -106,6 +111,7 @@ final class CoreLibsLoggingErrorMessagesTest extends TestCase
 				'str' => $str,
 				'id' => '',
 				'target' => '',
+				'target_style' => '',
 				'highlight' => [],
 			],
 			$em->getLastErrorMsg()
@@ -139,6 +145,7 @@ final class CoreLibsLoggingErrorMessagesTest extends TestCase
 				'level' => 'info',
 				'str' => 'INFO MESSAGE',
 				'target' => '',
+				'target_style' => '',
 				'highlight' => [],
 			],
 			$em->getLastErrorMsg()
@@ -154,6 +161,7 @@ final class CoreLibsLoggingErrorMessagesTest extends TestCase
 					'level' => 'info',
 					'str' => 'INFO MESSAGE',
 					'target' => '',
+					'target_style' => '',
 					'highlight' => [],
 				]
 			],
@@ -171,6 +179,7 @@ final class CoreLibsLoggingErrorMessagesTest extends TestCase
 				'level' => 'error',
 				'str' => 'ERROR MESSAGE',
 				'target' => '',
+				'target_style' => '',
 				'highlight' => [],
 			],
 			$em->getLastErrorMsg()
@@ -186,6 +195,7 @@ final class CoreLibsLoggingErrorMessagesTest extends TestCase
 					'level' => 'info',
 					'str' => 'INFO MESSAGE',
 					'target' => '',
+					'target_style' => '',
 					'highlight' => [],
 				],
 				[
@@ -193,6 +203,7 @@ final class CoreLibsLoggingErrorMessagesTest extends TestCase
 					'level' => 'error',
 					'str' => 'ERROR MESSAGE',
 					'target' => '',
+					'target_style' => '',
 					'highlight' => [],
 				]
 			],
@@ -232,6 +243,22 @@ final class CoreLibsLoggingErrorMessagesTest extends TestCase
 				'log_error' => true,
 				'expected' => '<ERROR> OTHER ERROR MESSAGE',
 			],
+			'notice' => [
+				'id' => '100',
+				'level' => 'notice',
+				'str' => 'NOTICE MESSAGE',
+				'message' => null,
+				'log_error' => null,
+				'expected' => '<NOTICE> NOTICE MESSAGE',
+			],
+			'notice, message' => [
+				'id' => '100',
+				'level' => 'notice',
+				'str' => 'NOTICE MESSAGE',
+				'message' => 'OTHER NOTICE MESSAGE',
+				'log_error' => null,
+				'expected' => '<NOTICE> OTHER NOTICE MESSAGE',
+			],
 			'crash' => [
 				'id' => '300',
 				'level' => 'crash',