Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#168 fix lost groups to choose while restore #169

Conversation

adamtppaw
Copy link
Contributor

No description provided.

@NicoAlexH NicoAlexH merged commit b533827 into ndunand:master Jan 30, 2023
@NicoAlexH
Copy link
Collaborator

Should fix #168 and #185, thanks @adamtppaw !

@cdipe
Copy link

cdipe commented Jan 31, 2023

@NicoAlexH I tested the new version now, and the link to the Group choice in copied course is now gone - but it looks like this: The two already added groups are still under "Available groups". (Nothing happens when selecting them and clicking Add)
group choice in restored course
groups in restored course

@cdipe
Copy link

cdipe commented Jan 31, 2023

@NicoAlexH @adamtppaw I tested this in my local MAMP Moodle 4.1.1 too, same problem - shows already added groups as selectable.

@NicoAlexH
Copy link
Collaborator

@cdipe I'm sorry, I can't seem to reproduce ; are you sure you unchecked the "groups and groupings" option when restoring the course ?

To clarify, this is what I did on my part :

1 - Create a course with students, groups and a choicegroup module
2 - Have the students select groups in the choicegroup module
3 - Backup the course without the students
4 - Restore to a new course (unchecking the "groups and groupings" option)

=> In the newly created course, there are no groups.

I also tried to restore the course, this time checking the option "groups and groupings" (4th step).

=> In the new course, I see the same groups that existed in the original course, and the "Add" button for adding participants works fine.

May I suggest you to open a new issue if you think this is abnormal, including the steps to reproduce the bug ? Thank you !

@cdipe
Copy link

cdipe commented Feb 2, 2023

Hi @NicoAlexH !
It works fine. We do the restore with the groups and groupings included (no users, though) and it still works fine, the right groups are selectable for the students in the restored course's Group Choice.

I think I just explained badly, and I may remember wrong how Group Choice used to work:

  • When adding groups in Group Choice administration, from Available Groups to Selected groups, they still remain in the Available groups menu.

  • The expected behaviour would be that when added to Selected groups, they disappear from Available groups. And, if removed from Selected groups, they should show under Available groups again.

The behaviour is the same if it's a copied course with Group Choice, or an new Group Choice in an new course.
I'll report this as a new issue, or, if it's always been this way, as an improvement suggestion.
Allt the best,
Peter

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants