Italico76, no sé que tan válido sea tu argumento respecto a condicionales dentro de un ciclo, tampoco he implementado la interfase de iterador en objetos, que es lo que está manejando el usuario; pero me parece interesante tu solución.
Luego, sólo como práctica, si la asignación de
$flag en el código original queda después de la estructura del if, funciona correctamente:
Código PHP:
Ver originalforeach( $items_gallery as $objeto){
echo '<a class="fancybox" rel="gallery1" href="'.base_url().'upload/mi_gabbanelli/'.$objeto->photo.'" ></a>';
// Nunca se va a realizar esta asignación
// $flag=1;
}
// Aquí es donde se debe definir $flag
$flag = 1;
}
Tal vez la única impresión proviene de algún código anterior y de ahí la necesidad de omitir el primer echo... entonces creo que sí estaríamos hablando de muy malas prácticas.