Skip to content

Commit d5081cc

Browse files
committed
Issues-217: check member group permissions
1 parent 41101d2 commit d5081cc

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

config/vanilla/bootstrap.before.php

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -362,13 +362,15 @@ function watchButton($categoryID) {
362362
/**
363363
* Check group permission for the current user
364364
* @param $groupID
365-
* @param null $permission null - any permission for a group
365+
* @param null $category
366+
* @param null $permissionCategoryID
367+
* @param null $permission null - any permission for a group
366368
* @param bool $fullMatch
367369
* @return bool return true if user has a permission
368370
*/
369-
function checkGroupPermission($groupID,$permission = null, $fullMatch = true) {
371+
function checkGroupPermission($groupID, $category = null , $permissionCategoryID = null , $permission = null, $fullMatch = true) {
370372
$groupModel = new GroupModel();
371-
return $groupModel->checkPermission(Gdn::session()->UserID,$groupID, $permission, $fullMatch);
373+
return $groupModel->checkPermission(Gdn::session()->UserID,$groupID, $category,$permissionCategoryID , $permission, $fullMatch);
372374
}
373375
}
374376

vanilla/applications/vanilla/models/class.categorymodel.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -905,7 +905,7 @@ public static function checkPermission($category, $permission, $fullMatch = true
905905
}
906906

907907
if($groupID && $groupID > 0) {
908-
$result = checkGroupPermission($groupID);
908+
$result = checkGroupPermission($groupID, $category, $permissionCategoryID, $permission, $fullMatch);
909909
} else {
910910
$result = Gdn::session()->checkPermission($permission, $fullMatch, 'Category', $permissionCategoryID)
911911
|| Gdn::session()->checkPermission($permission, $fullMatch, 'Category', $categoryID);

0 commit comments

Comments
 (0)