Disallow drop on non editable sheet

CHANGELOG.md

@@ -7,6 +7,7 @@
   - Courts of Stone
   - Path of Waves
   - Celestial Realms
+- English cleanup, thanks to Mark Zeman !
 - Fix js error when Advancement is not embed in a actor
 - Click on a weapon show the DicePicker with the weapon skill selected
 - Display Rarity in Compendiums for Items, Armors and Weapons

system/scripts/actors/base-sheet.js

@@ -73,6 +73,11 @@ export class BaseSheetL5r5e extends ActorSheet {
      * Handle dropped data on the Actor sheet
      */
     async _onDrop(event) {
+        // *** Everything below here is only needed if the sheet is editable ***
+        if (!this.options.editable) {
+            return;
+        }
+
         // Check item type and subtype
         const item = await game.l5r5e.HelpersL5r5e.getDragnDropTargetObject(event);
         if (

system/scripts/actors/twenty-questions-dialog.js

@@ -217,6 +217,11 @@ export class TwentyQuestionsDialog extends FormApplication {
      * Handle dropped items
      */
     async _onDropItem(type, event) {
+        // *** Everything below here is only needed if the sheet is editable ***
+        if (!this.options.editable) {
+            return;
+        }
+
         if (!["item", "technique", "peculiarity"].includes(type)) {
             return;
         }

system/scripts/items/item-sheet.js

@@ -133,6 +133,11 @@ export class ItemSheetL5r5e extends ItemSheet {
      * Also a property canot be on another property
      */
     async _onDrop(event) {
+        // Everything below here is only needed if the sheet is editable
+        if (!this.options.editable) {
+            return;
+        }
+
         // Check item type and subtype
         const item = await game.l5r5e.HelpersL5r5e.getDragnDropTargetObject(event);
         if (!item || item.entity !== "Item" || item.data.type !== "property" || this.item.type === "property") {