So I was skimming the patch diffs like I do every Monday night (sick addiction) and I noticed something that looks wrong with the code. There’s a duplicate costume_war_f_011
when it should probably say costume_wiz_m_011
. I may very well be wrong, and sorry if I am, but I think this code is saying that any male wizards opening this costume pack will receive the female warrior costume instead.
I’d hold off on opening this if you’re a male wizard. There may be another patch they push between now and the servers coming up, but that’s unusual unless they happen to notice this issue before then.
function SCR_USE_ALICEPACK_2016(pc)
local job = GetClassString('Job', pc.JobName, 'CtrlType')
local jobList = { 'Warrior', 'Wizard', 'Archer', 'Cleric' }
local i, j, jobNum = 0, 0, 0
for j = 1, table.getn(jobList) do
if job == jobList[j] then
jobNum = j
break
end
end
local costume = {
{ 'costume_war_m_011', 'costume_war_f_011' }, <- costume_war_f_011
{ 'costume_war_f_011', 'costume_wiz_f_011' }, <- costume_war_f_011 duplicate
{ 'costume_arc_m_012', 'costume_arc_f_012' },
{ 'costume_clr_m_012', 'costume_clr_f_012' }
}
local tx = TxBegin(pc);
TxGiveItem(tx, costume[jobNum][pc.Gender], 1, 'ALICEPACK_2016');
TxGiveItem(tx, 'AliceHairBox_2016', 1, 'ALICEPACK_2016');
TxGiveItem(tx, 'Premium_Enchantchip', 10, 'ALICEPACK_2016');
local ret = TxCommit(tx);
if ret ~= "SUCCESS" then
SendSysMsg(pc, "DataError");
return;
end
end