-
Notifications
You must be signed in to change notification settings - Fork 35
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
All threads from different dialogues in different courses visible in every Dialogue #91
Comments
Even stranger: When giving the right mod/dialogue:viewany to the roles, this makes the user see only the coversations in that dialogue, not from other Dialogues. But the intended function is also added, to see any conversation in that Dialogue. |
I haven't been able to reproduce this - please let me know if you work out the conditions that allow me to reproduce this with a fresh install of dialogue. thanks! |
I installed 3.9.7 locally om my mac and cannot reproduce. We'll investigate our live sites further. Dialogue was working fine in 3.8.6, that we upgraded from. Any ideas where to start investigating? |
Hi! We had a moodle partner helping us with the problem, and they made this report and code changes. |
Great! - any chance that Moodle partner could submit a Pull request with the change? - not ideal supplying patches in pdf files... |
I also note that the pdf suggests that this plugin hasn't received any updates since 2017, which is obviously incorrect and they didn't even look at the github codebase... |
@danmarsden I'll ask the Moodle partner if they will submit a pull request, I don't know how to do it properly. |
HI @cdipe - I'm closing this as I cannot reproduce this with the latest code - if you can find a way for me to reproduce this with the latest version including the steps required to do this please let me know and I'll take another look. |
Same problem on moodle 3.11 with the latest mod_dialogue at this time (3.11.1) Go to course A Code change required to fix this: Add below line 93: Disclaimer: This change was not extensively tested yet, but it seems to fix the problem for our 400+ teachers. |
Hi @on8pz thanks for the info - feel free to submit a PR with the change and I can take a look. |
Hi Dan - I have installed the 4.0 version of the plugin on my 3.11 Moodle and the same problem is occurring, so doesn't appear to have fixed it, which is a shame. I am not planning to upgrade the Moodle to 4.x until next summer. Is it worth me trying to edit the code as per the suggestions by on8pz? |
hmmm - that doesn't sound right... the Moodle 4.0 version of this plugin has a "requires" version set to Moodle 4.0 and higher, so it shouldn't let you install it on your Moodle 3.11 site at all. I'll see if I can find some time to backport the patch to the 3.11 branch sometime today. |
Hi, we just upgraded to moodle 3.9.7 and using dialogue 3.4 2017111300, and suddenly the users can see all their dialogue threads from any dialogue, in any dialogue, in any course. They start a dialogue in one course and can see it in another dialogue in another course.
This doesn't happen to moodle administrators, only teacher roles and students.
It's a big problem, since we didn't discover this until after the upgrade of the production environment.
https://moodle.org/mod/forum/discuss.php?d=423447
The text was updated successfully, but these errors were encountered: