Moodle

Mnet: Remote courses don't show in myMoodle list

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 2.0
  • Fix Version/s: 2.0
  • Component/s: My Moodle
  • Labels:
    None
  • Environment:
    All
  • Affected Branches:
    MOODLE_20_STABLE
  • Fixed Branches:
    MOODLE_20_STABLE

Description

Moodle Networking - local students enrolled in courses on remote Moodle see a list of "Remote Courses" in their local "My Courses" block. There is no equivalent listing in their myMoodle listing.

All our users are directed to their myMoodle page upon login. The inconsistency between the myMoodle list and the My Courses block will be confusing. Screenshot attached.

Temporary work around - have made the My Courses block a sticky block for myMoodle.

  1. MDL-13379_M19.diff
    09/Nov/10 11:05 PM
    2 kB
    Hubert Chathi
  2. MDL-13379_M19.diff
    09/Nov/10 10:23 AM
    2 kB
    Hubert Chathi
  3. MDL-13379_M2.diff
    16/Nov/10 1:12 AM
    4 kB
    Hubert Chathi
  4. MDL-13379_M2.diff
    09/Nov/10 11:05 PM
    3 kB
    Hubert Chathi
  5. MDL-13379_M2.diff
    09/Nov/10 10:23 AM
    3 kB
    Hubert Chathi
  1. overview of my courses_no_mnet_courses.jpeg
    92 kB
    08/Feb/08 1:53 PM

Activity

Hide
Penny Leach added a comment -

I don't seem to be able to assign bugs to Hubert (who is looking at redoing my moodle in 2.0) but I've added him as a watcher so he sees this.

Show
Penny Leach added a comment - I don't seem to be able to assign bugs to Hubert (who is looking at redoing my moodle in 2.0) but I've added him as a watcher so he sees this.
Hide
Hubert Chathi added a comment -

Yeah, I guess I'm not a developer, so you can't assign issues to me. (I also can't edit issues myself...). Maybe the best way, for now, to make sure that this issue doesn't get lost is to assign it to Mike Churchward, and to assign it to the "My Moodle" component.

Just a quick comment on this issue: listing remote courses shouldn't be too hard. But adding any extra information (e.g. number of unread forum posts) would require extra MNet calls, which we don't currently have, (and may introduce performance issues).

Show
Hubert Chathi added a comment - Yeah, I guess I'm not a developer, so you can't assign issues to me. (I also can't edit issues myself...). Maybe the best way, for now, to make sure that this issue doesn't get lost is to assign it to Mike Churchward, and to assign it to the "My Moodle" component. Just a quick comment on this issue: listing remote courses shouldn't be too hard. But adding any extra information (e.g. number of unread forum posts) would require extra MNet calls, which we don't currently have, (and may introduce performance issues).
Hide
Penny Leach added a comment -

Hubert I got Jordan to make you a developer in the tracker so now we can assign stuff to you! Yay!

Show
Penny Leach added a comment - Hubert I got Jordan to make you a developer in the tracker so now we can assign stuff to you! Yay!
Hide
Brent Lee added a comment -

Hi Hubert,

Let me know if I can help with this issue in any way. I was thinking about coding the hack myself, if you have any suggestions.

Cheers,

Brent.

Show
Brent Lee added a comment - Hi Hubert, Let me know if I can help with this issue in any way. I was thinking about coding the hack myself, if you have any suggestions. Cheers, Brent.
Hide
Minh-Tam Nguyen added a comment -

we're considering a second instance of Moodle for our organisation and this would remove many of the perceived barriers for students.

Show
Minh-Tam Nguyen added a comment - we're considering a second instance of Moodle for our organisation and this would remove many of the perceived barriers for students.
Hide
Hubert Chathi added a comment -

Attached are two patches (one for Moodle 2 and one for Moodle 1.9) that adds MNet courses to the My Moodle page.

Show
Hubert Chathi added a comment - Attached are two patches (one for Moodle 2 and one for Moodle 1.9) that adds MNet courses to the My Moodle page.
Hide
Hubert Chathi added a comment -

I've attached two new patches that tweak the output a bit.

Show
Hubert Chathi added a comment - I've attached two new patches that tweak the output a bit.
Hide
David Mudrak added a comment -

+1 for having it in 2.0, comments sent to Hubert. More testing and review next week, committing then

Show
David Mudrak added a comment - +1 for having it in 2.0, comments sent to Hubert. More testing and review next week, committing then
Hide
Hubert Chathi added a comment -

Updated the Moodle 2 version of the patch based on David's suggestions (and also fixed the other hard-coded HTML in that function)

Show
Hubert Chathi added a comment - Updated the Moodle 2 version of the patch based on David's suggestions (and also fixed the other hard-coded HTML in that function)
Hide
David Mudrak added a comment -

Fixed in CVS HEAD. Thanks for the patch Hubert! I just changed it a bit to use moodle_url instead of string URLs and to use s() instead of format_string() when putting the course fullname into title="" attribute value. Good to see that print_overview() API has not changed so this should not break eventual custom plugins.

Show
David Mudrak added a comment - Fixed in CVS HEAD. Thanks for the patch Hubert! I just changed it a bit to use moodle_url instead of string URLs and to use s() instead of format_string() when putting the course fullname into title="" attribute value. Good to see that print_overview() API has not changed so this should not break eventual custom plugins.

People

Dates

  • Created:
    Updated:
    Resolved: