From: David Mudrak <david.mudrak@gmail.com>
Date: Mon, 4 Jan 2010 17:55:23 +0000 (+0000)
Subject: Puttting a TODO note - return to this code once working on the Scheduled allocation
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=8d54587610a5df73f1bd85a5554eb338b3585817;p=moodle.git

Puttting a TODO note - return to this code once working on the Scheduled allocation
---

diff --git a/mod/workshop/allocation/random/allocator.php b/mod/workshop/allocation/random/allocator.php
index db8dada198..a710b3e000 100644
--- a/mod/workshop/allocation/random/allocator.php
+++ b/mod/workshop/allocation/random/allocator.php
@@ -90,9 +90,12 @@ class workshop_random_allocator implements workshop_allocator {
             $reviewers          = $this->workshop->get_grouped($reviewers);
             $assessments        = $this->workshop->get_assessments();
 
-            $newallocations     = array();      // array of (reviewer,reviewee) tuples
+            $newallocations     = array();      // array of array(reviewer => reviewee)
 
             if ($numofreviews) {
+                // TODO MDL-19870 rewrite this part to make it easier to maintain and extend.
+                // $removecurrent -> remove it at the beginning and stop doing the magic with unkept allocation
+                // (leading to possible bugs)
                 if ($removecurrent) {
                     // behave as if there were no current assessments
                     $curassessments = array();