gunnar: server/kolab-webadmin/kolab-webadmin/php/admin/include form.class.php, 1.30, 1.31

cvs at kolab.org cvs at kolab.org
Thu May 24 12:17:26 CEST 2007


Author: gunnar

Update of /kolabrepository/server/kolab-webadmin/kolab-webadmin/php/admin/include
In directory doto:/tmp/cvs-serv8816/kolab-webadmin/kolab-webadmin/php/admin/include

Modified Files:
	form.class.php 
Log Message:
Fixed kolab/issue1610 (https://intevation.de/roundup/kolab/issue1610)


Index: form.class.php
===================================================================
RCS file: /kolabrepository/server/kolab-webadmin/kolab-webadmin/php/admin/include/form.class.php,v
retrieving revision 1.30
retrieving revision 1.31
diff -u -d -r1.30 -r1.31
--- form.class.php	24 May 2007 09:49:58 -0000	1.30
+++ form.class.php	24 May 2007 10:17:24 -0000	1.31
@@ -97,12 +97,13 @@
       case 'input':
       case 'text':
 	$str .= '<tr>';
-	$str .= '<td>'.$value['name'].'</td>';
 	if( ereg( 'readonly', $value['attrs'] ) ) {
+	  $str .= '<td><label>'.$value['name'].'</label></td>';
 	  $str .= '<td><p class="ctrl">'.MySmarty::htmlentities($value['value']).'</p><input name="'
 	    .$key.'" type="hidden" value="'.MySmarty::htmlentities($value['value']).'" /></td>';
 	} else {
-	  $str .= '<td><input name="'.$key.'" type="'.$value['type'].'" value="'.MySmarty::htmlentities($value['value']).'" '
+	  $str .= '<td><label for="'.$key.'">'.$value['name'].'</label></td>';
+	  $str .= '<td><input name="'.$key.'" id="'.$key.'" type="'.$value['type'].'" value="'.MySmarty::htmlentities($value['value']).'" '
 	    .MySmarty::htmlentities($value['attrs']).' size="'.$size.'" /></td>';
 	}
 	$str .= '<td>'.KolabForm::comment_helper($value).'</td>';
@@ -110,7 +111,6 @@
 	break;
       case 'email':
 	$str .= '<tr>';
-	$str .= '<td>'.$value['name'].'</td>';
 	if( strpos($value['value'],'@')===false ) {
 	  $uname = $value['value'];
 	  $domain = '';
@@ -122,13 +122,14 @@
 	    MySmarty::htmlentities($uname).'" /><input name="domain_'.$key.'" type="hidden" value="'.
 	    MySmarty::htmlentities($domain).'" /></td>';
 	} else {
-	  $str .= '<td><input name="user_'.$key.'" type="text" value="'.MySmarty::htmlentities($uname).'" '
+	  $str .= '<td><label for="user_'.$key.'">'.$value['name'].'</label></td>';
+	  $str .= '<td><input name="user_'.$key.'" id="user_'.$key.'" type="text" value="'.MySmarty::htmlentities($uname).'" '
 	    .$value['attrs'].' size="'.($size-40).'" />';
 	  if( count($value['domains']) == 1 ) {
 	    $str .= '<input name="domain_'.$key.'" type="hidden" value="'.MySmarty::htmlentities($value['domains'][0]).'" />';
 	    $str .= '<span class="ctrl">@'.MySmarty::htmlentities($value['domains'][0]).'</span></td>';
 	  } else {
-	    $str .= '@<select name="domain_'.$key.'" '.$value['attrs']." >\n";
+	    $str .= '<label for="domain_'.$key.'">@</label><select name="domain_'.$key.'" id="domain_'.$key.'" '.$value['attrs']." >\n";
 	    foreach( $value['domains'] as $dom ) {
 	      if( $dom == $domain ) $s = 'selected="selected"';
 	      else $s = '';
@@ -150,29 +151,31 @@
 	break;
       case 'textarea':
 	$str .= '<tr>';
-	$str .= '<td>'.$value['name'].'</td>';
 	if( ereg( 'readonly', $value['attrs'] ) ) {
+	  $str .= '<td><label>'.$value['name'].'</label></td>';
 	  $str .= '<td><p class="ctrl">'.MySmarty::htmlentities($value['value']).'</p></td>';
 	} else {
-	  $str .= '<td><textarea name="'.$key.'" rows="5" cols="'.$size.'" '.$value['attrs'].' onkeypress="javascript:textareakeypress()">'.MySmarty::htmlentities($value['value']).'</textarea></td>';
+	  $str .= '<td><label for="'.$key.'">'.$value['name'].'</label></td>';
+	  $str .= '<td><textarea name="'.$key.'" id="'.$key.'" rows="5" cols="'.$size.'" '.$value['attrs'].' onkeypress="javascript:textareakeypress()">'.MySmarty::htmlentities($value['value']).'</textarea></td>';
 	}
 	$str .= '<td>'.KolabForm::comment_helper($value).'</td>';
 	$str .= '</tr>'."\n";
 	break;	
       case 'checkbox':
 	$str .= '<tr>';
-	$str .= '<td>'.$value['name'].'</td>';
 	if( ereg( 'readonly', $value['attrs'] ) ) {
+	  $str .= '<td><label>'.$value['name'].'</label></td>';
 	  $str .= '<td><span class="ctrl">'.($value['value']?_('Yes'):_('No')).'</span></td>';
 	} else {
-	  $str .= '<td><input name="'.$key.'" type="'.$value['type'].'" value="on" '.($value['value']?'checked':'').' '.$value['attrs'].' /></td>';
+	  $str .= '<td><label for="'.$key.'">'.$value['name'].'</label></td>';
+	  $str .= '<td><input name="'.$key.'" id="'.$key.'" type="'.$value['type'].'" value="on" '.($value['value']?'checked':'').' '.$value['attrs'].' /></td>';
 	}
 	$str .= '<td>'.KolabForm::comment_helper($value).'</td>';
 	$str .= '</tr>'."\n";
 	break;	
       case 'checklist':
 	$str .= '<tr>';
-	$str .= '<td>'.$value['name'].'</td>';
+	$str .= '<td><label>'.$value['name'].'</label></td>';
 	if( ereg( 'readonly', $value['attrs'] ) ) {
 	  $str .= '<td><span class="ctrl">'.(join('<br/>',$value['options'])).'</span></td>';
 	} else {
@@ -191,12 +194,13 @@
 	break;			
       case 'select':
 	$str .= '<tr>';
-	$str .= '<td>'.$value['name'].'</td>';
 	if( ereg( 'readonly', $value['attrs'] ) ) {
+	  $str .= '<td><label>'.$value['name'].'</label></td>';
 	  $str .= '<td><p class="ctrl">'.MySmarty::htmlentities($value['options'][$value['value']]).
 	    '<input type="hidden" name="'.$key.'" value="'.MySmarty::htmlentities($value['value']).'" /></p></td>';
 	} else {
-	  $str .= '<td><select name="'.$key.'" '.$value['attrs'].' >'."\n";
+	  $str .= '<td><label for="'.$key.'">'.$value['name'].'</label></td>';
+	  $str .= '<td><select name="'.$key.'" id="'.$key.'" '.$value['attrs'].' >'."\n";
 	  
 	  for( $i = 0; $i < count($value['options']); ++$i) {
 	    if( $i == $value['value'] ) $s = 'selected="selected"';
@@ -211,15 +215,16 @@
 	break;
       case 'foldertypeselect':
 	$str .= '<tr>';
-	$str .= '<td>'.$value['name'].'</td>';
 	$entries = array ( '' => _('Unspecified'), 'mail' => _('Mails'), 'task' => _('Tasks'), 
 			   'journal' => _('Journals'), 'event' => _('Events'), 
 			   'contact' => _('Contacts'), 'note' => _('Notes'));
 	if( ereg( 'readonly', $value['attrs'] ) ) {
+	  $str .= '<td><label>'.$value['name'].'</label></td>';
 	  $str .= '<td><p class="ctrl">'.MySmarty::htmlentities($entries[$value['value']]).
 	    '<input type="hidden" name="'.$key.'" value="'.MySmarty::htmlentities($value['value']).'" /></p></td>';
 	} else {
-	  $str .= '<td><select name="'.$key.'" '.$value['attrs'].' >'."\n";
+	  $str .= '<td><label for="'.$key.'">'.$value['name'].'</label></td>';
+	  $str .= '<td><select name="'.$key.'" id="'.$key.'" '.$value['attrs'].' >'."\n";
 	  foreach ($entries as $id => $title) {
 	    if ( $value['value'] == $id )
 	      $s = 'selected="selected"';
@@ -236,11 +241,12 @@
 	break;
       case 'aclselect': // Special Kolab entry for ACLs
 	$str .= '<tr>';
-	$str .= '<td>'.$value['name'].'</td>';
 	if( ereg( 'readonly', $value['attrs'] ) ) {
+	  $str .= '<td><label>'.$value['name'].'</label></td>';
 	  if( $value['user'] ) $str .= '<td><span class="ctrl">'.MySmarty::htmlentities($value['user']).'</span> <span class="ctrl">'.$value['perm'].'</span></td>';
 	} else {
-	  $str .= '<td><input name="user_'.$key.'" type="text" size="'.($size-15).'" value="'
+	  $str .= '<td><label for="user_'.$key.'">'.$value['name'].'</label></td>';
+	  $str .= '<td><input name="user_'.$key.'" id="user_'.$key.'" type="'.$value['type'].'" size="'.($size-15).'" value="'
 	    .MySmarty::htmlentities($value['user']).'" '.$value['attrs'].' />';
 	  $str .= '<select name="perm_'.$key.'">'."\n";
 	  if( $value['perm'] ) $selected_perm = $value['perm'];
@@ -269,7 +275,7 @@
 	debug("resourcepolicy");
 	$ro = ereg( 'readonly', $value['attrs'] );
 	$str .= '<tr>';
-	$str .= '<td>'.$value['name'].'</td>';
+	$str .= '<td><label>'.$value['name'].'</label></td>';
 	$str .= '<td>';
 	$str .= '<table>';
 	$i = 0;





More information about the commits mailing list