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

User registration: Enable extra field validation for user registration in Chamilo #5541

Open
wants to merge 2 commits into
base: 1.11.x
Choose a base branch
from

Conversation

juancp-contidosdixitais

This pull request introduces a new functionality to Chamilo that allows the use of a user extra field to validate user registrations. For instance, if we have an extra field to store passport numbers, we can ensure that there are no duplicate values.

registro_manual

This functionality also works on batch user registrations.

resultado_importacion

To utilize this functionality, you need to specify in "extra_fields_to_validate_on_user_registration" within configuration.php the extra fields you want to validate. For example:

$_configuration['extra_fields_to_validate_on_user_registration'] = [
'extra_fields' => [
'passport', 'employee_id'
]
];

@juancp-contidosdixitais
Copy link
Author

Updated to add the same logic to the user add/edit actions from administration

@ywarnier
Copy link
Member

I see the point of this PR, but the code comparison is a complete mess (probably not your fault - some mix of bad detections by Github and of adding changes to many small places, and maybe some parallel history of the same files in the last 5 months after your PR), so in the current state I cannot safely approve it (it would have to be rewritten with less intrusive changes).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants