Note it required some mods in gacl itself and now need to place the return_value(if applicable) within the acl_check() function. I showed how to do this in the demographics scripts in the commit. Testing well, so will likely commit this over next couple days unless any issues are brought up.
This is testing very well. Since it’s involving a core component of OpenEMR, hoping to get another set of eyes looking at the code before I commit it to sourceforge.
Seems OK, but I’ve always thought it was an error to design access policy so that a given request can have multiple hits. My thought was that usually, a user should be in only one group. Never much cared for bringing another project into OpenEMR and then start modifying it, though have been guilty of that myself.