Author: ftoledo Date: 2009-04-24 17:29:35 -0300 (Fri, 24 Apr 2009) New Revision: 6124 Modified: prosistem/alba/trunk/alba/apps/principal/modules/ciclolectivo/actions/actions.class.php prosistem/alba/trunk/alba/apps/principal/modules/ciclolectivo/templates/agregarTurnosYPeriodosSuccess.php prosistem/alba/trunk/alba/apps/principal/modules/ciclolectivo/validate/grabarTurnosYPeriodos.yml prosistem/alba/trunk/alba/apps/principal/templates/_flashes.php prosistem/alba/trunk/alba/web/css/main.css Log: fix delete de turnos y periodos fix estilos de flashes fix flashes Modified: prosistem/alba/trunk/alba/apps/principal/modules/ciclolectivo/actions/actions.class.php =================================================================== --- prosistem/alba/trunk/alba/apps/principal/modules/ciclolectivo/actions/actions.class.php 2009-04-24 20:29:17 UTC (rev 6123) +++ prosistem/alba/trunk/alba/apps/principal/modules/ciclolectivo/actions/actions.class.php 2009-04-24 20:29:35 UTC (rev 6124) @@ -58,16 +58,16 @@ foreach($aCiclolectivo as $ciclolectivo) { $optionsCiclolectivo[$ciclolectivo->getId()] = $ciclolectivo->getDescripcion(); } - + $this->optionsCiclolectivo = $optionsCiclolectivo; - if($ciclolectivo_id) { + if($ciclolectivo_id) { $ciclolectivo = CiclolectivoPeer::retrieveByPk($ciclolectivo_id); - + $c = new Criteria(); $c->add(TurnoPeer::FK_CICLOLECTIVO_ID, $ciclolectivo_id); $aTurnos = TurnoPeer::doSelect($c); - + $c = new Criteria(); $c->add(PeriodoPeer::FK_CICLOLECTIVO_ID, $ciclolectivo_id); $aPeriodo = PeriodoPeer::doSelect($c); @@ -78,21 +78,32 @@ $this->aPeriodo = $aPeriodo; $this->aTurnos = $aTurnos; $this->ciclolectivo = $ciclolectivo; - + } public function executeDeleteTurno ($request) { $this->turnos = TurnoPeer::retrieveByPk($this->getRequestParameter('idTurno')); $this->forward404Unless($this->turnos); - $this->turnos->delete(); - return $this->forward('ciclolectivo','agregarTurnosYPeriodos','id/'.$this->getRequestParameter('id')); + try { + $this->turnos->delete(); + } + catch(Exception $e) { + $this->getUser()->setFlash('error', 'No se puede eliminar el Turno porque está siendo utilizado.'); + } + + return $this->redirect('ciclolectivo/agregarTurnosYPeriodos?id='.$this->getRequestParameter('id')); } - + public function executeDeletePeriodo ($request) { $this->periodo = PeriodoPeer::retrieveByPk($this->getRequestParameter('idPeriodo')); $this->forward404Unless($this->periodo); - $this->periodo->delete(); - return $this->forward('ciclolectivo','agregarTurnosYPeriodos','id/'.$this->getRequestParameter('id')); + try { + $this->periodo->delete(); + } + catch (Exception $e) { + $this->getUser()->setFlash('error', 'No se puede eliminar el Período porque está siendo utilizado.'); + } + return $this->redirect('ciclolectivo/agregarTurnosYPeriodos?id='.$this->getRequestParameter('id')); } @@ -101,93 +112,93 @@ if(is_numeric($ciclolectivo['id'])) { $this->ciclolectivo = CiclolectivoPeer::retrieveByPk($ciclolectivo['id']); if(isset($ciclolectivo['descripcion'])) $this->ciclolectivo->setDescripcion($ciclolectivo['descripcion']); - if(isset($ciclolectivo['fecha_fin'])) { - if($ciclolectivo['fecha_fin']) { + if(isset($ciclolectivo['fecha_fin'])) { + if($ciclolectivo['fecha_fin']) { list($d, $m, $y) = sfContext::getInstance()->getI18N()->getDateForCulture($ciclolectivo['fecha_fin'], $this->getUser()->getCulture()); $this->ciclolectivo->setFechaFin("$y-$m-$d"); } - } + } - if(isset($ciclolectivo['fecha_inicio'])) { - if($ciclolectivo['fecha_inicio']) { + if(isset($ciclolectivo['fecha_inicio'])) { + if($ciclolectivo['fecha_inicio']) { list($d, $m, $y) = sfContext::getInstance()->getI18N()->getDateForCulture($ciclolectivo['fecha_inicio'], $this->getUser()->getCulture()); $this->ciclolectivo->setFechaInicio("$y-$m-$d"); } } - - $this->ciclolectivo->save(); + + $this->ciclolectivo->save(); } - + $aTurnos = $this->getRequestParameter('turnos'); - if(is_array($aTurnos)) { + if(is_array($aTurnos)) { foreach($aTurnos as $turnos) { - + $horaInicio = $this->_add_zeros($turnos['hora_inicio']['hour'],2).":".$this->_add_zeros($turnos['hora_inicio']['minute'],2)." ".$turnos['hora_inicio']['ampm']; $horaFin = $this->_add_zeros($turnos['hora_fin']['hour'],2).":".$this->_add_zeros($turnos['hora_fin']['minute'],2)." ".$turnos['hora_fin']['ampm']; - + if($turnos['descripcion'] AND ($horaInicio != $horaFin) ) { - + if(isset($turnos['id']) AND is_numeric($turnos['id'])) { $this->turnos = TurnoPeer::retrieveByPk($turnos['id']); } else { $this->turnos = new Turno(); } - + if(isset($turnos['descripcion'])) { $this->turnos->setDescripcion($turnos['descripcion']); } - + $this->turnos->setHoraInicio($horaInicio); $this->turnos->setHoraFin($horaFin); $this->turnos->setFkCiclolectivoId($ciclolectivo['id']); $this->turnos->save(); } - } + } } $aPeriodo = $this->getRequestParameter('periodo'); - if(is_array($aPeriodo)) { + if(is_array($aPeriodo)) { foreach($aPeriodo as $periodo) { if($periodo['descripcion'] AND ($periodo['fecha_inicio'] OR $periodo['fecha_fin'])) { - + if(isset($periodo['id']) AND is_numeric($periodo['id'])) { $this->periodo = PeriodoPeer::retrieveByPk($periodo['id']); } else { $this->periodo = new Periodo(); } - + if(isset($periodo['descripcion'])) $this->periodo->setDescripcion($periodo['descripcion']); - - if(isset($periodo['fecha_inicio'])) { - if($periodo['fecha_inicio']) { + + if(isset($periodo['fecha_inicio'])) { + if($periodo['fecha_inicio']) { list($d, $m, $y) = sfContext::getInstance()->getI18N()->getDateForCulture($periodo['fecha_inicio'], $this->getUser()->getCulture()); $this->periodo->setFechaInicio("$y-$m-$d"); } - } - - if(isset($periodo['fecha_fin'])) { - if($periodo['fecha_fin']) { + } + + if(isset($periodo['fecha_fin'])) { + if($periodo['fecha_fin']) { list($d, $m, $y) = sfContext::getInstance()->getI18N()->getDateForCulture($periodo['fecha_fin'], $this->getUser()->getCulture()); $this->periodo->setFechaFin("$y-$m-$d"); } - } + } $this->periodo->setFkCiclolectivoId($ciclolectivo['id']); $this->periodo->save(); } - } + } } - return $this->redirect('ciclolectivo/agregarTurnosYPeriodos?id='.$ciclolectivo['id']); + return $this->redirect('ciclolectivo/agregarTurnosYPeriodos?id='.$ciclolectivo['id']); } public function handleErrorGrabarTurnosYPeriodos() { $ciclolectivo = $this->getRequestParameter('ciclolectivo'); $this->forward('ciclolectivo','agregarTurnosYPeriodos','id='.$ciclolectivo['id']); } - + private function _add_zeros($string, $strlen) { if ($strlen > strlen($string)) { for ($x = strlen($string); $x < $strlen; $x++) { @@ -201,7 +212,7 @@ $c->add(CiclolectivoPeer::FK_ESTABLECIMIENTO_ID, $this->getUser()->getAttribute('fk_establecimiento_id')); } - protected function saveCiclolectivo($ciclolectivo) { + protected function saveCiclolectivo($ciclolectivo) { //si se guarda el ciclo y se marca como actual //los demas ciclo del establecimiento tiene que quedar como ACUAL = false @@ -213,32 +224,32 @@ $c1->add(CiclolectivoPeer::FK_ESTABLECIMIENTO_ID,$this->getUser()->getAttribute('fk_establecimiento_id')); $c2 = new Criteria(); $c2->add(CiclolectivoPeer::ACTUAL,false); - BasePeer::doUpdate($c1,$c2,$con); + BasePeer::doUpdate($c1,$c2,$con); } - $ciclolectivo->setFkEstablecimientoId($this->getUser()->getAttribute('fk_establecimiento_id')); - $ciclolectivo->save(); + $ciclolectivo->setFkEstablecimientoId($this->getUser()->getAttribute('fk_establecimiento_id')); + $ciclolectivo->save(); $con->commit(); //cambio el attributo porque se cambio el ciclo actual $this->getUser()->setAttribute('fk_ciclolectivo_id',$ciclolectivo->getId()); - $this->getUser()->setAttribute('ciclolectivo_descripcion',$ciclolectivo->getDescripcion()); + $this->getUser()->setAttribute('ciclolectivo_descripcion',$ciclolectivo->getDescripcion()); } catch (Exception $e){ $con->rollBack(); throw $e; } } - + /** - * Cambia el ciclo lectivo actual - */ + * Cambia el ciclo lectivo actual + */ public function executeCambiar() { $c = new Criteria (); $c->add(CiclolectivoPeer::FK_ESTABLECIMIENTO_ID,$this->getUser()->getAttribute('fk_establecimiento_id')); $this->cicloslectivos = CiclolectivoPeer::doSelect($c); - $this->referer = $this->getRequest()->getReferer(); - } - + $this->referer = $this->getRequest()->getReferer(); + } + /** * pone como actual elciclo lectivo seleccionado */ @@ -252,17 +263,17 @@ $this->getUser()->setAttribute('ciclolectivo_descripcion',$ciclolectivo->getDescripcion()); // $this->getUser()->setFlash('notice','Se ha cambiado correctamente de ciclo lectivo.'); } - + return $this->redirect($this->getRequestParameter('referer', '@homepage')); - + } /** - * Elimino el ciclo lectivo si es el actual + * Elimino el ciclo lectivo si es el actual * el usuario debe quedar con ciclolectivo "No seleccionado" */ public function executeDelete($request) { - + $this->ciclolectivo = CiclolectivoPeer::retrieveByPk($this->getRequestParameter('id')); $this->forward404Unless($this->ciclolectivo); try { @@ -276,10 +287,10 @@ echo $e->getMessage(); die(); } - + return $this->redirect('ciclolectivo/list'); } - + function validateDelete($request) { $this->ciclolectivo = CiclolectivoPeer::retrieveByPk($this->getRequestParameter('id')); $this->forward404Unless($this->ciclolectivo); @@ -295,13 +306,13 @@ $this->error = "El ciclo lectivo contiene Feriados asociados"; return false; } - - return true; + + return true; } - + public function executeSinciclolectivo() { $this->modulo = $this->getRequestParameter('m'); - } + } Modified: prosistem/alba/trunk/alba/apps/principal/modules/ciclolectivo/templates/agregarTurnosYPeriodosSuccess.php =================================================================== --- prosistem/alba/trunk/alba/apps/principal/modules/ciclolectivo/templates/agregarTurnosYPeriodosSuccess.php 2009-04-24 20:29:17 UTC (rev 6123) +++ prosistem/alba/trunk/alba/apps/principal/modules/ciclolectivo/templates/agregarTurnosYPeriodosSuccess.php 2009-04-24 20:29:35 UTC (rev 6124) @@ -1,42 +1,27 @@ <?php use_helper('DateForm','I18N','Validation') ?> - - <script> - function linkTo() { - var obj = document.getElementById('id'); - location.href = "<?php echo url_for('ciclolectivo/agregarTurnosYPeriodos', false);?>/id/"+obj.options[obj.selectedIndex].value; - } - </script> +<script> + function linkTo() { + var obj = document.getElementById('id'); + location.href = "<?php echo url_for('ciclolectivo/agregarTurnosYPeriodos', false);?>/id/"+obj.options[obj.selectedIndex].value; + } +</script> + <div id="sf_admin_container"> +<?php include_partial('global/flashes')?> <?php if(count($optionsCiclolectivo) > 0) { ?> <?php echo form_tag('ciclolectivo/grabarTurnosYPeriodos', 'onSubmit="selectItem()"'); ?> <div id="content"> -<br> +<br> <h1>Ciclo Lectivo <?php echo select_tag('id', options_for_select($optionsCiclolectivo, $sf_params->get('id')), 'onChange=linkTo()'); ?></h1> <div id="sf_admin_header"></div> <div id="sf_admin_content"> - -<?php if ($sf_request->hasErrors()): ?> - <div class="form-errors"> - <h2><?php echo __('There are some errors that prevent the form to validate') ?></h2> - <ul> - <?php foreach ($sf_request->getErrorNames() as $name): ?> - <li><?php echo $sf_request->getError($name) ?></li> - <?php endforeach; ?> - </ul> - </div> -<?php elseif ($sf_user->hasFlash('notice')): ?> - <div class="save-ok"> - <h2><?php echo __($sf_user->getFlash('notice')) ?></h2> - </div> -<?php endif; ?> - <table cellspacing="0" class="sf_admin_list"> <thead> @@ -50,14 +35,14 @@ <tbody> <tr class="sf_admin_row_0"> <td> - <div class="content<?php if ($sf_request->hasError('ciclolectivo{descripcion}')): ?> form-error<?php endif; ?>"> - <?php if ($sf_request->hasError('ciclolectivo{descripcion}')): ?> - <?php echo form_error('ciclolectivo{descripcion}', array('class' => 'form-error-msg')) ?> - <?php endif; ?> + <div class="content<?php if ($sf_request->hasError('ciclolectivo{descripcion}')): ?> form-error<?php endif; ?>"> + <?php if ($sf_request->hasError('ciclolectivo{descripcion}')): ?> + <?php echo form_error('ciclolectivo{descripcion}', array('class' => 'form-error-msg')) ?> + <?php endif; ?> <?php echo input_tag('ciclolectivo[descripcion]',$ciclolectivo->getDescripcion());?> - </div> + </div> </td> - <td> + <td> <div class="content<?php if ($sf_request->hasError('ciclolectivo{fecha_inicio}')): ?> form-error<?php endif; ?>"> <?php if ($sf_request->hasError('ciclolectivo{fecha_inicio}')): ?> @@ -69,7 +54,7 @@ </td> - <td> + <td> <div class="content<?php if ($sf_request->hasError('ciclolectivo{fecha_fin}')): ?> form-error<?php endif; ?>"> <?php if ($sf_request->hasError('ciclolectivo{fecha_fin}')): ?> @@ -91,9 +76,9 @@ </div> </th> </tr> -</tfoot> - -</table> +</tfoot> + +</table> <br> @@ -125,15 +110,15 @@ </td> </tr> <?php $i++; }?> - + <tr class="sf_admin_row_0"> <td><?php echo input_tag("turnos[$i][descripcion]",'');?></td> <td><?php echo select_time_tag("turnos[$i][hora_inicio]",array() , array('include_second' => false, '12hour_time' => true));?></td> - <td><?php echo select_time_tag("turnos[$i][hora_fin]", array(), array('include_second' => false, '12hour_time' => true));?></td> + <td><?php echo select_time_tag("turnos[$i][hora_fin]", array(), array('include_second' => false, '12hour_time' => true));?></td> <td></td> - </tr> - - + </tr> + + </tbody> <tfoot> <tr> @@ -144,7 +129,7 @@ </div> </th> </tr> -</tfoot> +</tfoot> </table> @@ -163,7 +148,7 @@ </thead> <tbody> <?php - $i = 0; + $i = 0; foreach($aPeriodo as $periodo){ ?> <tr class="sf_admin_row_0"> @@ -176,37 +161,37 @@ <li><?php echo link_to(image_tag(sfConfig::get('sf_admin_web_dir').'/images/delete_icon.png', array('alt' => __('delete'), 'title' => __('delete'))), 'ciclolectivo/deletePeriodo?idPeriodo='.$periodo->getId().'&id='.$sf_params->get('id')); ?> </ul> </td> - + </tr> <?php $i++; }?> - + <tr class="sf_admin_row_0"> <td><?php echo input_tag("periodo[$i][descripcion]",'');?></td> <td><?php echo input_date_tag("periodo[$i][fecha_inicio]",'',"rich=true calendar_button_img=/sf/sf_admin/images/date.png");?></td> <td><?php echo input_date_tag("periodo[$i][fecha_fin]",'',"rich=true calendar_button_img=/sf/sf_admin/images/date.png");?></td> <td></td> - </tr> - + </tr> + </tbody> <tfoot> <tr> <th colspan="9"> </th> </tr> - </tfoot> + </tfoot> </table> <div> <ul class="sf_admin_actions"> <li> - <?php echo submit_tag('submit', 'class=sf_admin_action_save value=Grabar'); ?> + <?php echo submit_tag('submit', 'class=sf_admin_action_save value=Grabar'); ?> </li> <li><?php echo button_to("Listado de Ciclo Lectivos","ciclolectivo/list",'class=sf_admin_action_list')?></li> </ul> </div> - + </form> <?php } ?> -</div> \ No newline at end of file +</div> Modified: prosistem/alba/trunk/alba/apps/principal/modules/ciclolectivo/validate/grabarTurnosYPeriodos.yml =================================================================== --- prosistem/alba/trunk/alba/apps/principal/modules/ciclolectivo/validate/grabarTurnosYPeriodos.yml 2009-04-24 20:29:17 UTC (rev 6123) +++ prosistem/alba/trunk/alba/apps/principal/modules/ciclolectivo/validate/grabarTurnosYPeriodos.yml 2009-04-24 20:29:35 UTC (rev 6124) @@ -1,9 +1,9 @@ methods: - post: + post: - "ciclolectivo{descripcion}" - "ciclolectivo{fecha_inicio}" - "ciclolectivo{fecha_fin}" - + names: ciclolectivo{descripcion}: required: Yes @@ -18,8 +18,8 @@ required: Yes required_msg: El campo fecha fin es obligatorio validators: lastDay - - + + caracteresMaximos255: class: sfStringValidator param: @@ -34,12 +34,12 @@ compare: ciclolectivo[fecha_fin] operator: "<" compare_error: La fecha inicio debe ser anterior a la fecha fin - + lastDay: class: albaDateValidator param: - date_error: EL campo debe ser una fecha + date_error: El campo debe ser una fecha compare: ciclolectivo[fecha_inicio] operator: ">" compare_error: La fecha fin debe ser posterior a la fecha inicio - + Modified: prosistem/alba/trunk/alba/apps/principal/templates/_flashes.php =================================================================== --- prosistem/alba/trunk/alba/apps/principal/templates/_flashes.php 2009-04-24 20:29:17 UTC (rev 6123) +++ prosistem/alba/trunk/alba/apps/principal/templates/_flashes.php 2009-04-24 20:29:35 UTC (rev 6124) @@ -5,3 +5,13 @@ <?php if ($sf_user->hasFlash('error')): ?> <div class="error"><?php echo __($sf_user->getFlash('error'), array(), 'sf_admin') ?></div> <?php endif; ?> +<?php if ($sf_request->hasErrors()): ?> +<div class="form-errors"> + <h2><?php echo __('There are some errors that prevent the form to validate') ?></h2> + <ul> + <?php foreach ($sf_request->getErrorNames() as $name): ?> + <li title="<?php echo $name?>"><?php echo $sf_request->getError($name) ?></li> + <?php endforeach; ?> + </ul> +</div> +<?php endif;?> Modified: prosistem/alba/trunk/alba/web/css/main.css =================================================================== --- prosistem/alba/trunk/alba/web/css/main.css 2009-04-24 20:29:17 UTC (rev 6123) +++ prosistem/alba/trunk/alba/web/css/main.css 2009-04-24 20:29:35 UTC (rev 6124) @@ -24,11 +24,11 @@ border: 1px dashed; } /* sobreescribo styles del admin generator */ -ul.sf_admin_actions -{ - margin: 10px; - list-style-type: none; - text-align: left; +ul.sf_admin_actions +{ + margin: 10px; + list-style-type: none; + text-align: left; } .sf_admin_filters { float: left @@ -94,4 +94,26 @@ /* left: 200px; */ /* top: 10px; */ width: 250px; -} \ No newline at end of file +} +/* MAIN ERROR */ +#sf_admin_container div.error +{ + background: #f33 url(../sf/sf_admin/images/error.png) no-repeat 10px 4px; + color: #fff; +} + +#sf_admin_container div.notice { + border-top: 1px solid #ddd; + border-bottom: 1px solid #ddd; + background: #ffc url(../sf/sf_admin/images/tick.png) no-repeat 10px 4px; + color: #222; +} +/* COMMON */ +#sf_admin_container div.error, +#sf_admin_container div.notice { + margin: 10px 0; + padding: 7px 0 7px 35px; + font-size: 1.2em; + font-weight: bold; +} +/**/