Patient statement in v5.0.0(3)

OpenEMR Development
21

I thought it always worked that way because when I did send to portal statement I had to allow for it and just send last page(summary). This was back in Dec. '16.

22

i saw your comment, no, in 4.1.3 it wouldn’t create multilple statements for the same person

23

Seems to me that makes more sense. Do you think it is a bug or was meant to be? I’m pretty sure if Terry did it, he would put in a global to turn on or off.

24

it’s a bug, i think i remember Terry saying so

25

i see this


but i don’t see any html form input attributes with form_invpids

26

Line 954 is the selector for eob’s. So help me out, we only want a one page report per pid regardless if one encounter is selected or more than one?

27

got it here

28

that’s right

29

I think there is a PR here that was working on this(didn’t get in the codebase, though):

30

@stephenwaite the code you’re looking at is hidden for portal and should not effect normal work flow however the only reason it is there was because of the existing behavior of statement print at the time. I’ll remove it once we figure out the bug or not bug. I’m still unclear on that.

31

Okay, so the PR is the solution then?

32

don’t think so, just tested venambati’s file, no love

33

If you guys can wait 'till tomorrow or so, I’ll fix it and also add the image path fix.

34

that would be great, thanks Jerry

35

could you also add CVV to the credit card image? (not sure if that’s your creation) thanks

36

Not mine but i’ll see what I can do

37

Yes Stephen I got it fixed
Thanks

38

Is there a PR or do I still need to do it?

39

I have no PR for this.

40

@sjpadgett in sl_eob_search.php change line 526 forward to this.

        ftruncate($fhprint,0);
   }
    else    continue;
}

} // end while

if (!empty($stmt)) ++$stmt_count;
fwrite($fhprint, make_statement($stmt));

That is the change I made to remove the duplicate printing within the statement.