New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Проблема с проверкой нажатий на кнопку СПАСИБО #704

Closed
ArmGono opened this Issue Oct 6, 2017 · 31 comments

Comments

Projects
None yet
5 participants
@ArmGono

ArmGono commented Oct 6, 2017

Сейчас на dru можно неограниченное кол-во раз поблагодарить 1 пост.
Тут можно посмотреть пример как "накрутил" и убрал отметки спасибо без особых заморочек https://youtu.be/E3fzAqs4zMw

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Oct 6, 2017

Contributor

Да, спасибо. Известный баг.
@awd-studio у нас есть другие issue под это дело?

Contributor

mbaev commented Oct 6, 2017

Да, спасибо. Известный баг.
@awd-studio у нас есть другие issue под это дело?

@ArmGono

This comment has been minimized.

Show comment
Hide comment
@ArmGono

ArmGono Oct 6, 2017

dru_tnx-multiple-thanks-bug-fix-patch.txt
Временное решение, не могу потестить)

ArmGono commented Oct 6, 2017

dru_tnx-multiple-thanks-bug-fix-patch.txt
Временное решение, не могу потестить)

@ArmGono

This comment has been minimized.

Show comment
Hide comment
@ArmGono

ArmGono Oct 6, 2017

не идеально, но не позволит делать то, что я на видео проделал))

ArmGono commented Oct 6, 2017

не идеально, но не позволит делать то, что я на видео проделал))

@awd-studio

This comment has been minimized.

Show comment
Hide comment
@awd-studio

awd-studio Oct 6, 2017

Contributor

у нас есть другие issue под это дело?

Тока #432
Я сегодня постараюсь исправить.

@ArmGono а патчик загрузился? у меня пустой чот открывается.

Contributor

awd-studio commented Oct 6, 2017

у нас есть другие issue под это дело?

Тока #432
Я сегодня постараюсь исправить.

@ArmGono а патчик загрузился? у меня пустой чот открывается.

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Oct 6, 2017

Contributor

А почему не можете потестить?

Contributor

mbaev commented Oct 6, 2017

А почему не можете потестить?

@ArmGono

This comment has been minimized.

Show comment
Hide comment
@ArmGono

ArmGono Oct 6, 2017

@awd-studio

