Mirroristas/rathena
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Removed the option ID requirement for countitem3, rentitem3, getitem3, makeitem3 and delitem3 (#4270)

Browse Source 
* These functions now accept the option ID array fill with 0
* They don't throw a warning anymore when the item doesn't have option ID
This commit is contained in:
Atemo
2019-08-16 14:52:09 +02:00
committed by GitHub
parent bc15f5ce14
commit 378e44d85d
1 changed files with 13 additions and 50 deletions
Download Patch File Download Diff File Expand all files Collapse all files

63
src/map/script.cpp
View File

@@ -6737,7 +6737,7 @@ static int script_getitem_randomoption(struct script_state *st, struct map_sessi
// If no player is attached
if( !script_rid2sd(sd) ){
ShowError( "buildin_%s: variable \"%s\" was not a server variable, but no player was attached.\n", funcname, opt_id_var );
return false;
return SCRIPT_CMD_FAILURE;
}
}
@@ -6756,7 +6756,7 @@ static int script_getitem_randomoption(struct script_state *st, struct map_sessi
// If no player is attached
if( !script_rid2sd(sd) ){
ShowError( "buildin_%s: variable \"%s\" was not a server variable, but no player was attached.\n", funcname, opt_val_var );
return false;
return SCRIPT_CMD_FAILURE;
}
}
@@ -6775,7 +6775,7 @@ static int script_getitem_randomoption(struct script_state *st, struct map_sessi
// If no player is attached
if( !script_rid2sd(sd) ){
ShowError( "buildin_%s: variable \"%s\" was not a server variable, but no player was attached.\n", funcname, opt_param_var );
return false;
return SCRIPT_CMD_FAILURE;
}
}
@@ -6792,11 +6792,6 @@ static int script_getitem_randomoption(struct script_state *st, struct map_sessi
opt_id_ref = reference_getref(opt_id);
opt_id_n = script_array_highest_key(st, sd, opt_id_var, opt_id_ref);
if (opt_id_n < 1) {
ShowError("buildin_%s: No option id listed.\n", funcname);
return SCRIPT_CMD_FAILURE;
}
opt_val_ref = reference_getref(opt_val);
opt_param_ref = reference_getref(opt_param);
@@ -6900,8 +6895,8 @@ int script_countitem_sub(struct item *items, struct item_data *id, int size, boo
* Returns number of items in inventory
* countitem(<nameID>{,<accountID>})
* countitem2(<nameID>,<Identified>,<Refine>,<Attribute>,<Card0>,<Card1>,<Card2>,<Card3>{,<accountID>}) [Lupus]
* countitem3(<item id>,<identify>,<refine>,<attribute>,<card1>,<card2>,<card3>,<card4>,<RandomIDArray>,<RandomValueArray>,<RandomParamArray>)
* countitem3("<item name>",<identify>,<refine>,<attribute>,<card1>,<card2>,<card3>,<card4>,<RandomIDArray>,<RandomValueArray>,<RandomParamArray>)
* countitem3(<item id>,<identify>,<refine>,<attribute>,<card1>,<card2>,<card3>,<card4>,<RandomIDArray>,<RandomValueArray>,<RandomParamArray>{,<accountID>})
* countitem3("<item name>",<identify>,<refine>,<attribute>,<card1>,<card2>,<card3>,<card4>,<RandomIDArray>,<RandomValueArray>,<RandomParamArray>{,<accountID>})
*/
BUILDIN_FUNC(countitem)
{
@@ -6917,16 +6912,8 @@ BUILDIN_FUNC(countitem)
random_option = true;
}
if (script_hasdata(st, aid)) {
if (!(sd = map_id2sd(script_getnum(st, aid)))) {
ShowError("buildin_%s: player not found (AID=%d).\n", command, script_getnum(st, aid));
st->state = END;
return SCRIPT_CMD_FAILURE;
}
} else {
if (!script_rid2sd(sd))
return SCRIPT_CMD_FAILURE;
}
if (!script_accid2sd(aid, sd))
return SCRIPT_CMD_FAILURE;
struct item_data *id;
@@ -6964,16 +6951,8 @@ BUILDIN_FUNC(cartcountitem)
if (command[strlen(command) - 1] == '2')
aid = 10;
if (script_hasdata(st, aid)) {
if (!(sd = map_id2sd(script_getnum(st, aid)))) {
ShowError("buildin_%s: player not found (AID=%d).\n", command, script_getnum(st, aid));
st->state = END;
return SCRIPT_CMD_FAILURE;
}
} else {
if (!script_rid2sd(sd))
return SCRIPT_CMD_FAILURE;
}
if (!script_accid2sd(aid, sd))
return SCRIPT_CMD_FAILURE;
if (!pc_iscarton(sd)) {
ShowError("buildin_%s: Player doesn't have cart (CID:%d).\n", command, sd->status.char_id);
@@ -7017,16 +6996,8 @@ BUILDIN_FUNC(storagecountitem)
if (command[strlen(command) - 1] == '2')
aid = 10;
if (script_hasdata(st, aid)) {
if (!(sd = map_id2sd(script_getnum(st, aid)))) {
ShowError("buildin_%s: player not found (AID=%d).\n", command, script_getnum(st, aid));
st->state = END;
return SCRIPT_CMD_FAILURE;
}
} else {
if (!script_rid2sd(sd))
return SCRIPT_CMD_FAILURE;
}
if (!script_accid2sd(aid, sd))
return SCRIPT_CMD_FAILURE;
struct item_data *id;
@@ -7069,16 +7040,8 @@ BUILDIN_FUNC(guildstoragecountitem)
if (command[strlen(command) - 1] == '2')
aid = 10;
if (script_hasdata(st, aid)) {
if (!(sd = map_id2sd(script_getnum(st, aid)))) {
ShowError("buildin_%s: player not found (AID=%d).\n", command, script_getnum(st, aid));
st->state = END;
return SCRIPT_CMD_FAILURE;
}
} else {
if (!script_rid2sd(sd))
return SCRIPT_CMD_FAILURE;
}
if (!script_accid2sd(aid, sd))
return SCRIPT_CMD_FAILURE;
struct item_data *id;