Merge lp:~inddiana/diana-addons/imp_he_biostar into lp:diana-addons/6.0

[Mauricio Jose Sanchez Puerta]

cambiado todos los read por browse en los archivos horas_marcaje_wizard.py, consultar_retardos_wizard.py
para evitar errores en memoria

[lp:~inddiana/diana-addons/imp_he_biostar]

76. By Mauricio Sanchez <email address hidden> on 2013-09-03

eliminado todos los print

=== modified file 'diana_integracion_biostar/model/horas_extras.py'
--- diana_integracion_biostar/model/horas_extras.py	2013-05-13 11:40:26 +0000
+++ diana_integracion_biostar/model/horas_extras.py	2013-09-03 16:22:03 +0000
@@ -145,13 +145,15 @@
     _description = 'horas_extra'
     _columns = {
         'empleado_id': fields.many2one('hr.employee', 'Trabajador', required=True),
-        'fecha': fields.date('Fecha', required=True,),
+        #~ 'fecha': fields.date('Fecha', required=True,),
         'state': fields.selection([('draft','Borrardor'), ('confirmed','Confirmado'),('revised','Revisado'), ('exception','Excepcion'), ('done','Listo'), ('cancel','Cancelado')],'Estado', readonly=True),
-        'hora_desde': fields.time('Hora desde', required=False, readonly=True, states={'draft':[('readonly',False)]}),
-        'hora_hasta': fields.time('Hora hasta', required=True, readonly=True, states={'draft':[('readonly',False)]}),
+        #~ 'hora_desde': fields.time('Hora desde', required=False, readonly=True, states={'draft':[('readonly',False)]}),
+        'hora_desde': fields.datetime('Hora desde', required=False, readonly=True, states={'draft':[('readonly',False)]}),
+        #~ 'hora_hasta': fields.time('Hora hasta', required=True, readonly=True, states={'draft':[('readonly',False)]}),
+        'hora_hasta': fields.datetime('Hora hasta', required=True, readonly=True, states={'draft':[('readonly',False)]}),
         'horas_extras_id': fields.many2one('horas.extras', 'Horas extra',readonly=True, required=True),
-        'hora_marcaje_entrada': fields.time('Hora de marcaje entrada', readonly=True), 
-        'hora_marcaje': fields.time('Hora de marcaje', readonly=True), 
+        'hora_marcaje_entrada': fields.datetime('Hora de marcaje entrada', readonly=True), 
+        'hora_marcaje': fields.datetime('Hora de marcaje salida', readonly=True), 
         'create_uid': fields.integer('create uid'), 
         'observacion': fields.text('Observacion', readonly=True),
     }
@@ -162,8 +164,8 @@
     } 
   
     
-    
     def _check_fecha(self, cr, uid, ids, context=None):
+            formato = '%Y-%m-%d %H:%M:%S'
             hel_brw = self.browse(cr, uid, ids)[0]
             periodos_brw = hel_brw.horas_extras_id.concepto_id.hr_process_id.hr_period_ids
             payment_calendar_id = hel_brw.empleado_id.hr_payroll_id.hr_payment_calendar_id.id
@@ -171,8 +173,10 @@
             for i in periodos:
                 perid_obj = self.pool.get('hr.period')
                 brw = perid_obj.browse(cr, uid, i.id-1)
-                if hel_brw.fecha <= brw.date_end and hel_brw.fecha >= brw.date_start:
-                    return True 
+                date = datetime.datetime.strptime(hel_brw.hora_desde, formato)
+                fecha = date.strftime(formato[0:8])
+                if fecha <= brw.date_end and fecha >= brw.date_start:
+                   return True 
                     
             return False 
     
@@ -180,12 +184,26 @@
         hel_brw = self.browse(cr, uid, ids)
         for i in hel_brw:
             if i.hora_desde:
-               if datetime.datetime.strptime(i.hora_hasta, "%H:%M:%S") <= datetime.datetime.strptime(i.hora_desde, "%H:%M:%S"):
+               if datetime.datetime.strptime(i.hora_hasta, "%Y-%m-%d %H:%M:%S") <= datetime.datetime.strptime(i.hora_desde, "%Y-%m-%d %H:%M:%S"):
                   return False
-        return True             
+        return True    
+    
+    def _check_sobretiempo(self, cr, uid, ids, conetxt=None):
+        formato = '%Y-%m-%d %H:%M:%S'
+        hel_brw = self.browse(cr, uid, ids)    
+        for i in hel_brw:
+            dif = abs(datetime.datetime.strptime(i.hora_hasta, formato) - datetime.datetime.strptime(i.hora_desde, formato))
+            hours = dif.days * 24 
+            if hours > 24 :
+                return False 
+        return True         
+             
+                     
                 
-    _constraints = [(_check_fecha,'La fecha no pertenece al anterior periodo', ['fecha'])]
+    _constraints = [(_check_fecha,'La fecha no pertenece al periodo anterior', ['hora_hasta'])]
     _constraints = [(_check_hora,'La hora desde no puede ser mayor que la hora hasta', ['hora_desde'])]
+    _constraints = [(_check_sobretiempo,'La difencia entre las horas no puede ser mayor de 24 horas', ['hora_desde'])]
+
     
    
     
@@ -234,16 +252,18 @@
         ih = ids
        
         for i in t_brw:
-            fh = i.fecha
             ha = i.hora_hasta 
             hd = i.hora_desde 
             cod = i.empleado_id.cod_number
+            fh_hasta = self.format_date(i.hora_hasta )
+            
             
             if hd:
-               self.query2(cr, uid, i.id, cod, fh, hd, ha, t, ih, context = None)
+               fh_desde = self.format_date(hd)
+               self.query2(cr, uid, i.id, cod, fh_desde[0], fh_desde[1], fh_hasta[1], t, ih,  fh_hasta[0],context = None)
            
             else:          
-                self.query(cr, uid, i.id, cod, fh, ha, t, ih, context = None)
+                self.query(cr, uid, i.id, cod, fh_hasta[0], fh_hasta[1], t, ih, context = None)
         
         self.review(cr, uid, ids, context = None) 
     
@@ -275,7 +295,16 @@
           raise osv.except_osv('ERROR', "Fallo de conexión") 
         
            
-    
+    def format_date(self, fecha):
+        formato = '%Y-%m-%d %H:%M:%S'
+        date = datetime.datetime.strptime(fecha, formato)
+        hora = date.strftime(formato[9:])
+        fecha = date.strftime(formato[0:8])
+        return [fecha, hora]
+        
+
+
+        
     def query(self, cr, uid, ids, cod, fh, hora_hasta, t, ids_horas_extras, context = None):
         
         """
@@ -315,7 +344,7 @@
           raise osv.except_osv('ERROR',err)
            
         for row in rows:
-            over = self.add_overtime(cr, uid, ids, row[0], hora_hasta, context = None)
+            over = self.add_overtime(cr, uid, ids, row[0], hora_hasta, fh, context = None)
             
         self.write(cr, uid,  ids_horas_extras, {'state': 'revised', }) 
         return True 
@@ -324,7 +353,7 @@
     # sql nueva Funcion fue agregada para poder hacer validaciones 
     # contra el biometrico cuando existe informacion del campo hora_desde 
     
-    def query2(self, cr, uid, ids, cod, fecha, hora_desde, hora_hasta, t, ih, context = None):
+    def query2(self, cr, uid, ids, cod, fecha_inicio, hora_desde, hora_hasta, t, ih, fecha_fin, context = None):
         
         """
         Funcion para ejecutar la consulta en el BioStar si ahi una hora desde
@@ -346,10 +375,10 @@
         JOIN TB_READER AS READER ON EVENT.nReaderIdn = READER.nReaderIdn JOIN TB_EVENT_DATA AS E_DATA ON EVENT.nEventIdn = E_DATA.nEventIdn 
         JOIN TB_USER_CUSTOMINFO As C ON C.nUserIdn = U.nUserIdn
         WHERE E_DATA.nEventIdn = 55
-        AND CONVERT(nVarchar(10), DATEADD(s, EVENT.nDateTime, '1970-01-01'), 20) = '%s'
+        AND CONVERT(nVarchar(10), DATEADD(s, EVENT.nDateTime, '1970-01-01'), 20)  BETWEEN '%s' and '%s'
         AND READER.nReaderIdn IN %s 
         AND C.sFieldValue5 =  %s
-        """ %(fecha, t, cod)
+        """ %(fecha_inicio, fecha_fin, t, cod)
         
     
         cnx = self.conexion(cr, uid, ids, context = None)
@@ -365,7 +394,7 @@
           raise osv.except_osv('ERROR',err)
            
         for row in rows:
-            validar = self.validar_horas(cr, uid, ids, row, hora_desde, hora_hasta, context=None)
+            validar = self.validar_horas(cr, uid, ids, row, hora_desde, hora_hasta, fecha_inicio, fecha_fin, context=None)
             
         self.write(cr, uid, ih, {'state': 'revised', }) 
         
@@ -373,7 +402,7 @@
     #Esta funcion fue agregada el 09/05/2013 para hacer comparacion de horas 
     #si el campo hora_hasta posee datos     
     
-    def validar_horas(self, cr, uid, ids, h_sql, h_desde, h_hasta, context = None):    
+    def validar_horas(self, cr, uid, ids, h_sql, h_desde, h_hasta, fecha_inicio, fecha_fin, context = None):    
         
         """
         Función que valida si las horas del biostar son nulas, si la 
@@ -391,6 +420,8 @@
         obhl_read = objhl.read(cr, uid, ids)
         sts_now = obhl_read.get('state')
         
+        entrada = "%s %s"%(fecha_inicio, h_sql[0])
+        salida = "%s %s"%(fecha_fin, h_sql[1])
         
         if  sts_now != 'cancel' and  sts_now != 'done':
             
@@ -400,15 +431,15 @@
                return False 
             
             elif  self.unlike_hours(cr, uid, h_sql[0], h_desde) == False:
-                  objhl.write(cr, uid, ids, {'hora_marcaje_entrada': h_sql[0], 'hora_marcaje': h_sql[1], 'state': 'exception',})
+                  objhl.write(cr, uid, ids, {'hora_marcaje_entrada': entrada, 'hora_marcaje': salida, 'state': 'exception',})
                   return False
             
             elif  self.unlike_hours(cr, uid, h_sql[1], h_hasta) == False:
-                  objhl.write(cr, uid, ids, {'hora_marcaje_entrada': h_sql[0], 'hora_marcaje': h_sql[1], 'state': 'exception',})
+                  objhl.write(cr, uid, ids, {'hora_marcaje_entrada': entrada, 'hora_marcaje': salida, 'state': 'exception',})
                   return False
                   
             else:    
-                objhl.write(cr, uid, ids, {'hora_marcaje_entrada': h_sql[0], 'hora_marcaje': h_sql[1], 'state': 'done',})
+                objhl.write(cr, uid, ids, {'hora_marcaje_entrada': entrada, 'hora_marcaje': salida, 'state': 'done',})
                 return True 
         
         return True
@@ -416,7 +447,7 @@
         
                 
     
-    def add_overtime(self, cr, uid, ids, hr, ha, context = None):
+    def add_overtime(self, cr, uid, ids, hr, ha, fecha, context = None):
         """
         Función que valida si las horas del biostar son nulas, si la 
         diferencia de horas no sale del rango y cambio los estados 
@@ -431,6 +462,7 @@
         obhl_read = objhl.read(cr, uid, ids)
         sts_now = obhl_read.get('state')
         
+        formato = '%s %s'
         if  sts_now != 'cancel' and  sts_now != 'done':
                    
             if hr == None:
@@ -438,11 +470,11 @@
                return False 
                
             elif  self.unlike_hours(cr, uid, hr, ha) == False:
-                  objhl.write(cr, uid, ids, {'hora_marcaje': hr, 'state': 'exception',})
+                  objhl.write(cr, uid, ids, {'hora_marcaje': formato%(fecha, hr), 'state': 'exception',})
                   return False
                   
             else:    
-                objhl.write(cr, uid, ids, {'hora_marcaje': hr, 'state': 'done',})
+                objhl.write(cr, uid, ids, {'hora_marcaje': formato%(fecha, hr), 'state': 'done',})
                 return True
         
     def unlike_hours(self, cr, uid, hr, ha):
@@ -493,6 +525,9 @@
     def convert_hours(self, hours):
         return datetime.datetime.strptime(hours, '%H:%M:%S')
     
+    def convert_datetime(self, date):
+        return datetime.datetime.strptime(date, '%Y-%m-%d %H:%M:%S')    
+    
     def turn_validate(self, date, h):
         """
         Funcion que valida si estas dentro de tu horario 
@@ -502,15 +537,15 @@
         @return False: si estas Fuera de tu turno
         """
         hr = self.convert_hours(h) 
-        hr0 = self.convert_hours(date[0]) 
-        hr1 = self.convert_hours(date[1])
-        if hr1 > hr0:
-          if hr >= hr0 and hr <= hr1:
+        hr_entrada = self.convert_hours(date[0]) 
+        hr_salida = self.convert_hours(date[1])
+        if hr_salida > hr_entrada:
+          if hr >= hr_entrada and hr <= hr_salida:
              return True
         else:
-            if hr >= hr1 and hr >= hr0:
+            if hr >= hr_salida and hr >= hr_entrada:
                return True 
-            if  hr <= hr0 and hr <= hr1:
+            if  hr <= hr_entrada and hr <= hr_salida:
                 return True      
         return False 
         
@@ -530,8 +565,7 @@
             return total
              
             
-            
-        
+              
         
     def review(self, cr, uid, ids, context = None):
         horas_extras = self.browse(cr, uid, ids)
@@ -540,10 +574,10 @@
         horas_extras_lineas = horas_extras[0].lineas_ids
     
         for i in horas_extras_lineas:
-            fh = i.fecha
-            ha = i.hora_hasta
+            fh = self.format_date(i.hora_hasta )
+            ha = fh[1]
             cod = i.empleado_id.cod_number
-            date = self.get_date_year(fh, i.empleado_id.shift_id)
+            date = self.get_date_year(fh[0], i.empleado_id.shift_id)
            
              
             if self.turn_validate(date, ha):
@@ -555,6 +589,13 @@
                return False 
                
               
+    
+    def search_schedule_concept(self, cr, uid, ids, objeto, hora):
+        tur_id = objeto.search(cr, uid, ['&', ('hour_start','<=', hora) , ('hour_end','>=', hora)])
+        tur_brw = objeto.browse(cr, uid, tur_id)[0].concepto_id.id
+        #~ tur_brw = objeto.browse(cr, uid, tur_id)[0].concepto_id.id
+        return tur_brw
+          
                 
     
     def ready(self, cr, uid, ids, context = None):            
@@ -565,60 +606,75 @@
         
         horas_extras_lineas = horas_extras[0].lineas_ids
         variable_id = horas_extras[0].concepto_id.id
-        
+        args = "{'employee_id':  employee_id, 'value': '', 'payroll_id': False, 'variable_concept_id':   variable_id  , 'type': 'qty', 'concept_id': ''}"
         
         
         for i in horas_extras_lineas:
-            fh = i.fecha
-            ha = i.hora_hasta
+            datos = self.format_date(i.hora_hasta)
+            fh = datos[0]
+            ha = datos[1]
             hd = i.hora_desde
+            
             cod = i.empleado_id.cod_number
             employee_id = i.empleado_id.id
             date = self.get_date_year(fh, i.empleado_id.shift_id)
-          
-            
-            if i.state == 'exception':
-               raise osv.except_osv('ERROR', "No puedes insertar la linea si tienes lineas en execepcion")  
-            
-        
-            hora_markaje = self.convert_hours(ha) 
+            hora_turno_entrada = self.convert_hours(date[0]) # variable que tiene la hora real de entrada del trabajador en formato datetime.datetime
+            hora_turno_salida = self.convert_hours(date[1]) # variable que tiene la hora de salida real del trabajador en formato datetime.datetime
             tur_id = turn_obj.search(cr, uid, ['&', ('hour_start','<=', date[1]) , ('hour_end','>=', date[1])])
-            tur_brw = turn_obj.browse(cr, uid, tur_id)
+            tur_brw = turn_obj.browse(cr, uid, tur_id)[0]
             hora_ext_id = turn_obj.search(cr, uid, ['&', ('hour_start','<=', ha) , ('hour_end','>=', ha)])
             hora_ext_brw =  turn_obj.browse(cr, uid, hora_ext_id)
+            args = eval(args)
+            
+            if i.state == 'exception':
+               raise osv.except_osv('ERROR', "No puedes insertar la linea si tienes lineas en execepcion")  
             
             if hd:
+               datos = self.format_date(hd)
+               fd = datos[0]
+               hd = datos[1]
                hora_markaje_entrada = self.convert_hours(hd)
+               hora_markaje_salida = self.convert_hours(ha)
                hora_entrada_antes_id = turn_obj.search(cr, uid, ['&', ('hour_start','<=', hd) , ('hour_end','>=', hd)])
-               hora_entrada_antes_brw = turn_obj.browse(cr, uid, hora_entrada_antes_id)
-               if hora_markaje_entrada < self.convert_hours(date[0]):
-                  dif =  self.subtract_hours(hd, date[0], 3600)
+               hora_entrada_antes_brw = turn_obj.browse(cr, uid, hora_entrada_antes_id)[0]
+              
+               dif =  self.subtract_hours(hd, date[0], 3600)
+               args['value'] = dif 
+               args['concept_id'] = hora_entrada_antes_brw.concepto_id.id 
+               
+               if hora_markaje_entrada < hora_turno_entrada and fd == fh and  hora_markaje_salida == hora_turno_salida:
+                  
                   if dif > 0:
-                     concept_obj.create(cr, uid, {'employee_id':  employee_id, 'value': dif, 'payroll_id': False, 'variable_concept_id':   variable_id  , 'type': 'qty', 'concept_id':  hora_entrada_antes_brw[0].concepto_id.id} , context=None)             
-
+                     concept_obj.create(cr, uid, args, context=None) 
+                     self.write(cr, uid, ids, {'state': 'done', })            
+                  return True 
+               else:
+                   if dif > 0:
+                      concept_obj.create(cr, uid, args, context=None) 
+                    
             if not self.turn_validate(date, ha): #Valida que estes fuera de tu horario de trabajo
             
-               if hora_markaje > self.convert_hours(tur_brw[0].hour_end):
-                  dif1 =  self.subtract_hours(tur_brw[0].hour_end, date[1], 3600)
-                  dif2 =  self.subtract_hours(ha, tur_brw[0].hour_end, 3600)
+               if hora_markaje_salida > self.convert_hours(tur_brw.hour_end):
+                  dif1 =  self.subtract_hours(tur_brw.hour_end, date[1], 3600)
+                  dif2 =  self.subtract_hours(ha, tur_brw.hour_end, 3600)
                   if dif1 > 0: 
-                     concept_obj.create(cr, uid, {'employee_id':  employee_id, 'value': dif1, 'payroll_id': False, 'variable_concept_id':   variable_id  , 'type': 'qty', 'concept_id':  tur_brw[0].concepto_id.id} , context=None)  
+                     concept_obj.create(cr, uid, {'employee_id':  employee_id, 'value': dif1, 'payroll_id': False, 'variable_concept_id':   variable_id  , 'type': 'qty', 'concept_id':  tur_brw.concepto_id.id} , context=None)  
                   if dif2 > 0:
                      concept_obj.create(cr, uid, {'employee_id':  employee_id, 'value': dif2, 'payroll_id': False, 'variable_concept_id':   variable_id  , 'type': 'qty', 'concept_id':  hora_ext_brw[0].concepto_id.id} , context=None)  
                   #~ return True 
                 
-               elif hora_markaje > self.convert_hours(date[1]) and  hora_markaje < self.convert_hours(tur_brw[0].hour_end):
+               elif hora_markaje_salida > self.convert_hours(date[1]) and  hora_markaje_salida < self.convert_hours(tur_brw.hour_end):
                     dif = self.subtract_hours(ha, date[1], 3600)
                     if dif > 0:
                        concept_obj.create(cr, uid, {'employee_id':  employee_id, 'value': dif, 'payroll_id': False, 'variable_concept_id':   variable_id  , 'type': 'qty', 'concept_id':  hora_ext_brw[0].concepto_id.id} , context=None)  
                     #~ return True 
                 
-               elif hora_markaje < self.convert_hours(date[0]) : 
-                     dif1 =  self.subtract_hours(tur_brw[0].hour_end, date[1], 3600)
-                     dif2 =  self.subtract_hours(tur_brw[0].hour_end, '23:59:59', 3600)
+               elif hora_markaje_salida < self.convert_hours(date[0]) : 
+                     dif1 =  self.subtract_hours(tur_brw.hour_end, date[1], 3600)
+                     dif2 =  self.subtract_hours(tur_brw.hour_end, '23:59:59', 3600)
                      dif3 =  self.subtract_hours(ha, '00:00:00', 3600)
                      if dif1 > 0:
-                        concept_obj.create(cr, uid, {'employee_id':  employee_id, 'value': dif1, 'payroll_id': False, 'variable_concept_id':   variable_id  , 'type': 'qty', 'concept_id': tur_brw[0].concepto_id.id} , context=None)  
+                        concept_obj.create(cr, uid, {'employee_id':  employee_id, 'value': dif1, 'payroll_id': False, 'variable_concept_id':   variable_id  , 'type': 'qty', 'concept_id': tur_brw.concepto_id.id} , context=None)  
                      if dif2 > 0:
                         concept_obj.create(cr, uid, {'employee_id':  employee_id, 'value': dif2, 'payroll_id': False, 'variable_concept_id':   variable_id  , 'type': 'qty', 'concept_id':  hora_ext_brw[0].concepto_id.id} , context=None)  
                      if dif3 > 0:
@@ -629,15 +685,18 @@
             
             else:
                 if self.convert_hours(date[1]) > self.convert_hours(date[0]):
-                    dif1 =  self.subtract_hours(tur_brw[0].hour_end, date[1], 3600)
-                    dif2 =  self.subtract_hours(tur_brw[0].hour_end, '23:59:59', 3600)
+                    dif1 =  self.subtract_hours(tur_brw.hour_end, date[1], 3600)
+                    dif2 =  self.subtract_hours(tur_brw.hour_end, '23:59:59', 3600)
+                    conced1 = self.search_schedule_concept(cr, uid, ids, turn_obj,  '23:59:59')
                     dif3 =  self.subtract_hours(date[0], '00:00:00', 3600)
+                    conced2 = self.search_schedule_concept(cr, uid, ids, turn_obj,  '00:00:00')
+
                     if dif1 > 0:
-                       concept_obj.create(cr, uid, {'employee_id':  employee_id, 'value': dif1, 'payroll_id': False, 'variable_concept_id':   variable_id  , 'type': 'qty', 'concept_id': tur_brw[0].concepto_id.id} , context=None)  
+                       concept_obj.create(cr, uid, {'employee_id':  employee_id, 'value': dif1, 'payroll_id': False, 'variable_concept_id':   variable_id  , 'type': 'qty', 'concept_id': tur_brw.concepto_id.id} , context=None)  
                     if dif2 > 0:
-                       concept_obj.create(cr, uid, {'employee_id':  employee_id, 'value': dif2, 'payroll_id': False, 'variable_concept_id':   variable_id  , 'type': 'qty', 'concept_id':  hora_ext_brw[0].concepto_id.id} , context=None)  
+                       concept_obj.create(cr, uid, {'employee_id':  employee_id, 'value': dif2, 'payroll_id': False, 'variable_concept_id':   variable_id  , 'type': 'qty', 'concept_id': conced1} , context=None)  
                     if dif3 > 0:
-                       concept_obj.create(cr, uid, {'employee_id':  employee_id, 'value': dif3, 'payroll_id': False, 'variable_concept_id':   variable_id  , 'type': 'qty', 'concept_id':  hora_ext_brw[0].concepto_id.id} , context=None)  
+                       concept_obj.create(cr, uid, {'employee_id':  employee_id, 'value': dif3, 'payroll_id': False, 'variable_concept_id':   variable_id  , 'type': 'qty', 'concept_id':   conced2} , context=None)  
                     #~ return True                 
                 else:
                     tur_hourn_id = turn_obj.search(cr, uid, ['&', ('hour_start','>=', date[1]) , ('hour_end','<=', date[0])])
@@ -648,7 +707,9 @@
                        concept_obj.create(cr, uid, {'employee_id':  employee_id, 'value': dif1, 'payroll_id': False, 'variable_concept_id':   variable_id  , 'type': 'qty', 'concept_id': tur_hourn_brw[0].concepto_id.id} , context=None) 
                     if dif2 > 0:
                        concept_obj.create(cr, uid, {'employee_id':  employee_id, 'value': dif2, 'payroll_id': False, 'variable_concept_id':   variable_id  , 'type': 'qty', 'concept_id':  hora_ext_brw[0].concepto_id.id} , context=None)  
-                    #~ return True
+                    #~ return True  
+          
+          
                 
         self.write(cr, uid, ids, {'state': 'done', })            
         return True      

=== modified file 'diana_integracion_biostar/view/horas_extras_view.xml'
--- diana_integracion_biostar/view/horas_extras_view.xml	2013-05-09 19:03:07 +0000
+++ diana_integracion_biostar/view/horas_extras_view.xml	2013-09-03 16:22:03 +0000
@@ -15,7 +15,9 @@
           <field name="lineas_ids" nolabel="1" colspan='4' attrs= "{'readonly':[('state','!=',  'draft')]}" >
               <tree editable='top'>
             <field name='empleado_id'  domain="[('state', 'not in', ['borrador', 'egresado'])]" />
+<!--
             <field name='fecha' />
+-->
             <field name='hora_desde'/>
             <field name='hora_hasta'/>
             <field name='hora_marcaje_entrada' />
@@ -76,7 +78,9 @@
     <field name="arch" type="xml">
       <tree string="Lineas del las horas extra">
         <field name="empleado_id" select="1" />
+<!--
         <field name="fecha" select="2" />
+-->
         <field name="hora_hasta" />
         <field name='state' />
       </tree>

=== modified file 'diana_integracion_biostar/wizard/consultar_retardos_view.xml'
--- diana_integracion_biostar/wizard/consultar_retardos_view.xml	2013-06-06 19:34:49 +0000
+++ diana_integracion_biostar/wizard/consultar_retardos_view.xml	2013-09-03 16:22:03 +0000
@@ -23,8 +23,16 @@
       <field name="view_mode">form</field>
       <field name="target">new</field>
     </record>
+<!--
     <menuitem name="PCP" id="consultar_marcaje" parent="hr.menu_hr_root" sequence="18" />
