Issue information

Issue ID
#5469
Status
Fixed
Severity
None
Started
Hercules Elf Bot
Mar 18, 2012 20:12
Last Post
Hercules Elf Bot
May 26, 2012 20:59
Confirmation
Yes (4)
No (1)

Hercules Elf Bot - Mar 18, 2012 20:12

Originally posted by [b]Vali[/b]
Hi,

Some of the players from my server did the quest to get the Eden Weapon 2 and got the dagger. It seems that the code is not checking if the player job correctly,

Tested with a merchant, pries, and high priest. All of them got a dagger after the quest.

Vali~

Hercules Elf Bot - Mar 19, 2012 1:05

Originally posted by [b]Slim[/b]
I tested it and it seem'd to work fine for merchant, priest, thief, etc. Though I did notice that it's not giving the correct ones to the Swordman, Knight and Crusader.

Correct me if I'm wrong, just to make sure that we're on the same page.

[code]

if ((BaseJob == Job_Swordman) || (((BaseJob == Job_Knight) || (BaseJob == Job_Crusader)) && (Upper != 1))) {
mes "[Michael]";
mes "What kind of equipment do you want?";
mes "We have slayer and sabre.";
mes "Here are the options.";
next;
mes "[Michael]";
mes "Eden Saber II: one hand sword. attack 170.";
mes "Eden Slayer II: Two hand sword. attack 185.";
mes "Both of them are Lv 2 weapons and required lv is 40.";
mes "Also you can't trade with other players or refine them.";
next;
switch(select("Eden Saber II:Eden Slayer II")) {
case 1:
mes "[Michael]";
mes "You chose the Eden Sabre II.";
mes "Additionally we have boots and uniform.";
next;
if (para_suv02 == 0) {
mes "[Michael]";
mes "You don't have record of getting any equipment";
mes "so, I will give Eden Group manteau and hat also.";
set para_suv01,38;
set para_suv02,3;
getitem 13051,1; //P_Dagger2
getitem 2458,1; //Para_Team_Boots3
getitem 15011,1; //Para_Team_Uniform3
getitem 5583,1; //Para_Team_Hat
getitem 2560,1; //Para_Team_Manteau
next;
mes "[Michael]";
mes "Check your equipment again and treat them well.";
close;
}
else {
mes "[Michael]";
mes "Weapon, Uniform, Boots all 3 things.";
mes "Please check it again.";
set para_suv01,38;
set para_suv02,3;
getitem 13051,1; //P_Dagger2
getitem 2458,1; //Para_Team_Boots3
getitem 15011,1; //Para_Team_Uniform3
next;
mes "[Michael]";
mes "I hope they're useful to you.";
close;
}
break;[/code]

Hercules Elf Bot - Mar 19, 2012 7:45

Originally posted by [b]Vali[/b]
Yes, we are on the same page, Really, the eden quest files are doing strange things. I testes on my server with a merch and I got a dagger :S. I do not know the reason of why you got the mace and i thw dagger.

Vali~

Hercules Elf Bot - Mar 19, 2012 8:00

Originally posted by [b]5511[/b]
LINE 3133
(Bas[b]s[/b]eJob == Job_Novice)

btw, you should keep your eye clear~

This post has been edited by 5511 on Mar 19, 2012 8:03

Hercules Elf Bot - Mar 19, 2012 13:16

Originally posted by [b]Aleos[/b]
Looks like the issue is because the creator copy-pasted the rewards and didn't go back to modify them per class.

More Information: http://irowiki.org/wiki/Eden_Group_Equipments_Quests#Weapons_by_Classes

Hercules Elf Bot - Mar 19, 2012 19:28

Originally posted by [b]Mysterious[/b]
LMAO
[CODE]else if ((BasseJob == Job_Novice) || (BaseJob == Job_Supernovice) || (Class == Job_Soul_linker) || (Class == Job_Ninja)) {
mes "[Michael]";[/CODE]


Why the hell is it Basse? >>

Hercules Elf Bot - Mar 19, 2012 21:32

Originally posted by [b]Slim[/b]
I see the problem, i'll work on this.

Hercules Elf Bot - May 26, 2012 14:55

Originally posted by [b]Masao[/b]
Fixed in [rev=16151]