Change StatSheetProperties constructor to handle default location

There were several callers that had to check for a `null` location and replace it with the default of
`BOTTOM_LEFT`. That is no longer necessary since the `StatSheetProperties` constructor will do so.

Author: kwvanderlinde

src/main/java/net/rptools/maptool/client/ui/campaignproperties/TokenPropertiesManagementPanel.java

@@ -481,9 +481,7 @@ private void populateStatSheetComboBoxes(String propertyType) {
     if (statSheetProperty == null) {
       id = ssManager.getDefaultStatSheetId();
       tokenTypeStatSheetMap.put(
-          propertyType,
-          new StatSheetProperties(
-              ssManager.getDefaultStatSheetId(), StatSheetLocation.BOTTOM_LEFT));
+          propertyType, new StatSheetProperties(ssManager.getDefaultStatSheetId(), null));
     } else {
       id = statSheetProperty.id();
     }

src/main/java/net/rptools/maptool/client/ui/token/dialog/create/NewTokenDialog.java

@@ -170,9 +170,6 @@ public boolean commit() {
     if (statSheet == null || (statSheet.name() == null && statSheet.namespace() == null)) {
       token.useDefaultStatSheet();
     } else {
-      if (location == null) {
-        location = StatSheetLocation.BOTTOM_LEFT;
-      }
       token.setStatSheet(new StatSheetProperties(ssManager.getId(statSheet), location));
     }
 

src/main/java/net/rptools/maptool/client/ui/token/dialog/edit/EditTokenDialog.java

@@ -910,9 +910,6 @@ public boolean commit() {
     } else {
       var ssManager = new StatSheetManager();
       var location = (StatSheetLocation) getStatSheetLocationCombo().getSelectedItem();
-      if (location == null) {
-        location = StatSheetLocation.BOTTOM_LEFT;
-      }
       token.setStatSheet(new StatSheetProperties(ssManager.getId(ss), location));
     }
 

src/main/java/net/rptools/maptool/model/CampaignProperties.java

@@ -50,7 +50,6 @@
 import net.rptools.maptool.client.ui.token.YieldTokenOverlay;
 import net.rptools.maptool.language.I18N;
 import net.rptools.maptool.model.drawing.DrawableColorPaint;
-import net.rptools.maptool.model.sheet.stats.StatSheetLocation;
 import net.rptools.maptool.model.sheet.stats.StatSheetManager;
 import net.rptools.maptool.model.sheet.stats.StatSheetProperties;
 import net.rptools.maptool.server.proto.CampaignPropertiesDto;
@@ -212,9 +211,7 @@ public Map<String, List<TokenProperty>> getTokenTypeMap() {
    */
   public StatSheetProperties getTokenTypeDefaultStatSheet(String propertyType) {
     return tokenTypeStatSheetMap.getOrDefault(
-        propertyType,
-        new StatSheetProperties(
-            StatSheetManager.LEGACY_STATSHEET_ID, StatSheetLocation.BOTTOM_LEFT));
+        propertyType, new StatSheetProperties(StatSheetManager.LEGACY_STATSHEET_ID, null));
   }
 
   /**

src/main/java/net/rptools/maptool/model/sheet/stats/StatSheetProperties.java

@@ -15,6 +15,7 @@
 package net.rptools.maptool.model.sheet.stats;
 
 import java.util.Objects;
+import javax.annotation.Nullable;
 import net.rptools.maptool.server.proto.StatSheetPropertiesDto;
 
 /**
@@ -33,11 +34,12 @@ public final class StatSheetProperties {
    * Creates a new instance of the class.
    *
    * @param id The id of the stat sheet.
-   * @param location The location of the stat sheet.
+   * @param location The location of the stat sheet, or {@code null} to use the default of {@link
+   *     StatSheetLocation#BOTTOM_LEFT}.
    */
-  public StatSheetProperties(String id, StatSheetLocation location) {
+  public StatSheetProperties(String id, @Nullable StatSheetLocation location) {
     this.id = id;
-    this.location = location;
+    this.location = Objects.requireNonNullElse(location, StatSheetLocation.BOTTOM_LEFT);
   }
 
   /**