Tech Works 1.4

[FokS]

там только при смене карты конфиг теха остается
При полной перезагрузки соответственно выставляется параметр defailt 0

Хотя ща секунду ща будет)

админ с любыми правами остаётся на сервере? Просто у меня админы без флага "z" всё равно остаются на сервере, можно это исправить? И да в админ меню нет параметра, работает только через команду в консоль.

 

[Felton]

Забил на этот плагин за ненадобностью так как меня устраивает переменная sv_password "")
Хочеш ботов можно установить bot_quota 4 или наоборот убрать bot_quota 0 для проверки или отладки плагина. После перезагрузке сервера а бывает и не раз нужна перезагрузка на сервер без пароля не пападёш .
А когда закончил дела на сервере можно и пароль снимать и всё ок!Всем удачи!

 

[d4Ck]

@parusstar,
Щас всё нормально но при смене карты ботов не кикнуло.Но иногда они нужны.Плагин нужно допиливать чтобы техработы были с ботами или без ботов.Перезагрузил сервер снял с себя права админа и я спокойно зашол.Об этом я писал на первой страницы.
Как я писал выше пока админ сам не снимет эту защиту никто не должен зайти на сервер даже после перезагрузки сервера

Омфг, ты хоть мой плагин то пробовал?

 

[Kruzya]

Вообще, я редко вмешиваюсь в "обновления ресурса" от сторонних разрабов, но увидев отзыв на 1 звезду и выложенный насильно в треде обновление плагина от Райского, не мог не удержаться посмотреть в код Райского и @jasmine, оценить код и выложить своё мнение на этот счёт.
Начнём с кода Райского. Оффтоп

А он начинающий в этом деле.

Спойлер: Райский

TopMenuObject admtex;

Ну и зачем? Я не видел ни в одном плагине сохранение указателя на объект в админ-меню. Тем более с ним ничего не делается.

g_cEnable = CreateConVar("sm_admtex_mode", "0", "Вкл/Выкл  технические работы на сервере!", FCVAR_DONTRECORD);

Зачем FCVAR_DONTRECORD? Тем более конвара и так не передаётся на клиенты. А даже если бы и передавалась, то клиент отвергал бы её, ибо подобной конвары нет у него самого.

WorksTime = CreateConVar("sm_admtex_time", "10", "Сколько секунд оповещать о начале технических работ ? Def: 10", _, true, 0.0, true, 60.0);

Зачем написано дефолтное значение в Description?
Зачем висит максимальный лимит на время об оповещении о начале тех работ? Может, я хочу в течении 5-и минут оповещать своих игроков, но из-за потолка в минуту не могу?

AutoExecConfig(true, "AdminTex");

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

FormatEx(buffer, maxlength, techworks ? "Перейти в игровой режим" : "Перейти в технический режим");

В том варианте, который я наблюдаю сейчас, логичнее было бы заюзать strcopy().
А если и юзать FormatEx(), то так:

FormatEx(buffer, maxlength, "Перейти в %s режим", techworks ? "игровой" : "технический");

if(object_id == admtex) ClientCommand(param, "sm_admtex");

Такие костыли поискать ещё, конечно, надо.
Не проще сделать ли прямой вызов каллбека команды? Она такая же функция, как и все.

if(sec <= GetConVarInt(WorksTime))

GetConVarInt() - довольно ресурсозатратный вызов. Как и ConVar.IntValue.get, и ConVar.GetInt(). Лучше хукнуть, и отслеживать изменения.

ServerCommand("sm_admtex_mode 1");

Тоже тот ещё костыль... Лучше так:

g_cEnable.BoolValue = true;

    if(newValue[0] == '1' && techworks == false)
   {
       techworks = true;
       GSTATUS();
       if(Start != null)
       {
           Start = null;
           sec = 0;
       }
   }
   else if(newValue[0] == '0' && techworks == true)
   {
       techworks = false;
   }

У меня фейспалм наидичайший...

    techworks = (newValue[0] != '0');
    if (techworks)
    {
        GSTATUS();
        Start = null;
    }

public void OnClientPostAdminCheck(int client)
{
   if(techworks && !IsFakeClient(client) && !IsClientAdmin(client))
       KickClient(client, "На сервере идут Технические работы.\nВход только для Администрации!\nПриносим свои извинения за доставленные неудобства.");
}

void GSTATUS()
{
   for(int i = 1; i <= MaxClients; i++)
   {
       if(IsClientInGame(i) && !IsFakeClient(i))
       {
           OnClientPostAdminCheck(i);
       }
   }
}

Для тех, кто не понял, за что я здесь зацепился:

void GSTATUS()
{
    /* ... */
    if(/* ... */ && !IsFakeClient(i))
    /* ... */
}

public void OnClientPostAdminCheck(int client)
{
   if(/* ... */ && !IsFakeClient(client) && /* ... */)
   // ...

Зачем проверять игрока на бота два раза?! Много времени тик длится, что-ли?!
Снести нахер проверку из GSTATUS().

public void OnLibraryRemoved(const char[] name) {if(StrEqual(name, "adminmenu")) g_TopMenu = null;}

А смысл? Указатель всё равно изменится.

void PrintToChats()
{
	PrintToChatAll("Сервер переведён в %s!", techworks ? "технический режим" : "игровой режим");
}

По примеру выше, слово "режим" можно вынести из форматирования.

void PrintToChats()
{
	PrintToChatAll("Сервер переведён в %s режим!", techworks ? "технический" : "игровой");
}

Спойлер: jasmine aka d4Сk

Вообще, я ненавижу KeyValues, но в этот раз закрою глаза на него. Если он, конечно, правильно использоваться будет.

Handle g_hKV = INVALID_HANDLE;

Ну, INVALID_HANDLE так-то устарел. Ему на замену пришёл null. И переменная и так по дефолту будет равна ему. Зачем явно присваивать - не ясно.

bool g_bServerLock = false;

Тоже самое. bool из коробки уже false.

		TopMenu hTopMenu;
		hTopMenu = GetAdminTopMenu();

А не проще сразу присвоить? Меньше опкода сгенерируется.

public void OnLibraryRemoved(const char[] szName)
{
    if (StrEqual(szName, "adminmenu")) 
		g_hAdminMenu = null;
}

Наследие от Райского, которое тоже по сути не нужно. Те, кто читали спойлер про Райского - поймут.

FormatEx(sBuffer, maxlength, g_bServerLock ? "Открыть сервер" : "Закрыть сервер");

Тот же самый косяк, что и у Райского. См. спойлер про него.

ServerCommand("sm_sl_lock");

Опять этот великий костыль... Та же проблема, что и у Райского.

public void OnPluginStart() {
  // ...
  RegAdminCmd("sm_sl_lock", cmdLockAdmin, ADMFLAG_ROOT);
  // ...
  RegAdminCmd("sm_sl_reload", cmdReloadAdmin, ADMFLAG_ROOT);
  // ...
}

public Action cmdLockAdmin(int client, int args)
{
  ServerCommand("sm_sl_lock");
}

public Action cmdReloadAdmin(int client, int args)
{
  ServerCommand("sm_sl_lock");
}

Вот это вообще поворот. Две казалось бы разные команды, а делают одно и то же. Да и внешне вообще на рекурсию похоже. Особенно без проверки на клиента.
Ещё return Plugin_Handled; нет. Печально. В этом плане d4Ck скатился.

	LoadCfg();
	
	KvSetNum(g_hKV, "server_lock", (g_bServerLock ? 0 : 1));
	KvSetString(g_hKV, "white_list", g_sWhiteList);
	KeyValuesToFile(g_hKV, g_sPatch); 
	
	LoadCfg();

Ну и зачем повторно загружать конфиг?

		if(StrContains(g_sWhiteList, sAuth, false) == -1)
			KickClient(client, "Доступ к серверу временно ограничен!\nПожалуйста попробуйте позже");

А вот тут я вижу баг. Если в вайтлисте есть юзер с SteamID STEAM_0:0:55665612, а на сервер зайдёт, допустим, юзер с SteamID STEAM_0:0:556656121, то его пропустит.

Краткий вердикт для тех, кому лень читать спойлеры: код у d4Ck местами хуже, чем у Райского. Много "наследия" от Райского. И ещё больше косяков.
В такие моменты невольно задаёшься вопросом, не проще ли сделать свою, адекватную реализацию?

 

[d4Ck]

Пользователь jasmine обновил ресурс Server Lock новой записью:

Обновление.

• Небольшие правки

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

 

[Kailo]

Пользователь Kailo обновил ресурс Tech Works новой записью:

Обновление.

• Добавлено закрытие сервера на время.
• Причины киков при технических работах.
• Логирование.
• Звук для обратного отсчёта.

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

 

[Kailo]

Пользователь Kailo обновил ресурс Tech Works новой записью:

Обновление.

Исправлены важные недоработки, рекомендуется обновление.

• Исправлена работа обратного отсчёта.
• Исправлен режим techworkstime когда он получал 0 и сохранял ничего =D.
• Добавлена переменная enablesound (Вкл/выкл звук при отсчёте)
• Мелкие исправления.

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

 

[xXMaXimXx]

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

 

[Kailo]

Пользователь Kailo обновил ресурс Tech Works новой записью:

Обновление.

• Оптимизация кода.
• В Панели администратора (Управление сервером) появился пункт Управление TechWorks.
• Изменена функция timer (Закрытие сервера на время).
• При кике игрока покажет H:M:S когда сервер будет доступен
• Добавлена функция перезагрузки сервера без потери игроков

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

 

[StormX]

Эх думал заведется на 34-ке sourcemod 1.9

Спойлер

L 04/22/2018 - 11:32:58: [SM] Exception reported: Invalid key value handle 0 (error 4)
L 04/22/2018 - 11:32:58: [SM] Blaming: TechWorks.smx
L 04/22/2018 - 11:32:58: [SM] Call stack trace:
L 04/22/2018 - 11:32:58: [SM] [0] KeyValues.GetNum
L 04/22/2018 - 11:32:58: [SM] [1] Line 257, C:\Users\SOURCEPAWN-AIKI\Desktop\scripting\TechWorks.sp::status
L 04/22/2018 - 11:32:58: [SM] [2] Line 55, C:\Users\SOURCEPAWN-AIKI\Desktop\scripting\TechWorks.sp::loads
L 04/22/2018 - 11:32:58: [SM] [3] Line 72, C:\Users\SOURCEPAWN-AIKI\Desktop\scripting\TechWorks.sp::OnPluginStart
L 04/22/2018 - 11:32:58: [SM] Unable to load plugin "TechWorks.smx": Error detected in plugin startup (see error logs)

 

[Kailo]

Эх думал заведется на 34-ке sourcemod 1.9

Спойлер

L 04/22/2018 - 11:32:58: [SM] Exception reported: Invalid key value handle 0 (error 4)
L 04/22/2018 - 11:32:58: [SM] Blaming: TechWorks.smx
L 04/22/2018 - 11:32:58: [SM] Call stack trace:
L 04/22/2018 - 11:32:58: [SM] [0] KeyValues.GetNum
L 04/22/2018 - 11:32:58: [SM] [1] Line 257, C:\Users\SOURCEPAWN-AIKI\Desktop\scripting\TechWorks.sp::status
L 04/22/2018 - 11:32:58: [SM] [2] Line 55, C:\Users\SOURCEPAWN-AIKI\Desktop\scripting\TechWorks.sp::loads
L 04/22/2018 - 11:32:58: [SM] [3] Line 72, C:\Users\SOURCEPAWN-AIKI\Desktop\scripting\TechWorks.sp::OnPluginStart
L 04/22/2018 - 11:32:58: [SM] Unable to load plugin "TechWorks.smx": Error detected in plugin startup (see error logs)

Решение: Если файл "<sourcemod>/configs/techworks.ini" существует все будет работать без ошибок. Если его нет, то при первом запуске он создаст его, а потом кинет эту ошибку. Раз файл теперь будет существовать, просто перезагрузите плагин и он будет работать.
Технические подробности: Да, автор чего-то начудил. Если файл отсутствует, то он создает новый, а затем почему-то удаляет Handle, исправим.

 

[StormX]

Решение: Если файл "<sourcemod>/configs/techworks.ini" существует все будет работать без ошибок. Если его нет, то при первом запуске он создаст его, а потом кинет эту ошибку. Раз файл теперь будет существовать, просто перезагрузите плагин и он будет работать.
Технические подробности: Да, автор чего-то начудил. Если файл отсутствует, то он создает новый, а затем почему-то удаляет Handle, исправим.

Спасибо, завелся).

 

