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

  • #006751

  • 0 - None Assigned

  • Fixed

Issue Confirmations

  • Yes (0)No (0)
Photo

RK Nosiege Rune

Posted by Hercules Bot on 03 October 2012 - 11:25 PM

Originally posted by xaxixi
Nosiege rune: Refresh - Changed the rune stone to be usable while in Frozen, Stun, Deep Sleep, Sleep, Petrification and Crystallization status.

src:http://forums.irowik...hp/t-95629.html

Originally posted by Beret
Is expected to fix ?

Originally posted by MarkZD
The piece below is incomplete.
I think it's enough:
Open src/map/pc.c
Find:
if( sd->sc.count && (
  sd->sc.data[SC_BERSERK] || sd->sc.data[SC__BLOODYLUST] ||
  (sd->sc.data[SC_GRAVITATION] && sd->sc.data[SC_GRAVITATION]->val3 == BCT_SELF) ||
  sd->sc.data[SC_TRICKDEAD] ||
  sd->sc.data[SC_HIDING] ||
  sd->sc.data[SC__SHADOWFORM] ||
  sd->sc.data[SC__MANHOLE] ||
  sd->sc.data[SC_KAGEHUMI] ||
  (sd->sc.data[SC_NOCHAT] && sd->sc.data[SC_NOCHAT]->val1&MANNER_NOITEM)
))
  return 0;
// Store information for later use before it is lost (via pc_delitem) [Paradox924X]
nameid = sd->inventory_data[n]->nameid;
Replace with:
// Store information for later use before it is lost (via pc_delitem) [Paradox924X]
nameid = sd->inventory_data[n]->nameid;

if( sd->sc.count && (
  sd->sc.data[SC_BERSERK] || sd->sc.data[SC__BLOODYLUST] ||
  (sd->sc.data[SC_GRAVITATION] && sd->sc.data[SC_GRAVITATION]->val3 == BCT_SELF) ||
  sd->sc.data[SC_TRICKDEAD] ||
  sd->sc.data[SC_HIDING] ||
  sd->sc.data[SC__SHADOWFORM] ||
  sd->sc.data[SC__MANHOLE] ||
  sd->sc.data[SC_KAGEHUMI] ||
  sd->sc.data[SC_WHITEIMPRISON] || // Fix to the list
  (sd->sc.data[SC_NOCHAT] && sd->sc.data[SC_NOCHAT]->val1&MANNER_NOITEM && nameid != 12725) //You can use Nosiege under some status
))
  return 0;

Edited by MarkZD, 24 November 2012 - 04:17 AM.


Originally posted by Beret
up.

Originally posted by Beret
still not to use when frozen, stun and deep sleep MarkZD.

Originally posted by MarkZD

still not to use when frozen, stun and deep sleep MarkZD.

You're right, my fix is incomplete.

Edited by MarkZD, 24 November 2012 - 04:08 AM.


Originally posted by Lighta
should be fixed in [rev=16959]

Originally posted by MarkZD

should be fixed in [rev=16959]


I tried a similar fix this morning, after I post the message above, and it didn't work as well as yours.

For some reason it doesn't reach clif_parse_useitem when the user is under Deep Sleep and Stone curse.

Edited by MarkZD, 24 November 2012 - 01:27 PM.


Originally posted by Lighta
Did you removed the option check in clif_parse_UseItem ? and the status and option before asking to do the skill ? (need to remove directly option or it will be delayed to 20ms)
Maybe we can discuss more on irc or pm to not flood the issue

Originally posted by MarkZD
Also, in the way you changed it's possible to use the Rune under Berserk, manhole, hiding etc which possibly need more info, as the rune may not work in this way.

Originally posted by MarkZD

Did you removed the option check in clif_parse_UseItem ? and the status and option before asking to do the skill ? (need to remove directly option or it will be delayed to 20ms)Maybe we can discuss more on irc or pm to not flood the issue


Yep, I deleted both files I changed and updated with yours to test.

I'm on irc.

Edited by MarkZD, 24 November 2012 - 01:43 PM.


Originally posted by Beret
Still no fix lighta, if the Rune is in the range, and the rune kinght is frozen to use the Rune even crashing it is defrosted.

Originally posted by MarkZD

Still no fix lighta, if the Rune is in the range, and the rune kinght is frozen to use the Rune even crashing it is defrosted.

We(lighta and me) know, the item is taking out the effects even when it's under the cooldown, it'll be fixed soon.



About using it under Deep Sleep, Sleep and Stone curse, apparently it's a client behavior. (At least it didn't work on some 2010.
I burlated the options so the client can use it under Deep Sleep while I tested it, but it doesn't seem it should be the way to go).

rAthena still doesn't work with 08/2012+ clients, which are the one that get this feature in sakray, so I don't have a very good way to test if it's supposed to work only in those clients.

This is what I found while I tested, maybe someone who has more knowledge of changeoption can argument about it, as it seems the setting that is blocking the client from using runes when under the status mentioned in this post.

Edited by MarkZD, 25 November 2012 - 06:18 PM.


Originally posted by Lighta
Followup in [rev=16969]

Originally posted by Beret
Seems to work now lighta.

Originally posted by Rytech
All runes should be usable while in those kind of status's according to a tester's results. I wonder if their usable while in Berserk as well.

1. any rune can be used while you are on silence status, cursed circle, root, oblivion curse, ETC.

2. any rune Skill can be use in middle of the aftercast dealy of other skills, but if you use a rune in middle of the cast time of other skill the rune could be consumed and do nothing.

Source: http://forums.irowik...29&postcount=73

Originally posted by MarkZD

1. any rune can be used while you are on silence status, cursed circle, root, oblivion curse, ETC.

It's this way now.
http://trac.rathena....changeset/16969

I've sent another patch to lighta wich will fix status being removed when refresh is used, just need to wait his answer.


but if you use a rune in middle of the cast time of other skill the rune could be consumed and do nothing.
Source: http://forums.irowik...29&postcount=73

Currently it's just not used/consumed, if you try to use it while you're casting.
Is it still this way?
So all item skill are this way or just runes?
We need confirmation on official.

Edited by MarkZD, 27 November 2012 - 06:36 PM.


Originally posted by MarkZD
Fixed at: r16979.

Originally posted by xaxixi
i can't use refresh in stone or sleep.

Originally posted by MarkZD

i can't use refresh in stone or sleep.


Unfortunately, it's a client behavior which blocks you from using item when you're in these states.