Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Arrow equip #1155

Closed
Atemo opened this issue Apr 9, 2016 · 16 comments
Closed

Arrow equip #1155

Atemo opened this issue Apr 9, 2016 · 16 comments
Labels
component:core A fault that lies within the main framework of rAthena mode:prerenewal A fault that exists within the pre-renewal mode mode:renewal A fault that exists within the renewal mode priority:low A fault that affects rAthena in one piece of functionality and is self-contained type:bug Issue that is a bug within rAthena

Comments

@Atemo
Copy link
Contributor

Atemo commented Apr 9, 2016

Hash 2305a66
On official renewal you can't equip arrow without bow equipped (so on rArthena you can. Assassin class can't equip them ; skill fail for ranger if no bow equipped).
Also on official if you unequip the bow the arrow is automatically unequipped

@cydh
Copy link
Contributor

cydh commented Apr 9, 2016

Depends on which official server u refers to :P

But maybe, this behavior was added in the MT when Assassin cannot equip an arrow any longer.

@Atemo
Copy link
Contributor Author

Atemo commented Apr 9, 2016

lol you're right, I refer to iro we need to test on kro

@RagnarokNova
Copy link
Contributor

Confirmed on kRO. Unequipping a bow removes your arrows. It's really uncomfortable.

@secretdataz
Copy link
Member

This should be made optional with official behavior as default imo.

@Playtester
Copy link
Member

If we made an option for that please make also one for ammo of gunslingers. There the official behavior is that you can't even equip weapons when the ammo doesn't match, very annoying. I'd rather have that auto-unequip that arrows have lol.

@MrAntares
Copy link

What happens on bow switch? Will it de-equip the arrows?

@Atemo
Copy link
Contributor Author

Atemo commented Apr 11, 2016

No it doesn't unequip the arrow when you switch the bow.

@stealth62
Copy link

this will break pvp mechanics when you switch bow and arrow automatically unequip I remember iro revert this changes a lot of player complain switching bow will unequip arrow.

@Playtester
Copy link
Member

Even more reason to make it optional.

@MrAntares
Copy link

And default off ;)

@Playtester
Copy link
Member

Default is always kRO. =3

@joelolopez
Copy link

i cant equip new type of guns coz the bullets is still equipped, such a hassle

@Playtester
Copy link
Member

Yeah that's official behavior though. I'd like to see it configurable. Should just automatically unequip the bullets and arrows when no suitable weapon equipped.

@joelolopez
Copy link

oh i see , lets keep it like that ahahha i love official behaviors , always keep messing ur game ahaha xo

@MrAntares
Copy link

MrAntares commented Apr 18, 2016

Much work for a feature that will rarely be used <3
Btw what happens if you switch from bow to dagger?

benching added a commit to benching/rathena that referenced this issue May 6, 2016
@benching
Copy link

benching commented May 13, 2016

On assasin , baby and parent/next jobs,
is venom knife(1771) can be equipped even without weapon?

@aleos89 aleos89 added status:confirmed Issue that has been validated by a developer to affect rAthena component:core A fault that lies within the main framework of rAthena priority:low A fault that affects rAthena in one piece of functionality and is self-contained mode:renewal A fault that exists within the renewal mode mode:prerenewal A fault that exists within the pre-renewal mode type:bug Issue that is a bug within rAthena labels Apr 19, 2018
aleos89 added a commit that referenced this issue Aug 28, 2018
* Fixes #1155.
* Added specific checks for weapons when equipping ammo.
* Ammo is removed if the required weapon is removed.
* Added battle configs to disable behavior.
Thanks to @Atemo, @exneval, and @benching!
@aleos89 aleos89 closed this as completed in d3d8f3c Sep 5, 2018
@aleos89 aleos89 removed the status:confirmed Issue that has been validated by a developer to affect rAthena label Sep 5, 2018
SeravySensei pushed a commit to SeravySensei/rathena that referenced this issue Jan 26, 2019
* Fixes rathena#1155.
* Added specific checks for weapons when equipping ammo.
* Ammo is removed if the required weapon is removed.
* Added battle configs to disable behavior.
Thanks to @Atemo, @exneval, and @benching!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:core A fault that lies within the main framework of rAthena mode:prerenewal A fault that exists within the pre-renewal mode mode:renewal A fault that exists within the renewal mode priority:low A fault that affects rAthena in one piece of functionality and is self-contained type:bug Issue that is a bug within rAthena
Projects
None yet
Development

No branches or pull requests

10 participants