Skip to content
Snippets Groups Projects
Commit c5eeedf0 authored by Prasad's avatar Prasad
Browse files

Merge branch '41062520_relatedModule_Permission' into 'master'

Fixes #1146 ::Uma::related module permission will be checked before process trigger

See merge request !363
parents 05ce2258 b17331c5
No related branches found
No related tags found
No related merge requests found
...@@ -516,7 +516,9 @@ class Vtiger_Detail_View extends Vtiger_Index_View { ...@@ -516,7 +516,9 @@ class Vtiger_Detail_View extends Vtiger_Index_View {
} }
if($targetControllerClass) { if($targetControllerClass) {
$targetController = new $targetControllerClass(); $targetController = new $targetControllerClass();
return $targetController->process($request); if($targetController->checkPermission($request)){
return $targetController->process($request);
}
} }
} }
......
...@@ -9,6 +9,17 @@ ...@@ -9,6 +9,17 @@
*************************************************************************************/ *************************************************************************************/
class Vtiger_RelatedList_View extends Vtiger_Index_View { class Vtiger_RelatedList_View extends Vtiger_Index_View {
function checkPermission(Vtiger_Request $request) {
$relatedModuleName = $request->get('relatedModule');
$relatedModulePermission = Users_Privileges_Model::isPermitted($relatedModuleName, 'DetailView');
if(!$relatedModulePermission) {
throw new AppException(vtranslate('LBL_PERMISSION_DENIED'));
}
return true;
}
function process(Vtiger_Request $request) { function process(Vtiger_Request $request) {
$moduleName = $request->getModule(); $moduleName = $request->getModule();
$relatedModuleName = $request->get('relatedModule'); $relatedModuleName = $request->get('relatedModule');
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment