Updated for VASSAL 3.6.7 and VASL 6.6.5.

Also added 6.6.3.1 to the list of supported versions.

.pylintrc

@@ -54,88 +54,14 @@ confidence=
 # --enable=similarities". If you want to run only the classes checker, but have
 # no Warning level messages displayed, use"--disable=all --enable=classes
 # --disable=W"
-disable=print-statement,
-        parameter-unpacking,
-        unpacking-in-except,
-        old-raise-syntax,
-        backtick,
-        long-suffix,
-        old-ne-operator,
-        old-octal-literal,
-        import-star-module-level,
-        non-ascii-bytes-literal,
-        invalid-unicode-literal,
+disable=
         raw-checker-failed,
         bad-inline-option,
         locally-disabled,
-        locally-enabled,
         file-ignored,
         suppressed-message,
         useless-suppression,
         deprecated-pragma,
-        apply-builtin,
-        basestring-builtin,
-        buffer-builtin,
-        cmp-builtin,
-        coerce-builtin,
-        execfile-builtin,
-        file-builtin,
-        long-builtin,
-        raw_input-builtin,
-        reduce-builtin,
-        standarderror-builtin,
-        unicode-builtin,
-        xrange-builtin,
-        coerce-method,
-        delslice-method,
-        getslice-method,
-        setslice-method,
-        no-absolute-import,
-        old-division,
-        dict-iter-method,
-        dict-view-method,
-        next-method-called,
-        metaclass-assignment,
-        indexing-exception,
-        raising-string,
-        reload-builtin,
-        oct-method,
-        hex-method,
-        nonzero-method,
-        cmp-method,
-        input-builtin,
-        round-builtin,
-        intern-builtin,
-        unichr-builtin,
-        map-builtin-not-iterating,
-        zip-builtin-not-iterating,
-        range-builtin-not-iterating,
-        filter-builtin-not-iterating,
-        using-cmp-argument,
-        eq-without-hash,
-        div-method,
-        idiv-method,
-        rdiv-method,
-        exception-message-attribute,
-        invalid-str-codec,
-        sys-max-int,
-        bad-python3-import,
-        deprecated-string-function,
-        deprecated-str-translate-call,
-        deprecated-itertools-function,
-        deprecated-types-field,
-        next-method-defined,
-        dict-items-not-iterating,
-        dict-keys-not-iterating,
-        dict-values-not-iterating,
-        deprecated-operator-function,
-        deprecated-urllib-function,
-        xreadlines-attribute,
-        deprecated-sys-function,
-        exception-escape,
-        comprehension-escape,
-    bad-whitespace,
-    bad-continuation,
     invalid-name,
     wrong-import-position,
     global-statement,
@@ -144,7 +70,14 @@ disable=print-statement,
     duplicate-code, # can't get it to shut up about @pytest.mark.skipif's :-/
     no-else-return,
     len-as-condition,
-    consider-using-enumerate
+    consider-using-enumerate,
+    import-outside-toplevel,
+    isinstance-second-argument-not-valid-type,
+    consider-using-f-string,
+    consider-using-max-builtin,
+    use-implicit-booleaness-not-comparison,
+    unnecessary-lambda-assignment,
+    duplicate-string-formatting-argument,
 
 # Enable the message, report, category or checker with the given id(s). You can
 # either give multiple identifier separated by comma (,) or put this option
@@ -289,13 +222,6 @@ max-line-length=120
 # Maximum number of lines in a module
 max-module-lines=1000
 
-# List of optional constructs for which whitespace checking is disabled. `dict-
-# separator` is used to allow tabulation in dicts, etc.: {1  : 1,\n222: 2}.
-# `trailing-comma` allows a space between comma and closing bracket: (a, ).
-# `empty-line` allows space-only lines.
-no-space-check=trailing-comma,
-               dict-separator
-
 # Allow the body of a class to be on the same line as the declaration if body
 # contains single statement.
 single-line-class-stmt=no
@@ -511,7 +437,7 @@ max-public-methods=20
 max-returns=10
 
 # Maximum number of statements in function / method body
-max-statements=80
+max-statements=100
 
 # Minimum number of public methods for a class (see R0903).
 min-public-methods=2

Dockerfile

@@ -1,61 +1,55 @@
 # NOTE: Use the run-container.sh script to build and launch this container.
 
-# We do a multi-stage build (requires Docker >= 17.05) to install everything, then copy it all
-# to the final target image.
+# NOTE: Multi-stage builds require Docker >= 17.05.
+FROM rockylinux:8.5 AS base
 
-FROM centos:8 AS base
+# update packages and install requirements
+RUN dnf -y upgrade-minimal && \
+    dnf install -y python39
 
-# update packages
-RUN dnf -y upgrade-minimal
+# NOTE: We don't need the following stuff for the build step, but it's nice to not have to re-install
+# it all every time we change the requirements :-/
 
-# install Python
-RUN dnf install -y python36 && pip3 install --upgrade pip
+# install Java
+ARG JAVA_URL=https://download.oracle.com/java/17/archive/jdk-17.0.2_linux-x64_bin.tar.gz
+RUN curl -s "$JAVA_URL" | tar -xz -C /usr/bin/
 
-# - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
+# install Firefox
+ARG FIREFOX_URL=https://ftp.mozilla.org/pub/firefox/releases/94.0.2/linux-x86_64/en-US/firefox-94.0.2.tar.bz2
+RUN dnf install -y bzip2 xorg-x11-server-Xvfb gtk3 dbus-glib && \
+    curl -s "$FIREFOX_URL" | tar -jx -C /usr/local/ && \
+    ln -s /usr/local/firefox/firefox /usr/bin/firefox && \
+    echo "exclude=firefox" >>/etc/dnf/dnf.conf
+
+# install geckodriver
+ARG GECKODRIVER_URL=https://github.com/mozilla/geckodriver/releases/download/v0.31.0/geckodriver-v0.31.0-linux64.tar.gz
+RUN curl -sL "$GECKODRIVER_URL" | tar -xz -C /usr/bin/
+
+# clean up
+RUN dnf clean all
 
-# NOTE: We install the Python dependencies into a temporary intermediate stage, then copy everything over
-# to the final image, because some modules (e.g. grpcio) need pre-requisites to be installed.
-# This saves us about 300 MB for the final image and, importantly, adding a requirement doesn't cause us
-# to re-install Java and Firefox below, when we start building the final image.
+# - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
 
 FROM base AS build
 
 # set up a virtualenv
-RUN python3 -m venv /opt/venv && pip3 install --upgrade pip
+RUN python3 -m venv /opt/venv
 ENV PATH="/opt/venv/bin:$PATH"
+RUN pip install --upgrade pip
 
 # install the application requirements
-COPY requirements.txt requirements-dev.txt ./
-RUN pip3 install -r requirements.txt
+COPY requirements.txt requirements-dev.txt /tmp/
+RUN pip3 install -r /tmp/requirements.txt
 ARG CONTROL_TESTS_PORT
 RUN if [ -n "$CONTROL_TESTS_PORT" ]; then \
-    dnf install -y gcc-c++ python3-devel && \
-    pip3 install -r requirements-dev.txt \
+    pip3 install -r /tmp/requirements-dev.txt \
 ; fi
 
 # - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
 
 FROM base
 
-# install Java
-RUN url="https://download.java.net/java/GA/jdk15.0.1/51f4f36ad4ef43e39d0dfdbaf6549e32/9/GPL/openjdk-15.0.1_linux-x64_bin.tar.gz" ; \
-    curl -s "$url" | tar -C /usr/bin/ -xz
-
-# install Firefox
-RUN dnf install -y wget bzip2 xorg-x11-server-Xvfb gtk3 dbus-glib && \
-    wget -qO- "https://download.mozilla.org/?product=firefox-latest-ssl&os=linux64&lang=en-US" \
-        | tar -C /usr/local/ -jx && \
-    ln -s /usr/local/firefox/firefox /usr/bin/firefox && \
-    echo "exclude=firefox" >>/etc/dnf/dnf.conf
-
-# install geckodriver
-RUN url=$( curl -s https://api.github.com/repos/mozilla/geckodriver/releases/latest | grep -Poh 'https.*linux64\.tar\.gz(?!\.)' ) && \
-    curl -sL "$url" | tar -C /usr/bin/ -xz
-
-# clean up
-RUN dnf clean all
-
-# install the application requirements
+# copy the virtualenv from the build image
 COPY --from=build /opt/venv /opt/venv
 ENV PATH="/opt/venv/bin:$PATH"
 
@@ -67,12 +61,26 @@ COPY setup.py requirements.txt requirements-dev.txt LICENSE.txt ./
 RUN pip3 install --editable .
 
 # install the config files
+COPY vasl_templates/webapp/config/logging.yaml.example ./vasl_templates/webapp/config/logging.yaml
 COPY docker/config/ ./vasl_templates/webapp/config/
 
 # create a new user
+# NOTE: It would be nice to just specify the UID/GID in the "docker run" command, but VASSAL has problems
+# if there is no user :-/ We could specify these here, but that would bake them into the image.
+# In general, this is not a problem, since the application doesn't need to access files outside the container,
+# but if the user wants to e.g. keep the cached scenario index files outside the container, and they are
+# running with a non-default UID/GID, they will have to manage permissions themselves. Sigh...
 RUN useradd --create-home app
 USER app
 
+# FUDGE! We need this to stop spurious warning messages:
+#   Fork support is only compatible with the epoll1 and poll polling strategies
+# Setting the verbosity to ERROR should suppress these, but doesn't :-/
+#   https://github.com/grpc/grpc/issues/17253
+#   https://github.com/grpc/grpc/blob/master/doc/environment_variables.md
+ENV GRPC_VERBOSITY=NONE
+
+# run the application
 EXPOSE 5010
 COPY docker/run.sh ./
 CMD ./run.sh

MANIFEST.in

@@ -1,3 +1,5 @@
+recursive-include vasl_templates/ui *.*
+
 recursive-include vasl_templates/webapp/config *.*
 recursive-include vasl_templates/webapp/data *.*
 recursive-include vasl_templates/webapp/static *.*

README.md

@@ -1,16 +1,14 @@
 # VASL Templates
 
-<a href="https://github.com/pacman-ghost/vasl-templates/raw/master/vasl_templates/webapp/static/help/images/hill-621.png" target="_blank">
-<img src="https://github.com/pacman-ghost/vasl-templates/raw/master/vasl_templates/webapp/static/help/images/hill-621.small.png" width="200" align="right" hspace="10">
-</a>
+[<img src="vasl_templates/webapp/static/help/images/hill-621.small.png" width="200" align="right" hspace="10">](vasl_templates/webapp/static/help/images/hill-621.png)
 
 *VASL Templates* makes it easy to set up attractive VASL scenarios, with loads of useful information embedded to assist with game play.
 
 Simply enter the scenario information into the UI, and the program will generate HTML snippets that you can transfer into VASL labels in your scenario.
 
-<img src="https://github.com/pacman-ghost/vasl-templates/raw/master/vasl_templates/webapp/static/help/images/ob_setup.png" width="200">
+[<img src="vasl_templates/webapp/static/help/images/ob_setup.png" width="200">](vasl_templates/webapp/static/help/images/ob_setup.png)
 
-You can find more examples of the program in action [here](https://github.com/pacman-ghost/vasl-templates/tree/master/examples/).
+You can find more examples of the program in action [here](examples/).
 
 ### Documentation
 

conftest.py

@@ -4,8 +4,8 @@ import os
 import threading
 import json
 import re
-import logging
 import tempfile
+import logging
 import urllib.request
 from urllib.error import URLError
 import pytest
@@ -18,6 +18,8 @@ from vasl_templates.webapp.tests.control_tests import ControlTests
 
 FLASK_WEBAPP_PORT = 5011
 
+_pytest_options = None
+
 # ---------------------------------------------------------------------
 
 def pytest_addoption( parser ):
@@ -40,7 +42,7 @@ def pytest_addoption( parser ):
         help="Run the tests headless."
     )
     parser.addoption(
-        "--window-size", action="store", dest="window_size", default="1000x700",
+        "--window-size", action="store", dest="window_size", default="1020x700",
         help="Browser window size."
     )
 
@@ -59,6 +61,22 @@ def pytest_addoption( parser ):
         help="Use the clipboard to get snippets."
     )
 
+# - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
+
+def pytest_configure( config ):
+    """Called after command-line options have been parsed."""
+    global _pytest_options
+    _pytest_options = config.option
+    import vasl_templates.webapp.tests
+    vasl_templates.webapp.tests.pytest_options = config.option
+
+# - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
+
+@pytest.fixture( scope="session" )
+def monkeypatch():
+    """Override the default monkeypatch fixture."""
+    assert False, "Don't use monkeypatch!" # it won't work when testing against a remote server
+
 # ---------------------------------------------------------------------
 
 _webapp = None
@@ -85,11 +103,11 @@ def _make_webapp():
     """Create the global webapp fixture."""
 
     # initialize
-    webapp_url = pytest.config.option.webapp_url #pylint: disable=no-member
+    webapp_url = _pytest_options.webapp_url
     if webapp_url and not webapp_url.startswith( "http://" ):
         webapp_url = "http://" + webapp_url
     app.base_url = webapp_url if webapp_url else "http://localhost:{}".format( FLASK_WEBAPP_PORT )
-    logging.disable( logging.CRITICAL )
+    _disable_console_logging()
 
     # initialize
     # WTF?! https://github.com/pallets/flask/issues/824
@@ -103,11 +121,11 @@ def _make_webapp():
             # stop the browser from checking for a dirty scenario when leaving the page
             kwargs["disable_close_window_check"] = 1
             # check if the tests are being run headless
-            if pytest.config.option.headless: #pylint: disable=no-member
+            if _pytest_options.headless:
                 # yup - there is no clipboard support :-/
-                pytest.config.option.use_clipboard = False #pylint: disable=no-member
+                _pytest_options.use_clipboard = False
             # check if we should disable using the clipboard for snippets
-            if not pytest.config.option.use_clipboard: #pylint: disable=no-member
+            if not _pytest_options.use_clipboard:
                 # NOTE: It's not a bad idea to bypass the clipboard, even when running in a browser,
                 # to avoid problems if something else uses the clipboard while the tests are running.
                 kwargs["store_clipboard"] = 1
@@ -125,7 +143,7 @@ def _make_webapp():
         # This means that the webapp doesn't get a chance to shutdown properly (in particular,
         # clean up the gRPC service), but since we send an EndTests message at the of each test,
         # the remote server gets a chance to clean up then. It's not perfect (e.g. if the tests fail
