Project

General

Profile

Bug #6701

No permite modificar el perfil de un usuario responsable de recursos humanos

Added by Laura Guirao over 8 years ago. Updated over 7 years ago.

Status:
Bug resolved
Priority:
Normal
Assignee:
-
Category:
Administration
Target version:
Start date:
13/09/2013
Due date:
24/04/2014
% Done:

100%

Estimated time:
Complexity:
Normal
SCRUM pts - complexity:
?

Description

Al intentar modificar desde la administración cualquier parámetro del perfil, por ejemplo el nombre de usuario o la dirección de e-mail, aparece un mensaje indicando que "El estado de este usuario no puede ser cambiado por el de Administrador de Recursos Humanos" cuando en realidad no se estaba cambiando el estado, pues el usuario ya tenía ese rol anteriormente asignado.


Files

c.jpg (33.1 KB) c.jpg Laura Guirao, 13/09/2013 13:54
c.jpg (50.4 KB) c.jpg Laura Guirao, 29/11/2013 12:48
administrador_RH.jpg (80.3 KB) administrador_RH.jpg Yoselyn Castillo, 13/12/2013 00:12
administrador_RH_modificado.jpg (82.4 KB) administrador_RH_modificado.jpg Yoselyn Castillo, 13/12/2013 00:12

Associated revisions

Revision 6055b3ac (diff)
Added by Yoselyn Castillo about 8 years ago

Fix for avoiding HRM rol to be registered in course -refs #6701

Revision 561076ff
Added by Julio Montoya about 8 years ago

Merge pull request #173 from ycastillo/yoselyn.6701

Fix for avoiding HRM rol to be registered in course -refs #6701

History

#1

Updated by Laura Guirao over 8 years ago

  • Category set to Administration
#2

Updated by Julio Montoya about 8 years ago

  • Status changed from New to Assigned
  • Assignee set to Yoselyn Castillo

Hola Yoselyn,

Pudes revisar la tarea?

#3

Updated by Yoselyn Castillo about 8 years ago

  • Status changed from Assigned to Needs testing

This should be already solved in stable.chamilo.org. Please try it and notify me

#4

Updated by Laura Guirao about 8 years ago

I just try in stable.chamilo.org but It doesn't works.
You can see it:

#5

Updated by Yoselyn Castillo about 8 years ago

  • Status changed from Needs testing to Assigned

As I have been testing..
I could create a user with Human Resources Manager rol
Later, I could modified the profile of that user since administration: eg. login, name, email etc
However, When i try to modify other user's rol to Human Resources Manager i could see the error message you have mentioned.
Checking..

#6

Updated by Yoselyn Castillo about 8 years ago

Bueno, he revisado a profundidad, y al editar un perfil de usuario, este no puede pasar a rol de administrador de recursos humanos si ya esta inscrito en algun curso, sin embargo si ya tiene el rol de administrador de recursos humanos, si puedo cambiar perfectamente los parametros desde la administracion en stable.chamilo.org.
Ver usuario administrador de Recursos Humanos

Luego lo modifique siendo administrador de RH

Ahora, si bien es cierto que no se puede asignar rol de administrador de recursos humanos a un usuario que esta inscrito en algun curso, entonces se deberia verificar que no sea administrador de recursos humanos cuando se vaya a inscribir a un curso, cosa que actualmente no sucede...

#7

Updated by Laura Guirao about 8 years ago

Hola Yoselyn

Cuando yo voy a stable.chamilo.org y hago lo siguiente:

1. Voy a administración y busco a "alumno tres" que es Responsable de recursos Humanos
2. Pulso el botón de editar su perfil
3. Intento cambiar su dirección de e-mail por otra y le doy a guardar
4. Me sigue apareciendo el error de que no es posible asignar el perfil de Recursos Humanos y no guarda los cambios. Es lo que aparece en la captura que envié.

Por lo que tu dices, no te ocurre lo mismo, pero no lo entiendo, porque a mi no me deja modificar los datos. ¿cómo lo haces tu para que sí te permita hacerlo?

#8

Updated by Yoselyn Castillo about 8 years ago

  • % Done changed from 0 to 60

AL parecer ocurre solamente si el usuario responsable de Recursos humanos esta asociado a algun curso.
main/admin/user_edit.php line 315
if ($user['status'] == DRH && $is_user_subscribed_in_course) {
$error_drh = true;
Prueba creando un usuario desde cero con el rol de Recursos Humanos, y luego trata de modificarlo y en ese caso si se puede. Es por eso que envie este pull para evitar que los usuarios con ese rol se les inscriba en algun curso, puesto que esta la restriccion al editar el usuario.
https://github.com/chamilo/chamilo-lms/pull/173

#9

Updated by Yannick Warnier over 7 years ago

  • Status changed from Assigned to Needs testing
  • Assignee deleted (Yoselyn Castillo)
  • Target version set to 1.9.8

Laura, al parecer esta espera tu validación. Debería estar corregido en stable.chamilo.org. Si no lo es, entonces hay que volver a revisarla.

#10

Updated by Yannick Warnier over 7 years ago

  • Due date set to 24/04/2014
#11

Updated by Laura Guirao over 7 years ago

  • Status changed from Needs testing to Bug resolved
  • % Done changed from 60 to 100

Correcto. Comprobado en stable.chamilo.org

Cierro el bug

Also available in: Atom PDF