diff --git a/profiles/drupalru/modules/dru_tnx/dru_tnx.pages.inc b/profiles/drupalru/modules/dru_tnx/dru_tnx.pages.inc
index a501a090..15af3987 100644
--- a/profiles/drupalru/modules/dru_tnx/dru_tnx.pages.inc
+++ b/profiles/drupalru/modules/dru_tnx/dru_tnx.pages.inc
@@ -77,6 +77,18 @@ function dru_tnx_data($vars) {
 
   // Counting tnx by entity
   $tnx_count = $vars['entity']->tnx;
+
+  // Check user thanked or not.
+  $user_is_tnx = db_select('tnx', 't')
+    ->condition('t.entity_id', $vars['id'])
+    ->condition('t.uid', $vars['user']->uid)
+    ->condition('t.entity_type', $vars['type']);
+  if($user_is_tnx && $vars['tnx'] == 'tnx'){
+    $vars['tnx'] = 'untnx';
+  }elseif(!$user_is_tnx && $vars['tnx'] == 'untnx'){
+    $vars['tnx'] = 'tnx';
+  }
+
   if ($vars['tnx'] == 'tnx') {
     $tnx_count++;
 

ArmGono commented Oct 6, 2017

@awd-studio

diff --git a/profiles/drupalru/modules/dru_tnx/dru_tnx.pages.inc b/profiles/drupalru/modules/dru_tnx/dru_tnx.pages.inc
index a501a090..15af3987 100644
--- a/profiles/drupalru/modules/dru_tnx/dru_tnx.pages.inc
+++ b/profiles/drupalru/modules/dru_tnx/dru_tnx.pages.inc
@@ -77,6 +77,18 @@ function dru_tnx_data($vars) {
 
   // Counting tnx by entity
   $tnx_count = $vars['entity']->tnx;
+
+  // Check user thanked or not.
+  $user_is_tnx = db_select('tnx', 't')
+    ->condition('t.entity_id', $vars['id'])
+    ->condition('t.uid', $vars['user']->uid)
+    ->condition('t.entity_type', $vars['type']);
+  if($user_is_tnx && $vars['tnx'] == 'tnx'){
+    $vars['tnx'] = 'untnx';
+  }elseif(!$user_is_tnx && $vars['tnx'] == 'untnx'){
+    $vars['tnx'] = 'tnx';
+  }
+
   if ($vars['tnx'] == 'tnx') {
     $tnx_count++;
 

@awd-studio awd-studio self-assigned this Oct 6, 2017

@awd-studio awd-studio added the Bug label Oct 6, 2017

@ArmGono

This comment has been minimized.

Show comment
Hide comment
@ArmGono

ArmGono Oct 6, 2017

@mbaev по времени ограничен сильно =)
Попытался найти базу, чтобы поднять копию сайта, но не получилось.

ArmGono commented Oct 6, 2017

@mbaev по времени ограничен сильно =)
Попытался найти базу, чтобы поднять копию сайта, но не получилось.

@mbaev

This comment has been minimized.

Show comment
Hide comment
Contributor

mbaev commented Oct 6, 2017

@ArmGono

This comment has been minimized.

Show comment
Hide comment
@ArmGono

ArmGono Oct 6, 2017

@mbaev спасибо, его бы в README.md на репо. Что-то не подумал в wiki заглянуть.
через пару недель освободится время, смогу помочь по коду.

ArmGono commented Oct 6, 2017

@mbaev спасибо, его бы в README.md на репо. Что-то не подумал в wiki заглянуть.
через пару недель освободится время, смогу помочь по коду.

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Oct 6, 2017

Contributor

@ArmGono да, будет обязательно. Я только дописал локальную развёртку на днях. Сейчас ещё допишу тот скрипт для настройки окружения и ликбез о том, как развернуть копию. Потом перенесу в readme.

Contributor

mbaev commented Oct 6, 2017

@ArmGono да, будет обязательно. Я только дописал локальную развёртку на днях. Сейчас ещё допишу тот скрипт для настройки окружения и ликбез о том, как развернуть копию. Потом перенесу в readme.

@awd-studio awd-studio added this to the 6-ое ноября 2017 milestone Oct 6, 2017

@Director-cemetery

This comment has been minimized.

Show comment
Hide comment
@Director-cemetery

Director-cemetery Oct 6, 2017

Вот теперь ролик в массы уйдёт, ведь договаривались до исправления бага не говорить никому.

Director-cemetery commented Oct 6, 2017

Вот теперь ролик в массы уйдёт, ведь договаривались до исправления бага не говорить никому.

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Oct 6, 2017

Contributor

После фикса будет пересчет ( я надеюсь ) и тогда всё станет как должно.

Contributor

mbaev commented Oct 6, 2017

После фикса будет пересчет ( я надеюсь ) и тогда всё станет как должно.

@ArmGono

This comment has been minimized.

Show comment
Hide comment
@ArmGono

ArmGono Oct 6, 2017

@Director-cemetery

  1. ролик доступен только по ссылке
  2. Я искал баг в issue и не нашел, решил наглядно показать, если бы описал подробно, эффект был бы тот же.
  3. Я ни с кем не договаривался, нашел баг, написал фикс(хоть и временный, причины выше в комментариях) и создал issue.

@mbaev в принципе, если я правильно понял работу модуля, он в отдельной таблице ведет учет, соответственно пересчет можно будет сделать.

ArmGono commented Oct 6, 2017

@Director-cemetery

  1. ролик доступен только по ссылке
  2. Я искал баг в issue и не нашел, решил наглядно показать, если бы описал подробно, эффект был бы тот же.
  3. Я ни с кем не договаривался, нашел баг, написал фикс(хоть и временный, причины выше в комментариях) и создал issue.

@mbaev в принципе, если я правильно понял работу модуля, он в отдельной таблице ведет учет, соответственно пересчет можно будет сделать.

@ArmGono

This comment has been minimized.

Show comment
Hide comment
@ArmGono

ArmGono Oct 6, 2017

Давайте обсудим правильное поведение при нажатии кнопки Спасибо(второй раз) и я поправлю модуль.
Варианты:

  1. Ничего не делать
  2. Снимать отметку спасибо
  3. ???

ArmGono commented Oct 6, 2017

Давайте обсудим правильное поведение при нажатии кнопки Спасибо(второй раз) и я поправлю модуль.
Варианты:

  1. Ничего не делать
  2. Снимать отметку спасибо
  3. ???
@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Oct 6, 2017

Contributor

Думаю, надо так же как для комментов

Contributor

mbaev commented Oct 6, 2017

Думаю, надо так же как для комментов

@awd-studio

This comment has been minimized.

Show comment
Hide comment
@awd-studio

awd-studio Oct 6, 2017

Contributor

Ребят, я уже развернул - клепаю.

Contributor

awd-studio commented Oct 6, 2017

Ребят, я уже развернул - клепаю.

@Director-cemetery

This comment has been minimized.

Show comment
Hide comment
@Director-cemetery

Director-cemetery Oct 6, 2017

В Мозилле баг не пашет, только в Хроме.
ArmGono сорри, я что по ссылке, не знал. Хотя и по ссылке у нас много дурачков похвастаться найдётся, если честно, знают двое - знает свинья.

Director-cemetery commented Oct 6, 2017

В Мозилле баг не пашет, только в Хроме.
ArmGono сорри, я что по ссылке, не знал. Хотя и по ссылке у нас много дурачков похвастаться найдётся, если честно, знают двое - знает свинья.

@Director-cemetery

This comment has been minimized.

Show comment
Hide comment
@Director-cemetery

Director-cemetery Oct 6, 2017

<<Я искал баг в issue>>
То ли там был закамуфлированный разговор, то ли в закрытом чате долго мусолили с год наверное назад ещё.
Про слив минусами все помнят, кстати?

Director-cemetery commented Oct 6, 2017

<<Я искал баг в issue>>
То ли там был закамуфлированный разговор, то ли в закрытом чате долго мусолили с год наверное назад ещё.
Про слив минусами все помнят, кстати?

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Oct 6, 2017

Contributor

Антоха фиксил, вроде. Чекните, кто знает.

Contributor

mbaev commented Oct 6, 2017

Антоха фиксил, вроде. Чекните, кто знает.

@ArmGono

This comment has been minimized.

Show comment
Hide comment
@ArmGono

ArmGono Oct 6, 2017

В Мозилле баг не пашет, только в Хроме.

Я воспроизводил в FIrefox Developer Edition (v57.beta точную версию не знаю) на ОС Kubuntu. И притом случайно наткнулся.
Так же надо будет пофиксить обновление количества и замена ссылки при нажатии.

ArmGono commented Oct 6, 2017

В Мозилле баг не пашет, только в Хроме.

Я воспроизводил в FIrefox Developer Edition (v57.beta точную версию не знаю) на ОС Kubuntu. И притом случайно наткнулся.
Так же надо будет пофиксить обновление количества и замена ссылки при нажатии.

@Director-cemetery

This comment has been minimized.

Show comment
Hide comment
@Director-cemetery

Director-cemetery Oct 6, 2017

сорри mbaev не так тебя понял

Director-cemetery commented Oct 6, 2017

сорри mbaev не так тебя понял

@Director-cemetery

This comment has been minimized.

Show comment
Hide comment
@Director-cemetery

Director-cemetery Oct 6, 2017

55.0.3 (32-бит) баг есть (очень странно) обновляю мозиллу до 56.0 (32-бит) тоже есть.
Год не было, я честно пытался )) То ли релиз виноват (хотя вряд ли) то ли Мозилла что то обновила и пошло как в Хроме.

