search

chamilo / chamilo-lms

Chamilo is a learning management system focused on ease of use and accessibility
https://chamilo.org
GNU General Public License v3.0
798 stars 480 forks source link

Users update through CSV does not take send_two_inscription_confirmation_mail into account #4956

Closed ywarnier closed 11 months ago

ywarnier commented 12 months ago

Setting $_configuration['send_two_inscription_confirmation_mail'] = true; should always ensure that Chamilo sends two e-mails (one with the username, one with the password), but when using main/admin/user_update_import.php with the "Automatically generate a new password" option checked, the setting is not taken into account.

The same goes for manual user edition, if it includes a password change, or any other user update (through web services or through plugins).

The code for the user creation taking this setting into account is at usermanager.lib.php +653

BT#21146

ywarnier commented 11 months ago

This should now work, thanks to this commit https://github.com/chamilo/chamilo-lms/commit/fe74d0d34ad30d11877a308f0bbfddcb46047e72