Skip to content

Commit

Permalink
Merge pull request #130 from zendtech/alanseiden-patch-7
Browse files Browse the repository at this point in the history
Explicitly check for false in makeDbCall()
  • Loading branch information
chukShirley authored Sep 18, 2020
2 parents 464bbdd + 045a809 commit e9852fb
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion ToolkitApi/Toolkit.php
Original file line number Diff line number Diff line change
Expand Up @@ -906,7 +906,7 @@ protected function makeDbCall($internalKey, $plugSize, $controlKeyString, $input

// can return false if prepare or exec failed.
$outputXml = $this->db->execXMLStoredProcedure($this->conn, $sql, $bindArray);
if (!$outputXml) {
if (false === $outputXml) {
// if false returned, was a database error (stored proc prepare or execute error)
// @todo add ODBC SQL State codes

Expand Down

0 comments on commit e9852fb

Please sign in to comment.