Release 1.5.5 #760
Merged
Release 1.5.5 #760
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Fix so that token ids stay the same after server start - Add javadocs info for Zone & Campaign constructors - Close #619
Fix token ids being reset on server start
- Fix so that Cut & paste doesn't change token id - Subsequent pastes will result in different token ids - Close #624
Fix cut & paste changes token id
- Fix "Paste" menu not getting updated after performing the Copy/Cut action from popup menu - Close #621
- Change how the measurer tool cursor updates, now based on attach/detach instead on paintOverlay - Fix cursor not updating right away when switching from measurer to pointer - Fix cursor not updating right away when switching to measurer from label, fog or VBL tool - Close #601
Fix "paste" option remaining grayed out after Copy/Cut token
Fix switching from/to the measure tool not updating cursor right away
Fix bug created by dragging token from library to map
- New message: Second argument to "assert": "{second parameter}" must be of type String
- Close #637
- New class FunctionUtil to handle checks and type conversions for macro functions - New internal methods checkNumberParam, getTokenFromParam, paramAsBigDecimal, paramAsBoolean, paramAsInteger, paramAsDouble, paramAsFloat, paramAsJson, paramAsJsonObject and paramAsJsonArray - Change TokenLightFunctions hasLightSource, clearLights, setLight, and getLights to use these new methods - Methods tested: checkNumberParam, getTokenFromParam, and paramAsBigDecimal - First step to solve #613
Translate a few UI elements to french
Fix incorrect message for assert() when the second parameter is number
Consolidate utility methods for macro functions in FunctionUtil
- Add parameter allowString: if set to false, raises exception when parameter is a string that could be converted to number( ex: "0", "5", etc).
- Functions getZoom, setZoom, getViewArea, setViewArea, and getViewCenter now use FunctionUtil - Fix functions getZoom and setViewArea not displaying errors when given too many parameters - Fix javadocs errors - Remove extra delimiter at the end of getViewCenter
Change ZoomFunction macro functions to use FunctionUtil parameter methods.
- Select Map button (top right of screen) is now properly translated - Tested for English and French
Add French translation support for "Select Map" button
- Change wrong number of parameters error messages to always display name of function - Builds on RPTools/parser#3 to work - Close #629
Change wrong number of parameters error messages to display the name of the macro function that was used.
Change parser implementation to 1.5.4
- Function takes a "json" and a "path" argument - Path should be defined according to https://github.com/json-path/JsonPath - Support inline predicates - Errors are printed to chat but not according to our standards - Progress on #612
- Fix so macro changes are now properly updated to the server - Close #713
Fix campaign macro changes not received when connecting to server
- Fix bug where the default metric is the user's preferred one instead of the server's. - Function affected: getDistance, getDistanceToXY, getTokens - Fix issue mentioned in #684
Fix bug introduced by #707 with default metrics
Add options "gm-self", "self", & "all" to broadcast
- Change so getViewArea and getViewCenter return the actual pixels as used in setViewArea, for example. - Close #724 Co-Authored-By: Phergus <[email protected]>
- Add new parameter "players" to execLink. All players specified will have the macro run for them. Default: self. - The parameter "players" can be a string, a list, or json array. Use new delim parameter for list or array (defaults to ","). - execLink also accepts "gm", "gm-self", "self", "all", and "none". - Close #716
- Add new class "JScrollPopupMenu" to handle scrolling - Class is fully functional but not complete. Other methods relying on indexes of the components should be reimplemented before they are used - Maximum of maps displayed at a time: 20 (scrolling will reveal them all) - Close #356
Functions getViewArea and getViewCenter will now return results in unzoomed map pixels.
- keys become variable names - values become the content of the variables - Close #591
- Change: parameter "suffixed" is now replaced by "prefix" and "suffix", which can add a prefix and/or suffix to the names. - Change: spaces in keys are now turned into underscore - Change: characters not alpha-numerical other than "." and "_" are deleted from var names - Change: function now return a JSONArray with the variable names
Add function json.toVars to turn jsonObject into variables
- Fix javadocs errors; it's now possible to generate the full javaDocs - No actual code changed - Warnings still present, but the errors should be fixed - Close #731
Fix javadocs errors
First pass.
* Update ChangeLog for 1.5.5 release First pass. * Fix ChangeLog and update credits.html Fixed typos and missing linkage in ChangeLog. Updated credits.html file. Issue #735
…738) Updated getInfo("server") macro function to pull JSON object from ServerPolicy instead of having duplicated code. Updated ServerPolicy.toJSON() to include isAutoRevealMovement and isUseIndividualFOW.
* jsonify "null", "true" and "false" into null true and false * fix null handling, using the library Null object * jsonify should be static and public * json unitary tests * use a resource fiel for json rather that constructed within the test code * spotless on json tests
Junit tests for Json function (#741)
Removed self dependency in Parser project.
Update parser to 1.5.5
build.gradle updated to pull in dicelib v1.5.5 Completes issue #746.
Due to how the uberJar is packaged, we need to register JAI imageIO for jpeg2000 The same fix was applied to TokenTool for the same issue. Signed-off-by: JamzTheMan <[email protected]>
Fix for jpeg2000 image extraction for PDF's
* Update ChangeLog for 1.5.5 release First pass. * Fix ChangeLog and update credits.html Fixed typos and missing linkage in ChangeLog. Updated credits.html file. Issue #735 * Add missing issue 670 Updated with issue 670. * Update ChangeLog and Credits Updated Change Log for issues 731, 746 & 751. Updated MapTool credits with recent contributor and missing `</td>` tags.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This change is