Forums / National / Russian / Пакет улучшений для users

Hurep
#1 2009-04-03 01:00
Переделывая движок для сайта, в файлах 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'] = 'Часовй пояс:';
Ratibor
#2 2009-04-03 01:24
# Hurep : 5. sed-light.ru.lang.php
Немного не в тему, но все же... Опечатка в [240] $skinlang['usersedit']['Timezone'] = 'Часовй пояс:';
Это поправил.
Не задавай глупых вопросов, не услышишь вранья.