WarnSystem 1.6.5 (фиксы модулей)

[pentaxist]

зачем тебе это в сурсбанс прописывать?
Ты в punish.cfg а конкретно в sm_ws_sourcebans_type установи цифру соответсвующую твоей бан системе

или я дурак или хз...
cfg/warnsystem/sourcebans.cfg я ничего не прописывал в сб, конфиг создается автоматически при старте варнов, я только убрал 2 как ты советовал и все.
а в punish.cfg у меня только это

 

[m0rbit]

или я дурак или хз...
cfg/warnsystem/sourcebans.cfg я ничего не прописывал в сб, конфиг создается автоматически при старте варнов, я только убрал 2 как ты советовал и все.
а в punish.cfg у меня только это

Посмотреть вложение 69059

/csgo/cfg/warnsystem

Спойлер: скрины

Что у тебя в sourcebans.cfg ?)

 

[September]

или я дурак или хз...
cfg/warnsystem/sourcebans.cfg я ничего не прописывал в сб, конфиг создается автоматически при старте варнов, я только убрал 2 как ты советовал и все.
а в punish.cfg у меня только это

Посмотреть вложение 69059

дождись обновы от @Rabb1t

 

[pentaxist]

/csgo/cfg/warnsystem

Спойлер: скрины

Посмотреть вложение 69075
Посмотреть вложение 69076

Что у тебя в sourcebans.cfg ?)

sourcebans.cfg

Спойлер

 

[m0rbit]

ну тут я хз( у меня по другому..

 

[Rabb1t]

Rabb1t обновил(а) ресурс WarnSystem новой записью:

Перезалив и небольшие фиксы

В ходе небольших тестовых работ с @September (за что ему спасибо) выяснили, что модуль Punish и SourceBans не стоит совмещать, иначе могут возникнуть проблемы.
Также стоит удалить конфиги (/cfg/sourcemod...) от модулей, дабы создались новые.
Прочие фиксы.

Узнать больше об этом обновлении...

 

[pentaxist]

Rabb1t обновил(а) ресурс WarnSystem новой записью:

Перезалив и небольшие фиксы



Узнать больше об этом обновлении...

L 12/21/2020 - 11:05:17: [WarnSystem/WS_WarnAdmin.smx] Database failure: Could not find database config "sourcebans". See FAQ: SB-MaterialAdmin/Web

// Type of using sourcebans, where 0 - sb (old), 1 - sb++ (new), 2 - MaterialAdmin (MA FORK)
// -
// Default: "2"
sm_ws_sourcebans_type "2"

 

[Rabb1t]

@pentaxist, попробуй перекомпилировать без библиотек sb и sb++. Этот модуль не использует квары на поддержку типа sb, то, что ты показал либо из sourcebans.cfg, либо punish.cfg.
----------------------------------------------------------------------
Rabb1t обновил(а) ресурс WarnSystem новой записью:

Фиксы модулей

Исправлен баг, когда модуль Punish выдавал наказание админу, выдающему это самое наказание. (Спасибо @September за инфу о баге).
Исправлен баг в модуле WS_Admin (логические операции).

Узнать больше об этом обновлении...

 

[JDW]

Хочу дать рекомендации для вас, так как проект не плохой, но в мелочах тоже нужно быть внимательнее, возможно где-то я буду не компетентный, так как опыта работы на sourcepawn нет, но я ради интереса хочу провести рецензирование кода(таким образом учусь), так как sourcepawn является процедурным языком программирования и по большей части (синтаксис и подход к написанию программ) соответствует программам, которые пишутся на C (не по спецификации, кто в теме, тот поймет).

Спойлер: #1

Rabb1tof/WarnSystem странно, что для Engine_Left4Dead, Engine_Left4Dead2 используется Rabb1tof/WarnSystem

Спойлер: #2

Rabb1tof/WarnSystem а где проверка для iAdmin, не логично же, если мы для iClient делаем проверку Rabb1tof/WarnSystem
А лучше прибегать к userId, если игрок вышел с игры, то при вызове функции alliedmodders/sourcemod

sourcemod/plugins/include/clients.inc at 978fa6b25285b2c380547042e185773b62d83316 · alliedmodders/sourcemod

SourceMod - Source Engine Scripting and Administration - alliedmodders/sourcemod

github.com

Аналогично Rabb1tof/WarnSystem

Спойлер: #3

У вас есть Rabb1tof/WarnSystem зачем тогда в функциях выше вы не применяете данную функцию?) А еще раз лучше проверять по userId

