summaryrefslogtreecommitdiff
path: root/src/map
diff options
context:
space:
mode:
authortoms <toms@54d463be-8e91-2dee-dedb-b68131a5f0ec>2006-07-29 23:06:03 +0000
committertoms <toms@54d463be-8e91-2dee-dedb-b68131a5f0ec>2006-07-29 23:06:03 +0000
commit7f024c8c6073c4fa1a4130c499a7f783e5a55ca1 (patch)
tree0ce117b29e4080cc80e7becafe43c0b4f215dbd6 /src/map
parent84e7380a76a1e3f5740c0a144d79405ddc4c82a2 (diff)
downloadhercules-7f024c8c6073c4fa1a4130c499a7f783e5a55ca1.tar.gz
hercules-7f024c8c6073c4fa1a4130c499a7f783e5a55ca1.tar.bz2
hercules-7f024c8c6073c4fa1a4130c499a7f783e5a55ca1.tar.xz
hercules-7f024c8c6073c4fa1a4130c499a7f783e5a55ca1.zip
Removed ugly struct cast in login.c
New version of buildin_query_sql which accept more than one column and can return the number of rows. See script_commands.txt for more details. git-svn-id: https://rathena.svn.sourceforge.net/svnroot/rathena/trunk@7975 54d463be-8e91-2dee-dedb-b68131a5f0ec
Diffstat (limited to 'src/map')
-rw-r--r--src/map/script.c76
1 files changed, 55 insertions, 21 deletions
diff --git a/src/map/script.c b/src/map/script.c
index a6ad56ffe..25f943d96 100644
--- a/src/map/script.c
+++ b/src/map/script.c
@@ -11186,41 +11186,75 @@ int buildin_setd(struct script_state *st)
#ifndef TXT_ONLY
int buildin_query_sql(struct script_state *st) {
- char *name, *query;
- int num, i = 0;
+ char *name = NULL, *query;
+ int num, i = 0,j, nb_rows;
+ struct { char * dst_var_name; char type; } row[32];
struct map_session_data *sd = (st->rid)? script_rid2sd(st) : NULL;
query = conv_str(st,& (st->stack->stack_data[st->start+2]));
- strcpy(tmp_sql, query);
+ strcpy(tmp_sql, query);
if(mysql_query(&mmysql_handle,tmp_sql)){
ShowSQL("DB error - %s\n",mysql_error(&mmysql_handle));
ShowDebug("at %s:%d - %s\n", __FILE__,__LINE__,tmp_sql);
+ push_val(st->stack,C_INT,0);
return 1;
}
- if(st->end > st->start+3) {
- if(st->stack->stack_data[st->start+3].type != C_NAME){
- ShowWarning("buildin_query_sql: 2nd parameter is not a variable!\n");
- } else {
- num=st->stack->stack_data[st->start+3].u.num;
- name=(char *)(str_buf+str_data[num&0x00ffffff].str);
- if((sql_res = mysql_store_result(&mmysql_handle))){
+ // If some data was returned
+ if((sql_res = mysql_store_result(&mmysql_handle))){
+ // Count the number of rows to store
+ nb_rows = mysql_num_fields(sql_res);
+
+ // Can't store more row than variable
+ if (nb_rows > st->end - (st->start+3))
+ nb_rows = st->end - (st->start+3);
+
+ if (!nb_rows)
+ {
+ push_val(st->stack,C_INT,0);
+ return 0; // Nothing to store
+ }
+
+ if (nb_rows > 32)
+ {
+ ShowWarning("buildin_query_sql: too much rows!\n");
+ push_val(st->stack,C_INT,0);
+ return 1;
+ }
+
+ memset(row, 0, sizeof(row));
+ // Verify argument types
+ for(j=0; j < nb_rows; j++)
+ {
+ if(st->stack->stack_data[st->start+3+j].type != C_NAME){
+ ShowWarning("buildin_query_sql: Parameter %d is not a variable!\n", j);
+ push_val(st->stack,C_INT,0);
+ return 0;
+ } else {
+ // Store type of variable (string = 0/int = 1)
+ num=st->stack->stack_data[st->start+3+j].u.num;
+ name=(char *)(str_buf+str_data[num&0x00ffffff].str);
if(name[strlen(name)-1] != '$') {
- while(i<128 && (sql_row = mysql_fetch_row(sql_res))){
- setd_sub(st,sd, name, i, (void *)atoi(sql_row[0]),st->stack->stack_data[st->start+3].ref);
- i++;
- }
- } else {
- while(i<128 && (sql_row = mysql_fetch_row(sql_res))){
- setd_sub(st,sd, name, i, (void *)sql_row[0],st->stack->stack_data[st->start+3].ref);
- i++;
- }
+ row[j].type = 1;
}
- mysql_free_result(sql_res);
+ row[j].dst_var_name = name;
}
}
+ // Store data
+ while(i<128 && (sql_row = mysql_fetch_row(sql_res))){
+ for(j=0; j < nb_rows; j++)
+ {
+ if (row[j].type == 1)
+ setd_sub(st,sd, row[j].dst_var_name, i, (void *)atoi(sql_row[j]),st->stack->stack_data[st->start+3+j].ref);
+ else
+ setd_sub(st,sd, row[j].dst_var_name, i, (void *)sql_row[j],st->stack->stack_data[st->start+3+j].ref);
+ }
+ i++;
+ }
+ // Free data
+ mysql_free_result(sql_res);
}
-
+ push_val(st->stack, C_INT, i);
return 0;
}