-        # or otherwise finish eearly before they get a chance to send the EndTests message), but
+        # or otherwise finish early before they get a chance to send the EndTests message), but
         # we can live with it.
         thread = threading.Thread(
             target = lambda: app.run( host="0.0.0.0", port=FLASK_WEBAPP_PORT, use_reloader=False ),
@@ -136,8 +154,9 @@ def _make_webapp():
         def is_ready():
             """Try to connect to the webapp server."""
             try:
-                resp = urllib.request.urlopen( app.url_for("ping") ).read()
-                assert resp.startswith( b"pong: " )
+                url = app.url_for( "ping" )
+                with urllib.request.urlopen( url ) as resp:
+                    assert resp.read().startswith( b"pong: " )
                 return True
             except URLError:
                 return False
@@ -147,20 +166,31 @@ def _make_webapp():
 
     # set up control of the remote webapp server
     try:
-        resp = json.load(
-            urllib.request.urlopen( app.url_for( "get_control_tests" ) )
-        )
+        url = app.url_for( "get_control_tests" )
+        with urllib.request.urlopen( url ) as resp:
+            resp_data = json.load( resp )
     except urllib.error.HTTPError as ex:
         if ex.code == 404:
-            raise RuntimeError( "Can't get the test control port - has remote test control been enabled?" )
+            raise RuntimeError( "Can't get the test control port - has remote test control been enabled?" ) from ex
         raise
-    port_no = resp.get( "port" )
+    port_no = resp_data.get( "port" )
     if not port_no:
         raise RuntimeError( "The webapp server is not running the test control service." )
     mo = re.search( r"^http://(.+):\d+$", app.base_url )
     addr = "{}:{}".format( mo.group(1), port_no )
     app.control_tests = ControlTests( addr )
 
+    # NOTE: We set the back-end webdriver to be the of the same type (Firefox or Chrome) as the browser
+    # being used to drive the tests, which, strictly speaking, doesn't make sense, since the two things
+    # don't have anything to do with each other. However, this is a convenient way to switch the backend
+    # webdriver's and exercise both of them. The webdriver binary must be on the path, but if it's not,
+    # we won't have even got this far, since it needs to be there to drive the browser.
+    # NOTE: This will have no effect if we're talking to a remote server, but we can live with that.
+    if _pytest_options.webdriver == "firefox":
+        app.config[ "WEBDRIVER_PATH" ] = "geckodriver"
+    elif _pytest_options.webdriver == "chrome":
+        app.config[ "WEBDRIVER_PATH" ] = "chromedriver"
+
     return app
 
 # - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
@@ -168,7 +198,7 @@ def _make_webapp():
 @pytest.fixture( scope="session" )
 def test_client():
     """Return a test client that can be used to connect to the webapp."""
-    logging.disable( logging.CRITICAL )
+    _disable_console_logging()
     return app.test_client()
 
 # - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
@@ -186,31 +216,22 @@ def webdriver( request ):
     # initialize
     driver = request.config.getoption( "--webdriver" )
     from selenium import webdriver as wb
+    log_fname = os.path.join( tempfile.gettempdir(), "webdriver-pytest.log" )
     if driver == "firefox":
         options = wb.FirefoxOptions()
-        options.set_headless( headless = pytest.config.option.headless ) #pylint: disable=no-member
-        driver = wb.Firefox(
-            firefox_options = options,
-            log_path = os.path.join( tempfile.gettempdir(), "geckodriver.log" )
-        )
+        options.headless = _pytest_options.headless
+        service = wb.firefox.service.Service( log_path=log_fname )
+        driver = wb.Firefox( options=options, service=service )
     elif driver == "chrome":
         options = wb.ChromeOptions()
-        options.set_headless( headless = pytest.config.option.headless ) #pylint: disable=no-member
-        driver = wb.Chrome( chrome_options=options )
-    elif driver == "ie":
-        # NOTE: IE11 requires a registry key to be set:
-        #   https://github.com/SeleniumHQ/selenium/wiki/InternetExplorerDriver#required-configuration
-        options = wb.IeOptions()
-        if pytest.config.option.headless: #pylint: disable=no-member
-            raise RuntimeError( "IE WebDriver cannot be run headless." )
-        options.IntroduceInstabilityByIgnoringProtectedModeSettings = True
-        options.EnsureCleanSession = True
-        driver = wb.Ie( ie_options=options )
+        options.headless = _pytest_options.headless
+        options.add_argument( "--disable-gpu" )
+        driver = wb.Chrome( options=options )
     else:
         raise RuntimeError( "Unknown webdriver: {}".format( driver ) )
 
     # set the browser size
-    words = pytest.config.option.window_size.split( "x" ) #pylint: disable=no-member
+    words = _pytest_options.window_size.split( "x" )
     driver.set_window_size( int(words[0]), int(words[1]) )
 
     # return the webdriver to the caller
@@ -218,3 +239,19 @@ def webdriver( request ):
         yield driver
     finally:
         driver.quit()
+
+# ---------------------------------------------------------------------
+
+def _disable_console_logging():
+    """Disable Python logging to the console.
+
+    We do this when running tests because:
+    (1) pytest's output is voluminous enough without including our stuff in there as well (and it tends to be
+        not that helpful, anyway)
+    (2) pytest captures all output and shows it when the test ends i.e. we don't get to see messages in real-time.
+    """
+    for logger in utils.get_all_loggers():
+        # NOTE: FileHandler derives from StreamHandler, and we want to keep those, so we can't use isinstance().
+        handlers = [ h for h in logger.handlers if type( h ) is logging.StreamHandler ] #pylint: disable=unidiomatic-typecheck
+        for h in handlers:
+            logger.removeHandler( h )

docker/config/.gitignore

@@ -0,0 +1 @@
+logging.yaml

docker/config/logging.yaml

@@ -1,43 +0,0 @@
-version: 1
-
-formatters:
-    standard:
-        format: "%(asctime)s.%(msecs)03d | %(message)s"
-        datefmt: "%H:%M:%S"
-
-handlers:
-    console:
-        class: "logging.StreamHandler"
-        formatter: "standard"
-        stream: "ext://sys.stdout"
-    file:
-        class: "logging.FileHandler"
-        formatter: "standard"
-        filename: "/tmp/vasl-templates.log"
-        mode: "w"
-
-loggers:
-    werkzeug:
-        level: "WARNING"
-        handlers: [ "file" ]
-    vasl_mod:
-        level: "WARNING"
-        handlers: [ "file" ]
-    update_vsav:
-        level: "WARNING"
-        handlers: [ "file" ]
-    analyze_vsav:
-        level: "WARNING"
-        handlers: [ "file" ]
-    analyze_vlog:
-        level: "WARNING"
-        handlers: [ "file" ]
-    webdriver:
-        level: "WARNING"
-        handlers: [ "file" ]
-    downloads:
-        level: "WARNING"
-        handlers: [ "file" ]
-    control_tests:
-        level: "INFO"
-        handlers: [ "console", "file" ]

docker/config/site.cfg

@@ -2,5 +2,5 @@
 
 IS_CONTAINER = 1
 
-JAVA_PATH = /usr/bin/jdk-15.0.1/bin/java
+JAVA_PATH = /usr/bin/jdk-17.0.2/bin/java
 WEBDRIVER_PATH = /usr/bin/geckodriver

examples/Hill 621 (Scenario E).json

@@ -1,16 +1,12 @@
 {
-    "SCENARIO_NAME": "Hill 621",
-    "SCENARIO_ID": "ASL E",
-    "SCENARIO_LOCATION": "Near Minsk, Russia",
+    "COMPASS": "right",
     "SCENARIO_DATE": "1944-07-01",
     "SCENARIO_WIDTH": "",
-    "PLAYER_1_DESCRIPTION": "Retreating elements of 170th Infantry Division",
-    "PLAYER_2_DESCRIPTION": "Elements of 5th Guards Army",
     "ASA_ID": "56512",
     "ROAR_ID": "129",
     "PLAYERS_WIDTH": "",
-    "VICTORY_CONDITIONS_WIDTH": "240px",
-    "SSR_WIDTH": "500px",
+    "VICTORY_CONDITIONS_WIDTH": "",
+    "SSR_WIDTH": "400px",
     "OB_VEHICLES_WIDTH_1": "",
     "OB_VEHICLES_MA_NOTES_WIDTH_1": "300px",
     "OB_ORDNANCE_WIDTH_1": "",
@@ -19,7 +15,12 @@
     "OB_VEHICLES_MA_NOTES_WIDTH_2": "300px",
     "OB_ORDNANCE_WIDTH_2": "",
     "OB_ORDNANCE_MA_NOTES_WIDTH_2": "300px",
-    "VICTORY_CONDITIONS": "The Russians win at Game End if they Control ≥ five Level 3 hill hexes on Board 2.",
+    "SCENARIO_NAME": "Hill 621",
+    "SCENARIO_ID": "ASL E",
+    "SCENARIO_LOCATION": "Near Minsk, Russia",
+    "PLAYER_1_DESCRIPTION": "Retreating elements of 170th Infantry Division",
+    "PLAYER_2_DESCRIPTION": "Elements of 5th Guards Army",
+    "VICTORY_CONDITIONS": "The Russians win at Game End if they Control<br>≥ five Level 3 hill hexes on Board 2.",
     "SCENARIO_THEATER": "ETO",
     "PLAYER_1": "russian",
     "PLAYER_1_ELR": "4",
@@ -27,6 +28,15 @@
     "PLAYER_2": "german",
     "PLAYER_2_ELR": "3",
     "PLAYER_2_SAN": "4",
+    "TURN_TRACK": {
+        "NTURNS": "10",
+        "WIDTH": "",
+        "VERTICAL": false,
+        "SHADING": "",
+        "REINFORCEMENTS_1": "2,5",
+        "REINFORCEMENTS_2": "1,2,4,5,8",
+        "SWAP_PLAYERS": false
+    },
     "SSR": [
         "EC are Moderate, with no wind at start.",
         "After \"At Start\" placement, each German infantry unit must take a TC. The only possible consequence of failure is that the unit must begin the scenario broken. Those units which break during this pre-game TC are not subject to DM in the initial German RPh.",
@@ -106,7 +116,7 @@
     ],
     "SCENARIO_NOTES": [
         {
-            "caption": "Download the scenario card from <a href=\"http://www.multimanpublishing.com/Support/ASLASLSK/ASLOfficialDownloads/tabid/109/Default.aspx\">Multi-Man Publishing</a> (ASL Classic pack).",
+            "caption": "Download the scenario card from <a href=\"https://mmpgamers.com/asl-downloads-ezp-3#scenarios\">Multi-Man Publishing</a> (ASL Classic pack).",
             "width": "300px",
             "id": 1
         }
@@ -173,7 +183,7 @@
             "id": 2
         }
     ],
-    "_app_version": "v1.3",
-    "_last_update_time": "2020-09-27T04:13:08.719Z",
+    "_app_version": "v1.10",
+    "_last_update_time": "2022-09-12T02:46:18.035Z",
     "_creation_time": "2020-09-27T03:46:56.089Z"
 }