Спойлер: #4

WarnSystem/scripting/WarnSystem/database.sp at 202b6d573fbf0b2c5049f90310e97e191d34d0b9 · Rabb1tof/WarnSystem

Universal warn system for Sourcemod. Contribute to Rabb1tof/WarnSystem development by creating an account on GitHub.

github.com

Давай перепишем вот так, но здесь дело вкуса конечно)

public void SQL_CheckData(Database hDatabase, DBResultSet hDatabaseResults, const char[] sError, int iClient)
{
    if (hDatabaseResults == INVALID_HANDLE || sError[0] || !IsValidClient(iClient))
    {
        //LogWarnings("[WarnSystem] SQL_CheckData - error while working with data (%s)", sError);
        return;
    }

        char dbQuery[513], szName[64], sEscapedClientName[129];
        GetClientName(iClient, szName, sizeof(szName));
        SQL_EscapeString(g_hDatabase, szName, sEscapedClientName, sizeof(sEscapedClientName));
        if (hDatabaseResults.RowCount == 0) {
            FormatEx(dbQuery, sizeof(dbQuery), g_sSQL_UploadData, g_iAccountID[iClient], sEscapedClientName, g_iWarnings[iClient]);
            if(g_bLogQuery)
                LogQuery("SQL_UnWarnPlayer::g_sSQL_UnwarnPlayerW: %s", dbQuery);
            g_hDatabase.Query(SQL_UploadData, dbQuery, iClient);
            return;
        }
        else {
            FormatEx(dbQuery, sizeof(dbQuery), g_sSQL_UpdateData, g_iAccountID[iClient], sEscapedClientName, g_iAccountID[iClient]);
            if(g_bLogQuery)
                LogQuery("SQL_UnWarnPlayer::g_sSQL_UnwarnPlayerW: %s", dbQuery);
            g_hDatabase.Query(SQL_UpdateData, dbQuery, iClient);
            return;
        }
}

Спойлер: #5

WarnSystem/scripting/WarnSystem/commands.sp at 202b6d573fbf0b2c5049f90310e97e191d34d0b9 · Rabb1tof/WarnSystem

Universal warn system for Sourcemod. Contribute to Rabb1tof/WarnSystem development by creating an account on GitHub.

github.com

Если я не ошибаюсь, то при return Plugin_Continue будет писать, что команда не найдена, следовательно при любом выходе из функции надо использовать return Plugin_Handled;

Спойлер: #6

Rabb1tof/WarnSystem Наверное лень проверить и выдать исключение, как в других нативах ?)

Спойлер: #7

WarnSystem/scripting/WarnSystem/api.sp at 202b6d573fbf0b2c5049f90310e97e191d34d0b9 · Rabb1tof/WarnSystem

Universal warn system for Sourcemod. Contribute to Rabb1tof/WarnSystem development by creating an account on GitHub.

github.com

sourcemod/plugins/include/core.inc at 978fa6b25285b2c380547042e185773b62d83316 · alliedmodders/sourcemod

SourceMod - Source Engine Scripting and Administration - alliedmodders/sourcemod

github.com

Делаем логичный вывод: из перечисления Action Plugin_Continue имеет значение 0, следовательно при создание переменной типа Int <> view_as<Action>(int) будет находиться значение 0, так как при создание переменной в отличии от ЯП С здесь будет инициализация, даже если программист этого указывать не будет

Спойлер: #8

WarnSystem/scripting/WarnSystem/api.sp at 202b6d573fbf0b2c5049f90310e97e191d34d0b9 · Rabb1tof/WarnSystem

Universal warn system for Sourcemod. Contribute to Rabb1tof/WarnSystem development by creating an account on GitHub.

github.com

WarnSystem/scripting/WarnSystem.sp at 202b6d573fbf0b2c5049f90310e97e191d34d0b9 · Rabb1tof/WarnSystem

Universal warn system for Sourcemod. Contribute to Rabb1tof/WarnSystem development by creating an account on GitHub.

github.com

WarnSystem/scripting/WarnSystem.sp at 202b6d573fbf0b2c5049f90310e97e191d34d0b9 · Rabb1tof/WarnSystem

