check if passed id > 0 in getWorkflow(), getWorkflowAction(), getWorkflowState()

This commit is contained in:
Uwe Steinmann 2021-09-27 10:08:20 +02:00
parent 79bcdb6086
commit 88e0dcc58d

View File

@ -2728,7 +2728,7 @@ class SeedDMS_Core_DMS {
* @return SeedDMS_Core_Workflow|bool of instances of {@link SeedDMS_Core_Workflow}, null if no workflow was found or false * @return SeedDMS_Core_Workflow|bool of instances of {@link SeedDMS_Core_Workflow}, null if no workflow was found or false
*/ */
function getWorkflow($id) { /* {{{ */ function getWorkflow($id) { /* {{{ */
if (!is_numeric($id)) if (!is_numeric($id) || $id < 1)
return false; return false;
$queryStr = "SELECT * FROM `tblWorkflows` WHERE `id`=".intval($id); $queryStr = "SELECT * FROM `tblWorkflows` WHERE `id`=".intval($id);
@ -2807,7 +2807,7 @@ class SeedDMS_Core_DMS {
* @return bool|SeedDMS_Core_Workflow_State or false * @return bool|SeedDMS_Core_Workflow_State or false
*/ */
function getWorkflowState($id) { /* {{{ */ function getWorkflowState($id) { /* {{{ */
if (!is_numeric($id)) if (!is_numeric($id) || $id < 1)
return false; return false;
$queryStr = "SELECT * FROM `tblWorkflowStates` WHERE `id` = " . (int) $id; $queryStr = "SELECT * FROM `tblWorkflowStates` WHERE `id` = " . (int) $id;
@ -2907,7 +2907,7 @@ class SeedDMS_Core_DMS {
* @return SeedDMS_Core_Workflow_Action|bool instance of {@link SeedDMS_Core_Workflow_Action} or false * @return SeedDMS_Core_Workflow_Action|bool instance of {@link SeedDMS_Core_Workflow_Action} or false
*/ */
function getWorkflowAction($id) { /* {{{ */ function getWorkflowAction($id) { /* {{{ */
if (!is_numeric($id)) if (!is_numeric($id) || $id < 1)
return false; return false;
$queryStr = "SELECT * FROM `tblWorkflowActions` WHERE `id` = " . (int) $id; $queryStr = "SELECT * FROM `tblWorkflowActions` WHERE `id` = " . (int) $id;