[Alba-desarrollo] SVN Alba r6124 - in prosistem/alba/trunk/alba: apps/principal/modules/ciclolectivo/actions apps/principal/modules/ciclolectivo/templates apps/principal/modules/ciclolectivo/validate apps/principal/templates web/css

commits en pressenter.com.ar commits en pressenter.com.ar
Vie Abr 24 22:29:36 CEST 2009


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;
+}
+/**/




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