Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Blocker
-
Resolution: Fixed
-
Affects Version/s: 2.3.2
-
Fix Version/s: 2.3.3
-
Component/s: Assignment
-
Testing Instructions:
-
Affected Branches:MOODLE_23_STABLE
-
Fixed Branches:MOODLE_23_STABLE
-
Pull from Repository:
-
Pull Master Branch:
-
Rank:44090
Description
If you take a assignment and import (or restore it) in into a new course, students cannot submit to it. After more research, mdl_assign_plugin_config is not populated, meaning no submission/feedback plugins are enabled, and will reset to defaults once the assign is edited and saved.
To reproduce:
You need two courses (A and B), B needs to have at least one student.
- In course A, create an assignment with default settings.
- In course B, select import. Find course A and import only the create assignment.
- Become one of the students in course B and enter the assignment. You will see "This assignment does not require you to submit anything online"
- Become an admin or teach in B again.
- Edit the imported assignment, make no changes, but go down to the bottom and click Save.
- Become one of the students in course B and enter the assignment. You will be able to submit.
Note that the settings are not preserved at all, all of the lost settings are reset to their defaults, regardless of what was set.
Activity
I discovered that the settings are actually completely lost during the process. Because of this I have raised the priority of this to a blocker and updated the description.
Adding more - after dissecting a backup file, it appears that this data is not included in the backups at all.
Turns out the problem is that the assign_plugin_config structure was only included in backups if include user info was set. I've moved that particular segment out of the if statement to always be included along with the basic assign structure.
Damyon - could you please have a look at this.
Thanks for the patch Eric, taking a look now
This looks like the correct fix to me and the patch is fine.
The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.
TIA and ciao 
Integrated to 2.3 & master, thanks guys.
It passes, I've been able to reproduce the problem reverting the patch and check that the patch solves the problem
Tested in master
Congratulations, you've done it!
Thanks, this change is now in the latest weekly release!
Join the crowds of people tomorrow from 8am and download this Moodle release from your local apple store!
We have about 400 assignments with this problem. We are currently running Moodle 2.4.1+ (Build: 20130214).
Is there a way we can restore this in one go? Perhaps some database/sql statement?
Thanks for reporting that.