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

  • #006155

  • 0 - None Assigned

  • Working as Intended

Issue Confirmations

  • Yes (0)No (0)
Photo

Divest Helm & Full Divestment

Posted by Hercules Bot on 03 July 2012 - 12:08 AM

Originally posted by Daegaladh
Both should strip EQP_HEAD_TOP instead of EQP_HELM.


Tested on iRO.

Originally posted by malufett
#define EQP_HELM (EQP_HEAD_LOW|EQP_HEAD_MID|EQP_HEAD_TOP)

EQP_HELM is consist of EQP_HEAD_LOW, EQP_HEAD_MID, EQP_HEAD_TOP
so what should be??

Originally posted by Daegaladh
As I said, Divest Helm and Full Divestment only removes top headgears on official servers. It doesn't remove middle or low headgears. (but works on top+mid and top+mid+low headgears obviously)

Edited by Daegaladh, 03 July 2012 - 03:37 AM.


Originally posted by malufett
hmm..ok...thanks...

BTW I check latest SVN...and I test it..it doesn't strip the the mid and low...only the top...
and the code shows
i = sd->equip_index[EQI_HEAD_TOP];

so nothing to change...

Originally posted by Daegaladh
Oops sorry, I saw this in skill.c and I thought the bug was still there:


case RG_STRIPHELM:
location = EQP_HELM;
break;
case ST_FULLSTRIP:
location = EQP_WEAPON|EQP_SHIELD|EQP_ARMOR|EQP_HELM;
break;


moved issue from Skills