Return null from documentCollection() to avoid rendering all Drawings…

… in scene.
mcglincy · Jul 6, 2024 · baee67e · baee67e
1 parent eadb134
commit baee67e
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 6 deletions.
diff --git a/modules/dungeondraw-bundle.min.js b/modules/dungeondraw-bundle.min.js
diff --git a/modules/dungeondraw-bundle.min.js.map b/modules/dungeondraw-bundle.min.js.map
diff --git a/src/dungeonlayer.js b/src/dungeonlayer.js
@@ -109,11 +109,22 @@ function createDataOffsetPoints(createData) {
 export class DungeonLayer extends PlaceablesLayer {
   static LAYER_NAME = "dungeon";
 
+  // TODO: figure out what documentName / embeddedName / type we should be using
+  /** @inheritdoc */
+  static documentName = "Drawing";
+
   constructor() {
     super();
     this.dungeon = null;
   }
 
+  get documentCollection() {
+    // avoid returning all Drawings in the scene, as we
+    // don't want DungeonLayer to draw them during draw().
+    // TODO: we need to stop re-using Drawing for the documentName.
+    return null;
+  }
+
   /** @inheritdoc */
   static get layerOptions() {
     return foundry.utils.mergeObject(super.layerOptions, {
@@ -126,10 +137,6 @@ export class DungeonLayer extends PlaceablesLayer {
     });
   }
 
-  // TODO: figure out what documentName / embeddedName / type we should be using
-  /** @inheritdoc */
-  static documentName = "Drawing";
-
   /**
    * Get initial data for a new drawing.
    * Start with some global defaults, apply user default config, then apply mandatory overrides per tool.