Prevents each arrow from independently proccing when double / triple firing with a bow #85
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Right now, when double / triple attacking with a bow, each arrow the bow fires rolls procs independently. This means that each time the ranged attack timer elapses with a bow, you can proc up to 18 times (6x from weapon, 12x from spell procs).
This is pretty different from double attacking / triple attacking with a two handed weapon, for example, where each combat round you get one opportunity to proc, regardless of the whether or not your double and triple attack chances succeed. The equivalent combat round for a two handed weapon can proc up to 6 times.
Some random guy on Discord said this behavior wasn't intentional / desired (I don't know if this is accurate):
![image](https://private-user-images.githubusercontent.com/730349/390161929-be352d97-c352-48b1-9c6d-f52531ac7b84.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzkyMDQyNTQsIm5iZiI6MTczOTIwMzk1NCwicGF0aCI6Ii83MzAzNDkvMzkwMTYxOTI5LWJlMzUyZDk3LWMzNTItNDhiMS05YzZkLWY1MjUzMWFjN2I4NC5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUwMjEwJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MDIxMFQxNjEyMzRaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT04Y2JkMDkzZjdiODVkNjE3YmJjZGI0YmFjMjk5MmI2OTc4MzQwN2U5YjhlZTVkNDU5ZjY2MGM3ZWQ5Y2Q3ZTNiJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.V5Pi3yGM764MFyqzFoAgANs6Fi5FSMD5jJ9ZIWjOU5E)
This fixes that behavior by skipping proc attempts on the arrows fired by successful double and triple bow shots.
4.0f
here is the default value for the optionalspeed
param ofDoArcheryAttackDmg
.