+-->
+<!--
     <menuitem name="Consultar retardos" id="Consultar_marcaje_pcp" action="consultar_marcaje_next_action" parent="consultar_marcaje" sequence="18" />
+-->
+<!--
+    <menuitem name="Consultar retardos" id="Consultar_marcaje_pcp" action="consultar_marcaje_next_action" parent="hr_holidays.menu_open_ask_holidays" sequence="25" />
+-->
+    <menuitem name="Consultar retardos" id="Consultar_marcaje_pcp" action="consultar_marcaje_next_action" parent="hr_holidays.menu_open_ask_holidays" sequence="25" />
     <record id="view_revisar_entrada_lineas_wizard" model="ir.ui.view">
       <field name="name">revisar.entrada.lineas.wizard.form</field>
       <field name="model">revisar.entrada.lineas.wizard</field>

=== modified file 'diana_integracion_biostar/wizard/consultar_retardos_wizard.py'
--- diana_integracion_biostar/wizard/consultar_retardos_wizard.py	2013-07-01 19:17:55 +0000
+++ diana_integracion_biostar/wizard/consultar_retardos_wizard.py	2013-09-03 16:22:03 +0000
@@ -5,6 +5,7 @@
 import datetime
 from datetime import timedelta
 import time 
+import calendar
 
 class revisar_entrada_wizard(osv.osv_memory):
     _name = 'revisar.entrada.wizard'
@@ -33,7 +34,6 @@
         if context != None:
            res = {'fecha_desde':context['fecha_desde'], 'fecha_hasta': context['fecha_hasta'], 'state':context['state'], 'mensaje':context['mensaje'],'revisar_line_ids':context['revisar_line_ids']}
         else:
-            #~ context = {}
             res = {}
         return res  
         
@@ -75,7 +75,10 @@
    
     def consultar(self, cr, uid, ids, context=None):
         data = self.read(cr, 1, ids)[0]
-        con = self.conexion(cr, uid, ids, context=None)
+        try:
+            con = self.conexion(cr, uid, ids, context=None)
+        except:
+            raise osv.except_osv('ERROR', 'Conexion fallida' )
         cod = data['codigo']
         driver = self.reader_id(cr, uid, ids)
         if driver:
@@ -83,7 +86,7 @@
         else:
             raise osv.except_osv('ERROR', 'No existen capta huellas o no a seleccionado ninguno en la configuracion')        
                
-        cod_id = self.pool.get('hr.employee').search(cr, uid, [('cod_number', '=', cod)])
+        cod_id = self.pool.get('hr.employee').search(cr, uid, [('cod_number', 'like', cod)])
         cod_obj = self.pool.get('hr.employee').browse(cr, uid,  cod_id)
         
         try:
@@ -94,58 +97,72 @@
         
         dia = datetime.datetime.now()
          
-        #~ dia = datetime.datetime(2013, 04, 30)
-   
         periodo = self.create_period(dia)
-     
-        
-        periodo = [i for i in periodo if self.discount_case(cod_obj[0].shift_id, i)[0] != '00:00:00' and self.discount_case(cod_obj[0].shift_id, i)[1] != '00:00:00']
-        
-     
-           
         fd = periodo[0]
         fh = periodo[-1]
         hmw_id = self.pool.get('revisar.entrada.wizard').create(cr, uid, {'fecha_desde': fd,  'fecha_hasta':fh})
+        date_end = "%s-%s-%s" %(dia.year, dia.month, calendar.monthrange(dia.year, dia.month)[1]) 
+        date_end = datetime.datetime.strptime(date_end, '%Y-%m-%d')
+        periodo2 = self.create_period(date_end)
+      
+        
+        periodo = [i for i in periodo if self.discount_case(cod_obj[0].shift_id, i)[0] != '00:00:00' and self.discount_case(cod_obj[0].shift_id, i)[1] != '00:00:00']
+        if not periodo:
+           periodo2 = [i for i in periodo2 if self.discount_case(cod_obj[0].shift_id, i)[0] == '00:00:00' and self.discount_case(cod_obj[0].shift_id, i)[1] == '00:00:00']
+           periodo2 = str(periodo2)
+           periodo2 = periodo2.replace('[', '')
+           periodo2 = periodo2.replace(']', '')
+           raise osv.except_osv('ERROR', 'El trabajador %s con codigo %s no labora en estas fechas del mes %s'%(cod, nombre, periodo2) )       
+      
+           
+        #~ fd = periodo[0]
+        #~ fh = periodo[-1]
+      
         
         conex = con.cursor()        
         tol = 0
 
         dia = dia.strptime(dia.strftime( '%H:%M:%S'), '%H:%M:%S')
         date = self.discount_case(cod_obj[0].shift_id, dia.strftime("%Y-%m-%d"))  
-        #~ print date 
-        if dia < self.convert_hours(date[0]) + timedelta(minutes=15):
-            
-            h1 = self.convert_hours(date[0])+ timedelta(minutes=5) #esta es la hora normal de entrada a la que se adiciona 15 minutos
+        if dia <= self.convert_hours(date[0]) + timedelta(minutes=15):
+            h1 = self.convert_hours(date[0])+ timedelta(minutes=5) #esta es la hora normal de entrada a la que se adiciona 5 minutos
             h2 = self.convert_hours(date[0])+ timedelta(minutes=15) #esta es la hora normal de entrada a la que se adiciona 15 minutos
             sql = self.query(cod, fd, fh, driver)
-            conex.execute(sql)
+            try:
+                conex.execute(sql)
+            except:
+                raise osv.except_osv('ERROR', 'Conexion fallida' )
+                    
             rows = conex.fetchall()
-            if not rows:
-               raise osv.except_osv('ERROR', 'El codigo %s perteneciente al trabajador no existe en el BioStar'%cod )     
             for row in rows:
                 retardos = self.search_delays(row, h1, h2)
                 if retardos:
                     tol = tol +1  
                     crl = self.pool.get('revisar.entrada.lineas.wizard').create(cr, uid, {'hora':retardos[2], 'fecha':retardos[4],'codigo':cod,'nombre':nombre, 'capta_huella':retardos[0], 'tolerancia': tol,'horas_marcaje_id':hmw_id})
-              
-            brw = self.pool.get('revisar.entrada.wizard').read(cr, 1, ids)
-            brw_line = self.pool.get('revisar.entrada.lineas.wizard').read(cr, 1, ids)
-           
+         
+                 
+            brw = self.pool.get('revisar.entrada.wizard').browse(cr, 1, ids)[0]
+            brw_line = self.pool.get('revisar.entrada.lineas.wizard').browse(cr, 1, ids)
+            
+            cabecera = {'fecha_desde':brw.fecha_desde , 'revisar_line_ids': [i.id for i in brw.revisar_line_ids], 'fecha_hasta': brw.fecha_hasta, 'state': False, 'mensaje': False, 'id': brw.id}
            
             context.update({'active_id': hmw_id, 'active_model':'revisar.entrada.wizard'})
             context.update({'active_model':'revisar.entrada.wizard'})
-            context.update(brw[0])
+            context.update(cabecera)
+            
             
             if tol >= 3:
                context.update({'state':'denied'})
                context.update({'mensaje':'Acceso denegado por exceder su limite de tolerancias mensual'})
             else:
                 context.update({'state':'accepted'})
-                context.update({'mensaje':'usted ya posee  %s tolerancia el maximo aceptado son 3 mensuales' %tol})
+                context.update({'mensaje':'usted posee  %s tolerancia el maximo aceptado son 3 mensuales' %tol})
              
             
-            try:        
-                context.update(brw_line[0])
+            try:
+                line = brw_line[0]
+                lineas = {'hora':line.hora , 'tolerancia':line.tolerancia , 'fecha': line.fecha, 'capta_huella':line.capta_huella , 'horas_marcaje_id': line.horas_marcaje_id.id, 'nombre': line.nombre, 'codigo': line.codigo, 'id': line.id}
+                context.update(lineas)
             except:
                 pass 
         
@@ -154,8 +171,8 @@
             
         
         else:
-             context.update({'fecha_desde': fd,  'fecha_hasta': fh, 'state':'denied' , 'mensaje': 'Acceso denegado a excedido la hora limite de entrada', 'revisar_line_ids':[]})        
-    
+             context.update({'fecha_desde': fd,  'fecha_hasta': fh, 'state':'denied' , 'mensaje': 'Acceso denegado ha excedido la hora limite de entrada', 'revisar_line_ids':[]})        
+    #~ 
         
         conex.close()
         return {
@@ -170,7 +187,7 @@
                     'context': context,
               #~ 
                     }
-        
+        #~ 
         
         
     def reader_id(self, cr, uid, ids):
@@ -203,7 +220,6 @@
     def conexion(self, cr, uid, ids, context=None):
         bio_obj = self.pool.get('horas.extras')
         bio_conect = bio_obj.conexion(cr, uid, ids, context = None)
-        #~ cursor = bio_conect.cursor()  
         return  bio_conect    
         
     def query(self, codigo, fecha_inicio, fecha_fin, captahuellas):
@@ -227,6 +243,7 @@
     def search_delays(self, row, h1, h2):
         #~ consulta = [i for i in row if self.convert_hours(row[2]) >= self.convert_hours(h1) and self.convert_hours(row[2]) < self.convert_hours(h2)]
         consulta = [i for i in row if self.convert_hours(row[2]) >= h1 and self.convert_hours(row[2]) < h2]
+        
         if consulta:
            return  consulta
         

=== modified file 'diana_integracion_biostar/wizard/export_biostar.py'
--- diana_integracion_biostar/wizard/export_biostar.py	2013-06-21 19:07:53 +0000
+++ diana_integracion_biostar/wizard/export_biostar.py	2013-09-03 16:22:03 +0000
@@ -26,13 +26,11 @@
         idsw = [context['active_id']]
         if len(context['active_ids']) > 1:
             idsw = context['active_ids']
-            #~ print idsw
         cod_obj = self.pool.get('hr.employee').browse(cr, uid, idsw)    
         for idt in cod_obj:
             cod = idt.cod_number
             nombre = idt.name
             cedula = idt.identification_id
-            #~ print cod, nombre, cedula
             chk = self.user_check(cr, uid, ids, cod, context=None)
             if chk == None:
                 pass 
@@ -45,16 +43,13 @@
                 raise osv.except_osv('ERROR', 'Conexion fallida')        
             
             for row in rows:
-                #~ print "consulata 1", row 
                 id = int(row[0]) + 1
-                #~ print "id de sUserID",id           
             sql2 = sql2 %(nombre, id)
-            #~ print sql2
             try:
                 conex.execute(sql2) 
             except:
                 raise osv.except_osv('ERROR', 'Conexion fallida')    
-           
+               
             conex.execute(sql3)   
             rows2 = conex.fetchall()
             for row in rows2:
@@ -68,7 +63,6 @@
                 raise osv.except_osv('ERROR', 'Conexion fallida')   
             
             sql5 = sql5%id2  
-            #~ print sql5
             try:
                 conex.execute(sql5) 
             except:
@@ -87,7 +81,6 @@
     def conexion(self, cr, uid, ids, context=None):
         bio_obj = self.pool.get('horas.extras')
         bio_conect = bio_obj.conexion(cr, uid, ids, context = None)
-        #~ cursor = bio_conect.cursor()  
         return bio_conect
     
     def user_check(self, cr, uid, ids, codigo, context=None):    

=== modified file 'diana_integracion_biostar/wizard/horas_extras_execpcion_wizard.py'
--- diana_integracion_biostar/wizard/horas_extras_execpcion_wizard.py	2013-05-13 11:40:26 +0000
+++ diana_integracion_biostar/wizard/horas_extras_execpcion_wizard.py	2013-09-03 16:22:03 +0000
@@ -9,8 +9,8 @@
     _name = 'horas.extras.execepcion.wizard'
     _description = 'excepcion de horas extras lineas'
     _columns = {
-        'hora_desde': fields.time('hora desde'),
-        'hora_hasta': fields.time('hora hasta'),
+        'hora_desde': fields.datetime('hora desde'),
+        'hora_hasta': fields.datetime('hora hasta'),
         'confirmar': fields.boolean('Modificar hora'),
         'observacion': fields.text('Observacion', required=True),
     }
@@ -22,12 +22,15 @@
       
         if  data['confirmar'] == True:
             
-            if data['hora_desde']:
+            if data['hora_desde'] and not data['hora_hasta']:
                self.pool.get('horas.extras.lineas').write(cr, uid, idsw, {'hora_desde':data['hora_desde']})
             
-            elif  data['hora_hasta']:  
+            elif  data['hora_hasta'] and not data['hora_desde']:  
                   self.pool.get('horas.extras.lineas').write(cr, uid, idsw, {'hora_hasta':data['hora_hasta']})
             
+            elif  data['hora_desde'] and data['hora_hasta']:
+                  self.pool.get('horas.extras.lineas').write(cr, uid, idsw, {'hora_hasta':data['hora_hasta'], 'hora_desde':data['hora_desde']})    
+            
             else:
                  raise osv.except_osv('ERROR', "Debe especificar una hora ")   
     

=== modified file 'diana_integracion_biostar/wizard/horas_marcaje_wizard.py'
--- diana_integracion_biostar/wizard/horas_marcaje_wizard.py	2013-07-01 19:17:55 +0000
+++ diana_integracion_biostar/wizard/horas_marcaje_wizard.py	2013-09-03 16:22:03 +0000
@@ -51,8 +51,7 @@
             'active_id': ids
              }
              
