Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move methods to get user complete name to UserManager #2650

Merged
merged 3 commits into from Sep 6, 2018

Conversation

AngelFQC
Copy link
Member

No description provided.

@AngelFQC AngelFQC added this to the 2.0 milestone Aug 29, 2018
AngelFQC referenced this pull request Aug 29, 2018
Hide the username when showing the complete name for a user
@chamilo chamilo deleted a comment from FlintCIBot Aug 29, 2018
@jmontoyaa
Copy link
Member

The setting is not added in the Schema:

"platform.hide_username_with_complete_name"

It's not better to use the "Profile" Schema?

src/CoreBundle/Settings/ProfileSettingsSchema.php

@AngelFQC
Copy link
Member Author

@jmontoyaa Creo que está más relacionado con la plataforma en general que sólo con el perfil del usuario.
Desaparecería el username en varias herramientas, mientras que en ProfileSettingsSchema hay settings más relacionadas al perfil (y login) del usuario.

@jmontoyaa
Copy link
Member

Todo lo que va con el usuario debería estar en "Profile", ahí tenemos los settings globales relacionados al usuario (account_valid_duration, use_users_timezone, split_users_upload_directory, etc). No son necesariamente cosas de profile. Ponerlos en PlatformSettingsSchema es demasiado general.

Hay que cambiar estos settings también:

'hide_username_with_complete_name' => 'false',
'hide_username_in_course_chat' => 'false',

Agregados aquí:

d5ef4f1

Tal vez seria una buena idea de cambiar el nombre del archivo a UserSettingsSchema.php en lugar de
ProfileSettingsSchema.php Pero eso ya sería cosa de otra tarea.

@chamilo chamilo deleted a comment Sep 5, 2018
@chamilo chamilo deleted a comment Sep 5, 2018
@FlintCIBot
Copy link

Lint errors were found. A patch is also available.

Please see the report: https://flintci.io/repositories/1013/analyses/10398

This comment was posted by FlintCI. It can be disabled in the repository settings.

@chamilo chamilo deleted a comment from FlintCIBot Sep 5, 2018
@AngelFQC
Copy link
Member Author

AngelFQC commented Sep 5, 2018

Moví los settings a ProfileSettingsSchema

@chamilo chamilo deleted a comment Sep 5, 2018
@chamilo chamilo deleted a comment Sep 5, 2018
@jmontoyaa jmontoyaa merged commit 41deee8 into chamilo:master Sep 6, 2018
@jmontoyaa jmontoyaa moved this from To Do to Done in Chamilo v2 Core & Critical Sep 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants