Issue information

Issue ID
#8354
Status
Confirmed
Severity
None
Started
kyeme
Sep 20, 2014 2:25
Last Post
GrumpyPanda
Mar 10, 2015 7:58
Confirmation
N/A

kyeme - Sep 20, 2014 2:25

Here are the infos:

[color=rgb(0,0,255)]Official:[/color]
If the party member exceed more than 12 cell the Devotion and so as Paladin's buff (Reflect Shield, Auto Guard, Endure) will automatically break/disable upon hit.
[size=5][b]In the case of Defending Aura buff, upon first hit Defending Aura will take effect and then automatically be canceled.[/b][/size]
[url="https://www.youtube.com/watch?v=h3PKq64F_W8&feature=youtu.be"]https://www.youtube.com/watch?v=h3PKq64F_W8&feature=youtu.be[/url]

[color=rgb(255,0,0)]Hercules:[/color]
If the party member already exceed more than 12 cell, upon hit (ex. Champion used Asura Strike) the Devotion and so as Paladin's Reflect Shield still takes effect.
[url="https://www.youtube.com/watch?v=GOPxXLR3O5o&feature=youtu.be"]https://www.youtube.com/watch?v=GOPxXLR3O5o&feature=youtu.be[/url]


[size=5][b]Casting Range:[/b][/size]
[color=rgb(0,0,255)]Official:[/color]
Devotion cast range is 12 cell, if party member is in the 13th cell, you can still cast Devotion but "skill has failed" will be shown.

Im not quite sure about the cell counting. Kindly refer to this screenshot for further understanding. Thank you!
[img]http://i.imgur.com/K20R1BW.jpg[/img]

This post has been edited by kyeme on Sep 20, 2014 4:51

kyeme - Sep 20, 2014 2:25

The missing part on latest Revision are:
~ [b]Endure[/b] [size=3]([color=rgb(40,40,40)][font=helvetica, arial, sans-serif]If the party member exceed more than 12 cell the Devotion and so as Paladin's buff (Reflect Shield, Auto Guard, [b]Endure[/b]) will automatically break/disable upon hit.)[/font][/color][/size]
~ [b]Casting Range[/b]

This post has been edited by kyeme on Sep 20, 2014 2:27

Ind - Sep 21, 2014 4:59

I believe the devotion reflect part was fixed in [url="https://github.com/HerculesWS/Hercules/commit/76662cc551f4"]https://github.com/HerculesWS/Hercules/commit/76662cc551f4[/url]

kyeme - Sep 21, 2014 5:15

Yes sir, but the only problem now is :
1. The Endure is still there even the party member is out of range of Paladin

2. Casting Range

csnv - Sep 21, 2014 13:38

[quote name="kyeme" timestamp="1411276549"]
Yes sir, but the only problem now is :
1. The Endure is still there even the party member is out of range of Paladin

2. Casting Range[/quote]

I've got a fix for the first one (one I didn't like so I didn't PR'd it).

About the second, IIRC the default range in Hercules for Devotion is exactly 11 cells, that said, we'll have to add 1 extra cell to it in the DB files and then make it fail if the user is in the last cell. Kind of problematic to replicate official behavior, no?

GrumpyPanda - Mar 9, 2015 2:41

Was about to make a new one, but since its highly related and the videos showcase it perfectly I'd like to add paladin does not get hit animation on hercules.

kyeme - Mar 9, 2015 4:11

[quote name="GrumpyPanda" timestamp="1425868889"]
Was about to make a new one, but since its highly related and the videos showcase it perfectly I'd like to add paladin does not get hit animation on hercules.[/quote]
Confirmed.
Thanks :D

malufett - Mar 9, 2015 19:35

btw what more is missing? only the flinch effect?

:meow:

kyeme - Mar 10, 2015 0:54

[quote name="malufett" timestamp="1425929710"]
btw what more is missing? only the flinch effect?

:meow:[/quote]
The devotion range and Flinch effect sir

GrumpyPanda - Mar 10, 2015 7:58

There also seems to be some skill animation on the paladin right before the flinch when getting devoted damage. So I guess clif->skill_damage should be used instead of clif->damage for devoted damage.