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

  • #005694

  • 0 - None Assigned

  • Fixed

Issue Confirmations

  • Yes (0)No (0)
Photo

Renewal Megingjard bonus

Posted by Hercules Bot on 03 May 2012 - 11:27 PM

Originally posted by frenzmu06
isnt it in renewal megingjard has a 10% bonus physical damage to boss monster when the user has 120 or above base str?

confirmation of it is
05/03/2012 - kRO Maintenance

it proves that there really is this extra bonus for renewal megingjard

Originally posted by sizenine
that's not a bug.. you can't expect such recent updates to be implemented already..

Originally posted by ForteXX
Thats not a new effect, the update says tht they fixed it not working. The effect itself was added long ago.

Originally posted by frenzmu06
@forteXX yup your right it was added long time ago i think somewhat from the beginning of renewal, so it should be already on the database
@sizenine i was just pointing out "05/03/2012 - kRO Maintenance"  as a reference that will prove that there is an extra 10% bonus damage to boss from megingjard. I am not saying that they just recently added this bonus in the "05/03/2012 - kRO Maintenance" and asking to add it immediately on the svn
hope that makes all clear

Edited by frenzmu06, 05 May 2012 - 08:13 AM.


Originally posted by Kenpachi
Fixed in [rev=16245].

Originally posted by EvilPuncker
the correct is

if(readparam(bStr)>=120)

and not

if(readparam(bStr)==120)

Originally posted by Kenpachi
You (the description) are wrong:
item Magingiorde
event OnStartEquip:
  var temp = get[VAR_CLEVEL] / 5
  AddExtParam User VAR_STRAMOUNT temp
  if (get[VAR_STANDARD_STR] == 120)
	ClassAddDamage 1 Target 10
  endif
return
event OnFinishEquip:
  var temp = get[VAR_CLEVEL] / 5
  SubExtParam User VAR_STRAMOUNT temp
  if (get[VAR_STANDARD_STR] == 120)
	ClassSubDamage 1 Target 10
  endif
return

I know that's stupid, but that's how it is.

Originally posted by EvilPuncker
okay then, but can you show me 4236 item script just out of curiosity :P

Originally posted by Kenpachi
Even though it's not related to the bug:

item Amon_Ra_Card

event OnStartEquip:

  AddExtParam User VAR_STRAMOUNT 1

  AddExtParam User VAR_INTAMOUNT 1

  AddExtParam User VAR_DEXAMOUNT 1

  AddExtParam User VAR_AGIAMOUNT 1

  AddExtParam User VAR_LUKAMOUNT 1

  AddExtParam User VAR_VITAMOUNT 1

  SetAutoSpell_MLEATKED User PR_KYRIE 10 30

  if (get[VAR_STANDARD_INT] > 98)

   SetAutoSpell_MLEATKED User PR_KYRIE 10 100

  endif

return

event OnFinishEquip:

  SubExtParam User VAR_STRAMOUNT 1

  SubExtParam User VAR_INTAMOUNT 1

  SubExtParam User VAR_DEXAMOUNT 1

  SubExtParam User VAR_AGIAMOUNT 1

  SubExtParam User VAR_LUKAMOUNT 1

  SubExtParam User VAR_VITAMOUNT 1

  SetAutoSpell_MLEATKED User PR_KYRIE 0 0

return