Issue information

Issue ID
#2332
Status
Fixed
Severity
None
Started
Hercules Elf Bot
Oct 9, 2008 17:42
Last Post
Hercules Elf Bot
Mar 5, 2012 9:12
Confirmation
N/A

Hercules Elf Bot - Oct 9, 2008 17:42

Originally posted by [b]RedSunset[/b]
http://www.eathena.ws/board/index.php?autocom=bugtracker&showbug=2332

Well I think it's not right oo
If you are a Assassin or Rogue class u can get the Skills without collecting the items. Because brackets are missing around the job check.

line 31
CODE
if (countitem(7041) > 4 && (BaseJob == Job_Thief && JobLevel > 24) || BaseJob == Job_Assassin || BaseJob == Job_Rogue) {


should be
CODE
if (countitem(7041) > 4 && [color="#FF0000"]([/color](BaseJob == Job_Thief && JobLevel > 24) || BaseJob == Job_Assassin || BaseJob == Job_Rogue)[color="#FF0000"])[/color]  {


line 155
CODE
if (countitem(940) > 19  && (BaseJob == Job_Thief && JobLevel > 34) || BaseJob == Job_Assassin || BaseJob == Job_Rogue) {


should be
CODE
if (countitem(940) > 19  && [color="#FF0000"] ( [/color](BaseJob == Job_Thief && JobLevel > 34) || BaseJob == Job_Assassin || BaseJob == Job_Rogue)[color="#FF0000"] ) [/color] {


Line 234
CODE
if (countitem(912) > 0 && countitem(948) > 0 && countitem(908) > 4 && (BaseJob == Job_Thief && JobLevel > 19) || BaseJob == Job_Assassin || BaseJob == Job_Rogue) {


should be
CODE
if (countitem(912) > 0 && countitem(948) > 0 && countitem(908) > 4 && [color="#FF0000"]([/color](BaseJob == Job_Thief && JobLevel > 19) || BaseJob == Job_Assassin || BaseJob == Job_Rogue)[color="#FF0000"])[/color]  {


Line 326
CODE
if (countitem(910) > 1 && countitem(911) > 1 && (BaseJob == Job_Thief && JobLevel > 14) || BaseJob == Job_Assassin || BaseJob == Job_Rogue) {


should be
CODE
if (countitem(910) > 1 && countitem(911) > 1 && [color="#FF0000"]([/color](BaseJob == Job_Thief && JobLevel > 14) || BaseJob == Job_Assassin || BaseJob == Job_Rogue)[color="#FF0000"])[/color] {


Sorry for my bad english ^^"

This post has been edited by RedSunset: Oct 9 2008, 10:43 AM