gradebook MDL-23750 category and item page had extra cells due to par…

…tial refactoring
safatshahin · Aug 11, 2010 · 5174f3c · 5174f3c
1 parent a26f25a
commit 5174f3c
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/grade/edit/tree/lib.php b/grade/edit/tree/lib.php
@@ -1164,7 +1164,8 @@ public function get_item_cell($item, $params) {
         if ($params['itemtype'] != 'course' && $params['itemtype'] != 'category') {
             $itemselect = '<input class="itemselect" type="checkbox" name="select_'.$params['eid'].'" onchange="toggleCategorySelector();"/>'; // TODO: convert to YUI handler
         }
-        return '<td class="cell last selection">' . $itemselect . '</td>';
+        //html_writer::table() will wrap the item cell contents in a <TD> so don't do it here
+        return $itemselect;
     }
 
     public function is_hidden($mode='simple') {

diff --git a/lib/outputcomponents.php b/lib/outputcomponents.php
@@ -1194,6 +1194,7 @@ public static function table(html_table $table) {
 
         if (!empty($table->head)) {
             $countcols = count($table->head);
+
             $output .= html_writer::start_tag('thead', array()) . "\n";
             $output .= html_writer::start_tag('tr', array()) . "\n";
             $keys = array_keys($table->head);
@@ -1289,7 +1290,13 @@ public static function table(html_table $table) {
                     $keys2 = array_keys($row->cells);
                     $lastkey = end($keys2);
 
+                    $gotlastkey = false; //flag for sanity checking
                     foreach ($row->cells as $key => $cell) {
+                        if ($gotlastkey) {
+                            //This should never happen. Why do we have a cell after the last cell?
+                            mtrace("A cell with key ($key) was found after the last key ($lastkey)");
+                        }
+
                         if (!($cell instanceof html_table_cell)) {
                             $mycell = new html_table_cell();
                             $mycell->text = $cell;
@@ -1307,6 +1314,7 @@ public static function table(html_table $table) {
                         $cell->attributes['class'] .= ' cell c' . $key;
                         if ($key == $lastkey) {
                             $cell->attributes['class'] .= ' lastcol';
+                            $gotlastkey = true;
                         }
                         $tdstyle = '';
                         $tdstyle .= isset($table->align[$key]) ? $table->align[$key] : '';