Skip to content

Assorted bugfixes

Ceikry requested to merge assorted-bugfixes into master

What have you done in this MR?

  • Corrected a confidential issue involving random events (Closes #878)
  • Add lumbridge tree patch to the farming handler (Closes #904 (closed))
  • Rewrote seedling handling in new systems (Closes #919 (closed))
  • Fixed the incorrect definition for telegrab's spell ID (only caused issues after the decoder was fixed) (Closes #921 (closed))
  • Fixed an issue that prevented leaving a clan (Closes #922 (closed))
  • Close interfaces when applicable when a walk packet is received (Closes #925 (closed))
  • Made the penguin state more reliable (Closes #650 (closed))
  • Fixed a somewhat rare concurrency bug with the packet processor
  • Refactored DialogueOption packet to ContinueOption, due to a revelation that it was not particular to dialogue (it has exactly one known non-dialogue use) (Fixes a new, unreported issue with the stats interface)

Are there any tricky things testers should keep an eye out for?

  • No

  • 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.

Edited by Ceikry

Merge request reports