diff --git a/includes/runtime/Controller.php b/includes/runtime/Controller.php index fdaa4be3cb9b0700b2c32ad337f05e4ab2cbacad..962244a9f18accaf01a4c32a6b4548df9d99bb77 100644 --- a/includes/runtime/Controller.php +++ b/includes/runtime/Controller.php @@ -129,6 +129,7 @@ abstract class Vtiger_Action_Controller extends Vtiger_Controller { throw new AppException(vtranslate('LBL_PERMISSION_DENIED')); } } + return true; } } diff --git a/modules/Accounts/actions/TransferOwnership.php b/modules/Accounts/actions/TransferOwnership.php index b04eec39931d3d984b93ff9df7bcea827805849b..a8816b1ea7321761688f2bb561e9c1262df599a8 100644 --- a/modules/Accounts/actions/TransferOwnership.php +++ b/modules/Accounts/actions/TransferOwnership.php @@ -27,6 +27,7 @@ class Accounts_TransferOwnership_Action extends Vtiger_Action_Controller { $this->transferRecordIds[] = $recordId; } } + return true; } public function process(Vtiger_Request $request) { diff --git a/modules/Accounts/views/AccountHierarchy.php b/modules/Accounts/views/AccountHierarchy.php index 6e31b143f6c233a2e3153e766588dd785918dfee..c07916e7a3507dfad2d00b1eae69af44e42d962c 100644 --- a/modules/Accounts/views/AccountHierarchy.php +++ b/modules/Accounts/views/AccountHierarchy.php @@ -16,7 +16,7 @@ class Accounts_AccountHierarchy_View extends Vtiger_View_Controller { } public function checkPermission(Vtiger_Request $request) { - parent::checkPermission($request); + return parent::checkPermission($request); } function preProcess(Vtiger_Request $request, $display = true) { diff --git a/modules/Calendar/actions/ActivityReminder.php b/modules/Calendar/actions/ActivityReminder.php index b32e026552b6046a8180187fdf2140a506341565..fe1b4aee5eb5f009f48b01ff8196c96e5614cba3 100644 --- a/modules/Calendar/actions/ActivityReminder.php +++ b/modules/Calendar/actions/ActivityReminder.php @@ -35,7 +35,7 @@ class Calendar_ActivityReminder_Action extends Vtiger_Action_Controller{ } public function checkPermission(Vtiger_Request $request) { - parent::checkPermission($request); + return parent::checkPermission($request); } public function process(Vtiger_Request $request) { diff --git a/modules/Calendar/actions/CalendarActions.php b/modules/Calendar/actions/CalendarActions.php index 7256e8b644ffc3de0fb97d694fc9997e0b873e5b..aac5e9e412786db62048c3c69466ffa2012ac08b 100644 --- a/modules/Calendar/actions/CalendarActions.php +++ b/modules/Calendar/actions/CalendarActions.php @@ -29,7 +29,7 @@ class Calendar_CalendarActions_Action extends Vtiger_BasicAjax_Action { } public function checkPermission(Vtiger_Request $request) { - parent::checkPermission($request); + return parent::checkPermission($request); } public function process(Vtiger_Request $request) { diff --git a/modules/Calendar/actions/CalendarUserActions.php b/modules/Calendar/actions/CalendarUserActions.php index 41744c24e28557e44efc598e03fe91eef1bf5888..22fed381088f4203a74f4da1a667c2b02579ae2e 100755 --- a/modules/Calendar/actions/CalendarUserActions.php +++ b/modules/Calendar/actions/CalendarUserActions.php @@ -42,7 +42,7 @@ class Calendar_CalendarUserActions_Action extends Vtiger_Action_Controller{ } public function checkPermission(Vtiger_Request $request) { - parent::checkPermission($request); + return parent::checkPermission($request); } public function process(Vtiger_Request $request) { diff --git a/modules/Campaigns/actions/DetailAjax.php b/modules/Campaigns/actions/DetailAjax.php index 31f8f17344c18577391db906d1ad804eaf9c908d..5a75e4dfd191fd842e29050c3a452d75c8f0ddec 100644 --- a/modules/Campaigns/actions/DetailAjax.php +++ b/modules/Campaigns/actions/DetailAjax.php @@ -30,7 +30,7 @@ class Campaigns_DetailAjax_Action extends Vtiger_BasicAjax_Action { } public function checkPermission(Vtiger_Request $request) { - parent::checkPermission($request); + return parent::checkPermission($request); } public function process(Vtiger_Request $request) { $mode = $request->get('mode'); diff --git a/modules/Campaigns/actions/RelationAjax.php b/modules/Campaigns/actions/RelationAjax.php index 3cfd4fa8d4ba54c066341dbe647562b70fa28703..0af2bbfb26e19f677259e544392aa1e5709e8e3a 100644 --- a/modules/Campaigns/actions/RelationAjax.php +++ b/modules/Campaigns/actions/RelationAjax.php @@ -35,7 +35,7 @@ class Campaigns_RelationAjax_Action extends Vtiger_RelationAjax_Action { } public function checkPermission(Vtiger_Request $request) { - parent::checkPermission($request); + return parent::checkPermission($request); } /** diff --git a/modules/Vtiger/actions/Save.php b/modules/Vtiger/actions/Save.php index 60ea03eaadc0da368636c77b88bddcc59d3f0d12..874dc78468a3cea39fe76d24fa68c23f9b0428d8 100644 --- a/modules/Vtiger/actions/Save.php +++ b/modules/Vtiger/actions/Save.php @@ -28,7 +28,7 @@ class Vtiger_Save_Action extends Vtiger_Action_Controller { throw new AppException(vtranslate('LBL_PERMISSION_DENIED')); } } - parent::checkPermission($request); + return parent::checkPermission($request); } public function validateRequest(Vtiger_Request $request) { diff --git a/modules/Vtiger/views/Detail.php b/modules/Vtiger/views/Detail.php index f7745a5fbd97af4e6c830f0eb85c5d98411369a9..33852d1fd5dd827efba62004c29525f8739d9468 100644 --- a/modules/Vtiger/views/Detail.php +++ b/modules/Vtiger/views/Detail.php @@ -65,7 +65,7 @@ class Vtiger_Detail_View extends Vtiger_Index_View { throw new AppException(vtranslate('LBL_PERMISSION_DENIED')); } } - parent::checkPermission($request); + return parent::checkPermission($request); } function preProcess(Vtiger_Request $request, $display=true) { diff --git a/modules/Vtiger/views/Edit.php b/modules/Vtiger/views/Edit.php index 683376294e77ac7d8837947cc8665753b459a2d1..00762d4e03bd2bcaaa7f122fc55585c8c8bb3246 100644 --- a/modules/Vtiger/views/Edit.php +++ b/modules/Vtiger/views/Edit.php @@ -34,7 +34,7 @@ Class Vtiger_Edit_View extends Vtiger_Index_View { throw new AppException(vtranslate('LBL_PERMISSION_DENIED')); } } - parent::checkPermission($request); + return parent::checkPermission($request); } public function setModuleInfo($request, $moduleModel) { diff --git a/modules/Vtiger/views/RelatedList.php b/modules/Vtiger/views/RelatedList.php index a5a7552ec465994a691df913aecd102c57a316da..fa2e03e67a83ae35d60f922d77809dbad65aa414 100644 --- a/modules/Vtiger/views/RelatedList.php +++ b/modules/Vtiger/views/RelatedList.php @@ -17,7 +17,7 @@ class Vtiger_RelatedList_View extends Vtiger_Index_View { } public function checkPermission(Vtiger_Request $request) { - parent::checkPermission($request); + return parent::checkPermission($request); } function process(Vtiger_Request $request) { @@ -28,7 +28,6 @@ class Vtiger_RelatedList_View extends Vtiger_Index_View { $relatedModuleModel = Vtiger_Module_Model::getInstance($relatedModuleName); $moduleFields = $relatedModuleModel->getFields(); - $searchParams = $request->get('search_params'); if(empty($searchParams)) {