Функция обновления комментариев

#1 4 декабря 2015 в 21:27
В 2ке есть удобная кнопочка обновления комментариев. Есть проблема, она показывается и незарегистрированным.
Но в экшне actionCommentsRefresh стоит проверка

  1. if (!$user->is_logged) { cmsCore::error404(); }
Получается, что незарегистрированный пользователь, нажав на нее, увидит лишь вечно крутящуюся стрелочку.

В шаблоне вывода комментариев видим код

  1. <?php if ($is_karma_allowed || $is_guests_allowed){ ?>
  2. <div id="comments_refresh_panel">
  3. <a href="#refresh" class="refresh_btn" onclick="return icms.comments.refresh()" title="<?php echo LANG_COMMENTS_REFRESH; ?>"></a>
  4. </div>
  5. <?php } ?>
т.е. она показывается гостям принудительно (в случае разрешения комментирования гостями, конечно).

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

Убедиться в проблеме легко. Идем по ссылке. И тыкаем по кнопочке обновления комментариев.

В своем шаблоне я пока скрою кнопку от незарегистрированных.

Мне кажется это баг, нужно оформить тикет на гитхабе. У меня там нет учетки, к сожалению.
#2 4 декабря 2015 в 21:52

Идем по ссылке. И тыкаем по кнопочке обновления комментариев.

Zau4man
Завораживающее действие.
#3 4 декабря 2015 в 22:41

Завораживающее действие.

Loadырь
это вы о 404 в консоли ошибок?
#4 4 декабря 2015 в 23:25
или убрать ее туда, где она должна быть...

#5 5 декабря 2015 в 00:37
solitario84, г-н Zau4man не о её местонахождении глаголит
Они (г-н Zau4man) говорят о том что эта наволочь мало того что комментарии новые не выводит в этом случае, так ещё и вертится! zlo
#6 5 декабря 2015 в 01:07

Идем по ссылке.

Zau4man

Блин, кажется я купил кроватку...
Что теперь? Третьего заводить или внуков дожидаться? scratch
#7 5 декабря 2015 в 01:17

Что теперь?

HiAndy
rofl
#8 5 декабря 2015 в 02:18
Олег Васильевич я, roflубили… я, видимо недопонял о чем г-н Zau4man глаголит… просто у меня комментирование гостям излишнее удовольствие, поэтому все намного проще…
Вы не можете отвечать в этой теме.
Войдите или зарегистрируйтесь, чтобы писать на форуме.
Используя этот сайт, вы соглашаетесь с тем, что мы используем файлы cookie.