New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
limit access to reports for front and back office #4893
Conversation
acl_upgrade.php
Outdated
AclExtended::shiftAcl($accounting_view, 'Accounting', 'patients', 'Patients', 'pat_rep', 'Patient Report', 'view'); | ||
AclExtended::shiftAcl($frontoffice_view, 'Front Office', 'patients', 'Patients', 'pat_rep', 'Patient Report', 'view'); | ||
AclExtended::shiftAcl($frontoffice_wsome, 'Front Office', 'patients', 'Patients', 'trans', 'Transactions (write,wsome optional)', 'wsome'); | ||
AclExtended::shiftAcl($frontoffice_wsome, 'Front Office', 'patients', 'Patients', 'notes', 'Patient Notes (write,wsome optional)', 'wsome'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
acl_upgrade.php
Outdated
// Upgrade for acl_version 12 | ||
$upgrade_acl = 12; | ||
if ($acl_version < $upgrade_acl) { | ||
echo "<B>UPGRADING ACCESS CONTROLS TO VERSION " . $upgrade_acl . ":</B></BR>"; | ||
|
||
//Collect the ACL ID numbers. | ||
echo "<B>Checking to ensure all the proper ACL(access control list) are present:</B></BR>"; | ||
$accounting_view = AclExtended::getAclIdNumber('Accounting', 'view'); | ||
$frontoffice_view = AclExtended::getAclIdNumber('Front Office', 'view'); | ||
$frontoffice_wsome = AclExtended::getAclIdNumber('Front Office', 'wsome'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like modifying write acl and not the wsome acl in install_gacl() changes
@@ -1017,7 +1017,7 @@ public function install_gacl() | |||
// xl('Things that front office can read and partly modify') | |||
$gacl->add_acl( | |||
array( | |||
'patients' => array('appt', 'demo', 'trans', 'notes'), | |||
'patients' => array('appt', 'demo'), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
note these changes are in the front office write acl and not the wsome acl
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice spot!
} | ||
|
||
return; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
acl_upgrade.php
Outdated
AclExtended::shiftAcl($accounting_view, 'Accounting', 'patients', 'Patients', 'pat_rep', 'Patient Report', 'view'); | ||
AclExtended::shiftAcl($frontoffice_view, 'Front Office', 'patients', 'Patients', 'pat_rep', 'Patient Report', 'view'); | ||
AclExtended::shiftAcl($frontoffice_wsome, 'Front Office', 'patients', 'Patients', 'trans', 'Transactions (write,wsome optional)', 'write'); | ||
AclExtended::shiftAcl($frontoffice_wsome, 'Front Office', 'patients', 'Patients', 'notes', 'Patient Notes (write,wsome optional)', 'write'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
above 2 lines change to $frontoffice_write
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also need to increment $v_acl in version.php
if testing well after fixes, then ready for codebase :)
testing well |
Short description of what this resolves:
limit access to reports for front and back office