examples/Hube's Pocket (Scenario G).json

@@ -1,15 +1,11 @@
 {
-    "SCENARIO_NAME": "Hube's Pocket",
-    "SCENARIO_ID": "ASL G",
-    "SCENARIO_LOCATION": "Near Buchach, Southern Russia",
+    "COMPASS": "down",
     "SCENARIO_DATE": "1944-04-06",
     "SCENARIO_WIDTH": "",
-    "PLAYER_1_DESCRIPTION": "Advance elements of 5th Tank Corps",
-    "PLAYER_2_DESCRIPTION": "10th SS Panzer Division \"Frundsberg\" and the First Panzer Army ",
     "ASA_ID": "56514",
     "ROAR_ID": "131",
     "PLAYERS_WIDTH": "",
-    "VICTORY_CONDITIONS_WIDTH": "320px",
+    "VICTORY_CONDITIONS_WIDTH": "",
     "SSR_WIDTH": "330px",
     "OB_VEHICLES_WIDTH_1": "",
     "OB_VEHICLES_MA_NOTES_WIDTH_1": "300px",
@@ -19,7 +15,12 @@
     "OB_VEHICLES_MA_NOTES_WIDTH_2": "300px",
     "OB_ORDNANCE_WIDTH_2": "",
     "OB_ORDNANCE_MA_NOTES_WIDTH_2": "300px",
-    "VICTORY_CONDITIONS": "The Germans win immediately by exiting &ge; 10 vehicles <br>\noff the west edge in either one or two Convoys (see SSR 4).",
+    "SCENARIO_NAME": "Hube's Pocket",
+    "SCENARIO_ID": "ASL G",
+    "SCENARIO_LOCATION": "Near Buchach, Southern Russia",
+    "PLAYER_1_DESCRIPTION": "Advance elements of 5th Tank Corps",
+    "PLAYER_2_DESCRIPTION": "10th SS Panzer Division \"Frundsberg\" and the First Panzer Army",
+    "VICTORY_CONDITIONS": "The Germans win immediately by exiting ≥ 10 vehicles <br>\noff the west edge in either one or two Convoys (see SSR 4).",
     "SCENARIO_THEATER": "ETO",
     "PLAYER_1": "german",
     "PLAYER_1_ELR": "4",
@@ -27,6 +28,15 @@
     "PLAYER_2": "russian",
     "PLAYER_2_ELR": "3",
     "PLAYER_2_SAN": "2",
+    "TURN_TRACK": {
+        "NTURNS": "14",
+        "WIDTH": "5",
+        "VERTICAL": false,
+        "SHADING": "",
+        "REINFORCEMENTS_1": "1,5",
+        "REINFORCEMENTS_2": "1",
+        "SWAP_PLAYERS": true
+    },
     "SSR": [
         "The SPW 251/sMG inherent HS is a 3-4-8.",
         "German inherent crews have a morale of 9.",
@@ -84,7 +94,7 @@
     ],
     "SCENARIO_NOTES": [
         {
-            "caption": "Download the scenario card from <a href=\"http://www.multimanpublishing.com/Support/ASLASLSK/ASLOfficialDownloads/tabid/109/Default.aspx\">Multi-Man Publishing</a> (ASL Classic pack).",
+            "caption": "Download the scenario card from <a href=\"https://mmpgamers.com/asl-downloads-ezp-3#scenarios\">Multi-Man Publishing</a> (ASL Classic pack).",
             "width": "",
             "id": 1
         }
@@ -110,7 +120,7 @@
     ],
     "OB_NOTES_1": [],
     "OB_NOTES_2": [],
-    "_app_version": "v1.3",
-    "_last_update_time": "2020-09-27T04:33:18.643Z",
+    "_app_version": "v1.10",
+    "_last_update_time": "2022-09-12T02:22:47.511Z",
     "_creation_time": "2020-09-27T04:11:07.200Z"
 }

