serios bugs en select_tag

Comentarios

• rogerjose81

  agosto 2007 edited 1:35

  "emilio.rst&quot wrote:

  ...

  ...
  	              if(is_object($opts[1][0])){
  
  
  
  	              } 
  

  
  OK, dos cosas:
  
  Por que hay un if vacio?
  
  Por que imprime el codigo resultante, si la convencion es retornarlo?
• skinner

  agosto 2007 edited 1:35

  Y ya que nos ponemos quisquillosos... <!-- s:twisted: --><img src="{SMILIES_PATH}/icon_twisted.gif" alt=":twisted:" title="Twisted Evil" /><!-- s:twisted: -->
  ¿Para qué sirve el segundo if en este fragmento?
  

  "emilio.rst&quot wrote:

  ...

  	if(is_array($opts)){
  
  		$code.="<select id='{$opts[0]}' name='{$opts[0]}' ";
  
  		if(is_array($opts)){
  
• emilio.rst

  agosto 2007 edited 1:35

  Ese es el codigo que venia en tags.php
  
  Yo solo corregi las partes inherentes a los bugs, es decir yo no escribi el codigo completo, por lo tanto no conozco todo su funcionamiento.
  
  Con respecto a la convencion, creo que deberia ser aplicada cuando se cambie a 0.5 a los tags correspondientes, para evitar problemas. Tambien fui de los primeros que hablo de la convencion para los helpers en el foro (cuchillo para mi garganta).
  
  Si lo comparas con el codigo del actual tags.php notaras que no hay muchos cambios
  
  Saludos.
• rogerjose81

  agosto 2007 edited 1:35

  "emilio.rst&quot wrote:

  Ese es el codigo que venia en tags.php
  
  Yo solo corregi las partes inherentes a los bugs, es decir yo no escribi el codigo completo, por lo tanto no conozco todo su funcionamiento.
  
  Con respecto a la convencion, creo que deberia ser aplicada cuando se cambie a 0.5 a los tags correspondientes, para evitar problemas. Tambien fui de los primeros que hablo de la convencion para los helpers en el foro (cuchillo para mi garganta).
  
  Si lo comparas con el codigo del actual tags.php notaras que no hay muchos cambios
  
  Saludos.

  
  Ok Emilio, tranquilo, sin "cosquillas" jeje.
  
  Nos queda pendiente seguir corrigiendo ese metodo.
• emilio.rst

  agosto 2007 edited 1:35

  Si es solo una solucion temporal, hay que optimizarlo, la mayoria de las preguntas que me hicieron, tambien me las hice en su momento.
• anthemfor182

  agosto 2007 edited 1:35

  Optimicemoslo y lo subimos al svn corregido.
  
  Saludos
• rogerjose81

  agosto 2007 edited 1:35

  "anthemfor182&quot wrote:

  Optimicemoslo y lo subimos al svn corregido.
  
  Saludos

  
  Aqui dejo una version optimizada.
  

  function select_tag($name='', $data=''){
     if(func_num_args()>1){
        $opts = get_params(func_get_args());
        $code = "<select id='{$opts[0]}' name='{$opts[0]}' ";      
  	  foreach($opts as $at => $val){
  		 if(!is_numeric($at)){
  		    $code.= "$at = '".$val."' ";
  		 }
  	  }
        $code.=">\r\n";
        if(is_array($opts[1])){
           if(!is_object($opts[1][0])){
              foreach($opts[1] as $key => $value){
                 $code.= "\t<option value='{$key}'>{$value}</option>\r\n";
  			}
           }
           $code.= "</select>\r\n";
        }
     } else {
        $code = "<select id='$opts' name='$opts'>";
     }
     return $code;
  }
  
• emilio.rst

  agosto 2007 edited agosto 2007

  Detecte un pequeño bug, pero de resto esta muy bien.
  
  Bug: cuando se pasa solo un parametro, el que corresponde al id y name del select, como no se ha cargado opts y opts es un array siempre, entonces no sustituye correctamente.
  
  Aqui esta la correccion:
  

  function select_tag($name='', $data='')
  {
  	if(func_num_args()>1)
  	{
  		$opts = get_params(func_get_args());
  		$code = "<select id='{$opts[0]}' name='{$opts[0]}' ";     
  		
  		foreach($opts as $at => $val)
  		{
  			if(!is_numeric($at))
  			{
  				$code.= "$at = '".$val."' ";
  			}
  		}
  		
  		$code.=">\r\n";
  		
  		if(is_array($opts[1]))
  		{
  			if(!is_object($opts[1][0]))
  			{
  				foreach($opts[1] as $key => $value)
  				{
  					$code.= "\t<option value='{$key}'>{$value}</option>\r\n";
  				}
  			}
  			$code.= "</select>\r\n";
  		}
  	}
  	else
  	{
  		$code = "<select id='$name' name='$name'>";
  	}
  	
  	return $code;
  }
  

  
  Muy buen trabajo roger. Saludos
• rogerjose81

  agosto 2007 edited 1:35

  Ok. Te recomiendo que coloques las llaves al lado de los if, ya que si pones la llave debajo incrementarás el numero de lineas espureamente, ademas colocar la llave al lado es casi un standard en muchos lenguajes.
• anthemfor182

  agosto 2007 edited 1:35

  Lo he revisado y me parece bien. Subelo al SVN. Recuerda que ya que esta el branch 0.5 en el SVN es necesario corregir los bugs tanto en la 0.4.x como en la 0.5.x por lo menos durante un buen tiempo hasta que 0.5 este estable.
  
  Saludos y buen trabajo
• emilio.rst

  agosto 2007 edited 1:35

  Gracias por las recomendaciones, lo de las llaves fue solo para hacerlo legible, lo estandarizare.
  
  Corregire entonces en las dos versiones.
  
  Saludos.