Merged requested to merge ByteEngineer/2009scape:sap-collect-listener into master
What have you done in this MR?
- Updated SapCollect plugin to listener
- Corrected interaction so that knife has to be used on tree instead of a bucket (https://runescape.wiki/w/Bucket_of_sap?oldid=2050551)
Are there any tricky things testers should keep an eye out for?
Yes, as follows:
I have tested these changes thoroughly.
This requires extra testing due to changes to architecture or other similarly risky changes.
I used the Thanos Tool for any JSON edits where possible, and have attached screenshots of any changes.
I acknowledge that this contribution will be released under the AGPL license.
Testing: With an empty bucket in inventory, using a knife on a valid tree (e.g. Evergreen tree) will give a bucket of sap. Also, emptying the bucket of sap should work.
Related to #230