From e32fbba991d88094b7980475a19b34f1123d3657 Mon Sep 17 00:00:00 2001
From: steinm <steinm@577cc39b-cfd1-4aa2-8bc3-eda205cea05a>
Date: Mon, 11 Feb 2013 07:46:02 +0000
Subject: [PATCH] - all methods ...ByName() return false if not name is passed

---
 LetoDMS_Core/Core/inc.ClassDMS.php | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/LetoDMS_Core/Core/inc.ClassDMS.php b/LetoDMS_Core/Core/inc.ClassDMS.php
index c0a9d9c85..372c28e34 100644
--- a/LetoDMS_Core/Core/inc.ClassDMS.php
+++ b/LetoDMS_Core/Core/inc.ClassDMS.php
@@ -1286,8 +1286,9 @@ class LetoDMS_Core_DMS {
 	 * @return object instance of {@link LetoDMS_Core_DocumentCategory}
 	 */
 	function getDocumentCategoryByName($name) { /* {{{ */
-		$queryStr = "SELECT * FROM tblCategory where name=".$this->db->qstr($name);
+		if (!$name) return false;
 
+		$queryStr = "SELECT * FROM tblCategory where name=".$this->db->qstr($name);
 		$resArr = $this->db->getResultArray($queryStr);
 		if (!$resArr)
 			return false;
@@ -1452,6 +1453,8 @@ class LetoDMS_Core_DMS {
 	 * @return object instance of {@link LetoDMS_Core_AttributeDefinition} or false
 	 */
 	function getAttributeDefinitionByName($name) { /* {{{ */
+		if (!$name) return false;
+
 		$queryStr = "SELECT * FROM tblAttributeDefinitions WHERE name = " . $this->db->qstr($name);
 		$resArr = $this->db->getResultArray($queryStr);
 
@@ -1582,6 +1585,8 @@ class LetoDMS_Core_DMS {
 	 * @return object of instances of {@link LetoDMS_Core_Workflow} or false
 	 */
 	function getWorkflowByName($name) { /* {{{ */
+		if (!$name) return false;
+
 		$queryStr = "SELECT * FROM tblWorkflows WHERE name=".$this->db->qstr($name);
 		$resArr = $this->db->getResultArray($queryStr);
 
@@ -1644,6 +1649,8 @@ class LetoDMS_Core_DMS {
 	 * @return object of instances of {@link LetoDMS_Core_Workflow_State} or false
 	 */
 	function getWorkflowStateByName($name) { /* {{{ */
+		if (!$name) return false;
+
 		$queryStr = "SELECT * FROM tblWorkflowStates WHERE name=".$this->db->qstr($name);
 		$resArr = $this->db->getResultArray($queryStr);
 
@@ -1737,8 +1744,7 @@ class LetoDMS_Core_DMS {
 	 * @return object instance of {@link LetoDMS_Core_Workflow_Action} or false
 	 */
 	function getWorkflowActionByName($name) { /* {{{ */
-		if (!is_numeric($id))
-			return false;
+		if (!$name) return false;
 
 		$queryStr = "SELECT * FROM tblWorkflowActions WHERE name = " . $this->db->qstr($name);
 		$resArr = $this->db->getResultArray($queryStr);