Cash receipt screen on version 2.9.1

xiaoanri wrote on Wednesday, February 04, 2009:

I am testing the new 2.9.1 version and found the warning msg on cash receipt by provider screen after refreshing the data:

Warning: strpos() [function.strpos]: Empty delimiter. in /var/www/openemr291/interface/billing/sl_receipts_report.php on line 28

Upon comparing the sl_receipt_report.php in 2.9.0 and 2.9.1, I found 2 lines were added in the new version, the differences are as the following:

2.9.0 version:

  include_once("…/forms/fee_sheet/codes.php");
  function is_clinic($code) {
    global $bcodes;
    return ($bcodes[‘CPT4’][xl(‘Lab’)][$code]     ||
      $bcodes[‘CPT4’][xl(‘Immunizations’)][$code] ||
      $bcodes[‘HCPCS’][xl(‘Therapeutic Injections’)][$code]);
  }

2.9.1 version:

  include_once("…/forms/fee_sheet/codes.php");
  function is_clinic($code) {
    global $bcodes;
    $i = strpos(’:’, $code);
    if ($i) $code = substr($code, 0, $i);
    return ($bcodes[‘CPT4’][xl(‘Lab’)][$code]     ||
      $bcodes[‘CPT4’][xl(‘Immunizations’)][$code] ||
      $bcodes[‘HCPCS’][xl(‘Therapeutic Injections’)][$code]);
}

I do not know the problem.  hopefully is an easy fix, such as a typo… Pls comment. 
Thank you, Hui

cfapress wrote on Wednesday, February 04, 2009:

Hi Hui,

This is just a mix up in the strpos command and has been fixed in the development tree.

Here is the line with the problem:
$i = strpos(’:’, $code);

… and here is what it should be
$i = strpos($code, ‘:’);

Jason

xiaoanri wrote on Thursday, February 05, 2009:

Hi, Jason,

got back to computer and put in the fix. It worked well.  Thanks a lot, 

Hui