Commit 3733a0d4c73862c624c5f8abf78c875041ee3456

Authored by Neil Blakey-Milner
1 parent a92bfa64

Both create and update methods on KTEntity classes can return false if

they haven't been moved over to use PEAR Errors instead.  Catch that
where they're used internally.


git-svn-id: https://kt-dms.svn.sourceforge.net/svnroot/kt-dms/trunk@6041 c91229c3-7414-0410-bfa2-8a42b809f60b
Showing 1 changed file with 9 additions and 0 deletions
lib/ktentity.inc
@@ -373,6 +373,9 @@ class KTEntity { @@ -373,6 +373,9 @@ class KTEntity {
373 if (PEAR::isError($ret)) { 373 if (PEAR::isError($ret)) {
374 return $ret; 374 return $ret;
375 } 375 }
  376 + if ($ret === false) {
  377 + return PEAR::raiseError(sprintf(_kt("update for class %s failed"), $sClassName));
  378 + }
376 return true; 379 return true;
377 } 380 }
378 381
@@ -598,6 +601,9 @@ class KTEntityUtil { @@ -598,6 +601,9 @@ class KTEntityUtil {
598 if (PEAR::isError($ret)) { 601 if (PEAR::isError($ret)) {
599 return $ret; 602 return $ret;
600 } 603 }
  604 + if ($ret === false) {
  605 + return PEAR::raiseError(sprintf(_kt("create for class %s failed"), $sClassName));
  606 + }
601 $meth = array($sClassName, 'get'); 607 $meth = array($sClassName, 'get');
602 return call_user_func($meth, $oObject->getId()); 608 return call_user_func($meth, $oObject->getId());
603 } 609 }
@@ -616,6 +622,9 @@ class KTEntityUtil { @@ -616,6 +622,9 @@ class KTEntityUtil {
616 if (PEAR::isError($ret)) { 622 if (PEAR::isError($ret)) {
617 return $ret; 623 return $ret;
618 } 624 }
  625 + if ($ret === false) {
  626 + return PEAR::raiseError(sprintf(_kt("update for class %s failed"), $sClassName));
  627 + }
619 return true; 628 return true;
620 } 629 }
621 630