From a713f8afac3544301da524e97e786a9bf23298b1 Mon Sep 17 00:00:00 2001
From: Ralf Becker <ralfbecker@outdoor-training.de>
Date: Fri, 22 Apr 2005 13:51:59 +0000
Subject: [PATCH] 1) autorepeated colums: only check if the max. column-number,
 reached so far, is exeeded, otherwise the rows have a differen number of
 cells and it saved a lot check 2) no <label> tags of readonly widgets or
 label-widgets

---
 etemplate/inc/class.uietemplate.inc.php | 15 +++++++++++----
 1 file changed, 11 insertions(+), 4 deletions(-)

diff --git a/etemplate/inc/class.uietemplate.inc.php b/etemplate/inc/class.uietemplate.inc.php
index 5ec79542b0..16fdcdff3f 100644
--- a/etemplate/inc/class.uietemplate.inc.php
+++ b/etemplate/inc/class.uietemplate.inc.php
@@ -427,6 +427,7 @@
 			{
 				$opts = array();
 			}
+			$max_cols = $grid['cols'];
 			for ($r = 0; $row = 1+$r /*list($row,$cols) = each($data)*/; ++$r)
 			{
 				if (!(list($r_key) = each($data)))	// no further row
@@ -466,10 +467,16 @@
 					$col = $this->num2chrs($c);
 					if (!(list($c_key) = each($cols)))		// no further cols
 					{
-						if (!$this->autorepeat_idx($cell,$c,$r,$idx,$idx_cname,True) ||
-							!$this->isset_array($content,$idx))
+						// only check if the max. column-number reached so far is exeeded
+						// otherwise the rows have a differen number of cells and it saved a lot checks
+						if ($c >= $max_cols)
 						{
-							break;	// no auto-col-repeat
+							if (!$this->autorepeat_idx($cell,$c,$r,$idx,$idx_cname,True) ||
+								!$this->isset_array($content,$idx))
+							{
+								break;	// no auto-col-repeat
+							}
+							$max_cols = $c+1;
 						}
 					}
 					else
@@ -1224,7 +1231,7 @@
 					$label = str_replace('&'.$accesskey[1],'<u>'.$accesskey[1].'</u>',$label);
 					$accesskey = $accesskey[1];
 				}
-				if ($label && ($accesskey || $label_for || $cell['name']))
+				if ($label && !$readonly && $type != 'label' && ($accesskey || $label_for || $cell['name']))
 				{
 					$label = $this->html->label($label,$label_for ? $this->form_name($cname,$label_for) : 
 						$form_name.($set_val?"[$set_val]":''),$accesskey);