Conjunto de cambios e44bd19 en seiven para usuario

Fecha y hora:

05/09/2016 16:26:35 (hace 8 años)

Autor:

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

Branches:

master, carga

Children:

52b5505

Parents:

595c7b5

Mensaje:

modificación en la autenticación de usuario referente a el estatus de activo del usuario de acuerdo a los niveles de acceso

Ubicación:

usuario

Ficheros:

• forms.py (modificado) (3 diferencias)
• models.py (modificado) (3 diferencias)
• views.py (modificado) (4 diferencias)

usuario/forms.py

@@ -16 +16 @@
 
 from base.constant import (
-    TIPO_DOCUMENTO_IDENTIFICACION, TIPO_DOCUMENTO_IDENTIFICACION_LIST, FORTALEZA_CONTRASENHA
+    TIPO_DOCUMENTO_IDENTIFICACION, TIPO_DOCUMENTO_IDENTIFICACION_LIST, FORTALEZA_CONTRASENHA, OCUPACION
 )
 from base.fields import TipoDocumentoField
@@ -25 +25 @@
 from django.forms import (
     ModelForm, TextInput, EmailInput, CharField, EmailField, PasswordInput,
-    Select, ModelChoiceField)
+    Select, ModelChoiceField, ChoiceField
+)
 from django.utils.encoding import python_2_unicode_compatible
 from django.utils.translation import ugettext_lazy as _
 
-from base.models import Institucion, Ocupacion
+from base.models import Institucion
 from base.widgets import TipoDocumentoWidgetReadOnly
 from .models import UserProfile
@@ -195 +196 @@
 
     ## Listado de instituciones registradas en el sistema
-    ocupacion = ModelChoiceField(
-        label=_(u"Ocupacion"), queryset=Ocupacion.objects.all(), empty_label=_(u"Seleccione..."),
+    ocupacion = ChoiceField(
+        label=_(u"Ocupacion"), choices=OCUPACION,
         widget=Select(attrs={
             'class': 'select2 select2-offscreen form-control', 'data-toggle': 'tooltip',
-            'title': _(u"Seleccione la ocupación o profesión del usuario a registrar")
+            'title': _(u"Seleccione la ocupación del usuario a registrar")
         })
     )

usuario/models.py

@@ -19 +19 @@
 from django.utils.translation import ugettext_lazy as _
 
-from base.constant import TIPO_DOCUMENTO_IDENTIFICACION
-from base.models import Institucion, Ocupacion
+from base.constant import TIPO_DOCUMENTO_IDENTIFICACION, OCUPACION, NIVELES_ACCESO
+from base.models import Institucion
 
 
@@ -46 +46 @@
     )
 
+    ## Contiene datos de la ocupación, oficio o profesión del usuario
+    ocupacion = models.CharField(max_length=2, choices=OCUPACION[1:])
+
     ## Establece la última fecha de modificación de la contraseña, lo cual permite establecer la caducidad de la misma
     fecha_modpass = models.DateTimeField(null=True, help_text=_("Fecha en la que se modificó la contraseña"))
@@ -52 +55 @@
     institucion = models.ForeignKey(Institucion, help_text=_("Institucion de la cual proviene el usuario"))
 
-    ## Contiene datos de la ocupación, oficio o profesión del usuario
-    ocupacion = models.ForeignKey(Ocupacion, help_text=_("Ocupación del usuario"))
+    ## Indica el nivel de acceso que tiene el usuario
+    nivel_acceso = models.PositiveSmallIntegerField(choices=NIVELES_ACCESO, null=True)
+
 
     ## Establece la relación entre el usuario y el perfil

usuario/views.py

@@ -77 +77 @@
     """
     form = AutenticarForm()
+    alert = None
 
     if request.method == "POST":
@@ -85 +86 @@
                 request.POST['tipo_documento_0'], request.POST['tipo_documento_1']
             )
-
-            usuario = authenticate(username=username, password=str(request.POST['clave']))
-
-            if usuario is not None:
-                login(request, usuario)
-                usr = User.objects.get(username=username)
-                usr.last_login = datetime.now()
-                usr.save()
+            if User.objects.filter(username=username, is_active=True):
+                usuario = authenticate(username=username, password=str(request.POST['clave']))
+
+                if usuario is not None:
+                    login(request, usuario)
+                    usr = User.objects.get(username=username)
+                    usr.last_login = datetime.now()
+                    usr.save()
+                else:
+                    logger.error(str(_("Error al autenticar el usuario [%s]") % username))
+
+                logger.info(str(_("Acceso al sistema por el usuario [%s]") % username))
+                return HttpResponseRedirect(urlresolvers.reverse("inicio"))
             else:
-                logger.error(str(_("Error al autenticar el usuario [%s]") % username))
-
-            logger.info(str(_("Acceso al sistema por el usuario [%s]") % username))
-            return HttpResponseRedirect(urlresolvers.reverse("inicio"))
-
-    return render_to_response('base.template.html', {'form': form}, context_instance=RequestContext(request))
+                alert = str(_("Su usuario se encuentra inactivo. Intente más tarde..."))
+
+
+    return render_to_response('base.template.html', {'form': form, 'alert': alert}, context_instance=RequestContext(request))
 
 
@@ -267 +271 @@
         if calcular_diferencia_fechas(user.date_joined) <= CADUCIDAD_LINK_REGISTRO:
             if key.strip() == hash_user(user, is_new_user=True).decode():
-                user.is_active = True
-                user.save()
+                if UserProfile.objects.filter(user=user, ocupacion='ES'):
+                    # Si es estudiante el sistema activa automáticamente el usuario,
+                    # en caso contrario debe esperar por la confirmación del administrador
+                    user.is_active = True
+                    user.save()
+                    user_profile = UserProfile.objects.get(user=user)
+                    user_profile.nivel_acceso = 3
+                else:
+                    mensaje = str(_("El enlace fue verificado y el administrador esta evaluando sus credenciales para "
+                                    "otorgarle un nivel de acceso al sistema"))
                 login_url = "%s?userid=%s&key=%s" % (
                     urlresolvers.reverse('acceso'), user.username, hash_user(user, is_new_user=True)
@@ -321 +333 @@
         self.object.save()
 
+
         ## Crea el perfil del usuario
-
         UserProfile.objects.create(
             tipo_documento=form.cleaned_data['tipo_documento'],