- 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 token having duplicated ids if a cut was followed by dragging token to map then a paste (induced by #625)
- Fix dragging token from library to map not ungraying the "Paste" option (fully completes #630)

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

- Functions can modify a nested json array
- Path should be defined according to https://github.com/json-path/JsonPath
- Closes #612
- Details:

1. json.path.add(json, path, value) adds an element to a jsonArray at the path
2. json.path.set(json, path, value) sets an element in a jsonArray or jsonObject at the path
3. json.path.put(json, path, key, value) adds or sets an element in a jsonObject at the path
4. json.path.delete(json, path) deletes an element in a jsonArray or jsonObject at the path

- Replace checkNumberOfParameters by FunctionUtil.checkNumberParam
- Replace getTokenFromParam by FunctionUtil.getTokenFromParam
- Fix #613

Merge release changes back to develop

- Add ZOrder sorting of the tokens on the server at the end of bringTokensToFront / sendTokensToBack

Consolidate methods checkNumberOfParameters and getTokenFromParam

Fix bring to front/send to back need to be done twice

Add json.path.read, and json.path.add/set/put/delete (based on jayway)

compile is deprecated and replaced with implementation

 * Make partialPaths thread safe and Close #328

Signed-off-by: JamzTheMan <[email protected]>

Fix for ConcurrentModificationException

 * Closes #665 and Closes #663
 * Added abeille-maptool-forms.jfpr project file to buil-resources

Add new Server Option for GM to Reveal FoW for Unowned Tokens.  Add option plus hosting status to output of getInfo("server").

- Add playStream(uri, cycleCount, volume) to play audio from url or local file.
- cycleCount: number of times to play the resource. -1 for infinite loop. Default: 1.
- volume: volume value from 0-1. Default: 1.
- Add stopStream(uri, remove) to stop a stream. If remove is set to 1, unload the stream from memory. Default: 1. If no uri specified, stop all sounds.
- Add editStream(uri, cycleCount, volume) to change the cycleCount or volume of stream.
- Add AppPreference playStreams to enable/disable playing streams
- Ex:

[playStream("https://www.fesliyanstudios.com/musicfiles/2019-05-01_-_Undercover_Spy_Agent_-_David_Fesliyan.mp3", -1, 0.8)]

- Fix access HashMap to be thread safe (now accessed solely from the JavaFX app single thread)
- Fix so that disabling streams in settings also stops all streams

Add playStream, stopStream, and editStream macro functions for playing/streaming audio.

- Move streaming methods to class MediaPlayerAdapter, as discussed in #668
- Add new parameters: startTime and stopTime to playStream & editStream
- playStream with cycleCount 0 preloads the stream but does not play it
- Add function getStreamProperties as suggested in #667
- Add support for volume slider through setGlobalVolume, but the slider needs to be created

Add features to playStream functions

- Fix getStreamProperties to return status "STOPPED" after song finished playing
- Change parameters startTime and stopTime to take seconds instead of ms

Fix streams that had finished returning "PLAYING" in getStreamProperties() , change input in ms for start/stop time to secs.

- Icons provided by @JamzTheMan

Co-Authored-By: Jamz <[email protected]>

Adds Volume/Mute controls.

- Add optional fadeout (in seconds) before closing a stream
- 0: no fadeout (default)
- Stream will stop once the fadeout period is over
- Suggested by @dorpond in #615

Add optional fadeout to stopStream().

…nt (#611)

dispatcher.

- Add back,forward,next,pause,play,previous,record,record_on,repeat,shuffle,shuffle_on, and stop icons
- Move mute & volume icons to correct directory
- Icons will be used for #615
- Icons provided by @JamzTheMan

Co-Authored-By: Jamz <[email protected]>

Add sound icons to the github directory

- Add conversion of string to uri, replacing space into %20
- Add "FILE/:" at the start of uri if it is missing
- Fix issues raised in #667

- Change to using jitpack for clientserver
- Fix #679

Change to using jitpack for clientserver.  Should fix PR's failing in Travis test.

Now supports Windows file paths (and should support Mac & Linux as well) through URLDecoder class.

…682)

- Add special case so that "*" isn't modified
- Fix regression created by #678
- Fix #681

- Add new option "propertyType" to conditions in getTokens()
- Value can be a single type as string, or an array of types
- Simplify getTokens() code by removing the second loop. Now include/exclude is a filter property ("match")
- Improve performance of getTokens() by trimming the list continuously instead of constantly filtering the list of all tokens
- Close #676

…ernally (#672)

* jsonify "null", "true" and "false" into null true and false

* fix null handling, using the library Null object

* jsonify should be static and public

- Change so overlap is computed explicitely instead of using A*
- Much better performance
- Solves #683 for "area" but not for "range"

- Request only images in view from ImageManager
- Trigger repaint() from paint-affecting properties only if property
actually changes
- Calculate truncated caption heuristically
- Improve spacing
- Include caption in mouse capture area
- Antialias text
- Use interpolation when rendering scaled images

- Fix manhattan distance while moving tokens & in distance functions
- Fix getDistance, getDistanceToXY, getTokens ignoring metric parameter for isometric maps
- Close #688

- Fix issue raised in #612

Fix #700

- Improve speed dramatically while using getTokens() with a distance option
- Previous walker-based approach kept as it could be extended to take VBL & terrain into account
- Distance returned tested to be the same as previous code
- Close #683

Resolve #696 Implement dragTheshold in DefaultTool

Fix NullPointer exception when closing maptool

Change getDistance to use explicit closed-form

- Fix token "jumping" to a different spot once Snap-to-Grid is disabled
- Fix getDistance returning incorrect distance with NO_GRID metric
- Close #699

- Change so that getDistanceToXY and getDistance now calculate the distance from the center of the token when the token is large
- Add new "pixel" parameter to getDistanceToXY. It indicates if the x,y are for a cell or pixel (pixel: false by default)
- pixel false: getDistanceToXY gets the distance from the center of the token to the center of the specified cell
- Close #684

Fix to getDistanceToXY and getDistance with NO_GRID metric.  Add new pixels parameter to getDistanceToXY.

Fix two issues from moving token when "Snap Token While Dragging" is off

- Fix incorrect boolean check
- Fix missing CellPoint to ZonePoint conversion

- Change system sounds to be affected by volume slider & mute button
- Change system sounds to use JavaFX
- Close #709

Change volume slider to affect system sounds

- 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

Close #718

Fix bug introduced by #707 with default metrics