[Alba-desarrollo] SVN Alba r4493 - prosistem/alba/trunk/alba/apps/principal/modules/docenteHorario/actions

commits en pressenter.com.ar commits en pressenter.com.ar
Lun Mar 19 16:47:26 CET 2007


Author: josx
Date: 2007-03-19 12:47:25 -0300 (Mon, 19 Mar 2007)
New Revision: 4493

Modified:
   prosistem/alba/trunk/alba/apps/principal/modules/docenteHorario/actions/actions.class.php
Log:
Se soluciono un problema en docenteHorarios cuando no enviaban id docente

Modified: prosistem/alba/trunk/alba/apps/principal/modules/docenteHorario/actions/actions.class.php
===================================================================
--- prosistem/alba/trunk/alba/apps/principal/modules/docenteHorario/actions/actions.class.php	2007-03-19 14:59:17 UTC (rev 4492)
+++ prosistem/alba/trunk/alba/apps/principal/modules/docenteHorario/actions/actions.class.php	2007-03-19 15:47:25 UTC (rev 4493)
@@ -46,31 +46,39 @@
 
   public function executeList ()
   {
-    if($this->getRequestParameter('idDocente')) {
 
-        $aRepeticion  = RepeticionPeer::doSelect(new Criteria());
+    $optionsDocente = array();
+    $aMuestraRepeticion = array();
+    $aHorario = array();
 
-        $aMuestraRepeticion = array();
-        foreach($aRepeticion  as $repeticion) {
-            $aMuestraRepeticion[$repeticion->getId()] = $repeticion->getDescripcion();
-        }
-    
-        $c = new Criteria();
-        $c->add(DocenteHorarioPeer::FK_DOCENTE_ID, $this->getRequestParameter('idDocente'));
-        $aHorario  = DocenteHorarioPeer::doSelect($c);
-        $this->aHorario = $aHorario;
+    $c = new Criteria();
+    $aDocente  = DocentePeer::doSelect($c);
+    $optionsDocente = array();
+    foreach($aDocente as $docente) {
+        $optionsDocente[$docente->getId()] = $docente->getApellido().' '.$docente->getNombre();
+    }
 
-        $c = new Criteria();
-        $aDocente  = DocentePeer::doSelect($c);
-        $optionsDocente = array();
-        foreach($aDocente as $docente) {
-            $optionsDocente[$docente->getId()] = $docente->getApellido().' '.$docente->getNombre();
-        }
+    $aRepeticion  = RepeticionPeer::doSelect(new Criteria());
 
-        $this->aRepeticion = $aMuestraRepeticion;
-        $this->optionsDocente = $optionsDocente;
-    } 
+    $aMuestraRepeticion = array();
+    foreach($aRepeticion  as $repeticion) {
+        $aMuestraRepeticion[$repeticion->getId()] = $repeticion->getDescripcion();
+    }
 
+
+    if(count($optionsDocente) > 0) {
+        $docente_id = ($this->getRequestParameter('idDocente')) ? $this->getRequestParameter('idDocente') : key(current($optionsDocente)) ;
+    } else {
+        // error si no tiene docente_id y no hay cargados docentes.
+    }
+
+    $c = new Criteria();
+    $c->add(DocenteHorarioPeer::FK_DOCENTE_ID, $this->getRequestParameter('idDocente'));
+    $aHorario  = DocenteHorarioPeer::doSelect($c);
+
+    $this->aHorario = $aHorario;
+    $this->optionsDocente = $optionsDocente;
+    $this->aRepeticion = $aMuestraRepeticion;
   }
 
     public function executeDeleteHorario ()  {




Más información sobre la lista de distribución Alba-desarrollo