[Phpgroupware-cvs] phpgwapi/inc class.matrixview.inc.php

[Dave Hall]

CVSROOT:        /sources/phpgwapi
Module name:    phpgwapi
Changes by:     Dave Hall <skwashd>     06/12/28 10:38:44

Modified files:
        inc            : class.matrixview.inc.php 

Log message:
        more fixes - mostly markup

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/phpgwapi/inc/class.matrixview.inc.php?cvsroot=phpgwapi&r1=1.16&r2=1.17

Patches:
Index: class.matrixview.inc.php
===================================================================
RCS file: /sources/phpgwapi/phpgwapi/inc/class.matrixview.inc.php,v
retrieving revision 1.16
retrieving revision 1.17
diff -u -b -r1.16 -r1.17
--- class.matrixview.inc.php    28 Dec 2006 09:54:40 -0000      1.16
+++ class.matrixview.inc.php    28 Dec 2006 10:38:44 -0000      1.17
@@ -7,7 +7,7 @@
        * @license http://www.fsf.org/licenses/lgpl.html GNU Lesser General 
Public License
        * @package phpgwapi
        * @subpackage gui
-       * @version $Id: class.matrixview.inc.php,v 1.16 2006/12/28 09:54:40 
skwashd Exp $
+       * @version $Id: class.matrixview.inc.php,v 1.17 2006/12/28 10:38:44 
skwashd Exp $
        */
 
        /**
@@ -209,6 +209,7 @@
 
                        $this->out_header();
                        $this->out_ruler();
+                       echo '<tbody>';
 
                        // loop through number of items
                        for($z=0;$z<$this->items_count;$z++)
@@ -224,7 +225,8 @@
                                // loop through days of desired month
                                for($r=1;$r<$this->sumdays+1;$r++)
                                {
-                                       if($this->items_content[$z][$r] == 'x')
+                                       if( isset($this->items_content[$z][$r])
+                                               && $this->items_content[$z][$r] 
== 'x')
                                        {
                                                $color = $itemcolor;
                                        }
@@ -232,12 +234,12 @@
                                        {
                                                $color = 
$this->color_emptyfield;
                                        }
-                                       echo '<td bgcolor="' . $color . 
'">&nbsp;</td>' . "\n";
+                                       echo '<td bgcolor="' . $color . '" 
width="20">&nbsp;</td>' . "\n";
                                }
 
                                echo '</tr>' . "\n";
                        }
-
+                       echo '</tbody>';
                        echo '</table>';
                }
 
@@ -273,7 +275,7 @@
                function out_ruler ()
                {
                        $span = $this->sumdays + 1;
-                       echo "<tfoot>\n<tr>\n<td 
colspan=\"{$span}\">&nbsp;</td>\n</tr>\n</thead>\n";
+                       echo "<tfoot>\n<tr>\n<td 
colspan=\"{$span}\">&nbsp;</td>\n</tr>\n</tfoot>\n";
                }
 
                /**