Problems with group forums / topics

Bug #1752442 reported by Robert Lyon
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
High
Robert Lyon
16.10
Fix Released
High
Unassigned
17.04
Fix Released
High
Unassigned
17.10
Fix Released
High
Unassigned
18.04
Fix Released
High
Robert Lyon

Bug Description

In master:
- the edit/delete buttons for a forum display to normal members (even though they can't edit/delete a forum)
- the ability to subscribe / unsubscribe to a topic in a forum has gone away but the 'i' help still remains

These need to be fixed up

Robert Lyon (robertl-9)
Changed in mahara:
assignee: nobody → Robert Lyon (robertl-9)
Revision history for this message
Robert Lyon (robertl-9) wrote :

Actually the problem with the 'i' help icon is due to a form that displays when you are not subscribed to the forum allowing you to subscribe to only some of the topics

Revision history for this message
Steven (stevens-q) wrote :

Manually ran the following Behat test:

@javascript @core @core_view
Feature: Edit and delete Forum nbuttons are not displayed for normal users
    As a normal user and not the author
    I should not be able to see edit and delete buttons
    because I do not have permissions

Background:
Given the following "users" exist:
    | username | password | email | firstname | lastname | institution | authname | role |
    | UserA | Kupuhipa1 | <email address hidden> | Angela | User | mahara | internal | member |
    | UserB | Kupuhipa1 | <email address hidden> | Bob | User | mahara | internal | member |

And the following "groups" exist:
      | name | owner | description | grouptype | open | invitefriends | editroles | submittableto | allowarchives | members | staff |
      | GroupA | UserA | GroupA owned by UserA | standard | ON | ON | all | ON | ON | UserA, UserB | UserA |
      | GroupB | UserB | GroupB owned by UserB | standard | ON | ON | all | ON | ON | UserB, UserA | UserB |

Scenario: Normal members should not see edit/delete forum buttons
 Given I log in as "UserA" with password "Kupuhipa1"
 And I am on "/group/view.php?id=1"
 When I follow "Forums"
 # The following line does not run
 Then I should see "Forums | Group A"
 And I should not see "Edit \"General discussion\""
 And I should not see "Delete \"General discussion\""

 Scenario: Group owner should see edit/delete forum buttons
 Given I log in as "UserB" with password "Kupuhipa1"
 And I am on "/group/view.php?id=2"
 When I follow "Forums"
 # The following line does not run
 Then I should see "Forums | Group B"
  And I should see "Edit \"General discussion\""
 And I should see "Delete \"General discussion\""

Catalyst QA Approved ✔

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/8550
Committed: https://git.mahara.org/mahara/mahara/commit/af02288f85a039b75fe3ba3a3527b07189a2aa42
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit af02288f85a039b75fe3ba3a3527b07189a2aa42
Author: Robert Lyon <email address hidden>
Date: Thu Mar 1 11:20:44 2018 +1300

Bug 1752442: Showing edit/delete links to non group admins in forums

Also show a help icon to a form that is not shown

behatnotneeded

Change-Id: I603f42a757bd6b9617e59c1214c3e86e2930d995
Signed-off-by: Robert Lyon <email address hidden>

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "17.10_STABLE" branch: https://reviews.mahara.org/8562

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "17.04_STABLE" branch: https://reviews.mahara.org/8563

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "16.10_STABLE" branch: https://reviews.mahara.org/8564

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/8564
Committed: https://git.mahara.org/mahara/mahara/commit/5b0e6d53ceec2291b96663caec922262feae9a57
Submitter: Robert Lyon (<email address hidden>)
Branch: 16.10_STABLE

commit 5b0e6d53ceec2291b96663caec922262feae9a57
Author: Robert Lyon <email address hidden>
Date: Thu Mar 1 11:20:44 2018 +1300

Bug 1752442: Showing edit/delete links to non group admins in forums

Also show a help icon to a form that is not shown

behatnotneeded

Change-Id: I603f42a757bd6b9617e59c1214c3e86e2930d995
Signed-off-by: Robert Lyon <email address hidden>
(cherry picked from commit af02288f85a039b75fe3ba3a3527b07189a2aa42)

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Reviewed: https://reviews.mahara.org/8563
Committed: https://git.mahara.org/mahara/mahara/commit/ff109c21cdee312f8d667d38cbe090f3633cfa23
Submitter: Robert Lyon (<email address hidden>)
Branch: 17.04_STABLE

commit ff109c21cdee312f8d667d38cbe090f3633cfa23
Author: Robert Lyon <email address hidden>
Date: Thu Mar 1 11:20:44 2018 +1300

Bug 1752442: Showing edit/delete links to non group admins in forums

Also show a help icon to a form that is not shown

behatnotneeded

Change-Id: I603f42a757bd6b9617e59c1214c3e86e2930d995
Signed-off-by: Robert Lyon <email address hidden>
(cherry picked from commit af02288f85a039b75fe3ba3a3527b07189a2aa42)

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Reviewed: https://reviews.mahara.org/8562
Committed: https://git.mahara.org/mahara/mahara/commit/12d0a41121077d6efb8c303307f4678c881d73f0
Submitter: Robert Lyon (<email address hidden>)
Branch: 17.10_STABLE

commit 12d0a41121077d6efb8c303307f4678c881d73f0
Author: Robert Lyon <email address hidden>
Date: Thu Mar 1 11:20:44 2018 +1300

Bug 1752442: Showing edit/delete links to non group admins in forums

Also show a help icon to a form that is not shown

behatnotneeded

Change-Id: I603f42a757bd6b9617e59c1214c3e86e2930d995
Signed-off-by: Robert Lyon <email address hidden>
(cherry picked from commit af02288f85a039b75fe3ba3a3527b07189a2aa42)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.