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

  • #006396

  • 0 - None Assigned

  • Confirmed

Issue Confirmations

  • Yes (2)No (0)
Photo

Chemical Protection

Posted by Hercules Bot on 02 August 2012 - 06:09 AM

Originally posted by kyeme
#1 On Official Server: ITEM CP_Helm_Scroll and other chemical protection scroll are ITEM TYPE 11. And "skill has failed" if the users dont have Coating Bottle and NO EQUIPMENT.

#2 On Official Server: SKILL Chemical Protection of alchemist class "skill has failed" if the targetNO EQUIPMENT.

@edit #2 already fixed.

Originally posted by malufett

On Official Server: SKILL Chemical Protection of alchemist class "skill has failed" if the target is NO EQUIPMENT.

Fixed @ [rev='16560']

On Official Server: ITEM CP_Helm_Scroll and other chemical protection scroll are ITEM TYPE 11. And "skill has failed" if the users dont have Coating Bottle and NO EQUIPMENT.

I'll left this work to the db/script dev... /no1

Originally posted by kyeme
Thank you sir.

Originally posted by kyeme
I used the latest SVN, the only fixed AM_CP_WEAPON, AM_CP_SHIELD, AM_CP_ARMOR, and AM_CP_HELM but CR_FULLPROTECTION is still used in areas without equipment.. Thank you again sir.

Originally posted by malufett
ooops..sorry forgot..:)

Originally posted by kyeme
Its ok, thanks again. Hahaha hirap na ako mag ingles nyahaha.

Edited by kyeme, 26 November 2012 - 07:04 PM.


Originally posted by malufett
lols...I think you really need to listen to your English teacher better..
anyway I'll commit the follow up fixes tomorrow really need to get sleep now..:)

Originally posted by malufett

I used the latest SVN, the only fixed AM_CP_WEAPON, AM_CP_SHIELD, AM_CP_ARMOR, and AM_CP_HELM but CR_FULLPROTECTION is still used in areas without equipment.. Thank you again sir.

ok done in [rev='16567'] /no1

Originally posted by kyeme
Thank you very much! 143 /kis

Originally posted by malufett
Euphy its not yet fixed...

On Official Server: ITEM CP_Helm_Scroll and other chemical protection scroll are ITEM TYPE 11. And "skill has failed" if the users dont have Coating Bottle and NO EQUIPMENT.


must change the SC change script of the item...

Originally posted by Kenpachi
So the only thing left is to change the item type to 11?
If that's all I'd like to wait for the outcome of this report since it implies that item type 11 is kind of unneeded.

changed status to: Fixed

Still bug, the scroll is self skill. In official this scroll is item type 11 (target skills)

Hmm, I see. Let me see what can be done...

changed status to: Confirmed

Okay, I did some investigation and the only way to make this useable like the original skill (you can choose the target) is to use the itemskill command. Unfortunately this command does not check for required items (catalysts) so I had to create a new bug report. I'll update the item scripts when this is done...