server icon indicating copy to clipboard operation
server copied to clipboard

🐛 Throwing darts is mistakenly using the singing animation

Open almuth150 opened this issue 1 year ago • 7 comments

I affirm:

  • [x] I understand that if I do not agree to the following points by completing the checkboxes my issue will be ignored.
  • [x] I have read and understood the Contributing Guide and the Code of Conduct.
  • [x] I have searched existing issues to see if the issue has already been opened, and I have checked the commit log to see if the issue has been resolved since my server was last updated.

OS / platform the server is running (if known)

Branch affected by issue

base

Steps to reproduce

Equip a Dart, or any other throwing ammo equip that has no model, then do a ranged attack

Expected behavior

The animation shouldn't be the one used for singing a BRD song with no instrument. I think the proper animation is different from the one used for throwing shuriken, but I'm not 100% certain.

almuth150 avatar Jan 02 '25 21:01 almuth150

Can confirm, a few throwing weapons are missing model data, simple PR if anyone wants to take the time going through the item_equipment.sql and filling the MID's in, will need to confirm using retail captures for things like shurikens and grenades but I think it may only be a few items missing, darts can all be MID(model) 22 as they don't show in game and as far as I know there is only 1 MID for darts.

Omnione avatar Jan 13 '25 21:01 Omnione

Dart shows it already has model ID of 22 in the database. I think I might recall something similar happening to me before and it was related to style lock. Mind try going into ingame menu, disabling style lock, then try throwing again and see if it still does it?

UmeboshiXI avatar Jan 14 '25 02:01 UmeboshiXI

Tested with style lock, same thing happens if the throwing weapon being style locked has no MID in item_equipment.sql, Interestingly though the in game command /stylelock off does not seem to change to ammo MID back to what the player currently has equipped,

So in conclusion,

Adding the correct MID's to all the missing throwing equipment will stop the incorrect animation playing but it does not fix the issue, This needs further investigation into why the ammo slot is not resetting back to currently equipped ammo.

Omnione avatar Jan 14 '25 16:01 Omnione

I am not able to replicate the /lockstyle off issue:

  • equip aht_urhgan_dart (mid=0)
  • /lockstyle on
  • equip dart (mid=22)
  • /lockstyle off
  • /range
  • Proper range animation will play

I've requested retails caps of throwing skill items (skill 27) in item_weapon that has a item_equipment.mid of 0.

ayechonk avatar Mar 18 '25 18:03 ayechonk

After looking through the various darts in the item_equipment table (or at least items that end in "dart"), it turns out that only the "Dart" has a mid set - everything else, including the one I use in-game all has their mid set to 0. Doing what you described does indeed give a throwing animation that is appropriate for throwing darts, so it seems that the model of all of the other darts needs to be captured?

almuth150 avatar Mar 19 '25 04:03 almuth150

I think so. The cap is almost completed.

ayechonk avatar Mar 19 '25 12:03 ayechonk

Just 11 records left to be updated. once i get the rest i'll put in a PR

---EDIT--- 8 left. 2 are holiday items and 1 is not implemented (throwing coin was never implemented)

ayechonk avatar Mar 19 '25 12:03 ayechonk