Support for ACL return values

bradymiller wrote on Saturday, September 15, 2012:

Hi,

Think I got the support of return values working well with following commit:
http://github.com/bradymiller/openemr/commit/7acc2a7c1ac05e4656b48b238252aae417952a97

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.

-brady
OpenEMR

bradymiller wrote on Monday, September 17, 2012:

Hi,

Updated code to work with the Add/Edit event window (to require write or wsome) (no other changes):
http://github.com/bradymiller/openemr/commits/acl-support-return-value_2

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.

thanks,
-brady
OpenEMR

sunsetsystems wrote on Monday, September 17, 2012:

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.

Rod
www.sunsetsystems.com