Jump to content
Sign in to follow this  
jaBote

Suggestion for restricted equipment (making a new conf for it)

Recommended Posts

Hi everybody! I was thinking if it was possible to create a new conf value for the disabled equipment which make you decide whether you want to unequip a disabled item or just nullify its effect.

 

Yeah I know nullifying that effect is the official option and that it was implemented in 5b378058f5 and cc8e005f27 (thanks, kyeme!), but I know for sure that there'll be a lot of mess on servers because of unaware people (the most) seeing they can use their equip on those maps but without any effect and claiming it to be a critical bug, which is not correct. Most players and admins are used to being forced by the map server to unequip disabled equipment. Moreover, they know an item is disabled because they see it wear off, but with the new system you don't usually note one of your equipped items is disabled until you see it has no effect.

 

What I'm suggesting is to create a conf value just like the item_restricted_consumption_type one in /conf/battle/items.conf which should let server admins to decide whether they want to select the official behavior or the one that has been there for years.

 

Thanks a lot!

Share this post


Link to post
Share on other sites

Yes I know -I've been reading this forum and Hercules source commits for like a month before deciding to register-, and I think this is a great addition for the emulator, but when you disable any equipment in the db/{re or pre-re}/map_zone_db.txt it just gets... what it says, disabled, like you were equiping dummy equipment. But I think it'd be useful to make admins have the power of deciding if they want to just disable the equipment or do what Athena emulators were doing for ages, just unequip them where they are deactivated. It'd be logic.

Share this post


Link to post
Share on other sites

what it says, disabled, like you were equiping dummy equipment. But I think it'd be useful to make admins have the power of deciding if they want to just disable the equipment or do what Athena emulators were doing for ages, just unequip them where they are deactivated. It'd be logic.

this was changed recently when we found out that it was how official servers were doing it (nullify effect instead of unequip) i think we added a config to choose between the two hmmm....

edit:

aah no we added the setting only for the item-use behavior that was also changed with it...

// Will disabled consumables (disabled by map_zone_db.conf) be consumed when trying to use them?// 1 (official): yes// 0: noitem_restricted_consumption_type:1

Share this post


Link to post
Share on other sites

I have a question. I'm trying to restrict an item (equipment) not consumable, placed it in mapzone but it doesn't seem to work at all?

Share this post


Link to post
Share on other sites

http://upaste.me/f94d11052d9751e97

this one should go inside SVN

EDIT: suddenly I feel this one can be improve ....

 

ok this one better

http://upaste.me/d2ac11053eed06c03

Edited by AnnieRuru

Share this post


Link to post
Share on other sites

actually I'm thinking of writing both ... with this one

http://herc.ws/board/topic/4412-map-zone-dbconf-allows-to-restrict-an-item-type/

then only pull request

 

go ahead annie, both ideas are good and got support from community! they will be accepted for sure :P

Share this post


Link to post
Share on other sites

I read this topic before, when I was on my cellphone.

 

First of all, thanks Annie for the SRC ^_^ ! I feel this would be appropriate to add to the repo because it's a classic mechanic, but will refrain to do that until a dev agrees on committing it just to see if my opinion is followed.

Share this post


Link to post
Share on other sites

makes sense to me, i agree.

Hope Hercules to implement this fix , Cuz player still can wear  the restricted equipments of disabled_items

Share this post


Link to post
Share on other sites

pull request that's it ?

https://github.com/HerculesWS/Hercules/pull/330

 

for the other one ... its quite tough ...

it got 2 problems that needs time to tackle with

 

https://github.com/rathena/rathena/blob/master/conf/battle/items.conf#L91

I just realized rathena already have something similar ...

so we should get to do the same

Edited by AnnieRuru

Share this post


Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Loading...
Sign in to follow this  

×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.