И снова о рейтинге...

[webair 179]

@celsoft

Я уже писал о не логичности бд рейтинга.

 

Теперь о нелогичности кода рейтинга. На этот "баг" меня натолкнул клиент, которому не понравилось такое нелогичное поведение.

 

Стандартное поведение DLE:

Если выставляем рейтинг 5, то нельзя изменить свой рейтинг снова на 5, только на 1-4. Ну ок, выбираем 1, после чего невозможно выбрать снова 1, а только 2-5.

Надо бы определиться, мы либо ограничиваем изменение оценки или нет?

 

Ошибка "Вы уже выставили свою оценку для данной статьи"

По тексту ошибки ну никак не понятно, что мы пытаемся изменить свой голос с 5 на 5. А скорее что мы проголосовали и не можем поменять свой голос. Но, на самом деле можем, на 1-4 звезд.

 

Где логика? Клиенту пришлось доказывать, что я не верблюд)

 

 

Я бы вынес в настройки возможность выбрать вкл/выкл возможность изменения рейтинга. Наверное, в настройки групп это надо.

Изменено 28 мая 2020 пользователем webair

[alex32 944]

А в чем логика менять свой голос с 5 на 5?

[webair 179]

1 час назад, alex32 сказал:

А в чем логика менять свой голос с 5 на 5?

А какая разница скрипту, выполнить пару запросов к бд на обновление информации, когда уже выполнен запрос на проверку оценивал он или нет.

Получать ошибку "Вы уже выставили свою оценку для данной статьи" при выборе той же оценки и позволять проголосовать, если оценка отличается, это нормально?

Хотя бы тогда понятный текст ошибки выводить "Выберите оценку, которая отличается от предыдущей", а не так, как будто он вообще не может изменить оценку.

 

 

Изменено 28 мая 2020 пользователем webair

[celsoft 6228]

5 часов назад, webair сказал:

Если выставляем рейтинг 5, то нельзя изменить свой рейтинг снова на 5, только на 1-4. Ну ок, выбираем 1, после чего невозможно выбрать снова 1, а только 2-5.

Надо бы определиться, мы либо ограничиваем изменение оценки или нет?

Изменить, это значит что? Правильно поменять на другое. Как 5 изменить на 5 по вашей логике и по логике вашего клиента? Вы уже выставили 5 и собираетесь поменять на то что уже итак установлено.

 

4 часа назад, webair сказал:

Получать ошибку "Вы уже выставили свою оценку для данной статьи" при выборе той же оценки и позволять проголосовать, если оценка отличается, это нормально?

Абсолютно нормально. Потому как пользователь может поменять свое мнение. И имеет на это право. Потому как его мнение принадлежит ему, а не кому то еще. Равно как например ваше мнение принадлежит только вам и никто за вас не может решить можете ли вы его менять или нет. Это решаете только вы.

 

Никаких изменений в этом вопросе точно не планируется и не будет. Потому как предложение не логично.

4 часа назад, webair сказал:

Хотя бы тогда понятный текст ошибки выводить "Выберите оценку, которая отличается от предыдущей", а не так, как будто он вообще не может изменить оценку.

Если нужна какой то свой текст через систему плагинов можно поменять текст в языковом файле на любой свой.