-  
-             
+    
     def search_hour(self, cr, uid, ids, context=None):
         data = self.read(cr, 1, ids)[0]
         idsw = [context['active_id']]
@@ -81,15 +80,21 @@
             rows = conex.execute(sql)
             for row in rows:
                 crl = self.pool.get('horas.marcaje.lineas.wizard').create(cr, uid, {'hora':row[0], 'fecha':row[1],'codigo':row[3],'nombre':nombre, 'capta_huella':row[2], 'horas_marcaje_id':hmw_id})
-        brw = self.pool.get('horas.marcaje.wizard').read(cr, 1, ids)
-        brw_line = self.pool.get('horas.marcaje.lineas.wizard').read(cr, 1, ids)
-    
+        brw = self.pool.get('horas.marcaje.wizard').browse(cr, 1, ids)[0]
+        line = self.pool.get('horas.marcaje.lineas.wizard').browse(cr, 1, ids)[0]
+        
+        cabecera = {'fecha_desde':brw.fecha_desde , 'fecha_hasta': brw.fecha_hasta, 'h_line_ids':[i.id for i in brw.h_line_ids] , 'id': brw.id}
+        
         context.update({'active_id': hmw_id, 'active_model':'horas.marcaje.wizard'})
         context.update({'active_model':'horas.marcaje.wizard'})
-        context.update(brw[0])
-     
+        context.update(cabecera)
+        
+        lineas = {'hora':line.hora , 'fecha':line.fecha , 'capta_huella':line.capta_huella, 'horas_marcaje_id':line.horas_marcaje_id.id , 'nombre': line.nombre, 'codigo': line.codigo, 'id': line.id}  
+        
+
+        
         try:        
-            context.update(brw_line[0])
+            context.update(lineas)
         except:
             raise osv.except_osv('ERROR', 'No existen marcajes en el perido desde %s hasta %s' %(fd, fh))        
         
@@ -184,7 +189,7 @@
         'model': 'horas.marcaje.wizard',
             }
        
-        data = self.read(cr, uid, ids)
+        data = self.read(cr, 1, ids)
         data = data[0]
         data.update(d) 
        

=== modified file 'diana_integracion_biostar/wizard/horas_retardo_wizard.py'
--- diana_integracion_biostar/wizard/horas_retardo_wizard.py	2013-06-21 19:07:53 +0000
+++ diana_integracion_biostar/wizard/horas_retardo_wizard.py	2013-09-03 16:22:03 +0000
@@ -37,7 +37,6 @@
         concepto_jornada_no_trabajda_id = data.get('concepto_no_trabjada') 
         concepto_horas_no_trabajadas_id = data.get('concepto_tiempo_no_trabajda') 
        
-        #~ print concepto_id, jornada_no_trabajda, tiempo_no_trabajado
         idsw = [context['active_id']]
         
         hrvc_linea_obj = self.pool.get('hr.variable.concept.line')
@@ -55,7 +54,7 @@
              raise osv.except_osv('ERROR', 'Error al crear Cursor')            
       
         # PERIODO A BUSCAR PARA EL SQL  
-        hrvc_brw =    hrvc_obj.browse(cr, uid, idsw)
+        hrvc_brw = hrvc_obj.browse(cr, uid, idsw)
         period_id = hrvc_brw[0].hr_process_id.hr_period_ids[0].id  
         fhp = self.pool.get('hr.period').browse(cr, uid, int(period_id -1))
         fh = [fhp.date_start,  fhp.date_end]
@@ -76,48 +75,52 @@
         inicio = period2[0][0:8]+'01'          
         fin = datetime.datetime.strptime(period2[0], '%Y-%m-%d')  - timedelta(days=1)
         period1 = self.generate_period([inicio,  fin.strftime('%Y-%m-%d')])
-        
-        fecha_inicio = period1[0][0:8]+'01'          
+      
+        fecha_inicio = period1[0]          
         fecha_fin = period2[-1]    
         periodo = period2
-        
-        j = []
-        n = []
+        tolerancia = []
+        dias_trabajados = []
         retardos = []
         tiempo_no_laborado = []
-        asusencia = []
+        ausencia = []
         
         
 
         for i in cod:
             periodo = self.generate_valid_period(cr, uid, ids, i, periodo, hr_holidays_calendar)
-            #~ print periodo, i
             sql =  self.query(i, fecha_inicio, fecha_fin, t)
             cursor.execute(sql)
             rows = cursor.fetchall()
