[MDL-41166] assign -> the export to portfolio button should not be shown if the user has not the right to export - Moodle Tracker

XML

Word

Printable

Testing Instructions:
Hide

Enable portfolios in advanced settings

Enable the file download portfolio plugin

Create an assignment

As student submit a response to the assignment

As student, verify you can see the portfolio buttons to export your assignment files

Log out of student

As admin, edit the student role to remove the mod/assign:exportownsubmission capability

Log back in as student

VERIFY: you can no longer see the export buttons
Show
Enable portfolios in advanced settings Enable the file download portfolio plugin Create an assignment As student submit a response to the assignment As student, verify you can see the portfolio buttons to export your assignment files Log out of student As admin, edit the student role to remove the mod/assign:exportownsubmission capability Log back in as student VERIFY: you can no longer see the export buttons
Affected Branches:
MOODLE_25_STABLE, MOODLE_27_STABLE, MOODLE_28_STABLE, MOODLE_29_STABLE
Fixed Branches:
MOODLE_27_STABLE, MOODLE_28_STABLE
Pull from Repository:
git://github.com/danpoltawski/moodle.git
Pull Master Branch:
~~MDL-41166~~-master
Pull Master Diff URL:
https://github.com/danpoltawski/moodle/compare/eb1dc9f...MDL-41166-master

Enable and configure Mahara for the Moodleinstallation
students don't have the right to export their assignments to Mahara
-> mod/assign:exportownsubmission is not set
-> moodle/portfolio:export is not set
add an online assignment with all the default configuration

when a student adds a submission there is the button 'export to portfolio'

add to line 2152 in assign/locallib.php

if (has_capability('mod/assign:exportownsubmission', $this->context)) {

Testing discovered

MDL-48727 Assignment shouldn't offer to export someone elses files to portfolio

Participants:: CiBoT, Dan Poltawski, Eloy Lafuente (stronk7), Jetha Chan, Katja Krüger, Simey Lameze

Component watchers:: Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias