There are several regressions, unfinished code, roles related problems and also some old bugs in groups implementation.
I am working on these problems now...
Description
There are several regressions, unfinished code, roles related problems and also some old bugs in groups implementation.
I am working on these problems now...
Thanks Petr. I've patched a production site which makes extensive use of separated groups in chat and databases. Currently all seems working corerctly, we are monitoring the resultes in the next hours.
Andrea Bicciolo added a comment - 15/May/07 03:43 PM Thanks Petr. I've patched a production site which makes extensive use of separated groups in chat and databases. Currently all seems working corerctly, we are monitoring the resultes in the next hours.
Ray Lawrence added a comment - 15/May/07 03:54 PM Hi,
Just updated from anon CVS (UK). Resources still have groups settings (MDL-9365).
How long will it take for me to be able to check the 181_2.patch via anon CVS?
Petr, groups are all looking really good, apart from those few small interface things we talked about in Skype (moving menu to left again, and hiding groupings fully)
Martin Dougiamas added a comment - 15/May/07 05:30 PM Petr, groups are all looking really good, apart from those few small interface things we talked about in Skype (moving menu to left again, and hiding groupings fully)
Check it in!
Thanks for the excellent catch! :-D
Petr Skoda committed 19 files to 'Moodle CVS' on branch 'MOODLE_18_STABLE' - 16/May/07 02:23 AM
Petr/MartinD - I may be helpful to update http://docs.moodle.org/en/group/index#Groups_in_Moodle_1.8_onwards and http://docs.moodle.org/en/Groups_FAQ#Groupings so that it is consistent. From what I can tell the plan to implement groupings in 1.8 has been aborted (post release). This does not seem to be a trivial change. If folks were using 1.8 and had groupings defined, have steps been taken to create the enablegroupings record in mdl_config? I did not see any set_config statements so I am assuming not. I am concerned for those who use CVS to update their sites, all of a sudden groupings will no longer be working and I think some type of warning/documentation should exist to clarify what is happening. Thanks for your work and for keeping the Moodle code and documentation top notch. Peace - Anthony
Anthony Borrow added a comment - 23/May/07 05:40 AM Petr/MartinD - I may be helpful to update http://docs.moodle.org/en/group/index#Groups_in_Moodle_1.8_onwards and http://docs.moodle.org/en/Groups_FAQ#Groupings so that it is consistent. From what I can tell the plan to implement groupings in 1.8 has been aborted (post release). This does not seem to be a trivial change. If folks were using 1.8 and had groupings defined, have steps been taken to create the enablegroupings record in mdl_config? I did not see any set_config statements so I am assuming not. I am concerned for those who use CVS to update their sites, all of a sudden groupings will no longer be working and I think some type of warning/documentation should exist to clarify what is happening. Thanks for your work and for keeping the Moodle code and documentation top notch. Peace - Anthony
Petr/MartinD - I have updated the above referenced Docs - please review them to make sure that they are accurate and helpful to other users. Peace - Anthony
Anthony Borrow added a comment - 23/May/07 05:49 AM Petr/MartinD - I have updated the above referenced Docs - please review them to make sure that they are accurate and helpful to other users. Peace - Anthony
Is there a way for me to replace the groups tables in my 1.8 installation and isolate the group upgrade? I'm trying to troubleshoot why there are so many fewer records in mdl_groups_members than there are in mdl_groups_members_temp. Being able to isolate the group upgrade would greatly help in testing things. Any pointers would be appreciated.
Anthony Borrow added a comment - 23/May/07 06:01 AM Is there a way for me to replace the groups tables in my 1.8 installation and isolate the group upgrade? I'm trying to troubleshoot why there are so many fewer records in mdl_groups_members than there are in mdl_groups_members_temp. Being able to isolate the group upgrade would greatly help in testing things. Any pointers would be appreciated.
I am going to review the docs now.
The group upgrade is defined in one file only, you can have a look into group/db/upgrade.php function groups_transfer_db()
looking at the upgrade code it looks to be badly inefficient - it should have been done with SQL instead of loops with php functions
The problem could be that the script died during upgrade or if one group fails for some reason, the rest is skipped.
Petr Skoda added a comment - 23/May/07 05:18 PM I am going to review the docs now.
The group upgrade is defined in one file only, you can have a look into group/db/upgrade.php function groups_transfer_db()
looking at the upgrade code it looks to be badly inefficient - it should have been done with SQL instead of loops with php functions
The problem could be that the script died during upgrade or if one group fails for some reason, the rest is skipped.
All content on this web site is made available under the GNU General Public License, unless otherwise stated.