|
This is the error report with activated debug mode in 1.9:
Notice: Undefined index: assignment in /mydomain.de/moodle/backup/restorelib.php on line 5901 Notice: Undefined index: chat in /mydomain.de/moodle/backup/restorelib.php on line 5901 Notice: Undefined index: choice in /mydomain.de/moodle/backup/restorelib.php on line 5901 Notice: Undefined index: data in /mydomain.de/moodle/backup/restorelib.php on line 5901 Notice: Undefined index: glossary in /mydomain.de/moodle/backup/restorelib.php on line 5901 Notice: Undefined index: journal in /mydomain.de/moodle/backup/restorelib.php on line 5901 Notice: Undefined index: lesson in /mydomain.de/moodle/backup/restorelib.php on line 5901 Notice: Undefined index: quiz in /mydomain.de/moodle/backup/restorelib.php on line 5901 Notice: Undefined index: resource in /mydomain.de/moodle/backup/restorelib.php on line 5901 Notice: Undefined index: scorm in /mydomain.de/moodle/backup/restorelib.php on line 5901 Notice: Undefined index: survey in /mydomain.de/moodle/backup/restorelib.php on line 5901 Notice: Undefined index: workshop in /mydomain.de/moodle/backup/restorelib.php on line 5901
Notice: Undefined property: deleting in /mydomain.de/moodle/backup/restorelib.php on line 5963 Notice: Undefined property: deleting in /mydomain.de/moodle/backup/restorelib.php on line 5971
Notice: Undefined property: metacourse in /mydomain.de/moodle/backup/restorelib.php on line 6121 Notice: Undefined index: quiz in /mydomain.de/moodle/backup/restorelib.php on line 6143 Notice: Undefined property: user_files in /mydomain.de/moodle/backup/restorelib.php on line 6163
Notice: Undefined property: messages in /mydomain.de/moodle/backup/restorelib.php on line 6217
Notice: Undefined property: course_startdateoffset in /mydomain.de/moodle/mod/forum/restorelib.php on line 48 Notice: Undefined property: course_startdateoffset in /mydomain.de/moodle/mod/forum/restorelib.php on line 48 Notice: Undefined property: course_startdateoffset in /mydomain.de/moodle/mod/wiki/restorelib.php on line 38
Notice: Undefined property: logs in /mydomain.de/moodle/backup/restorelib.php on line 6386
Uhm... since the beginning the behaviour for section on restore has been:
1) If we are restoring to a NEW course (not existing yet), then restore the section information (summary...). 2) If we are restoring to an EXISTING course: So, it should work ok if you haven't the destination course created (situation 1). But, if you are using the import feature, instead of the restore one, I guess that then, the course exists, so situation 2 will be applied. I think we followed such approach to avoid overwriting information when restoring to EXISTING courses (2). Although, perhaps, it could be a bit more clever and, when restoring/importing to existing courses we could restore the summary information if it's EMPTY at destination. Just typing as I think....hope this helps...ciao We have gone through this with Matt Clarkson (CC'd) and it works correctly for us. As Eloy points out, terhaps it is related to restoring to an empty course vs letting the restore process create the new course.
As Eloy explained the behaviour seems to be correct, but the expected behaviour of an import process is an other one.
Let me explain the situations we often found. The process informations doesn't inform the teacher that some elements are not imported. The initial labels in the sections exists in each course as an empty area. My ideas for further developments: This issue still affects 1.9beta, as of August 24/07.
Please keep this ticket to the IMPORT function only - not restore.
This issue should be fixed - importing a course should import everything - including each module's summary. Something like this would allow the section summary to be updated when not set. Though its sub-optimal perf wise and also needs more sanity checking
 Hi Dan, sorry, but I'm not a developer. As far as I understand the code, it checks if there is a standard label entry.If there is no label it adds a content. I didn't see what happens if there is content.
Can you add a function in case of a section exists that a new label is created. Having the same problem in 1.8.4
Priority shouldn't be minor!!! Well, just trying to re-organize a bit the thoughts about this report:
1) Trivial but very important. We are talking about section summaries. Not labels nor anything else. 2) This is current behaviour: a) If we are restoring to a NEW course (not existing yet), section summaries ARE restored. 3) Given the previous behaviour, can all we agree what (a,b,c,d) to change, and how (always, based on setting, when empty...) ? Ciao I don't know if this would be the best solution, but what about (for c and d) you append the old summaries to the new ones?
Teacher mostly think that ignoring section summaries in case c) and d) are bugs. It will be more transparent if they are asked to add, replace or to ignore the section summaries from the imported course content while restoring/importing.
Idea from a non developer: I really don't know if it is possible.. Just tested the patch under Moodle 1.9.2 (Build: 20080711) and it solved the "import summaries" problem nicely. I see it's scheduled to be rolled into Moodle 1.9.3; is there any other tweaking/testing that's needed to get it there?
This patch also changes these two scenarios so what
c) If we are restoring to an EXISTING course, KEEPING previous data, section summaries ARE NOT restored. After the patch, restoring a course will bring in the topic summaries if those summaries are blank in both of these scenarios. This is the behavior we would expect and (in our case) want. I'd like to add something further which I just added in another similar ticket:
other options that are needed in import function: BLOCKS - it would be great to have these imported as well, or at least an option to if desired This would truly make the import function self-sustaining for faculty and cut down on the amount of work it takes to prepare courses from term to term for those like us that have new courses created by the SIS each term. I have noticed this issue on Moodle 1.9.2 (Build: 20080716)
When I import a course to an existing (empty) course (created based on a template using a database enrolment system) the summaries are not imported. I too would just like to advocate for including this into 1.9.4. The patch continues to apply against 1.9.3+ (Build: 20081224), albeit with quite a bit of fuzz. I believe this solves the problem of the original bug adequately (about the only small thing the patch could do additionally would be provide logging that stated that a summary was skipped and what was skipped).
I agree that Clark Shah-Nelson's comment that there could be significant feature enhancement of this, but I think that really belongs in a different bug. Again, thanks for this patch, it's working very well in our production system and greatly enhances the functionality of the "Import" button. We have also implemented this patch and it works very well, thanks for providing it. I think it would be greatly enhanced with a checkbox displayed on the import screen so that users could choose which summaries they want to import (to any empty section summary or (if possible) add to any non-empty summary)
Teresa Hello - just wondering about the fix version: it was set to 1.9.3, then changed to 1.9.4, then removed altogether. So does this just remain a patch or will it be added to a core version?
I really want to get this into our production server hosted on Moodlerooms. thanks! clark Resolving this as fixed. Patch applied to 18_STABLE, 19_STABLE and HEAD (will be available in next weekly).
Current behaviour is: a) If we are restoring to a NEW course (not existing yet), section summaries ARE restored. So section summaries are ALWAYS restored UNLESS there are some content in them. No new setting nor more options. Just new behaviour. Credit goes to Dan Poltawski for the patch (I forgot to credit you in the commit, Dan, sorry). Ciao |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Sub-task
Resolved
Minor
 |
|
Priority shouldn't be minor!!!