v1.9.1 - Advancements Bugfixes

CHANGELOG.md

@@ -1,6 +1,9 @@
 # Changelog
 Date format : day/month/year
 
+## 1.9.1 - 08/09/2022 - Advancements Bugfixes
+- Fix for advancements fail to add new items ("_id" readonly error. see #42).
+
 ## 1.9.0 - 01/09/2022 - Foundry v10 Compatibility
 __! Be certain to carefully back up any critical user data before installing this update !__
 - Updated the System to FoundryVTT v10.

README.md

@@ -2,6 +2,7 @@
 
 ![Banner Legend of the Five Rings](./l5rBan.jpg)
 [![Buy Me a Coffee](./tags-bymecoffee.svg)](https://ko-fi.com/vlyan)
+[![FoundryVTT version](https://img.shields.io/badge/FVTT-v10.x-informational)](https://foundryvtt.com/)
 [![Forge Installs](https://img.shields.io/badge/dynamic/json?label=Forge%20Installs&query=package.installs&suffix=%25&url=https%3A%2F%2Fforge-vtt.com%2Fapi%2Fbazaar%2Fpackage%2Fl5r5e&colorB=4aa94a)](https://forge-vtt.com/bazaar#package=l5r5e)
 [![Foundry Hub Endorsements](https://img.shields.io/endpoint?logoColor=white&url=https%3A%2F%2Fwww.foundryvtt-hub.com%2Fwp-json%2Fhubapi%2Fv1%2Fpackage%2Fl5r5e%2Fshield%2Fendorsements)](https://www.foundryvtt-hub.com/package/l5r5e/)
 [![Foundry Hub Comments](https://img.shields.io/endpoint?logoColor=white&url=https%3A%2F%2Fwww.foundryvtt-hub.com%2Fwp-json%2Fhubapi%2Fv1%2Fpackage%2Fl5r5e%2Fshield%2Fcomments)](https://www.foundryvtt-hub.com/package/l5r5e/)

system/scripts/actors/base-character-sheet.js

@@ -135,14 +135,14 @@ export class BaseCharacterSheetL5r5e extends BaseSheetL5r5e {
     async _onDrop(event) {
         // *** Everything below here is only needed if the sheet is editable ***
         if (!this.isEditable || this.actor.system.soft_locked) {
-            console.log("LR5E | Not editable");
+            console.log("L5R5E | This sheet is not editable");
             return;
         }
 
         // Check item type and subtype
         const item = await game.l5r5e.HelpersL5r5e.getDragnDropTargetObject(event);
         if (!item || !["Item", "JournalEntry"].includes(item.documentName) || item.type === "property") {
-            console.log(`LR5E | Wrong subtype ${item?.type}`, item);
+            console.log(`L5R5E | Wrong subtype ${item?.type}`, item);
             return;
         }
 
@@ -150,7 +150,7 @@ export class BaseCharacterSheetL5r5e extends BaseSheetL5r5e {
         if (item.documentName === "JournalEntry") {
             // npc does not have this
             if (!this.actor.system.identity?.school_curriculum_journal) {
-                console.log("LR5E | NPC won't go to school :'(");
+                console.log("L5R5E | NPC won't go to school :'(");
                 return;
             }
             this.actor.system.identity.school_curriculum_journal = {
@@ -180,7 +180,7 @@ export class BaseCharacterSheetL5r5e extends BaseSheetL5r5e {
                     return embedItem._id === item._id;
                 })
             ) {
-                console.log("LR5E | This item already exist in this actor", item.uuid);
+                console.log("L5R5E | This element has been ignored because it already exists in this actor", item.uuid);
                 return;
             }
 

system/scripts/item.js

@@ -216,8 +216,11 @@ export class ItemL5r5e extends Item {
         }
 
         // New id
-        if (newId) {
+        if (newId || !item._id) {
-            item._id = foundry.utils.randomID();
+            // Bypass the readonly for "_id"
+            const tmpData = item.toJSON();
+            tmpData._id = foundry.utils.randomID();
+            item = new ItemL5r5e(tmpData);
         }
 
         // Copy the parent permission to the sub item

system/system.json

@@ -7,11 +7,11 @@
     "changelog": "https://gitlab.com/teaml5r/l5r5e/-/blob/master/CHANGELOG.md",
     "license": "https://gitlab.com/teaml5r/l5r5e/-/blob/master/LICENSE.md",
     "manifest": "https://gitlab.com/teaml5r/l5r5e/-/raw/master/system/system.json",
-    "download": "https://gitlab.com/teaml5r/l5r5e/-/jobs/artifacts/v1.9.0/raw/l5r5e.zip?job=build",
+    "download": "https://gitlab.com/teaml5r/l5r5e/-/jobs/artifacts/v1.9.1/raw/l5r5e.zip?job=build",
-    "version": "1.9.0",
+    "version": "1.9.1",
     "compatibility": {
         "minimum": 10,
-        "verified": "10.283"
+        "verified": "10.284"
     },
     "manifestPlusVersion": "1.2.0",
     "socket": true,