Plots2: Скрыть ответы на комментарии, которые были отправлены спамом или еще не прошли модерацию

Кажется, это что-то в моей рулевой рубке! Я могу разбить это на части в тот момент, когда чувствую, что моя работа впереди.

Я тестировал это локально, чтобы разбить его на ftos, похоже, проблема уже исправлена. Может ли кто-нибудь подтвердить то же самое с их стороны ... @ noi5e, возможно, вы решили это где-то в своем проекте ..

@cesswairimu Хм !! Думаю, это сделал кто-то другой 😅

@cesswairimu Кто-нибудь может написать тест для этого, если я не доберусь до него? Это слишком сложно для FTO, но я думаю, что написание тестов может многому научить соискателей. Также, возможно, нам не следует закрывать эту проблему, пока не будет написан тест

@ noi5e Я согласен, я

Хм, я не уверен - может быть, сначала может пройти тест, чтобы подтвердить, действительно ли это исправлено?

Поскольку мы можем фильтровать только по использованию, а не в определении has_many , кажется, и я не вижу .where.not(status: 0) для них:

https://github.com/publiclab/plots2/blob/ebf57e0cdd1dcc9d5cc884a15e7cd5b5bbe78390/app/views/notes/_comments.html.erb#L8

https://github.com/publiclab/plots2/blob/401874455490c6300270ef54490fe9452e0ca522/app/views/questions/show.html.erb#L69

https://github.com/publiclab/plots2/blob/ebf57e0cdd1dcc9d5cc884a15e7cd5b5bbe78390/app/views/notes/_comment.html.erb#L162

Что вы думаете?

Спасибо всем!!! 💯

ах, и я думаю, что на этих includes() он должен быть назван, так что, может быть ... .where('comments.status != 0') ?

Да, я думаю, что это правильно, см. Использование здесь:

https://github.com/publiclab/plots2/blob/672d63ab11c2159e28d2cc502c47a3b3c83730b9/app/models/node.rb#L1114

@jywarren @cesswairimu @ noi5e Могу я поработать над написанием тестов для этого?

@ 17sushmita Да,

Похоже, ответы на комментарии не фильтруются на предмет спама. Мы должны увидеть это сообщение:

"Вы уверены? Пользователь больше не сможет входить в систему или публиковать, а их контент будет скрыт, кроме комментариев ». кажется неправильным

@jywarren Что это значит, когда говорится, что сообщение кажется неправильным? Разве он не должен отображаться, когда модератор пытается пометить комментарий как спам? Пожалуйста, дайте ему немного больше контекста.

@ 17sushmita Меня это тоже немного смущает. Я думаю, может быть, Джеффри имел в виду, что на изображении ниже текст отображается желтым, а не Moderate first-time comment ? Если так, то это кажется неправильным.

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

Для начала, вы можете вставить в тестовую базу данных спам / первый авторский комментарий, который является ответом на другой комментарий:

nodes(node_name).add_comment({
  uid: user_id,
  body: comment_text
  reply_to: parent_comment_id
  status: 0 # comment status: 0 for banned, 4 for first-time poster (a moderator has to approve the comment)
})

Затем напишите оттуда оставшуюся часть теста, убедившись, что метод, который мы используем для получения комментариев, не получает ответы на комментарии со статусом 0 или 4.

Что-то подобное! Вам, вероятно, придется провести небольшое исследование. Обязательно оставьте здесь комментарий, если вы застряли или вам нужны указатели, где искать в кодовой базе. Мы здесь, чтобы помочь!

@ noi5e , большое спасибо за помощь !! Мне нужно еще одно уточнение. Нужно ли мне создавать новый файл для тестов ответов на комментарии или вносить изменения, возможно, в /plots2/test/functional/admin_controller_test.rb или /plots2/test/system/spam2_test.rb или /plots2/test/functional/comment_controller_test.rb или любой другой файл?

@ 17sushmita Я не уверен на 100%, но думаю, /test/functional/comment_controller_test.rb . Другое место, о котором я подумал, было модульным тестом для комментариев. Мысли от кого-то еще?

А пока не стесняйтесь начинать работать над этим, пока мы ждем, пока другие выскажутся.

@ 17sushmita Я не уверен на 100%, но думаю, /test/functional/comment_controller_test.rb . Другое место, о котором я подумал, было модульным тестом для комментариев. Мысли от кого-то еще?

А пока не стесняйтесь начинать работать над этим, пока мы ждем, пока другие выскажутся.

Спасибо, я начал писать тесты, но я заметил одну вещь: даже после пометки комментария как спама он отображается, и в соответствии с программным обеспечением он является правильным, потому что при пометке его как спама он генерирует предупреждение, упомянутое @jywarren выше «Вы уверены? Пользователь больше не сможет входить в систему или публиковать сообщения, а их содержание будет скрыто, кроме комментариев ». но действительно ли уместно отображать комментарии, помеченные как спам? 🤔️

действительно ли уместно отображать комментарии, помеченные как спам? 🤔️

Я так не думаю!

Этот текст, который вы упомянули, довольно интересен:

«Вы уверены? Пользователь больше не сможет входить в систему или публиковать сообщения, а их содержимое будет скрыто, кроме комментариев ».

Это заставляет меня думать, что, возможно, когда-то мы не модерировали комментарии. Я думаю, что этот отрывок просто устарел.

В любом случае, я думаю, мы все согласны с тем, что мы определенно должны фильтровать все комментарии! Таким образом, мы можем избежать спам-ботов, о которых изначально упоминал @jywarren .

Я только что протестировал это локально, и мне кажется, что это не было исправлено. В настоящее время мы не фильтруем спам или впервые модерируемые комментарии ОТВЕТЫ ... Помните, что обычные комментарии (которые не являются ОТВЕТАМИ на другие комментарии) фильтруются, поэтому это относится только к ответам. (Я тестировал комментарии, которые НЕ являются ответами, и они проходят модерацию)

Разберем это немного дальше. Локально я создал новую учетную запись и опубликовал комментарий к заметке. Это то, что я вижу, когда оставляю комментарий. Все идет нормально:

Как пользователь сообщение _ "Ожидает утверждения модераторами сообщества" _ заставляет меня думать, что мой комментарий не будет опубликован.

Однако комментарий ДЕЙСТВИТЕЛЬНО публикуется! Когда я просматриваю ту же заметку в окне инкогнито, которое выходит из системы, комментарий отображается точно так же, как и пользователю _cisco_:

Запрет пользователя _cisco_, похоже, тоже ничего не меняет. Сообщение _ "Ожидает утверждения" _ исчезнет, ​​но комментарий по-прежнему можно будет просмотреть в окне в режиме инкогнито:

Так что да, это определенно нужно изменить.

@ 17sushmita Продолжайте работать над этим!

Кроме того, если вы еще не нашли его, у нас есть забаненный пользователь _spammer_, уже созданный в /test/fixtures/users.yml

@ noi5e Это было очень ясное объяснение. Я тоже очень похоже тестировал свою локальную систему, и результаты были такими же, что заставило меня так думать. Итак, я сначала поработаю над удалением спам-комментариев, а затем напишу для него тест, исправлен он или нет.

Кстати, похоже, что ответы на комментарии в настоящее время не отображаются в панели управления модерацией спама администратора:

И если пользователь cisco публикует ответ на комментарий, он не отображает их в Active Users:

Возможная проблема для проекта модерации спама Outreachy, упомянутого в № 9257?

Спасибо @ 17sushmita и @ noi5e за то, что вы так тщательно распаковали это. Приношу свои извинения за нечеткое примечание в моем исходном сообщении, но

фильтрация спама или комментариев, модерируемых впервые. ОТВЕТЫ

И отличный улов, который до сих пор не раскрыт для REPLIES.

Супер ценю вашу помощь, вы оба. 🎉

Отмечу проблему с панелью спама в описании нового проекта !!!

Привет @jywarren @ noi5e , Извините, что снова спрашиваю, но у меня возникла еще одна путаница. Должны ли мы отображать комментарий, который не является спамом, а пользователь забанен? Что делать в таких случаях?

Да, в настоящее время статус заблокированных пользователей не влияет на видимость их
Комментарии. Спасибо!

В среду, 24 марта 2021 г., 8:17 Sushmita @ . * > написал:

Привет @jywarren https://github.com/jywarren @ noi5e
https://github.com/noi5e , извините, что снова спросил, но у меня был еще один
путаница. Должны ли мы отображать комментарий, который не является спамом, но пользователь
забанен? Что делать в таких случаях?

-
Вы получаете это, потому что вас упомянули.
Ответьте на это письмо напрямую, просмотрите его на GitHub
https://github.com/publiclab/plots2/issues/8854#issuecomment-805774370 ,
или отказаться от подписки
https://github.com/notifications/unsubscribe-auth/AAAF6J4WSMFWF64K72GXR5TTFHJ6NANCNFSM4U4TWDIQ
.

Кстати, похоже, что ответы на комментарии в настоящее время не отображаются в панели управления модерацией спама администратора:

И если пользователь cisco публикует ответ на комментарий, он не отображает их в Active Users:

Возможная проблема для проекта модерации спама Outreachy, упомянутого в № 9257?

@ noi5e Я не могу воспроизвести проблему с ответами на комментарии локально. Я вошел в систему как новый пользователь, оставил комментарий, вышел из системы, а затем вошел в систему как администратор, а затем пометил ответ на комментарий как спам. Я зашел на страницу модерации спама и нашел ответ на комментарий под комментариями.

Что-то мне не хватает?

Привет, @TildaDares , я считаю, что эта проблема исправлена ​​в # 9381

Спасибо @jcads