examples/README.md

@@ -4,4 +4,4 @@ This directory contains examples of *VASL Templates* in action, with the `.json`
 
 The online versions contain images that will be loaded from the internet, which looks much better, but there will be a short delay when you open the scenario in VASSAL as the images are downloaded.
 
-These scenarios were taken from Multi-Man Publishing's [*ASL Classic* scenario pack](http://www.multimanpublishing.com/Support/ASLASLSK/ASLOfficialDownloads/tabid/109/Default.aspx).
+These scenarios were taken from Multi-Man Publishing's [*ASL Classic* scenario pack](https://mmpgamers.com/asl-downloads-ezp-3#scenarios).

examples/The Streets Of Stalingrad (Scenario C).json

@@ -1,15 +1,11 @@
 {
-    "SCENARIO_NAME": "The Streets Of Stalingrad",
-    "SCENARIO_ID": "ASL C",
-    "SCENARIO_LOCATION": "Stalingrad, Russia",
+    "COMPASS": "up",
     "SCENARIO_DATE": "1942-10-06",
     "SCENARIO_WIDTH": "",
-    "PLAYER_1_DESCRIPTION": "308th Rifle Division / 295th Rifle Division / 2nd Battalion, 37th Guards Division",
-    "PLAYER_2_DESCRIPTION": "389th Infantry Division",
     "ASA_ID": "56510",
     "ROAR_ID": "127",
     "PLAYERS_WIDTH": "",
-    "VICTORY_CONDITIONS_WIDTH": "400px",
+    "VICTORY_CONDITIONS_WIDTH": "450px",
     "SSR_WIDTH": "500px",
     "OB_VEHICLES_WIDTH_1": "",
     "OB_VEHICLES_MA_NOTES_WIDTH_1": "300px",
@@ -19,7 +15,12 @@
     "OB_VEHICLES_MA_NOTES_WIDTH_2": "300px",
     "OB_ORDNANCE_WIDTH_2": "",
     "OB_ORDNANCE_MA_NOTES_WIDTH_2": "300px",
-    "VICTORY_CONDITIONS": "Victory is based upon satisfying the Victory Conditions of Scenarios A and B:\n<ul style=\"margin:0 0 10px 10px;\">\n<li> If each side fulfills one Victory Condition, the game is a draw.\n<li> If a player fulfills one Victory Condition and draws the other, he wnis.\n<li> A decisive victory is achieved when a player fulfills both Victory Conditions.\n</ul>\n\n<p> <b>Scenario A:</b> The Russians win at Game End if they Control &ge; 2 more buildings initially occupied by the Germans than they lose of their own initially-held stone buildings to German Control, and/or have a favorable 3:1 ratio of unbroken squad-equivalents.\n<p> <b>Scenario B:</b> At Game End, the player with undisputed control of at least 6 hexes of building X3 wins. A hex containing a Melee is controlled by neither player. If only one player has an unbroken unit in the building at Game End, that player is the winner. Any other result is a draw.\n</ul>",
+    "SCENARIO_NAME": "The Streets Of Stalingrad",
+    "SCENARIO_ID": "ASL C",
+    "SCENARIO_LOCATION": "Stalingrad, Russia",
+    "PLAYER_1_DESCRIPTION": "308th Rifle Division / 295th Rifle Division / 2nd Battalion, 37th Guards Division",
+    "PLAYER_2_DESCRIPTION": "389th Infantry Division",
+    "VICTORY_CONDITIONS": "Victory is based upon satisfying the Victory Conditions of Scenarios A and B:\n<ul style=\"margin:0 0 10px 10px;\">\n<li> If each side fulfills one Victory Condition, the game is a draw.\n</li><li> If a player fulfills one Victory Condition and draws the other, he wnis.\n</li><li> A decisive victory is achieved when a player fulfills both Victory Conditions.\n</li></ul>\n\n<p> <b>Scenario A:</b> The Russians win at Game End if they Control ≥ 2 more buildings initially occupied by the Germans than they lose of their own initially-held stone buildings to German Control, and/or have a favorable 3:1 ratio of unbroken squad-equivalents.\n</p><p> <b>Scenario B:</b> At Game End, the player with undisputed control of at least 6 hexes of building X3 wins. A hex containing a Melee is controlled by neither player. If only one player has an unbroken unit in the building at Game End, that player is the winner. Any other result is a draw.\n</p>",
     "SCENARIO_THEATER": "ETO",
     "PLAYER_1": "russian",
     "PLAYER_1_ELR": "3",
@@ -27,6 +28,15 @@
     "PLAYER_2": "german",
     "PLAYER_2_ELR": "4",
     "PLAYER_2_SAN": "6",