Skip to content

Commit 8804834

Browse files
committed
Issues-490: Fixed permissions
1 parent e9f0ab0 commit 8804834

File tree

1 file changed

+3
-24
lines changed

1 file changed

+3
-24
lines changed

models/class.groupmodel.php

Lines changed: 3 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -1541,11 +1541,6 @@ public function canAddNewAnnouncement($group) {
15411541
*/
15421542
public function canEditDiscussion($discussion) {
15431543
$canEditDiscussion = DiscussionModel::canEdit($discussion) ;
1544-
$groupID= $this->findGroupIDFromDiscussion($discussion);
1545-
if($groupID && Gdn::session()->checkPermission(GroupsPlugin::GROUPS_MODERATION_MANAGE_PERMISSION)) {
1546-
return true;
1547-
}
1548-
15491544
return $canEditDiscussion;
15501545
}
15511546

@@ -1559,6 +1554,7 @@ public function canDismissDiscussion($discussion) {
15591554
&& !$discussion->Dismissed
15601555
&& Gdn::session()->isValid();
15611556

1557+
/*
15621558
$groupID = $this->findGroupIDFromDiscussion($discussion);
15631559
if($groupID) {
15641560
$group = $this->getByGroupID($groupID);
@@ -1567,8 +1563,9 @@ public function canDismissDiscussion($discussion) {
15671563
Gdn::session()->checkPermission(GroupsPlugin::GROUPS_MODERATION_MANAGE_PERMISSION)) {
15681564
return true;
15691565
}
1570-
}
1566+
} */
15711567
return $canDismissDiscussion;
1568+
15721569
}
15731570

15741571
/**
@@ -1638,16 +1635,6 @@ public function canMoveDiscussion($discussion) {
16381635
Gdn::session()->checkPermission('Garden.Moderation.Manage')) {
16391636
return true;
16401637
}
1641-
1642-
$groupID = $this->findGroupIDFromDiscussion($discussion);
1643-
if($groupID) {
1644-
$group = $this->getByGroupID($groupID);
1645-
$groupRole = self::getGroupRoleFor(Gdn::session()->UserID, $groupID);
1646-
if ($groupRole === GroupModel::ROLE_LEADER || Gdn::session()->UserID === $group->OwnerID
1647-
|| Gdn::session()->checkPermission(GroupsPlugin::GROUPS_MODERATION_MANAGE_PERMISSION)) {
1648-
return true;
1649-
}
1650-
}
16511638
return false;
16521639
}
16531640

@@ -1662,14 +1649,6 @@ public function canRefetchDiscussion($discussion) {
16621649
*/
16631650
public function canDeleteDiscussion($discussion) {
16641651
$canDeleteDiscussion = CategoryModel::checkPermission($discussion->CategoryID, 'Vanilla.Discussions.Delete');
1665-
/*
1666-
$groupID = $this->findGroupIDFromDiscussion($discussion);
1667-
if($groupID) {
1668-
$group = $this->getByGroupID($groupID);
1669-
if (Gdn::session()->UserID == $group->OwnerID) {
1670-
return true;
1671-
}
1672-
}*/
16731652
return $canDeleteDiscussion ;
16741653
}
16751654

0 commit comments

Comments
 (0)