[Alba-desarrollo] SVN Alba r3974 - in prosistem/alba/trunk/alba/apps/principal/modules/boletin: actions templates

commits en pressenter.com.ar commits en pressenter.com.ar
Vie Dic 29 22:31:13 CET 2006


Author: hsanchez
Date: 2006-12-29 18:31:12 -0300 (Fri, 29 Dec 2006)
New Revision: 3974

Modified:
   prosistem/alba/trunk/alba/apps/principal/modules/boletin/actions/actions.class.php
   prosistem/alba/trunk/alba/apps/principal/modules/boletin/templates/listConceptoSuccess.php
Log:
Corrigiendo bug, cuando no se selecciona concepto permitia grabar.

Modified: prosistem/alba/trunk/alba/apps/principal/modules/boletin/actions/actions.class.php
===================================================================
--- prosistem/alba/trunk/alba/apps/principal/modules/boletin/actions/actions.class.php	2006-12-29 20:36:23 UTC (rev 3973)
+++ prosistem/alba/trunk/alba/apps/principal/modules/boletin/actions/actions.class.php	2006-12-29 21:31:12 UTC (rev 3974)
@@ -83,8 +83,7 @@
                 }
 
                 $con->commit(); 
-             }
-             
+             }             
              catch (Exception $e){
                  $con->rollback();
                  throw $e;  
@@ -119,9 +118,6 @@
 
     }
 
-
-
-
     protected function getActividades($division_id) {
         $optionsActividad = array();
         $criteria = new Criteria();
@@ -167,9 +163,6 @@
             $optionsActividad = $this->getActividades($division_id);
         }
 
-       
-//        if ($this->getRequest()->getMethod() == sfRequest::POST) {
-
             $aAlumno = $this->getAlumnos($division_id);
             $criteria = new Criteria();
             $criteria->add(PeriodoPeer::FK_CICLOLECTIVO_ID, $this->getUser()->getAttribute('fk_ciclolectivo_id'));
@@ -186,7 +179,7 @@
 
            
             if(count($aAlumno) > 0) {
-// esto puede ser mejorado con solo una query bastante facilmente
+                // esto puede ser mejorado con solo una query bastante facilmente
                 foreach($aAlumno as $alumno) {
                     foreach($aPeriodo as $periodo) {
                         $criteria = new Criteria();
@@ -207,7 +200,6 @@
                 }
             }
 
-
             $criteria = new Criteria();
             $criteria->add(EscalanotaPeer::FK_ESTABLECIMIENTO_ID, $establecimiento_id);
             $aPosiblesNotas = EscalanotaPeer::doSelect($criteria);
@@ -219,9 +211,6 @@
             }
      
 
-  //      }
-
-
         // llenar variables a mostrar en el template
         $this->optionsDivision = $optionsDivision;
         $this->optionsActividad =$optionsActividad;
@@ -237,7 +226,6 @@
     }
 
 
-
     protected function getConcepto($establecimiento_id) {
         $optionsConcepto = array();
         $criteria = new Criteria();
@@ -317,8 +305,6 @@
                 }
             }
 
-
-
         }
 
         $criteria = new Criteria();
@@ -331,9 +317,6 @@
             }
         }
 
-
-
-
         // llenar variables a mostrar en el template
         $this->optionsDivision = $optionsDivision;
         $this->optionsConcepto =$optionsConcepto;
@@ -350,8 +333,6 @@
     }
 
 
-
-
     public function executeIndex() {
        return $this->forward('boletin', 'list');
     }
@@ -421,8 +402,6 @@
                     }
                 }            
 
-
-
                 $criteria = new Criteria();
                 $criteria->add(PeriodoPeer::FK_CICLOLECTIVO_ID, $this->getUser()->getAttribute('fk_ciclolectivo_id'));
                 $aPeriodo = PeriodoPeer::doSelect($criteria);
@@ -451,8 +430,6 @@
         $this->aAsistencia = $aAsistencia;
     }
 
-
-
     protected function getEscalanota($establecimiento_id) {
         $aDatosTablaEscalaNota = array();
         $criteria = new Criteria();
@@ -464,8 +441,6 @@
         return $aDatosTablaEscalaNota;
     }
 
-
-
     public function executeGrabarNotasConcepto() {
 
         // inicializando variables
@@ -522,8 +497,7 @@
                     }
                 }
                 $con->commit(); 
-             }
-             
+             }             
              catch (Exception $e){
                  $con->rollback();
                  throw $e;  
@@ -563,5 +537,4 @@
         return $aAsistencia;
     }
 
-
 }

Modified: prosistem/alba/trunk/alba/apps/principal/modules/boletin/templates/listConceptoSuccess.php
===================================================================
--- prosistem/alba/trunk/alba/apps/principal/modules/boletin/templates/listConceptoSuccess.php	2006-12-29 20:36:23 UTC (rev 3973)
+++ prosistem/alba/trunk/alba/apps/principal/modules/boletin/templates/listConceptoSuccess.php	2006-12-29 21:31:12 UTC (rev 3974)
@@ -41,7 +41,7 @@
 
 
 
-<? if (count($aAlumno) > 0) { ?>
+<? if (count($aAlumno) > 0 && $concepto_id ){ ?>
 <h1>Alumnos</h1>
 
 Posibles Notas para calificar: 
@@ -77,7 +77,7 @@
   </tbody>
 </table>
 
-<? } 
+<?// } 
 if($division_id) { ?>
 
  <ul class="sf_admin_actions">
@@ -86,6 +86,8 @@
   'class' => 'sf_admin_action_save',
 )) ?></li>
 </ul>
-<? } ?>
+<? }
+ }
+ ?>
 </fieldset>
 </form>
\ No newline at end of file




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