UNLOZE/sourcemod
10
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

SQL-Admin-Manager: style clean-up (#974)

Browse Source 
* Switched to sizeof instead of hard-coding for loop size

* Switched back to pre-increment.

* Dynamically pad variables
This commit is contained in:
⭐ B3none 2019-04-06 20:47:21 +02:00 committed by Kyle Sanderson
parent 318b36cdcf
commit 8f0527487f
1 changed files with 25 additions and 26 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

51
plugins/sql-admin-manager.sp
View File

@ -31,17 +31,16 @@
* Version: $Id$
*/
/* We like semicolons */
#pragma semicolon 1
#include <sourcemod>
/* We like semicolons */
#pragma semicolon 1
#pragma newdecls required
#define CURRENT_SCHEMA_VERSION 1409
#define SCHEMA_UPGRADE_1 1409
int current_version[4] = {1, 0, 0, CURRENT_SCHEMA_VERSION};
int current_version[] = {1, 0, 0, CURRENT_SCHEMA_VERSION};
public Plugin myinfo =
{
@ -88,7 +87,7 @@ Database Connect()
void CreateMySQL(int client, Database db)
{
char queries[7][] =
char queries[][] =
{
"CREATE TABLE sm_admins (id int(10) unsigned NOT NULL auto_increment, authtype enum('steam','name','ip') NOT NULL, identity varchar(65) NOT NULL, password varchar(65), flags varchar(30) NOT NULL, name varchar(65) NOT NULL, immunity int(10) unsigned NOT NULL, PRIMARY KEY (id))",
"CREATE TABLE sm_groups (id int(10) unsigned NOT NULL auto_increment, flags varchar(30) NOT NULL, name varchar(120) NOT NULL, immunity_level int(1) unsigned NOT NULL, PRIMARY KEY (id))",
@ -99,7 +98,7 @@ void CreateMySQL(int client, Database db)
"CREATE TABLE IF NOT EXISTS sm_config (cfg_key varchar(32) NOT NULL, cfg_value varchar(255) NOT NULL, PRIMARY KEY (cfg_key))"
};
for (int i = 0; i < 7; i++)
for (int i = 0; i < sizeof(queries); i++)
{
if (!DoQuery(client, db, queries[i]))
{
@ -124,7 +123,7 @@ void CreateMySQL(int client, Database db)
void CreateSQLite(int client, Database db)
{
char queries[7][] =
char queries[][] =
{
"CREATE TABLE sm_admins (id INTEGER PRIMARY KEY AUTOINCREMENT, authtype varchar(16) NOT NULL CHECK(authtype IN ('steam', 'ip', 'name')), identity varchar(65) NOT NULL, password varchar(65), flags varchar(30) NOT NULL, name varchar(65) NOT NULL, immunity INTEGER NOT NULL)",
"CREATE TABLE sm_groups (id INTEGER PRIMARY KEY AUTOINCREMENT, flags varchar(30) NOT NULL, name varchar(120) NOT NULL, immunity_level INTEGER NOT NULL)",
@ -135,7 +134,7 @@ void CreateSQLite(int client, Database db)
"CREATE TABLE IF NOT EXISTS sm_config (cfg_key varchar(32) NOT NULL, cfg_value varchar(255) NOT NULL, PRIMARY KEY (cfg_key))"
};
for (int i = 0; i < 7; i++)
for (int i = 0; i < sizeof(queries); i++)
{
if (!DoQuery(client, db, queries[i]))
{
@ -203,7 +202,7 @@ bool GetUpdateVersion(int client, Database db, int versions[4])
char version_numbers[4][12];
if (ExplodeString(version_string, ".", version_numbers, 4, 12) == 4)
{
for (int i = 0; i < 4; i++)
for (int i = 0; i < sizeof(version_numbers); i++)
{
versions[i] = StringToInt(version_numbers[i]);
}
@ -257,7 +256,7 @@ void UpdateSQLite(int client, Database db)
*/
if (versions[3] < SCHEMA_UPGRADE_1)
{
char queries[8][] =
char queries[][] =
{
"ALTER TABLE sm_admins ADD immunity INTEGER DEFAULT 0 NOT NULL",
"CREATE TABLE _sm_groups_temp (id INTEGER PRIMARY KEY AUTOINCREMENT, flags varchar(30) NOT NULL, name varchar(120) NOT NULL, immunity_level INTEGER DEFAULT 0 NOT NULL)",
@ -269,7 +268,7 @@ void UpdateSQLite(int client, Database db)
"CREATE TABLE IF NOT EXISTS sm_config (cfg_key varchar(32) NOT NULL, cfg_value varchar(255) NOT NULL, PRIMARY KEY (cfg_key))"
};
for (int i = 0; i < 8; i++)
for (int i = 0; i < sizeof(queries); i++)
{
if (!DoQuery(client, db, queries[i]))
{
@ -329,7 +328,7 @@ void UpdateMySQL(int client, Database db)
*/
if (versions[3] < SCHEMA_UPGRADE_1)
{
char queries[6][] =
char queries[][] =
{
"CREATE TABLE IF NOT EXISTS sm_config (cfg_key varchar(32) NOT NULL, cfg_value varchar(255) NOT NULL, PRIMARY KEY (cfg_key))",
"ALTER TABLE sm_admins ADD immunity INT UNSIGNED NOT NULL",
@ -339,7 +338,7 @@ void UpdateMySQL(int client, Database db)
"ALTER TABLE sm_groups DROP immunity"
};
for (int i = 0; i < 6; i++)
for (int i = 0; i < sizeof(queries); i++)
{
if (!DoQuery(client, db, queries[i]))
{
@ -482,7 +481,7 @@ public Action Command_SetAdminGroups(int client, int args)
char name[80];
int inherit_order = 0;
for (int i=3; i<=args; i++)
for (int i = 3; i <= args; i++)
{
GetCmdArg(i, name, sizeof(name));
@ -872,20 +871,20 @@ stock bool DoQuery(int client, Database db, const char[] query)
stock Action DoError(int client, Database db, const char[] query, const char[] msg)
{
char error[255];
SQL_GetError(db, error, sizeof(error));
LogError("%s: %s", msg, error);
LogError("Query dump: %s", query);
delete db;
ReplyToCommand(client, "[SM] %t", "Failed to query database");
return Plugin_Handled;
char error[255];
SQL_GetError(db, error, sizeof(error));
LogError("%s: %s", msg, error);
LogError("Query dump: %s", query);
delete db;
ReplyToCommand(client, "[SM] %t", "Failed to query database");
return Plugin_Handled;
}
stock Action DoStmtError(int client, Database db, const char[] query, const char[] error, const char[] msg)
{
LogError("%s: %s", msg, error);
LogError("Query dump: %s", query);
delete db;
ReplyToCommand(client, "[SM] %t", "Failed to query database");
return Plugin_Handled;
LogError("%s: %s", msg, error);
LogError("Query dump: %s", query);
delete db;
ReplyToCommand(client, "[SM] %t", "Failed to query database");
return Plugin_Handled;
}