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

  • #004347

  • 2 - Fair

  • Fixed

Issue Confirmations

  • Yes (0)No (0)
Photo

Arrow Shower not working on Land Protector + fix

Posted by Hercules Bot on 08 July 2010 - 06:57 PM

Originally posted by Damlug
http://www.eathena.w...er&showbug=4347

Hello

Arrow shower is supposed to work on Land Protector, but it doesn't on eAthena. I made a video on an Aegis server (fro) then you will see it works perfectly.

The video is here
http://www.megaupload.com/?d=T7FQLCIC


Here is a diff file you need to put in src/map)
http://baha922.free....rrowshower.diff

It works for me with trunk 14363, but you should test it too, we never know; or maybe you have a better way to fix it.

Thank you

This post has been edited by Damlug: Jul 13 2010, 08:34 AM

Originally posted by Ind
Fixed in [rev=15129]

Originally posted by mrjnumber1
This fix is incomplete, as it does not fix MA_SHOWER.
switch(group->skill_id)
{
  case AC_SHOWER:
  case MA_SHOWER:
   break;
  default:
   if( !(skill_get_inf2(group->skill_id)&(INF2_SONG_DANCE|INF2_TRAP)) && map_getcell(bl->m, bl->x, bl->y, CELL_CHKLANDPROTECTOR) )
    return 0; //AoE skills are ineffective. [Skotlex]
   break;
}
This would fix it however :)

Originally posted by Ind
Thanks for pointing out.
I've added a new skill info type so it gets easier to modify when new ones show up.
Fixed in [rev=15132]