Issue information

Issue ID
#7966
Status
Fixed
Severity
None
Started
N e s s
Jan 12, 2014 13:08
Last Post
malufett
Feb 10, 2014 7:03
Confirmation
N/A

N e s s - Jan 12, 2014 13:08

in my last rev over brand is working fine but now in new latest rev my overbrand is not working properly it only hit once in any cell. is it the official behavior?

malufett - Jan 12, 2014 13:32

here is the official path...
[img]http://i44.tinypic.com/33zg3t0.png[/img]
Pierce is the highest damage
Swing is where knockback happens + knockback bonus
Pierce and Swing is where 2 hits happen

:meow:

N e s s - Jan 12, 2014 14:26

i try any cell but it hit only once :(

malufett - Jan 12, 2014 14:48

check your copy maybe?
[code=auto:0] case LG_OVERBRAND: { uint8 dir = map->calc_dir(src, x, y); uint8 x2 = x = src->x, y2 = y = src->y; switch( dir ){ case 0: case 1: case 7: x2++; x--; y2 += 7; break; case 3: case 4: case 5: x2++; x--; y -= 7; break; case 2: y2++; y--; x -= 7;break; case 6: y2++; y--; x2 += 7;break; } map->foreachinarea(skill->area_sub, src->m, x, y, x2, y2, BL_CHAR, src, skill_id, skill_lv, tick, flag|BCT_ENEMY|SD_ANIMATION|1,skill->castend_damage_id); skill->addtimerskill(src,timer->gettick() + status_get_amotion(src), 0, 0, 0, LG_OVERBRAND_BRANDISH, skill_lv, dir, flag); } break; [/code]

[img]http://i40.tinypic.com/4gx77.jpg[/img]

:meow:

N e s s - Jan 12, 2014 15:02

so if the player whos having the damage not pushed over the wall it will only take 1 damage?

N e s s - Jan 12, 2014 15:05

and by the way is hercules now supports up to level 175?

malufett - Jan 12, 2014 15:07

[quote name="N e s s" timestamp="1389538965"]
so if the player whos having the damage not pushed over the wall it will only take 1 damage?[/quote]
no..actually the image I posted above should be 3 hits the piercing, swing and the knockback bonus..however only two captured...
and it will never be 1 hit even knockback except if outside the pierce+swing area or it misses..

this is the full 3 hits..
[img]http://i43.tinypic.com/1498cbd.jpg[/img]
:meow:

This post has been edited by malufett on Jan 12, 2014 15:11

malufett - Jan 12, 2014 15:18

[quote name="N e s s" timestamp="1389539132"]
and by the way is hercules now supports up to level 175?[/quote]
ah..in my screen shots...the suppose 'Level limit break' update to be commited soon is still intact in my latest hercules.. :P

:meow:

This post has been edited by malufett on Jan 12, 2014 15:19

N e s s - Jan 12, 2014 15:24

ok sir malufett :) thanks you sir! :)

N e s s - Jan 13, 2014 2:08

but when use it from players it only hit once in any cell why?

malufett - Jan 13, 2014 6:21

[img]http://i41.tinypic.com/5zpmwl.jpg[/img]
in this SS the target receive 2 hits..and no 3rd hit since already dead before collision to the wall...

:meow:

krabs - Jan 13, 2014 8:32

[URL=https://imageshack.com/i/mj7qxvj][img]http://imagizer.imageshack.us/v2/800x600q90/811/7qxv.jpg[/img][/URL]

[URL=https://imageshack.com/i/gvsarkj][img]http://imagizer.imageshack.us/v2/800x600q90/607/sark.jpg[/img][/URL]

same here, it only does the piercing damage, i believe this happens after applying the script for #overbrandhittingtargetsinhide

malufett - Jan 13, 2014 9:21

hmm..weird after updating my files now I produce the bug.. i didn't see any conflicts with Haru and Ind's commit..anyway let me check..

edit:
I know now the problem(skill_db.txt not updated)..my tortoise do trolling me again I have the same files in testing and commiting updates so no conflicts in confirming..it shows it was unmodified but eventually its not commited..haaaiiss..sorry for the inconvenience..

:meow:

This post has been edited by malufett on Jan 13, 2014 9:42

N e s s - Jan 14, 2014 3:14

sir! is OB already fixed? i mean the hit? :)

malufett - Jan 14, 2014 14:02

can't commit the fix atm..anyway here it is

@skill_db.txt

[color=#ff0000]-2317,1,6,2,-1,0x2,5,5,1,yes,0,0,0,none,3:4:5:6:7, LG_OVERBRAND,Over Brand //CHECK I know the splash is needed somehow for the strange AoE it gives.[/color]
[color=#00ff00]+2317,2,6,2,-1,0x2,0,5,1,yes,0,0,0,weapon,0, LG_OVERBRAND,Over Brand[/color]

2325,0,6,4,0,0x1,0,5,1,yes,0,0,0,none,0, LG_INSPIRATION,Inspiration
[color=#00ff00]+2519,1,6,1,-1,0x2,0,5,1,yes,0,0,0,weapon,3:4:5:6:7, LG_OVERBRAND_BRANDISH,Over Brand[/color]

:meow:

This post has been edited by malufett on Jan 14, 2014 14:05

N e s s - Jan 17, 2014 1:12

hi sir maluffet! is this patch been comitted already? :) because i want the official one! thank you sir!

malufett - Feb 10, 2014 7:03

Fixed @ 4c735df7d013901c5fc79984bd2206d2109f53d8

:meow: