| Hurep |
|
|---|---|
|
Переделывая движок для сайта, в файлах users.*.inc.php заметил несколько недоработок, которые неплохо было бы исправить.
1. users.register.inc.php При регистрации, если юзер где-то допустил ошибку, например e-mail не тот, или пароль короткий, то все данные, введенные в экстраполя обнуляются, а остальные поля остаются введенными. По этому вопросу я уже сделал в trac'е ticket. 2. users.register.inc.php, users.profile.inc.php, users.edit.inc.php Эта проблема похожа на предыдущую, отличие в том, что не сохраняются данные всех полей, а не только экстра. 3. users.profile.inc.php Код в этом файле написан таким образом, что может возникнуть следующая, на мой взгляд парадоксальная ситуация. Юзер решил поменять все данные, включая почту и пароль. Он все ввел, но ошибся в адресе e-mail. В итоге он видит ошибку о вводе e-mail'а, все поля у него вернулись в прежнее состояние (см. пункт выше), но пароль изменился на новый. ИМХО чтобы не путать людей, если возникает ошибка, то не обновлять ничего. Ну или на крайняк писать, что обновилось, а что нет. 4. users.profile.tpl и users.edit.tpl Не вполне понятно, зачем в этих файлах присутствуют данные строки: <input type="hidden" name="userid" value="{USERS_PROFILE_ID}" /> в users.profile.tpl <input type="hidden" name="id" value="{USERS_EDIT_ID}" /> в users.edit.tpl В соответствующих php файлах данные из них не считываются. 5. sed-light.ru.lang.php Немного не в тему, но все же... Опечатка в [240] $skinlang['usersedit']['Timezone'] = 'Часовй пояс:'; |
