SB-MaterialAdmin / Web

SourceBans Material Design
64 stars 45 forks source link

Управление блокировками при существующих банах #344

Open BlackYuzia opened 2 years ago

BlackYuzia commented 2 years ago

@CrazyHackGUT

Описание

Почему в СБ нет:

Скриншоты

Пример попытки редактировать бан, при этом у этого игрока есть еще баны на других серверах: image image

CrazyHackGUT commented 2 years ago

возможности выдавать баны игрокам у которых уже есть баны (а что, так можно было?)

Зачем?

возможности редактировать баны игрока при условии что у него уже есть другие баны (на других серверах)

Ты не можешь выдать более одного бана. Специфичность плагинов пока опускаем.

возможности выбрать сервер(-а) на котором ты хочешь забанить игрока

Потому что суть СБ - в единой системе банов, и нехрен мучать жопу, пытаясь присрать сюда ограничение по серверу для бана.

BlackYuzia commented 2 years ago

Зачем? Ты не можешь выдать более одного бана. Специфичность плагинов пока опускаем. Потому что суть СБ - в единой системе банов, и нехрен мучать жопу, пытаясь присрать сюда ограничение по серверу для бана.

Ну потому-что в настройках МА (плагин) можно выставить чтобы бан(ы) и мут(ы) на других серверах не считался, тем самым идет разделение по серверам (каждый сервер имеет свои баны и муты)

А в таком случае через ВЕБ можно забанить только в таких вариантах:

Если веб может указать один сервер, он может указать любой из (а значит выбрать можно). Веб бан я так понимаю будет сразу на всех.

Кстати, интересно что будет если один и тот же игрок будет сразу на двух и более серверах. Его забанит сразу на двоих?

К тому же, а смысл с того что веб-админ не может выдать более одного активного бана игрокам? В чем смысл то? Тот кто придумал эту логику сосет большой-большой шлен. Ну или тот кто добавил логику / возможность разделять баны по серверам, но без реализации этой же возможности на вебе.

CrazyHackGUT commented 2 years ago

Ну потому-что в настройках МА (плагин) можно выставить чтобы бан(ы) и мут(ы) на других серверах не считался, тем самым идет разделение по серверам (каждый сервер имеет свои баны и муты)

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

Кстати, интересно что будет если один и тот же игрок будет сразу на двух и более серверах. Его забанит сразу на двоих?

Это невозможная чисто технически ситуация.

К тому же, а смысл с того что веб-админ не может выдать более одного активного бана игрокам?

А в чём смысл выдавать два бана одному и тому же игроку? Подумай.

без реализации этой же возможности на вебе

Потому что умеет только один плагин. Смысл делать ради одного плагина это? Фрагментацию системы повышать? Она и так высокая, а тут можно будет уже вешаться.

BlackYuzia commented 2 years ago

Завези настройку и в старый плагин - тогда это будет иметь смысл.

Из нас двоих я вообще жуесер. Могу написать тебе на заборе шлен. А вот в этот ваш сорспавн я пока не лез. Возможно полезу когда-нибудь. Но не сейчас.

Это невозможная чисто технически ситуация.

Да, но что если? Условно ты добавил 2 сервера с разными ip но этот один и тот же сервер - тогда это возможно

А в чём смысл выдавать два бана одному и тому же игроку? Подумай.

Смысл появляется тогда когда у тебя идет разбитие банов / мутов по серверам. Подумай. Но лучше не думай и сделай или скажи где убрать проверки - я сцук даже PR сделаю ради такого. Если там все проще чем я думаю. А если нет - тогда сам делай. Я в этой вашей пыхе разбираться не хочу. Еще учить синтаксис пыхи :eyes:

Потому что умеет только один плагин. Смысл делать ради одного плагина это? Фрагментацию системы повышать? Она и так высокая, а тут можно будет уже вешаться.

У тебя система и так с 2008 года. (ладно, 12-13? не суть). Логикой больше, логикой меньше. Один Крузя по нормальному все бы переписать вообще.

Тут повторяюсь - добавляй ту же логику в старую версию плагина и допиливай веб

CrazyHackGUT commented 2 years ago

Условно ты добавил 2 сервера с разными ip но этот один и тот же сервер - тогда это возможно

Зачем?

BlackYuzia commented 2 years ago

Зачем?

Чтобы проверить что будет если игрок будет сразу на двух или более серверах.

CrazyHackGUT commented 2 years ago

Чтобы проверить что будет если игрок будет сразу на двух или более серверах.

На одном найдёт и перестанет искать, отправит команду кика. Емнип.

BlackYuzia commented 2 years ago

На одном найдёт и перестанет искать, отправит команду кика. Емнип

А если удалить код который отвечает за "остановиться" и добавить немного логики? Он создаст для каждого сервера свой (отдельный) бан?

rm-veil commented 2 years ago

А в чём смысл выдавать два бана одному и тому же игроку? Подумай.

Ну возьмём пример юзю у которого стоит AntiDLC от феникса который банит игрока только на одном сервере (всего серверов 15). Игрок получил бан от античита на одном сервере и пошёл играть на другой, там он нарушил правило и я как админ должен его забанить, НО, из-за того что у игрока уже есть действующий бан я этого сделать не смогу. Обычно таких мы просто перебаниваем веб баном и всё, но юзе этот вариант не нравится (это же юзя, к'мон) ибо "читер которому не повезло нарушить правила на другом сервере получит бан на всех, а другой читер который не нарушал правила на другом сервере будет в бане только на одном" Это конечно глупое суждение ибо читер должен быть в бане на всех серверах, а не только там где он решил с читом зайти но к'мон, это же юзя "Читер получил наказание? Получил, все Зачем ВЫБОРОЧНО выдавать баны?" (с) Юзя

CrazyHackGUT commented 2 years ago

Игрок получил бан от античита на одном сервере и пошёл играть на другой, там он нарушил правило и я как админ должен его забанить, НО, из-за того что у игрока уже есть действующий бан я этого сделать не смогу.

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

BlackYuzia commented 2 years ago

@CrazyHackGUT научи всех разделять баны. Ты вроде говорил что чисто из-за старого плагина не можешь это сделать.

Выкати обнову плагина (старого) добавив возможность банить отдельно по серверам

CrazyHackGUT commented 2 years ago

Дублируй исью в репу старого плагина.

BlackYuzia commented 2 years ago

Дублируй исью в репу старого плагина.

Сделай ты :eyes:

rm-veil commented 2 years ago

Кстати, интересно что будет если один и тот же игрок будет сразу на двух и более серверах. Его забанит сразу на двоих?

Это невозможная чисто технически ситуация.

Как тебе такое, Илон Маск👀 IMG_20220829_011755

BlackYuzia commented 2 years ago

Ну, все ясно, @CrazyHackGUT🏳️‍🌈 - круздабол

CrazyHackGUT commented 2 years ago

Если будете сейчас и в репозитории заниматься публично той хренью, что происходит в Телеге и Дискорде -- я церемониться не буду, и просто отправлю на банановые острова тут. Это начинает мне надоедать.