Loading...

XML

Word

Printable

Affected Branches:
MOODLE_21_STABLE
Fixed Branches:
MOODLE_21_STABLE, MOODLE_22_STABLE
Pull from Repository:
git@github.com:marinaglancy/moodle.git
Pull Main Branch:
wip-~~MDL-30709~~-master
Pull Main Diff URL:
https://github.com/marinaglancy/moodle/compare/master...wip-MDL-30709-master
Testing Instructions:
Hide

Create an upload assignment (type = uploadsingle)

Login as student

Upload a file to the assignment

Go someplace else and attempt to choose that uploaded file in the file picker under recent files.

Should be able to successfully use the recent file.

Create an upload assignment (type = upload)

Login as student

Upload a file to the assignment

Go someplace else and attempt to choose that uploaded file in the file picker under recent files.

Should be able to successfully use the recent file.
Show
Create an upload assignment (type = uploadsingle) Login as student Upload a file to the assignment Go someplace else and attempt to choose that uploaded file in the file picker under recent files. Should be able to successfully use the recent file. Create an upload assignment (type = upload) Login as student Upload a file to the assignment Go someplace else and attempt to choose that uploaded file in the file picker under recent files. Should be able to successfully use the recent file.

This appears to be the exact same problem as ~~MDL-30214~~, but for Assignment. Most likely have to create an assignment_get_file_info() method.

Other modules should be tested to see if they must implement MODNAME_get_file_info() method as well.

Discovered while testing

MDL-31661 Deprecate use of <modname>_get_file_info() and convert

has been marked as being related by

MDL-32378 Wiki lacks mod_wiki_get_file_info(), necessary for Recent files

MDL-31738 Missing function mod_glossary_get_file_info() : needed Recent Files

Testing discovered

MDL-30214 File picker: Recent files option does not work

Participants:: Andrew Lyons, Eloy Lafuente (stronk7), Marina Glancy, Mark Nielsen, Michael de Raadt, Rajesh Taneja, Sam Hemelryk