|
This is still a bug in 1.9, at least as of build 20080307.
I haven't tried it with the course creator role in Moodle 1.8.x; we don't make use of that particular one. I'll try and spend some time debugging this later this week or early next. I haven't tried it in Moodle 1.9 yet.
Here is a diff file of the changes that worked for me, at least so far, in 1.9.
Following the diff file for 1.8.3, I edited /backup/restore_form.html by moving several lines out of the if statement. The date line shows up for teachers and seems to work correctly so long as the specified date is greater than or equal to the original course start date.
matt greenwolfe made changes - 01/May/08 09:49 AM
I can confirm the problem still exists for teachers in Moodle 1.9.2. Matt's patch failed for me so I've generated a new one based on Moodle 1.9.2 (Build: 20080711).
I can not recreate the problem with the Course Creator role; when logged in as a course creator I was able to see the start data form (which is the way the original code was supposed to work, as that functionality was reserved for course creators, even though non-course creating teachers needed it as well).
Kenneth Newquist made changes - 17/Jul/08 10:58 PM
I think the patch is broken ". Text File 192_restore_form_patch.txt (2 kb)"
We tried the ". Text File 192_restore_form_patch.txt (2 kb)" from a 1.9.2 intallation, the patch while it did take into account the current date, it did not honor the settings that we set for course start date, if we set the start-date to May 1st, the course startdate after the restore is still July 20th. I'll re-check the patch later today.
EXCUSE THE CAPS, BUT I HOPE SOMEONE READS THIS THAT CAN PUT THE PATCH INTO THE DISTRIBUTION WE HAVE NOW A PROBLEM 171 ASSIGNMENTS IN OUR MOODLE INSTALL HAVE SILLY DUE DATES!!!
The patch has been verified and it works in 1.9.2, We had patched our 1.8.3 distribution but when we did the security upgrade to 1.8.6 we must have lost the patch, it wasnt included in the new distribution so if someone who has access to the main code repository might put this patch in (sorry I dont know how) then this might be avoided. We have successfully used this patch in 1.8.3 and in 1.9.2 I really dont want to get bit by this again! - Thanks. So, we checked, this still hasnt made it into Moodle 1.9.3, I'm glad I did my due diligence with this one.
Martin, what can I do to resolve this? Please advise. Eloy can you look at this please?
Martin Dougiamas made changes - 30/Dec/08 08:50 AM
Eloy Lafuente (stronk7) committed 2 files to 'Moodle CVS' - 03/Jan/09 02:59 AM
Eloy Lafuente (stronk7) committed 2 files to 'Moodle CVS' on branch 'MOODLE_18_STABLE' - 03/Jan/09 03:00 AM
Instead of using the patches above... I've continued with current approach, i.e. non-course-creators cannot roll dates at all.
I've committed fix for 18_STABLE, 19_STABLE and HEAD. Ciao
Eloy Lafuente (stronk7) made changes - 03/Jan/09 03:03 AM
Eloy Lafuente (stronk7) made changes - 03/Jan/09 03:03 AM
Tim Hunt made changes - 06/Jan/09 11:22 AM
Tim Hunt made changes - 06/Jan/09 11:23 AM
Hi Eloy,
Thank you for this fix. However, I have to disagree with the current practice of making this so only "course creators" are allowed to roll the dates. If you follow MDL-9367 (and other related trackers), you'll see that there is an overwhelming need by the community for faculty to be allowed to roll the dates in a course. If it is assumed only the admin is doing this, then an institution has to fund a whole team of folks with admin privileges (or "course creator") to do nothing but course restores from semester to semester. If instead, faculty (teacher role) are allowed to do restore and import with setting the course start date, then the burden is off the team and in the hands of the ones for which this function most effects: the instructors. Otherwise, faculty have to manually edit the dates on every activity every semester, thus defeating the purpose of this "roll forward" feature. Logically, as a systems administrator, I would WANT my faculty to be able to set their own course start date (especially as courses start on differing dates throughout the semester), but I wouldn't want my faculty to be assigned to the course creator role in order to make this happen - because I wouldn't want them to have the ability to create whatever courses they wanted on the fly. We only host courses that have gone through a process (such as curriculum committee, dept approval, etc.). If this fix could also be applied to at minimum the teacher role, this would greatly assist many institutions in the management of course rollover from semester to semester (term to term). Thank you for listening, Mary I do agree with Mary,
If we were to give all the instructors Course-creator privs this would definatly be problematic. and well as Martin L put it "unsustainable" We tend to have faculty..... yeah. In short I do agree with Mary, I also do know of many other institutions that cannot afford the resources hand-holding such as ours. Kind Regards, I agree with Mary and Rahim, if we are to make course restores a valuable self-service function for our instructors, the teacher role needs to be able to restore courses without getting wonky dates. Thank you for your help and work on this. I second Barron, Mary, and Rahim. I'm not about to give my faculty course-creator roles; that could be disasterous.
Eloy Lafuente (stronk7) made changes - 16/Jan/09 04:44 AM
Eloy Lafuente (stronk7) made changes - 16/Jan/09 04:44 AM
Eloy Lafuente (stronk7) made changes - 16/Jan/09 04:45 AM
So, one capability like:
'moodle/restore:rolldates' enabled by default to admins and course creators (to have backward compatibility) but able to be assigned to other roles... sounds like a good solution to everybody here? Ciao and that capability should be able to work at course level or only at category level? I ask this because sometimes, on restore we don't know the target course until later in the process (for example, when creating a new course) or when restoring to existing, deleting first, the "teacher" haven't still that capability allowed.
So, perhaps it should be a "category-based" capability? and not a course one?
Eloy Lafuente (stronk7) committed 3 files to 'Moodle CVS' on branch 'MOODLE_19_STABLE' - 06/Oct/09 09:13 AM
Eloy Lafuente (stronk7) committed 3 files to 'Moodle CVS' - 06/Oct/09 09:14 AM
Eloy Lafuente (stronk7) committed 3 files to 'Moodle CVS' on branch 'MOODLE_19_STABLE' - 06/Oct/09 09:22 AM
Eloy Lafuente (stronk7) committed 3 files to 'Moodle CVS' - 06/Oct/09 09:27 AM
Eloy Lafuente (stronk7) committed 1 file to 'Moodle CVS' on branch 'MOODLE_19_STABLE' - 06/Oct/09 09:44 AM
Eloy Lafuente (stronk7) committed 1 file to 'Moodle CVS' - 06/Oct/09 09:58 AM
Done. New capability 'moodle/restore:rolldates' is available and allowed to admins and course creators by default (old behaviour). Can be allowed to other roles (at any level) in order to permit them to roll dates on restore.
Ciao Please, test if and report any problem here ASAP, as this new capability will be available in tomorrow's weekly build and in next stable release (Moodle 1.9.6). TIA and re-ciao
martignoni committed 1 file to 'Lang CVS' - 23/Oct/09 05:35 AM
martignoni committed 1 file to 'Lang CVS' - 23/Oct/09 05:38 AM
martignoni committed 1 file to 'Lang CVS' - 23/Oct/09 05:40 AM
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Bug
Closed
Major
Course restores for non-admin users appear ~7 years early
 |
|
I tried a restore as an admin and I change the start date of the course during restore and I get wrong forum post dates (they are set in the future).
I test it on a 1.8.3 moodle with a 8.2.4 postgres installed in a SUSE Linux Enterprise Server 10 SP1.