Moodle

Mnet overhaul for Moodle 2.0

Details

  • Type: Bug Bug
  • Status: Open Open
  • Priority: Critical Critical
  • Resolution: Unresolved
  • Affects Version/s: 2.0
  • Fix Version/s: DEV backlog
  • Component/s: MNet
  • Labels:
  • Difficulty:
    Difficult
  • Affected Branches:
    MOODLE_20_STABLE

Description

Meta bug to help group 2.0 bugs

Activity

Hide
Petr Škoda (skodak) added a comment -

maybe we could get rid of the ugly $MNET global that is instantiated in the middle of library function, I say you already did something like that in portfolio code. The $MNET instance could be returned by some core function like "get_mnet_environment()" which would also load the general mnet library.

see mnet/lib.php:

$MNET = new mnet_environment();
$MNET->init();
Show
Petr Škoda (skodak) added a comment - maybe we could get rid of the ugly $MNET global that is instantiated in the middle of library function, I say you already did something like that in portfolio code. The $MNET instance could be returned by some core function like "get_mnet_environment()" which would also load the general mnet library. see mnet/lib.php:
$MNET = new mnet_environment();
$MNET->init();
Hide
Penny Leach added a comment -

Petr - that's being tracked in MDL-21276

I also want to get rid of MNET_REMOTE_CLIENT global everywhere too, it's super ugly. It can't be passed around as a function argument (I commented on this somewhere else, can't remember where), but it can be encapsulated in get_mnet_remote_client() call.

Show
Penny Leach added a comment - Petr - that's being tracked in MDL-21276 I also want to get rid of MNET_REMOTE_CLIENT global everywhere too, it's super ugly. It can't be passed around as a function argument (I commented on this somewhere else, can't remember where), but it can be encapsulated in get_mnet_remote_client() call.
Hide
Penny Leach added a comment -

Argh, there's also another global $MNETIDPJUMPURL

Show
Penny Leach added a comment - Argh, there's also another global $MNETIDPJUMPURL
Hide
Petr Škoda (skodak) added a comment -

David and Penny, what is the status of these issues?

Show
Petr Škoda (skodak) added a comment - David and Penny, what is the status of these issues?
Hide
David Mudrak added a comment -

Penny did a great job with organising these issues. Unfortunatelly, I won't be able to touch almost any of them until the second half of Sep so if 2.0 is really going to release by that time, the remaining issues here should be postponed to 2.1.

Show
David Mudrak added a comment - Penny did a great job with organising these issues. Unfortunatelly, I won't be able to touch almost any of them until the second half of Sep so if 2.0 is really going to release by that time, the remaining issues here should be postponed to 2.1.
Hide
Eloy Lafuente (stronk7) added a comment -

Moving this to the STABLE backlog... are there any plans? Do we need to fix any of them (really urgent/causing boom) for 2.0.x? Or can the whole thing be moved to DEV backlog?

Show
Eloy Lafuente (stronk7) added a comment - Moving this to the STABLE backlog... are there any plans? Do we need to fix any of them (really urgent/causing boom) for 2.0.x? Or can the whole thing be moved to DEV backlog?
Hide
David Mudrak added a comment -

My +1 for moving all this to DEV backlog for 2.1. MNet needs some significant refactoring anyway (for example naming XMLRPC methods or the way how plugins can implement MNet interface).

Show
David Mudrak added a comment - My +1 for moving all this to DEV backlog for 2.1. MNet needs some significant refactoring anyway (for example naming XMLRPC methods or the way how plugins can implement MNet interface).
Hide
Eloy Lafuente (stronk7) added a comment -

Moving to DEV backlog, as requested. Thanks!

Show
Eloy Lafuente (stronk7) added a comment - Moving to DEV backlog, as requested. Thanks!

People

Vote (1)
Watch (6)

Dates

  • Created:
    Updated: