Divest Helm & Full Divestment - Hercules bug tracker archive

Issue information

Issue ID

#6155

Status

Working as Intended

Severity

None

Started

Hercules Elf Bot

Jul 3, 2012 0:08

Last Post

Mysterious

Mar 27, 2014 21:30

Confirmation

N/A

Hercules Elf Bot - Jul 3, 2012 0:08

Originally posted by [b]Daegaladh[/b]
Both should strip EQP_HEAD_TOP instead of EQP_HELM.

Tested on iRO.

Hercules Elf Bot - Jul 3, 2012 3:22

Originally posted by [b]malufett[/b]
[CODE]#define EQP_HELM (EQP_HEAD_LOW|EQP_HEAD_MID|EQP_HEAD_TOP)[/CODE]

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

Hercules Elf Bot - Jul 3, 2012 3:34

Originally posted by [b]Daegaladh[/b]
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)

This post has been edited by Daegaladh on Jul 3, 2012 3:37

Hercules Elf Bot - Jul 3, 2012 3:57

Originally posted by [b]malufett[/b]
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
[CODE]i = sd->equip_index[EQI_HEAD_TOP];[/CODE]

so nothing to change...

This post has been edited by malufett on Jul 3, 2012 4:15

Hercules Elf Bot - Jul 3, 2012 11:27

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

[CODE]
case RG_STRIPHELM:
location = EQP_HELM;
break;
case ST_FULLSTRIP:
location = EQP_WEAPON|EQP_SHIELD|EQP_ARMOR|EQP_HELM;
break;[/CODE]