Universal warn system for Sourcemod. Contribute to Rabb1tof/WarnSystem development by creating an account on GitHub.

github.com

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

В общем это то, что я смог увидеть при своем уровни знаний. Автор ресурса, не пойми меня не правильно, это моя рецензия никакого негативного характера она не несет.

P.S Для проверки игрока на валидность при помощи GetClientUserId/GetClientOfUserId не стоит этим злоупотреблять, так как это является лишними тактами. Это имеет смысла быть при обработке асинхронных запросов (Например: при работе с базой данных, событиях и тд)

Например где смысла это не имеет:

public Action Command_Example(int client, int args)
{
    Print(client);
    return Plugin_Handle;
}

void Print(const int client)
{
    PrintToChat(client, "Hello, world!");
}

Ресурс бомба, так что рекомендую его к установке на сервер!

 

[Rabb1t]

@AnYY, спасибо, конечно, за всю рецензию, но...код на гитхабе ещё не был обновлён по некоторым причинам. Если хотите, можете связать со мной в телеграмме или дискорде (контакты в профиле), дам доступ к закрытому репозиторию, т.к. там проводилась работа над текущей версией. Что-то там безусловно осталось из того, что Вы выше привели в примеры, но тем не менее не всё.

 

[Punisher89]

@Rabb1t, Стеянаюсь спросить,а чем отличается от старого? Я разницы не увидел. Возможно я не так поставил,но,как не работала функция снятия предупреждения и как ни работала она с сайтом sourceban(material),так и не работает/

 

[ifx]

рабочая система предупреждений для css34, поддержка сурсбанс
взято с оленей, проверено, все ок

 

[justice]

@Rabb1t, Поставил на сервер в админ меню не появился, но все время циклится дублированная, рандомная вкладка, но уже не рабочая. Это могло произойти, что я phrases под себя настраивал?

 

[D1fox]

@Rabb1t, Поставил на сервер в админ меню не появился, но все время циклится дублированная, рандомная вкладка, но уже не рабочая. Это могло произойти, что я phrases под себя настраивал.

Если закосипорил что либо, то да, а если все по уму сделанно, то должно быть все оке!

 

[Rabb1t]

@Rabb1t, Поставил на сервер в админ меню не появился, но все время циклится дублированная, рандомная вкладка, но уже не рабочая. Это могло произойти, что я phrases под себя настраивал?

Так сразу сказать не могу, не зная что ты менял, а главное как. Ошибки вообще присутствуют какие (логи)?

 

[Limbo]

Здравствуйте,Подскажите что за ошибка пожалуйста

Спойлер: ошибка

nfo (map "workshop/913021229/de_dust2_rf") (file "/home/pve/csgoserver/csgo_public/csgo/addons/sourcemod/logs/errors_20210123.log")
L 01/23/2021 - 00:03:36: [WarnSystem/WS_WarnAdmin.smx] Database failure: Could not find database config "sourcebans". See FAQ: SB-MaterialAdmin/Web
L 01/23/2021 - 00:03:38: Error log file session closed.

Я пользуюсь МА

 

[Rabb1t]

Здравствуйте,Подскажите что за ошибка пожалуйста

Спойлер: ошибка

nfo (map "workshop/913021229/de_dust2_rf") (file "/home/pve/csgoserver/csgo_public/csgo/addons/sourcemod/logs/errors_20210123.log")
L 01/23/2021 - 00:03:36: [WarnSystem/WS_WarnAdmin.smx] Database failure: Could not find database config "sourcebans". See FAQ: SB-MaterialAdmin/Web
L 01/23/2021 - 00:03:38: Error log file session closed.

Я пользуюсь МА

А в конфиге модуля выбран режим работы с МА?

 

[Limbo]

А в конфиге модуля выбран режим работы с МА?

// Type of using sourcebans, where 0 - sb (old), 1 - sb++ (new), 2 - MaterialAdmin (MA FORK)
// -
// Default: "2"
sm_ws_sourcebans_type "2"

Правильно?

 

[kylc]

// Type of using sourcebans, where 0 - sb (old), 1 - sb++ (new), 2 - MaterialAdmin (MA FORK)
// -
// Default: "2"
sm_ws_sourcebans_type "2"

Правильно?

Да

 

[Limbo]

Да

Ну дак и почему ошибка в ерррор логах?))