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