[xXMaXimXx]

Долбит в консоль. Сервер лагать начинает.

Спойлер: Спойлер

SV_StartSound: *techworks.wav not precached (0)
SV_StartSound: *techworks.wav not precached (0)
SV_StartSound: *techworks.wav not precached (0)
SV_StartSound: *techworks.wav not precached (0)
SV_StartSound: *techworks.wav not precached (0)
SV_StartSound: *techworks.wav not precached (0)
SV_StartSound: *techworks.wav not precached (0)
SV_StartSound: *techworks.wav not precached (0)
SV_StartSound: *techworks.wav not precached (0)
SV_StartSound: *techworks.wav not precached (0)
SV_StartSound: *techworks.wav not precached (0)
SV_StartSound: *techworks.wav not precached (0)
SV_StartSound: *techworks.wav not precached (0)
SV_StartSound: *techworks.wav not precached (0)
SV_StartSound: *techworks.wav not precached (0)
SV_StartSound: *techworks.wav not precached (0)

 

[FokS]

Админы без ROOT прав остаются на сервере всё равно, как сделать чтобы их тоже кикало?

 

[Tomich]

В чем проблема?

L 05/08/2018 - 22:48:26: [SM] Blaming: TechWorks.smx
L 05/08/2018 - 22:48:26: [SM] Call stack trace:
L 05/08/2018 - 22:48:26: [SM] [0] KeyValues.GetNum
L 05/08/2018 - 22:48:26: [SM] [1] Line 257, C:\Users\SOURCEPAWN-AIKI\Desktop\scripting\TechWorks.sp::status
L 05/08/2018 - 22:48:26: [SM] [2] Line 55, C:\Users\SOURCEPAWN-AIKI\Desktop\scripting\TechWorks.sp::loads
L 05/08/2018 - 22:48:26: [SM] [3] Line 72, C:\Users\SOURCEPAWN-AIKI\Desktop\scripting\TechWorks.sp::OnPluginStart
L 05/08/2018 - 22:48:26: [SM] Unable to load plugin "TechWorks.smx": Error detected in plugin startup (see error logs)

 

[Livindege]

В чем проблема?

L 05/08/2018 - 22:48:26: [SM] Blaming: TechWorks.smx
L 05/08/2018 - 22:48:26: [SM] Call stack trace:
L 05/08/2018 - 22:48:26: [SM] [0] KeyValues.GetNum
L 05/08/2018 - 22:48:26: [SM] [1] Line 257, C:\Users\SOURCEPAWN-AIKI\Desktop\scripting\TechWorks.sp::status
L 05/08/2018 - 22:48:26: [SM] [2] Line 55, C:\Users\SOURCEPAWN-AIKI\Desktop\scripting\TechWorks.sp::loads
L 05/08/2018 - 22:48:26: [SM] [3] Line 72, C:\Users\SOURCEPAWN-AIKI\Desktop\scripting\TechWorks.sp::OnPluginStart
L 05/08/2018 - 22:48:26: [SM] Unable to load plugin "TechWorks.smx": Error detected in plugin startup (see error logs)

Какой соурс мод используйте?

 

[Kruzya]

@Tomich, логи полные пожалуйста приведите. Они всегда начинаются с Exception reported, а не Blaming.

 

[Tomich]

@Tomich, логи полные пожалуйста приведите. Они всегда начинаются с Exception reported, а не Blaming.

Замял почему то начало извиняюсь.

L 05/08/2018 - 22:48:26: [SM] Exception reported: Invalid key value handle 0 (error 4)
L 05/08/2018 - 22:48:26: [SM] Blaming: TechWorks.smx
L 05/08/2018 - 22:48:26: [SM] Call stack trace:
L 05/08/2018 - 22:48:26: [SM] [0] KeyValues.GetNum
L 05/08/2018 - 22:48:26: [SM] [1] Line 257, C:\Users\SOURCEPAWN-AIKI\Desktop\scripting\TechWorks.sp::status
L 05/08/2018 - 22:48:26: [SM] [2] Line 55, C:\Users\SOURCEPAWN-AIKI\Desktop\scripting\TechWorks.sp::loads
L 05/08/2018 - 22:48:26: [SM] [3] Line 72, C:\Users\SOURCEPAWN-AIKI\Desktop\scripting\TechWorks.sp::OnPluginStart
L 05/08/2018 - 22:48:26: [SM] Unable to load plugin "TechWorks.smx": Error detected in plugin startup (see error logs)

Какой соурс мод используйте?

SourceMod (1.9.0.6225) by AlliedModders LLC

 

[Livindege]

Замял почему то начало извиняюсь.

L 05/08/2018 - 22:48:26: [SM] Exception reported: Invalid key value handle 0 (error 4)
L 05/08/2018 - 22:48:26: [SM] Blaming: TechWorks.smx
L 05/08/2018 - 22:48:26: [SM] Call stack trace:
L 05/08/2018 - 22:48:26: [SM] [0] KeyValues.GetNum
L 05/08/2018 - 22:48:26: [SM] [1] Line 257, C:\Users\SOURCEPAWN-AIKI\Desktop\scripting\TechWorks.sp::status
L 05/08/2018 - 22:48:26: [SM] [2] Line 55, C:\Users\SOURCEPAWN-AIKI\Desktop\scripting\TechWorks.sp::loads
L 05/08/2018 - 22:48:26: [SM] [3] Line 72, C:\Users\SOURCEPAWN-AIKI\Desktop\scripting\TechWorks.sp::OnPluginStart
L 05/08/2018 - 22:48:26: [SM] Unable to load plugin "TechWorks.smx": Error detected in plugin startup (see error logs)


SourceMod (1.9.0.6225) by AlliedModders LLC

Скиньте ваш пример конфига techworks.ini

 

[Tomich]

Скиньте ваш пример конфига techworks.ini

"Tech Settings"
{
"mode" "0"
"modetime" "0"
"sectime" "10"
"enablesound" "1"
"sound" "techworks.wav"
"techtime" "0"
}
Standart
--- Добавлено позже ---
UPD
sm plugins reload TechWorks.smx
Помог. Спасибо Кайло.