|
Michael Blake made changes - 21/Aug/06 04:21 PM
> From (penny at catalyst.net.nz) Thursday, 10 March 2005, 04:18 AM:
> 1-Having a setting for groups could be a nightmare because modules running > under groups in the original course have to be modified to take care of this > setting to delete their group info (and group mode). It's possible to implement it > but perhaps it's too late for 1.5... Eloy wrote > I agree that it would be a nightmare, because of all the modules and how they work > with groups. Maybe for 1.6? I mean, I'd love it for 1.5 but it may be unrealistic. I would like the ability to turn off backup/restore groups. Especially in the case when the course contains no group settings. In other words initially the setting could be quite, violent. Tim Hi,
Just found this bug, and the option to include/exclude groups (and groupings?) on backup/restore is something we'd very much like at the OU. I agree we need to define what happens for activities in group mode. I'm not sure it can be done for 1.9, but I took the liberty of updating the "Affects versions" and Components information. Nick [OU Bug 3132 "groups are copied from previous course presentations on backup/restore" - Sophie]
Nick Freear made changes - 02/Oct/07 10:07 PM
Penny & Eloy,
I'm looking at a potential fix for this bug for HEAD/2.0, see the Moodle Wiki, http://docs.moodle.org/en/Development:Groupings_OU#Groups_optional_backup.2Frestore I'm investigating just giving the user an option on restore initially. This should be quicker to implement - I'm on a tight schedule to do this for the OU. Nick Finally a patch. It affects 9 files:
-backup/restore_form.html -lang/en_utf8/group.php -mod/chat/restorelib.php I would like to commit to HEAD in the next day or so, and possibly back port to 1.9 - definitely Skype beforehand! Comments welcome! Thanks
Nick Freear made changes - 06/Dec/07 01:58 AM
Nick Freear made changes - 06/Dec/07 02:22 AM
Looks correct. Cool!
Only 1thought (not critical) that have arrived while reviewing the code: Shouldn't activity group mode be "reseted" acordingly with the "reset" that code performs in discussions /entries.... ? not 100% sure, just one thought about what is more "logical". Ciao
Eloy Lafuente (stronk7) made changes - 06/Dec/07 03:06 AM
Version 8 core of the patch - 1 extra file + modifications:
In response to Eloy - it may be logical to reset group mode, but for us (OU) and hopefully others, it seems more useful to maintain activity group mode so that groups can be recreated. The new help file hopefully explains this, and the group restore options generally. Comments welcome! I'd like to commit this Monday 10th to HEAD, and maybe 1.9 - if I get the nod
Nick Freear made changes - 08/Dec/07 12:53 AM
Hi, I've committed revision 8 of the patch (affecting 10 files, see previous comments) to Moodle HEAD. Can I backport this to 1.9 branch now?
Thanks, Nick.
Nick Freear committed 10 files to 'Moodle CVS' - 10/Dec/07 07:26 PM
Mitsuhiro Yoshida committed 12 files to 'Lang CVS' - 11/Dec/07 05:09 AM
martignoni committed 2 files to 'Lang CVS' - 14/Dec/07 07:15 PM
Petr Skoda made changes - 09/Mar/08 09:42 PM
Petr Skoda made changes - 18/Mar/08 05:19 AM
taking over this one:
Petr Skoda committed 5 files to 'Moodle CVS' on branch 'MOODLE_19_STABLE' - 18/Mar/08 06:27 AM
Petr Skoda committed 2 files to 'Moodle CVS' - 18/Mar/08 06:33 AM
Petr Skoda committed 4 files to 'Moodle CVS' on branch 'MOODLE_19_STABLE' - 18/Mar/08 06:33 AM
Petr Skoda committed 2 files to 'Moodle CVS' on branch 'MOODLE_19_STABLE' - 18/Mar/08 06:54 AM
Petr Skoda committed 2 files to 'Moodle CVS' - 18/Mar/08 06:57 AM
Petr Skoda made changes - 18/Mar/08 07:01 AM
Petr Skoda committed 1 file to 'Moodle CVS' - 18/Mar/08 07:09 AM
Helen, could you please tweak the docs so that there is no PHP code in it, thanks!
Petr Skoda made changes - 21/Mar/08 04:58 AM
...and close this when done, thanks
wildgirl committed 1 file to 'Moodle CVS' on branch 'MOODLE_19_STABLE' - 21/Mar/08 05:48 AM
wildgirl committed 1 file to 'Moodle CVS' - 21/Mar/08 06:07 AM
wildgirl committed 1 file to 'Moodle CVS' on branch 'MOODLE_19_STABLE' - 21/Mar/08 06:07 AM
Petr Skoda committed 2 files to 'Moodle CVS' on branch 'MOODLE_19_STABLE' - 21/Mar/08 06:16 AM
Petr Skoda committed 2 files to 'Moodle CVS' - 21/Mar/08 06:17 AM
Thanks for everyone's work on this
PHP removed and help file reworded.
Helen Foster made changes - 21/Mar/08 06:18 AM
Mitsuhiro Yoshida committed 3 files to 'Lang CVS' - 21/Mar/08 10:05 AM
Anthony Borrow made changes - 23/Mar/08 11:15 AM
martignoni committed 1 file to 'Lang CVS' - 25/Mar/08 03:11 AM
martignoni committed 1 file to 'Lang CVS' - 25/Mar/08 03:13 AM
Closing as requested. Thanks!
Eloy Lafuente (stronk7) made changes - 16/Apr/08 08:09 AM
I see that his is only for 1.9 and above.
I still use 1.6 Does anyone have any suggestions on a hack for 1.6 to just ignore all backup and restore of groups? Timothy, Moodle 1.6 is not supported version any more (and we are going to stop supporting 1.7 on October 31). You should upgrade your server anyway as using 1.6 may be considered as a sort of hazard...
I don't think that I have that luxury. I would like to avoid the hazard.
The first of the two diffs looked like it was going to work since it is for 1.6.3 Tim 1.6.2+ - is that a joke? I hope it is not accessible from internet, is it?!
Ha ha!
It is accessible from the internet. The only thing that is accessible is the login page, to a IMAP server, and I trust my students not to hack, but maybe the loging is no longer secure. I wish that there were security patches available. I may upgrade to 1.6.3 but that is the limit of my TUI module, which cost me 2000 dollars, that I like. If I upgrade to 1.6.3 then then I can apply the patch above. I am not aware of other things that I want from moodle. Roles!? But I do need to fix this bug. There is nothing like "only login page is accessible" - you do not need any login in order to exploit some bugs. Upgrading to 1.6.3 does not solve anything, you must upgrade to latest MOODLE_16_STABLE cvs branch at least, but even that is not enough because 1.6.x is not maintained - no more bug&security fixes there
Well, as I have said in conversation before, I wish that would would tell us about the bug fixes rather than encourage us to upgrade.
I just looked at the TUI code available from the module database - it is very old code from 1.5.x times that is still relying on register globals, no new param cleaning, etc. - this module alone has probably many security issues itself.
I hacked TUI to work with 1.6. Register globals...when was that turned off?
Can people access TUI without login? Anyway, anyway, tell us the issues. I bet that there are very few. 5? 10? patchable. Please share the patches. I do not want to be on a windows-like bloatware upgrade escalator. Last time we had this conversation I looked through the security issues and it seemed to But I need this. Groups restoring even though you don't want them to, is a real pain. I am sorry, I am not going to review/fix dead code - I have to concentrate on 1.9.6 and 2.0 right now
It is kind of you to even consider it. Good luck with 2.0.
I will look into moving to 1.6 stable, and then applying the first of the two patches above. Tim |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Also, perhaps there should actually be a setting for groups names.
In looking at the code it seems like scales and events always get backed up/restored without asking for a preference as well.
From (penny at catalyst.net.nz) Friday, 4 March 2005, 12:28 PM:
Also, group images don't seem to be backed up/restored?
From Eloy Lafuente (stronk7 at moodle.org) Saturday, 5 March 2005, 08:54 AM:
Groups are always saved but group members only if you select to backup users.
Only used scales (course_scale_used() function) are included automatically, and course events too. Here it's an ordered list of my opinions:
1-Having a setting for groups could be a nightmare because modules running under groups in the original course have to be modified to take care of this setting to delete their group info (and group mode). It's possible to implement it but perhaps it's too late for 1.5...
2-About groups too, yes, images should be included in backup and restore! I've added it to my todo list (backup/CHANGES_14_15.txt). My fault!
3-Having a setting for scales it's really a wrong idea and I think that the current approach (export used scales) is the correct one. We can, anyway, improve it a bit to detect only scales used by modules being exported (currently all scales in any module in the course are being included). This involves creating a new function in backup/lib.php called backup_scale_used() really similar to the course based but skipping not selected modules. For 1.5?
4.-And finally, events. A setting for this could be really nice and easy to implement. As such info in not related to the rest of the backup (like groups or scales) having a check to decide it is really a good idea. For 1.5?
Ciao
From (penny at catalyst.net.nz) Monday, 7 March 2005, 04:29 AM:
Sorry, my bad, I just did cvs annotate on restorelib.php and saw that the check for backup->users for group members was added after our last merge. Should have checked that before posting the bug report.
From Eloy Lafuente (stronk7 at moodle.org) Tuesday, 8 March 2005, 01:38 AM:
NP!
Anyway, we have to decide about points 3 and 4 (Martin D added to the discussion).
Are my theories correct? Must I include them in my CHANGES_14_15 list?
Ciao
From (penny at catalyst.net.nz) Thursday, 10 March 2005, 04:18 AM:
> 1-Having a setting for groups could be a nightmare because modules running
> under groups in the original course have to be modified to take care of this
> setting to delete their group info (and group mode). It's possible to implement it
> but perhaps it's too late for 1.5...
I agree that it would be a nightmare, because of all the modules and how they work with groups. Maybe for 1.6? I mean, I'd love it for 1.5 but it may be unrealistic.
> 3-Having a setting for scales it's really a wrong idea and I think that the current
> approach (export used scales) is the correct one. We can, anyway, improve it a bit
> to detect only scales used by modules being exported (currently all scales in any
> module in the course are being included). This involves creating a new function in
> backup/lib.php called backup_scale_used() really similar to the course based but
> skipping not selected modules. For 1.5?
+1, but with a setting for it, like:
there are scales, but they're not used anywhere. Do you still want to export them?
> 4.-And finally, events. A setting for this could be really nice and easy to
> implement. As such info in not related to the rest of the backup (like groups or
> scales) having a check to decide it is really a good idea. For 1.5?
Yup! Sounds good. +1
From (penny at catalyst.net.nz) Wednesday, 12 October 2005, 10:44 AM:
bump! anything happened on this one?
From Eloy Lafuente (stronk7 at moodle.org) Friday, 14 October 2005, 03:52 AM:
Hey,
this was lost in the abyss of my bugs...
Should I try to do it or, perhaps, you have intentions to do so?
About 3, perhaps the setting should be a popup between:
About 4, a simple Y/N will be enough, I think.
From Eloy Lafuente (stronk7 at moodle.org) Wednesday, 3 May 2006, 03:48 AM:
Hehe,
what if....we postpone this some more months? :-P
...orr perhaps somebody else could try it after 1.6 branch... I estimate that my time availability next 5-6 moths is going to be really short!
But moodling, of course! B-)