search

pavel-pimenov / flylinkdc-r5xx

flylinkdc-r5xx
GNU General Public License v2.0
55 stars 26 forks source link

Бот хаба не показывается в списке пользователей #955

Closed pavel-pimenov closed 9 years ago

pavel-pimenov commented 9 years ago

From toss.Alexey on February 28, 2013 15:23:54

What steps will reproduce the problem? 1. Зайти на хаб flylinkdc.dyndns.org

  1. Попытаться найти в списке пользователей ник Invisible What is the expected output? What do you see instead? Ожидаю увидеть в списке ник бота, как это и происходит в других клиентах. Но его в списке нет. What version of the product are you using? On what operating system? -=[ FlylinkDC++ r502 -beta66 build 13050 HE Compiled on: 2013-02-27 ]=- -=[ FlylinkDC++ r502 -beta66 build 12954 Compiled on: 2013-02-21 ]=- Please provide any additional information below. Если это фича, то она должна быть отключаемой, разве нет?

Original issue: http://code.google.com/p/flylinkdc/issues/detail?id=916

pavel-pimenov commented 9 years ago

From Pavel.Pimenov@gmail.com on February 28, 2013 06:45:37

Это в Ежовой версии? он там что-то мудрил с ними. почему рассказано про билд 12954: у меня видно https://www.box.net/shared/8zj0eaa9hh8wlebsw2in

pavel-pimenov commented 9 years ago

From a.rain...@gmail.com on February 28, 2013 06:53:54

Да я и на ежовой это подтвердить не могу http://studia2000.sytes.net/image/Flylink/no-error-on-user-filtering.png покажите скрин что ли.

p.s:

он там что-то мудрил с ними. поиском по коду FLYLINKDC_HE - там всё одинаково ;)

Status:
Labels: -Type-Defect -Priority-Medium Type-Other

pavel-pimenov commented 9 years ago

From a.rain...@gmail.com on February 28, 2013 06:57:45

Тут в чатике разрабов высказали мнение, что вы не ОП, по этому и бота не видите - настройки хаба таковы.

Status: Done

pavel-pimenov commented 9 years ago

From Pavel.Pimenov@gmail.com on February 28, 2013 06:59:55

Вопрос.

  1. откуда вы узнали что он там есть?
  2. зачем вам его видеть?

У него статус не видимый - я вот например оп. но видеть его тоже не особо хочу. но объяснить и доказать это не могу.

Status: Accepted
Owner: Pavel.Pimenov@gmail.com

pavel-pimenov commented 9 years ago

From toss.Alexey on February 28, 2013 07:11:30

По скриншотам: бота нет в списке, пока вы не начинаете писать в окошке фильтра списка пользователей или пока выключите-включите показ списка пользователей. Попробуйте найти бота в списке перезайдя на хаб и не прибегая к этим фокусам? У меня не получилось (см. скриншот).

"Тут в чатике разрабов высказали мнение, что вы не ОП, по этому и бота не видите - настройки хаба таковы."

Мнение неправильное, ведь как я уже написал, другие клиенты (ApexDC++, EiskaltDC++, Greylink) нормально показывают бота в списке без лишних телодвижений.

"У него статус не видимый"

Нет такого статуса в NMDC.

Attachment: 20130228_FLDCPP_HubBot.png

pavel-pimenov commented 9 years ago

From toss.Alexey on February 28, 2013 07:17:11

"2. зачем вам его видеть?"

Конкретно этот бот мне не сильно нужен, я его взял просто для иллюстрации проблемы. А вообще боты могут выполнять разнообразные задачи: это и комнаты и всякие викторины и боты для управления хабом. Что-бы начать использовать такого бота, достаточно написать ему в приват, но как писать, если его нет в списке?

pavel-pimenov commented 9 years ago

From toss.Alexey on February 28, 2013 07:18:25

Вдобавок, совершенно непонятно, по каким критериям вы решаете скрыть или показать бота.

pavel-pimenov commented 9 years ago

From Pavel.Pimenov@gmail.com on February 28, 2013 07:28:34

CT_HIDDEN = 64 Вот код оригинального клиента

dcplusplus\dcpp\OnlineUser.h