-            if not rows:
-               raise osv.except_osv('ERROR', 'El codigo %s perteneciente al trabajador %s no existe en el BioStar'%(i, self.name_for_codnumber(cr, uid, ids, i)) )     
-            for row in rows:
-                n.append(row[-1])
-                d = self.descuento(cr, uid, ids, context, row, j, concepto_id, idsw, periodo, retardos, tiempo_no_laborado, concepto_horas_no_trabajadas_id, asusencia, concepto_jornada_no_trabajda_id)
-            asu = self.absence(cr, uid, ids, context, n, periodo, concepto_jornada_no_trabajda_id, idsw, i, asusencia)
-            j = []
-            n = []
-            retardos = []
-            tiempo_no_laborado = []
-            asusencia = []
-            
-           
-            if d[0]['value'] != 0:
-               hrvc_linea_obj.create(cr, uid, d[0] , context=None)
-           
-            if d[1]['value'] != 0:
-                hrvc_linea_obj.create(cr, uid, d[1] , context=None)
-           
-            
-            if asu['value'] != 0:
-               hrvc_linea_obj.create(cr, uid, asu , context=None)
-        
+            
+            if rows:
+                for row in rows:
+                    dias_trabajados.append(row[-1])
+                    d = self.descuento(cr, uid, ids, context, row, tolerancia, concepto_id, idsw, periodo, retardos, tiempo_no_laborado, concepto_horas_no_trabajadas_id, ausencia, concepto_jornada_no_trabajda_id)
+                aus = self.absence(cr, uid, ids, context, dias_trabajados, periodo, concepto_jornada_no_trabajda_id, idsw, i, ausencia)
+                
+                tolerancia = []
+                dias_trabajados = []
+                retardos = []
+                tiempo_no_laborado = []
+                ausencia = []
+                periodo = period2
+           
+                if d[0]['value'] != 0:
+                   hrvc_linea_obj.create(cr, uid, d[0] , context=None)
+               
+                if d[1]['value'] != 0:
+                    hrvc_linea_obj.create(cr, uid, d[1] , context=None)
+               
+                
+                if aus['value'] != 0:
+                   hrvc_linea_obj.create(cr, uid, aus , context=None)
+            
+            else:
+                aus2 = self.absence(cr, uid, ids, context, dias_trabajados, periodo, concepto_jornada_no_trabajda_id, idsw, i, ausencia)
+                hrvc_linea_obj.create(cr, uid, aus2 , context=None)
+                ausencia = []
+                periodo = period2
         
         return {
             'type': 'ir.actions.act_window_close',
@@ -176,7 +179,7 @@
         l.append(fh[1])    
         return l
     
-    def descuento(self, cr, uid, ids, context, row, j, concepto, variable_id, periodo, retardos, tiempo_no_laborado, concepto_horas_no_trabajadas_id, asusencia, concepto_jornada_no_trabajda_id):
+    def descuento(self, cr, uid, ids, context, row, tolerancia, concepto, variable_id, periodo, retardos, tiempo_no_laborado, concepto_horas_no_trabajadas_id, ausencia, concepto_jornada_no_trabajda_id):
 
         empleado_obj = self.pool.get('hr.employee')
         empleado_id = empleado_obj.search(cr, uid, [('cod_number', '=', row[0])])
@@ -184,39 +187,54 @@
         period = empleado_brw[0].shift_id
         dif_total = self.total_discount('mensual')
         d = self.discount_case(period, row[-1])
-        des1 = [i for i in row if row[1] < 2 if row[-1] in periodo]
-        des2 = [i for i in row  if row[1] >= 2  if self.cvr_hr(row[2]) >= self.cvr_hr(d[0]) + timedelta(minutes=15) if row[-1] in periodo]
-        des3 = [i for i in row if row[1] >= 2  if self.cvr_hr(row[2]) >= self.cvr_hr(d[0]) + timedelta(minutes=5)  and self.cvr_hr(row[2]) < self.cvr_hr(d[0]) + timedelta(minutes=15) ]
-        des4 = [i for i in row if row[1] >= 2  if self.cvr_hr(row[3]) < self.cvr_hr(d[1]) if row[-1] in periodo]                     
+        
+        
+        des1 = [i for i in row if row[1] < 2 if self.cvr_hr(d[0]) < self.cvr_hr(d[1]) if row[-1] in periodo]
+        des2 = []
+        if self.cvr_hr(d[0]) < self.cvr_hr(d[1]):
+            if  row[1] >= 2:
+                if self.cvr_hr(row[2]) >= self.cvr_hr(d[0]) + timedelta(minutes=15) and row[-1] in periodo:
+                    des2 = list(row)
+        else:
+            if self.cvr_hr(row[2]) >= self.cvr_hr(d[0]) + timedelta(minutes=15) and row[-1] in periodo:
+               des2 = list(row)           
+                
+       
+        des3 = []
+        if self.cvr_hr(d[0]) < self.cvr_hr(d[1]):
+           if  row[1] >= 2:
+               if self.cvr_hr(row[2]) >= self.cvr_hr(d[0]) + timedelta(minutes=5)  and self.cvr_hr(row[2]) < self.cvr_hr(d[0]) + timedelta(minutes=15) and row[-1] in periodo:
+                   des3 = list(row)
+        else:
+            if self.cvr_hr(row[2]) >= self.cvr_hr(d[0]) + timedelta(minutes=5)  and self.cvr_hr(row[2]) < self.cvr_hr(d[0]) + timedelta(minutes=15)  and row[-1] in periodo:
+               des3 = list(row)     
+        
+        des4 = [i for i in row if row[1] >= 2   if self.cvr_hr(d[0]) < self.cvr_hr(d[1]) if self.cvr_hr(row[3]) < self.cvr_hr(d[1]) if row[-1] in periodo]                     
+        
+                                       
         
         if des1:
-           asusencia.append(1) 
-           #~ print "descuento1", des1 
+           ausencia.append(1) 
 
         if des2:
-           asusencia.append(1) 
-           #~ print "descuento2", des2[2] 
+           ausencia.append(1) 
 
         if des3:
-           #~ print des3 
-           j.append(1)  
-           if len(j) > 3:
-              asusencia.append(1) 
-              #~ print "descuento totelacia"    
+           tolerancia.append(1)  
+           if len(tolerancia) > 3:
+              ausencia.append(1) 
            else:
                if des3[-1] in periodo:
                   dif = self.subtract_hours(des3[2], d[0], 3600)
                   retardos.append(dif) 
-                #~ print "tolerancia acumulada", len(j), des3, dif 
 
         if des4:
            dif = self.subtract_hours(des4[3], d[1], 3600)
            tiempo_no_laborado.append(dif) 
-           #~ print "descuento saldia" , dif, des4[3]           
             
         return  [{'employee_id':  empleado_id[0], 'value': sum(retardos), 'payroll_id': False, 'variable_concept_id':   variable_id[0], 'type': 'qty', 'concept_id':  concepto},
                  {'employee_id':  empleado_id[0], 'value': sum(tiempo_no_laborado), 'payroll_id': False, 'variable_concept_id':   variable_id[0], 'type': 'qty', 'concept_id':  concepto_horas_no_trabajadas_id},      
-                 {'employee_id':  empleado_id[0], 'value': sum(asusencia), 'payroll_id': False, 'variable_concept_id':   variable_id[0], 'type': 'qty', 'concept_id':  concepto_jornada_no_trabajda_id}]     
+                 {'employee_id':  empleado_id[0], 'value': sum(ausencia), 'payroll_id': False, 'variable_concept_id':   variable_id[0], 'type': 'qty', 'concept_id':  concepto_jornada_no_trabajda_id}]     
                          
     def cvr_hr(self, hours):
         """
@@ -293,15 +311,14 @@
         periodo = [i for i in fecha if i not in hr_holidays_calendar if self.discount_case(period, i)[0] != '00:00:00' and self.discount_case(period, i)[1] != '00:00:00']
         return  periodo   
     
-    def absence(self, cr, uid, ids, context, fecha, periodo, concepto, variable_id, codigo, asusencia):
+    def absence(self, cr, uid, ids, context, fecha, periodo, concepto, variable_id, codigo, ausencia):
         empleado_obj = self.pool.get('hr.employee')
         empleado_id = empleado_obj.search(cr, uid, [('cod_number', '=', codigo)])
         for x in periodo:
             if x not in fecha:
                dif_total = self.total_discount('mensual')
-               asusencia.append(1)
-               #~ print "descuento por el dia no trabajado",x, codigo
-        return {'employee_id':  empleado_id[0], 'value': int(sum(asusencia)), 'payroll_id': False, 'variable_concept_id':   variable_id[0], 'type': 'qty', 'concept_id':  concepto}  
+               ausencia.append(1)
+        return {'employee_id':  empleado_id[0], 'value': int(sum(ausencia)), 'payroll_id': False, 'variable_concept_id':   variable_id[0], 'type': 'qty', 'concept_id':  concepto}  
        
     def name_for_codnumber(self, cr, uid, ids, codigo):
         empleado_obj = self.pool.get('hr.employee')