Director-cemetery commented Oct 6, 2017

55.0.3 (32-бит) баг есть (очень странно) обновляю мозиллу до 56.0 (32-бит) тоже есть.
Год не было, я честно пытался )) То ли релиз виноват (хотя вряд ли) то ли Мозилла что то обновила и пошло как в Хроме.

@Director-cemetery

This comment has been minimized.

Show comment
Hide comment
@Director-cemetery

Director-cemetery Oct 6, 2017

Посмотрел ролик, первый раз вижу эту технологию ))
я по другому всё.
Хотя у линуксойдов свои приколы, лан.

Director-cemetery commented Oct 6, 2017

Посмотрел ролик, первый раз вижу эту технологию ))
я по другому всё.
Хотя у линуксойдов свои приколы, лан.

@ArmGono

This comment has been minimized.

Show comment
Hide comment
@ArmGono

ArmGono Oct 7, 2017

Посмотрел ролик, первый раз вижу эту технологию
Вроде ничего странного там нет, нажал Спасибо. Ссылка не изменилась, ничего не произошло.... еще раз нажал. Опять то же самое... еще пару раз, потом решил обновить страницу - смотрю на 4 голоса увеличилось... подумал вдруг это я? и начал проверять.
Кстати то же самое и на домашнем (win10 x64) Linux тут не причем

