bradymiller/openemr — comment on commit a0e572611acefc0b26d3eca4ffa6950880f00c47
File:  line 0
By bradymiller:

Comments from previous review:
Forgot to drink coffee before this observation, so got confused on the not stuff. I see what you are doing, but then you are using a acl control to "close" access to something. It is much better to do the other way around. For example, if a user has notes, notes_a, and notes_a_v all selected, you don't want the little guy (notes_a_v) to stop the big guys (notes_a and notes). So, will be better to figure out which controls need to be set to allow this:
Using the handy-dandy newly created acl wiki list:
http://open-emr.org/wiki/index.php/Access_Controls_Listing
Looks like super, notes(if this is this user's encounter), notes_a, and med (from the patients group) should suffice to control the AMC stuff.
-brady