/* One of possibly many identities of a user, mainly for UI purposes / class Identity : public Flags { public: enum IdentityFlagBits { GOT_INF_BIT, NMDC_PASSIVE_BIT }; enum IdentityFlags { GOT_INF = 1 << GOT_INF_BIT, NMDC_PASSIVE = 1 << NMDC_PASSIVE_BIT }; enum ClientType { CT_BOT = 1, CT_REGGED = 2, CT_OP = 4, CT_SU = 8, CT_OWNER = 16, CT_HUB = 32, CT_HIDDEN = 64 };

pavel-pimenov commented 9 years ago

From toss.Alexey on February 28, 2013 07:33:23

А этот код относится не к ADC?

pavel-pimenov commented 9 years ago

From Pavel.Pimenov@gmail.com on February 28, 2013 07:43:16

Нет - юзается в NMDC

D:\bz-src\dcplusplus\dcpp\NmdcHub.cpp

            OnlineUser& u = getUser(nick);

            // If he is already considered to be the hub (thus hidden), probably should appear in the UserList
            if(u.getIdentity().isHidden()) {
                    u.getIdentity().setHidden(false);
                    u.getIdentity().setHub(false);
pavel-pimenov commented 9 years ago

From Pavel.Pimenov@gmail.com on February 28, 2013 07:44:19

В оригинальном флае будет показывать везде. в Ежовом будет как раньше - прятаться https://code.google.com/p/flylinkdc/source/detail?r=13071#

pavel-pimenov commented 9 years ago

From a.rain...@gmail.com on February 28, 2013 07:51:21

Конкретно этот бот мне не сильно нужен, я его взял просто для иллюстрации проблемы. А вообще боты могут выполнять разнообразные задачи: это и комнаты и всякие викторины и боты для управления хабом. Что-бы начать использовать такого бота, достаточно написать ему в приват, но как писать, если его нет в списке? Скрываются только те боты для которых хаб прислал соответствующий флаг, если флага нет - бот будет в списке.

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

p.s: в ежовой починю баг с отображением таких ботов.

pavel-pimenov commented 9 years ago

From Pavel.Pimenov@gmail.com on February 28, 2013 07:54:31

Давайте еще уточним этот флаг есть в стандарте или нет?

pavel-pimenov commented 9 years ago

From a.rain...@gmail.com on February 28, 2013 08:22:29

В самом протоколе - точно нет. Однако с NMDC у всех разработчиков возникает попоболь в плане совместимости, из-за того, что есть куча расширений, которые все поголовно используют, но которые официально не приняты.

pavel-pimenov commented 9 years ago

From Pavel.Pimenov@gmail.com on February 28, 2013 08:44:42

Флаг мальчик-девочка и то был бы полезней :)

pavel-pimenov commented 9 years ago

From a.rain...@gmail.com on February 28, 2013 08:47:31

Паш, вот скажи, зачем видить ботов, которым не написать нельзя, не скачать с них ничего нельзя, и вообще ничего полезного с ними сделать нельзя? :)

pavel-pimenov commented 9 years ago

From Pavel.Pimenov@gmail.com on February 28, 2013 09:01:59

Тогда их не нужно создавать вообще в клиенте. а ты их героически прячешь выполняя всегда фильтрацию всех юзеров по этому флагу. Также если они такие бесполезные зачем они нужны?

pavel-pimenov commented 9 years ago

From a.rain...@gmail.com on February 28, 2013 09:14:38

Признак скрытости передаётся не сразу, по этому при обновлении инфы о юзере надо его проверять, и если юзер теперь скрыт - дельнуть из списка. В общем этот флаг необходим для полностью информационных ботов - они могут что то писать в чатик, но видить их в списке при этом нет необходимости. К примеру можно создать бота "Illegal search filter" который будет уведомлять пользователя о том, что он ищет что то запрещённое и по этому поиск не будет осуществлён, очевидно, что у такого бота не может быть настроек, или чего то подобного, по этому в списке он не нужен. Так же скрыт бот хаба, он тоже может писать в чатик, но более ничего делать не может, и как либо взаимодействовать с ним не имеет смысла. Возможно не стоит их создавать вообще, однако не уверен, что при этом будут корректно работать фильтры спама от фейковых юзеров - это надо посмотреть.

pavel-pimenov commented 9 years ago

From toss.Alexey on February 28, 2013 10:17:53

"Скрываются только те боты для которых хаб прислал соответствующий флаг, если флага нет - бот будет в списке."

Лично я из приведённого кода не понял, что понимается под таким флагом в NMDC? Можно пример в терминах NMDC?

pavel-pimenov commented 9 years ago

From Pavel.Pimenov@gmail.com on February 28, 2013 10:22:50

Я предложил Алексею просто таким ботам-нивидимкам иконку менять это будет намного эффективней не хочет - говорит невидимый значит никто не видит. кино про невидимку не смотрел наверно. кинуть ему чтоль магнит )

pavel-pimenov commented 9 years ago

From toss.Alexey on February 28, 2013 10:23:32

Скриншот клиента DC++, бот отображается нормально без всякой магии

Attachment: 20130228_DCPP_HubBot.PNG

pavel-pimenov commented 9 years ago

From Pavel.Pimenov@gmail.com on February 28, 2013 10:31:12

Это инициатива исклбючительно Алексея. он считает, что если невидимка, то значит не нужно видеть

pavel-pimenov commented 9 years ago

From a.rain...@gmail.com on February 28, 2013 10:47:54

Не совсем так. Эта инициатива SMT, т.е. ещё со времён Apex Speed Mode, просто я переделывал, как и многое другое в клиенте. Ну а так - да, имхо - инициатива правильная.

pavel-pimenov commented 9 years ago

From toss.Alexey on February 28, 2013 10:54:17

Бот отображается нормально в -=[ ApexDC++ s16.4 Compiled on: 2008-06-12 ]=-

Я ещё раз прошу показать, что вы считаете невидимкой в NMDC?

pavel-pimenov commented 9 years ago

From toss.Alexey on February 28, 2013 11:00:22

"Флаг мальчик-девочка и то был бы полезней :)"

Про этот флаг можно забыть в том виде, в котором он реализован в грейлинке - конфликтует с PtokaX 0.5.0.0.

pavel-pimenov commented 9 years ago

From Pavel.Pimenov@gmail.com on February 28, 2013 18:46:08

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

pavel-pimenov commented 9 years ago

From Pavel.Pimenov@gmail.com on March 01, 2013 06:35:38

Почему молчим? :)

pavel-pimenov commented 9 years ago

From a.rain...@gmail.com on March 02, 2013 11:27:57

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

pavel-pimenov commented 9 years ago

From a.rain...@gmail.com on March 02, 2013 11:32:44

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

pavel-pimenov commented 9 years ago

From zippoz...@gmail.com on March 14, 2013 03:29:37

Я волью капельку скипидара в ваши обсуждения:

После недавних действий со "скрытыми ботами" в транке (в серой ветке), в NMDC хабах я вижу ВСЕХ ботов, даже тех, которые не зарегистрированы хабом, то-есть для них не выполнялась команда Core.RegBot(). Эти боты обычно сидят молча, пока в менюшке не выбирают какой-либо пункт, например "Показать мне анекдот". Тогда такие боты под своим ником срут в чат или личку.

Теперь же, ВСЕ эти боты в списке пользователей. Ответьте мне,- нафига??? Сочиняя скрипт, я специально не публикую бота, чтобы его не торкали запросами списков файлов, личками и прочей хернёй. А тут нате - вывалили всё что непопадя наружу.

За сим, предлагаю вернуть всё как было и как есть в Ежовой сборке.

pavel-pimenov commented 9 years ago

From mike.kor...@gmail.com on March 19, 2013 05:35:42

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

pavel-pimenov commented 9 years ago

From a.rain...@gmail.com on May 01, 2013 05:13:06

Про расплодившихся ботов - Issue 944 .

pavel-pimenov commented 9 years ago

From Pavel.Pimenov@gmail.com on May 01, 2013 05:46:29

Исправлено - закрываю https://www.box.com/files/0/f/215289294/1/f_7837967230 Продолжение в Issue 944

Status: Done

pavel-pimenov commented 9 years ago

From toss.Alexey on September 27, 2013 11:53:23

Опять сломалось.

pavel-pimenov commented 9 years ago

From Pavel.Pimenov@gmail.com on September 27, 2013 11:56:07

В каком билде? я вроде в той части ничего не трогал...

Status: Accepted

pavel-pimenov commented 9 years ago

From toss.Alexey on September 27, 2013 12:14:23

В 72 бете ещё работало, в сотых бетах - уже нет, а в какой бете заметил первый раз - уже не помню. Где достать бинарников меньше 100 беты я не нашёл. Не из инсталляторов же выдирать.

pavel-pimenov commented 9 years ago

From toss.Alexey on September 27, 2013 14:14:35

Похоже, сломали в r14764 , так как за пару ревизий до него сабж на месте.

pavel-pimenov commented 9 years ago

From Pavel.Pimenov@gmail.com on September 27, 2013 15:17:50

ок. завтра утром посмотрю. p.s. Все билды сейчас лежат тут http://www.fly-server.ru/install/r5xx/src-bin

pavel-pimenov commented 9 years ago

From a.rain...@gmail.com on September 27, 2013 18:59:39

Подписался на апдейты по ишье.

Cc: a.rain...@gmail.com
Labels: Milestone-Release502 Usability

pavel-pimenov commented 9 years ago

From Pavel.Pimenov@gmail.com on December 19, 2013 09:50:11

Status: Started
Cc: -a.rain...@gmail.com

pavel-pimenov commented 9 years ago

From Pavel.Pimenov@gmail.com on December 19, 2013 10:29:59

Хаб

dchub://scalolaz.dyndns-server.com

В r422 бот мастерйода есть http://www.flickr.com/photos/96019675@N02/11453881553/ в r503 уже нет http://www.flickr.com/photos/96019675@N02/11453844556/ [22:24:33] Нет такого понятия "флаг невидимости". [22:24:43] В NMDC. [22:25:16] В ADC есть, но здесь вам не ADC!

pavel-pimenov commented 9 years ago

From Pavel.Pimenov@gmail.com on December 19, 2013 10:55:19

This issue was closed by revision r16252 .

Status: Fixed