ArmGono commented Oct 7, 2017

Посмотрел ролик, первый раз вижу эту технологию
Вроде ничего странного там нет, нажал Спасибо. Ссылка не изменилась, ничего не произошло.... еще раз нажал. Опять то же самое... еще пару раз, потом решил обновить страницу - смотрю на 4 голоса увеличилось... подумал вдруг это я? и начал проверять.
Кстати то же самое и на домашнем (win10 x64) Linux тут не причем

@ArmGono

This comment has been minimized.

Show comment
Hide comment
@ArmGono

ArmGono Oct 7, 2017

и еще в догонку бага...
1507418496479

ArmGono commented Oct 7, 2017

и еще в догонку бага...
1507418496479

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Oct 8, 2017

Contributor

@ArmGono это как раз оно

Contributor

mbaev commented Oct 8, 2017

@ArmGono это как раз оно

@awd-studio

This comment has been minimized.

Show comment
Hide comment
@awd-studio

awd-studio Oct 8, 2017

Contributor

Пересчет пока не делал, только фиксонул мультиспасибки и рефакторнул (плюс немного няшности добавил).

Позже напишу пересчет (ннада, кстати, придумать как всякие хук_апдейты запускать при деплое, и там модульки вкл/выкл делать).

Contributor

awd-studio commented Oct 8, 2017

Пересчет пока не делал, только фиксонул мультиспасибки и рефакторнул (плюс немного няшности добавил).

Позже напишу пересчет (ннада, кстати, придумать как всякие хук_апдейты запускать при деплое, и там модульки вкл/выкл делать).

@ArmGono

This comment has been minimized.

Show comment
Hide comment
@ArmGono

ArmGono Oct 8, 2017

(ннада, кстати, придумать как всякие хук_апдейты запускать при деплое, и там модульки вкл/выкл делать).

А в чем собственно проблема?
Создать hook_update_N и сменить версию модуля. Update.php запустит hook_update().

ArmGono commented Oct 8, 2017

(ннада, кстати, придумать как всякие хук_апдейты запускать при деплое, и там модульки вкл/выкл делать).

А в чем собственно проблема?
Создать hook_update_N и сменить версию модуля. Update.php запустит hook_update().

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Oct 8, 2017

Contributor

Да, это все настроено. Просто пишите hook_update_N и он накатится при апдейте.

Contributor

mbaev commented Oct 8, 2017

Да, это все настроено. Просто пишите hook_update_N и он накатится при апдейте.

@zvse

This comment has been minimized.

Show comment
Hide comment
@zvse

zvse Oct 21, 2017

под функционал прямо просится Flag

zvse commented Oct 21, 2017

под функционал прямо просится Flag

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Oct 31, 2017

Contributor

Исправлено

Contributor

mbaev commented Oct 31, 2017

Исправлено

@mbaev mbaev closed this Oct 31, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment