fix: x12837 billing 5 or 9 digit zip check (openemr#7760)

bradymiller · Oct 10, 2024 · 8493cde · 8493cde
1 parent 4852747
commit 8493cde
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/src/Billing/Claim.php b/src/Billing/Claim.php
@@ -1677,7 +1677,7 @@ public function referrerFirstName()
 
     public function referrerMiddleName()
     {
-        return $this->x12Clean(trim($this->referrer['mname']));
+        return $this->x12Clean(trim($this->referrer['mname'] ?? ''));
     }
 
     public function referrerNPI()

diff --git a/src/Billing/X125010837P.php b/src/Billing/X125010837P.php
@@ -1223,7 +1223,7 @@ public static function genX12837P(
             if (
                 !(
                     (strlen($claim->insuredZip($ins)) == 5)
-                    || (strlen($claim->insuredZip($ins) == 9))
+                    || (strlen($claim->insuredZip($ins)) == 9)
                 )
             ) {
                 $log .= "*** Other insco insured zip is not 5 or 9 digits.\n";
@@ -1285,7 +1285,7 @@ public static function genX12837P(
             if (
                 !(
                     (strlen($claim->payerZip($ins)) == 5)
-                    || (strlen($claim->payerZip() == 9))
+                    || (strlen($claim->payerZip($ins)) == 9)
                 )
             ) {
                 $log .= "*** Other payer zip is not 5 or 9 digits.\n";

diff --git a/src/Services/FormService.php b/src/Services/FormService.php
@@ -48,6 +48,7 @@ public function getFormByEncounter(
 
         $res = sqlStatement($sql, $arraySqlBind);
 
+        $all = [];
         for ($iter = 0; $row = sqlFetchArray($res); $iter++) {
             $all[$iter] = $row;
         }