Issue information

Issue ID
#6527
Status
Fixed
Severity
None
Started
Hercules Elf Bot
Aug 21, 2012 19:43
Last Post
kyeme
Nov 13, 2015 13:03
Confirmation
Yes (1)
No (1)

Hercules Elf Bot - Aug 21, 2012 19:43

Originally posted by [b]kyeme[/b]
In Official when TALKING TO NPC you can USE USABLE ITEMS, SKILLS (Self Skills only) and CHANGE EQUIPMENTS / EQUIP/UNEQUIP (you can Change Equip except Refiner NPC). However what we have is once we talk to the NPC it will not allow us to USE usable items, skills and change equipment.
~kyeme :)

This post has been edited by kyeme on Aug 21, 2012 20:10

Hercules Elf Bot - Aug 22, 2012 8:55

Originally posted by [b]malufett[/b]
and I think we need to create another script command for this
[quote](you can Change Equip except Refiner NPC)[/quote]

:meow:

Hercules Elf Bot - Aug 22, 2012 20:44

Originally posted by [b]Ind[/b]
[quote name='malufett' timestamp='1345625746' post='13620']
and I think we need to create another script command for this[quote](you can Change Equip except Refiner NPC)[/quote]:meow:
[/quote]
thats a rather dangerous thing to do -- it'd open exploits in many poorly scripted npcs.

Hercules Elf Bot - Aug 23, 2012 6:08

Originally posted by [b]malufett[/b]
so what shall we do? to hinder characters to change equipment/use skills/ and etc in a specific npc?? by not affecting general npc..:)

:meow:

Hercules Elf Bot - Aug 23, 2012 6:11

Originally posted by [b]Kenpachi[/b]
in AEGIS there is a script command to block the players ability to change equip... I'd say let's do it the official way.

Hercules Elf Bot - Aug 23, 2012 6:36

Originally posted by [b]malufett[/b]
[quote]in AEGIS there is a script command to block the players ability to change equip... I'd say let's do it the official way.[/quote]
+1

Hercules Elf Bot - Aug 23, 2012 17:56

Originally posted by [b]Lunar[/b]
I think we have to list all npc which could be vulnerable to this behavior, like card remover, [s]or armor enchanter[/s] (player have to take only one copy of armor)

This post has been edited by Lunar on Aug 23, 2012 18:00

Hercules Elf Bot - Aug 23, 2012 18:16

Originally posted by [b]Leeg[/b]
I'd like to point out that in iRO you can't use self-skills or items, but you can change your equipment.

If you try to, a message is shown in the general message tab:

'any work in progress (NPC dialog, manufacturing...) quit and try again',

Hercules Elf Bot - Aug 24, 2012 8:03

Originally posted by [b]malufett[/b]
[quote]I'd like to point out that in iRO you can't use self-skills or items, but you can change your equipment.[/quote]
as I checked kRO only non-self skill target cannot be used..but let me check if there is a error message when doing restricted moves...

:meow:

Hercules Elf Bot - Aug 24, 2012 20:47

Originally posted by [b]mkbu95[/b]
[code]*enable_items;
*disable_items;

These commands enable item usage while an NPC is running. When enable_items is
run, items can be used during scripts until disable_items is called.
To avoid possible exploits, when enable_items is invoked, it will only enable
item usage while running that script in particular. Note that if a different
script also calls enable_items, it will override the last call (so you may
want to call this command at the start of your script without assuming the
effect is still in effect).[/code]
?

Hercules Elf Bot - Sep 7, 2012 12:32

Originally posted by [b]Angezerus[/b]
[quote name='Ind' timestamp='1345668289' post='13658']
[quote name='malufett' timestamp='1345625746' post='13620']and I think we need to create another script command for this[quote](you can Change Equip except Refiner NPC)[/quote] :meow:[/quote]thats a rather dangerous thing to do -- it'd open exploits in many poorly scripted npcs.
[/quote]
As long as the criteria check is in the same "frame" as the "action", there shouldn't be any problem, and afaik every official npc checks in the exact moment when it does something with an item. Or am I wrong?

Hercules Elf Bot - Sep 7, 2012 15:43

Originally posted by [b]Brian[/b]
I think leave item/skill usage disabled and let users enable item use if they want, using 'enable_items' and 'disable_items'.

kyeme - Feb 7, 2013 6:20

Any news in kRO?
[quote name="Hercules Elf Bot" timestamp="1345745814"]
Originally posted by [b]Leeg[/b]
I'd like to point out that in iRO you can't use self-skills or items, but you can change your equipment.

If you try to, a message is shown in the general message tab:

'any work in progress (NPC dialog, manufacturing...) quit and try again',[/quote]

This behavior are now on pRO. Bifrost episode.
[url="http://www.youtube.com/watch?v=iMO_4BoGYIM&feature=youtu.be"]http://www.youtube.com/watch?v=iMO_4BoGYIM&feature=youtu.be[/url]

Beret - Feb 7, 2013 15:46

I confirm in iRO Sakray.

malufett - Feb 7, 2013 16:39

[quote]
Any news in kRO?[/quote]
same ways...

:meow:

kyeme - Feb 9, 2013 9:08

[color=rgb(40,40,40)][font=helvetica, arial, sans-serif][color=rgb(51,51,51)][font='lucida grande', tahoma, verdana, arial]If you are allowed to unequipping/equipping while talking to NPC, maybe what we should do is when there is written in the script/npc like[b]"successrefitem, failedrefitem, successremovecards"[/b], then.. equipping and unequipping should be automatically disabled when talking to that NPC. [/font][/color][/font][/color]
[color=rgb(51,51,51)][font=helvetica, arial, sans-serif][size=3][font='lucida grande', tahoma, verdana, arial]I[/font][/size][/font][/color][color=rgb(51,51,51)][font=helvetica, arial, sans-serif][size=3][font='lucida grande', tahoma, verdana, arial][size=2] [size=4]would suggest this if this is possible. :)[/size][/size][/font][/size][/font][/color]

This post has been edited by kyeme on Feb 9, 2013 9:20

Euphy - Feb 9, 2013 13:56

[quote name="kyeme" timestamp="1360400919"]
[color=#282828][font=helvetica, arial, sans-serif][color=#333333][font=tahoma, verdana, arial]If you are allowed to unequipping/equipping while talking to NPC, maybe what we should do is when there is written in the script/npc like[b]"successrefitem, failedrefitem, successremovecards"[/b], then.. equipping and unequipping should be automatically disabled when talking to that NPC. [/font][/color][/font][/color]
[color=#333333][font=helvetica, arial, sans-serif][size=3][font=tahoma, verdana, arial]I[/font][/size][/font][/color][color=#333333][font=helvetica, arial, sans-serif][size=3][font=tahoma, verdana, arial][size=2] [size=4]would suggest this if this is possible. :)[/size][/size][/font][/size][/font][/color][/quote]
There are commands in the official scripts that toggle this functionality -- that hasn't been the problem. Something like this will totally break custom scripts on a [i]lot[/i] of servers, potentially creating exploits everywhere. But I suppose it has to be done sometime. ._.

If somebody codes the source for this, I can add the commands to our scripts.

malufett - Feb 23, 2013 22:07

moving this to scripts

please identify all npc that needs disabling of equipping/unquipping of equipment and use this command.
[code=auto:0] *enable_items; *disable_items; [/code]
afterwards return it here so that we can enable by default 'enable_items'

Thank you..
:meow:

Euphy - Feb 24, 2013 4:23

I'll make a full script patch if you post a full core patch. XD[code=auto:0]cashitem\advholgren.sc 27 local\dewata\dewata_weapon.sc 360 local\dewata\dewata_weapon.sc 423 local\malangdo\mal_jerun.sc 48 local\malangdo\mal_jerun.sc 574 local\malangdo\mal_yong.sc 25 local\malangdo\mal_yong.sc 1912 quest\2004_hat.sc 829 quest\2005_hat.sc 1528 quest\2005_hat.sc 2288 quest\god\god_mjol.sc 1955 quest\god\god_mjol.sc 2310 quest\tutorial\para_suvquest.sc 3382 siglude\alberta_refine.sc 231 siglude\einbroch_refine.sc 232 siglude\lighthalzen_refine.sc 231 siglude\morocc_refine.sc 231 siglude\new_smelting.sc 27 siglude\new_smelting.sc 442 siglude\new_smelting.sc 857 siglude\payon_refine.sc 231 siglude\prontera_refine.sc 231 siglude\yuno_refine.sc 231 [/code]

This post has been edited by Euphy on Feb 24, 2013 4:24

Euphy - Feb 25, 2013 3:05

[attachment=103:r17160.patch]

This post has been edited by Euphy on Feb 25, 2013 3:05

malufett - Feb 26, 2013 18:24

Thanks...Fixed @ [url="https://github.com/HerculesWS/Hercules/commit/ad792f41994ce54dc9e45fbdaaf591f38fccecee"]https://github.com/HerculesWS/Hercules/commit/ad792f41994ce54dc9e45fbdaaf591f38fccecee[/url]

:meow:

kyeme - Mar 4, 2013 11:57

[quote name="malufett" timestamp="1361903042"]
Thanks...Fixed @ [url="https://github.com/HerculesWS/Hercules/commit/ad792f41994ce54dc9e45fbdaaf591f38fccecee"][url="https://github.com/HerculesWS/Hercules/commit/ad792f41994ce54dc9e45fbdaaf591f38fccecee"]https://github.com/HerculesWS/Hercules/commit/ad792f41994ce54dc9e45fbdaaf591f38fccecee[/url][/url]

:meow:[/quote]
[s]Commit failed? [/s] Nevermind.

This post has been edited by kyeme on Mar 4, 2013 13:37

kyeme - Nov 13, 2015 13:03

On pre-renewal you can: use usable items and self skills while talking to NPC.

This post has been edited by kyeme on Nov 13, 2015 14:16