Changeset de5a897 in sicp

Timestamp:

Mar 10, 2016, 10:21:51 AM (8 years ago)

Author:

Ing. Roldan D. Vargas G <rvargas@…>

Branches:

master

Children:

90186b5

Parents:

a902c05

Message:

se agregan instrucciones de log

Location:

apps/usuario

Files:

• forms.py (modified) (4 diffs)
• views.py (modified) (7 diffs)

apps/usuario/forms.py

@@ -116 +116 @@
         correo = self.cleaned_data['correo']
         if User.objects.filter(email=correo):
-            """logger.warning(_(u"Intento de registro de usuario con un correo ya registrado. "
-                              u"El correo introducido es [%s]" % self.cleaned_data['correo']))"""
+            logger.warning(
+                u"Intento de registro de usuario con un correo ya registrado. El correo introducido es [%s]"
+                % self.cleaned_data['correo']
+            )
             raise forms.ValidationError(_(u"El correo electronico indicado ya se encuentra registrado."))
         return correo
@@ -133 +135 @@
 
         if User.objects.filter(username=cedula):
-            """logger.warning(_(u"Intento de registro de usuario ya existente en el sistema. "
-                             u"El usuario que intenta registrar es %s" % cedula))"""
+            logger.warning(
+                u"Intento de registro de usuario ya existente en el sistema. El usuario que intenta registrar es %s"
+                % cedula
+            )
             raise forms.ValidationError(_(u"El número de cédula ya se encuentra registrado"))
 
@@ -262 +266 @@
         """
         if not User.objects.filter(email=self.cleaned_data['correo']):
-            """logger.warning(_(u"Intento de recuperación de contraseña para un correo no existente. "
-                              u"El correo introducido es [%s]" % self.cleaned_data['correo']))"""
+            logger.warning(
+                u"Intento de recuperación de contraseña para un correo no existente. El correo introducido es [%s]"
+                % self.cleaned_data['correo']
+            )
             raise forms.ValidationError(_(u"El correo electronico no se encuentra registrado en el sistema"))
 
@@ -389 +395 @@
         })
     )
+
+    def clean_correo(self):
+        """!
+        Método que permite validar los datos introducidos en el campo del correo electrónico
+
+        @author Ing. Roldan Vargas (rvargas at cenditel.gob.ve)
+        @copyright <a href='http://www.gnu.org/licenses/gpl-2.0.html'>GNU Public License versión 2 (GPLv2)</a>
+        @date 29-07-2015
+        @return Devuelve un mensaje de error en caso de que el correo electrónico especificado ya se encuentre registrado
+        """
+        correo = self.cleaned_data['correo']
+        cedula = self.data['cedula']
+        if User.objects.filter(email=correo).exclude(username=cedula):
+            logger.warning(u"Intento de modificación de correo electrónico del usuario [%s] con un correo ya "
+                           u"registrado en el sistema. El correo introducido es [%s]"
+                           % (cedula, self.cleaned_data['correo']))
+            raise forms.ValidationError(_(u"El correo electrónico indicado ya esta registrado a otro usuario."))
+        return correo
+
+    def clean_cedula(self):
+        """!
+        Método que permite validar los datos introducidos en el campo cédula
+
+        @author Ing. Roldan Vargas (rvargas at cenditel.gob.ve)
+        @copyright <a href='http://www.gnu.org/licenses/gpl-2.0.html'>GNU Public License versión 2 (GPLv2)</a>
+        @date 29-07-2015
+        @return Devuelve un mensaje de error en caso de que la cédula de identidad especificada ya se encuentre registrada
+        """
+        cedula = self.cleaned_data['cedula']
+
+        return cedula

apps/usuario/views.py

@@ -40 +40 @@
 from apps.usuario.functions import generar_password
 
-
+import os
 import logging
 
@@ -71 +71 @@
                 'hora': now.time(), 'dia': now.date(), 'ip': request.META.get('REMOTE_ADDR')
             })
-            #logger.info(_(u"Acceso al sistema por el usuario [%s]" % request.POST['usuario']))
+            logger.info(u"Acceso al sistema por el usuario [%s]" % request.POST['usuario'])
             return HttpResponseRedirect(urlresolvers.reverse("inicio"))
         else:
@@ -92 +92 @@
     if user.is_authenticated():
         logout(request)
+        salida = os.path.join(settings.BASE_DIR, "tmp/%s*.*" % str(user))
+        os.system("rm %s" % salida)
         #logger.info(_(u"El usuario [%s] salio del sistema" % user))
     return HttpResponseRedirect(urlresolvers.reverse("inicio"))
@@ -137 +139 @@
             form.instance.cargo = cargo
             self.object.save()
-            """logger.info(
-                _(u"Se ha registrado el usuario [%(usuario)s] de la institución [%(institucion)s] con el cargo de [%(cargo)s]")
+            logger.info(
+                u"Se ha registrado el usuario [%(usuario)s] de la institución [%(institucion)s] con el cargo de [%(cargo)s]"
                 % {'usuario':form.cleaned_data['cedula'], 'institucion':institucion.nombre, 'cargo':cargo.nombre}
-            )"""
+            )
             enviado = enviar_correo(correo, 'registro.mail', 'Registro de usuario en SICP', {
                 'usuario': cedula, 'clave': clave, 'emailapp': settings.EMAIL_FROM
             })
 
-            """if not enviado:
-                logger.warning(_(u"Ocurrió un inconveniente al enviar el correo de registro al usuario [%s]" % cedula))"""
+            if not enviado:
+                logger.warning(u"Ocurrió un inconveniente al enviar el correo de registro al usuario [%s]" % cedula)
 
             for user in User.objects.filter(is_staff=True):
@@ -152 +154 @@
                     'usuario': cedula, 'emailapp': settings.EMAIL_FROM
                 })
-                """if not env_admin:
-                    logger.warning(_(u"Ocurrió un inconveniente al enviar el correo al administrador del sistema [%s] sobre el registro de un nuevo usuario [%s]" % (user.username, cedula)))"""
+                if not env_admin:
+                    logger.warning(
+                        u"Ocurrió un inconveniente al enviar el correo al administrador del sistema [%s] sobre el "
+                        u"registro de un nuevo usuario [%s]" % (user.username, cedula)
+                    )
         except Exception, e:
-            print e
+            logger.error(
+                u"Ocurrio un error al registrar el usuario [%s]. Detalles: %s" % (form.cleaned_data['cedula'], e)
+            )
 
         return super(RegistroCreate, self).form_valid(form)
@@ -195 +202 @@
             'usuario': usr.username, 'clave': clave, 'emailapp': settings.EMAIL_FROM
         })
-        """if not enviado:
-            logger.warning(_(u"Ocurrió un inconveniente al enviar el correo con los datos de la nueva contraseña generada al usuario [%s]" % usr.username))
+        if not enviado:
+            logger.warning(
+                u"Ocurrió un inconveniente al enviar el correo con los datos de la nueva contraseña generada al "
+                u"usuario [%s]" % usr.username
+            )
         else:
-            logger.info(_(u"El usuario [%s] solicitó una nueva contraseña y le fue suministrada por correo electrónico"
-                           % usr.username))"""
+            logger.info(
+                u"El usuario [%s] solicitó una nueva contraseña y le fue suministrada por correo electrónico"
+                % usr.username
+            )
         return super(OlvidoClaveView, self).form_valid(form)
 
@@ -306 +318 @@
         self.object.save()
 
-        #logger.info(_(u"Se han actualizado los datos del usuario [%s]") % usuario)
+        logger.info(u"Se han actualizado los datos del usuario [%s]" % usuario)
 
         return super(ModificarPerfilView, self).form_valid(form)