Full Changelog: https://gist.github.com/Jonathing/c3ad28b2a048ac839a7baba5417ee870
The key features are:
- ES6 language support
- Thoroughly updated ASMAPI, with full documentation
- Bug fixes (some optional for backwards-compatibility)
- Partial internal code cleanup
- Request CoreMods to not apply fix for ASMAPI.findFirstInstructionBefore by default
Related dependency bumps:
- Updated ASM to 9.7.1
- Updated Nashorn to 15.4
- Updated Log4J to 2.19.0
* add supplier to FlowerBlock so it works with custom MobEffects
* address comments by @Matyrobbrt
* address comments by @ChampionAsh5357
* fix missing exc entry
* remove null check and rename constructor parameter
* Backport grindstone event from 1.19
* Fix several bugs in the original PR related to canceling the event
Fix canceling the event not clearing the result item, causing it to dupe whatever the result with the previous item was
Fix canceling the event running vanilla behavior if vanilla behavior ran at all since the menu was opened and no other listener changed the XP
* Add comments to the test mod explaining the expected behavior
* Clarify docs on behavior of the XP
Makes it clear XP is ignored in on place if the output is empty or the event is canceled
* Fix broadcast changes not being called if the event overrides grindstone behavior
Not sure its strictly needed, but every vanilla branch in the function calls it and its a trivial change.
* Add enabled flag to the test mod
* Improve javadocs
(cherry picked from commit e12c849cc0)
* Formatting fixes
* Apply javadoc suggestions from review
Co-authored-by: matyrobbrt <65940752+Matyrobbrt@users.noreply.github.com>
* Correct capitalization in event name
* Javadoc improvements
Remove some redundancy in certain javadoc comments
Add @see to the header
Inline single line javadocs
* Add some funny curly brackets next to returns
Seems the kids these days have invented a new way to write return statements in javadocs
* Remove extra dots and capital letters
The return tag apparently puts more dots at the end, and having two dots on each sentence is not in the budget. Have also reached the capital letter quota
* Apply suggestions from code review
Co-authored-by: matyrobbrt <65940752+Matyrobbrt@users.noreply.github.com>
Co-authored-by: ferriarnus <61201275+ferriarnus@users.noreply.github.com>
Co-authored-by: matyrobbrt <65940752+Matyrobbrt@users.noreply.github.com>
Jonathing