Jump to content

  •  

Bug Tracker Migration

June 3rd
Good news everyone! The staff has decided that it is time to slowly kill off this Bug Tracker. We will begin the process of slowly migrating from this Bug Tracker over to our Github Issues which can be found here: https://github.com/HerculesWS/Hercules/issues

Over the next couple of days, I will be closing off any opportunity to create new reports. However, I still will keep the opportunity to reply to existing Bug Reports. Doing this will allow us to slowly fix any bug reports we have listed here so that we can easily migrate over to our Issue Tracker.

Update - June 7th 2015: Creating new bug posts has been disabled. Please use our https://github.com/HerculesWS/Hercules/issues tracker to post bugs. Users are still able to reply to existing bug posts.

- Administration

Issue Information

  • #006305

  • 0 - None Assigned

  • Fixed

Issue Confirmations

  • Yes (7)No (0)
Photo

Brandish Spear bug for rune knight

Posted by Hercules Bot on 24 July 2012 - 06:39 AM

Originally posted by Brynner
rune knight class cannot use brandish spear. because it needs a peco. but rune knight don't have peco. they only have dragon mount. that's why it can't be used by rune knight. but if i use lord knight. i can use brandish spear without a problem.

Originally posted by Kenpachi
Indeed, in script_require_db.txt the field RequiredState can't hold more than one state but for Brandish Spear it should be riding AND dragon.

BTW; The legend is outdated. please edit it along with fixxing the bug.

Originally posted by Brynner
thanks for the confirm

Edited by Brynner, 24 July 2012 - 08:54 AM.


Originally posted by Brynner
any update for this?

Originally posted by Brynner
bump for this

Originally posted by Kenpachi
Dude, seriously. We are no machines....

Originally posted by Brynner

Dude, seriously. We are no machines....


since there is no one fix the brandish spear bug. here is my fix for this bug.

Edited by Brynner, 30 July 2012 - 10:07 PM.


Originally posted by Euphy
@Brynner: That's quite a workaround for a single skill edit.  What Kenpachi stated was that a field value should be expanded, not that the "Riding" status itself should be changed...

Originally posted by Brynner

@Brynner: That's quite a workaround for a single skill edit.  What Kenpachi stated was that a field value should be expanded, not that the "Riding" status itself should be changed...



Brandish Spear bug for rune knight is now working if you are using dragon mount. which means rune knight cannot use brandish spear because they are not using peco anymore. but on the solution that i made the brandish spear can now be use even you are riding on dragon.

Originally posted by malufett
Euphy has a point..
anyway I'll make it so support multiple state as sir Kenpachi suggested...

Originally posted by hendra814
any update for this problem, beside using patch?

Originally posted by hendra814
bump

Originally posted by hendra814
need update fot this bug.
Thanks

Originally posted by AppleGirl
buuump please fix..

Originally posted by Jhedzkie
are there any fix for this yet? :) thanks

Originally posted by hendra814
bump... please fix this problem
Thanks

Originally posted by MarkZD
There's no need to make it use more than one state as any skill you can use in peco you can use in Ferus.

In scr/map/skill.c
Just change:
case ST_RIDING:
  if(!pc_isriding(sd)) {
   clif_skill_fail(sd,skill,USESKILL_FAIL_LEVEL,0);
   return 0;
  }
To:
case ST_RIDING:
  if(!(pc_isriding(sd) || pc_isridingdragon(sd))) {
   clif_skill_fail(sd,skill,USESKILL_FAIL_LEVEL,0);
   return 0;
  }

Edited by MarkZD, 12 November 2012 - 04:19 AM.


Originally posted by exneval
it's just same, i made that since rA still working on expanding the RequireState

Edited by exneval, 13 November 2012 - 07:29 AM.


Originally posted by hendra814
ok, i'm wait the fixing for this bug.
Thanks Exneval

Edited by hendra814, 13 November 2012 - 07:37 AM.


moved issue from Skills

Originally posted by Kenpachi
Indeed, in script_require_db.txt the field RequiredState can't hold more than one state but for Brandish Spear it should be riding AND dragon.

BTW; The legend is outdated. please edit it along with fixxing the bug.

its not AND, its OR D: but i was mislead as well I didnt think it fully, wrote a faux fix :{ http://herc.ws/board...asted-time-t-t/