Refactor/crafting system manager app (#75)

Author: misterpotts

src/lang/en.json

@@ -124,15 +124,36 @@
         "content": "Delete {systemName}? This action cannot be undone.",
         "title": "Fabricate | Delete crafting system"
       },
+      "noSystemSelected": "No crafting system selected. Create a new system or select one from the left menu.",
       "readErrorPrompt": {
         "title": "Error Reading Crafting Systems",
         "content": "{errorMessage}. \n You can reset Fabricate to resolve the error, but this will delete all except the bundled crafting systems!"
       },
+      "prompts": {
+        "itemReadError":{
+          "title": "Remove {type} {itemId}?",
+          "content": "The {type} with UUID {itemId} was referenced by a Fabricate Item, but does not exist. Deleting references to it will allow Fabricate to load. Proceed?"
+        },
+        "removeComponent": {
+          "title": "Remove {name}?",
+          "content": "The component will be removed from the Crafting System, as well as any Recipes and Components that reference it. This action cannot be undone. Are you sure?"
+        },
+        "removeEssence": {
+          "title": "Remove {name}?",
+          "content": "The essence will be removed from the Crafting System, as well as any Recipes and Components that reference it. This action cannot be undone. Are you sure?"
+        },
+        "removeRecipe": {
+          "title": "Remove {name}?",
+          "content": "The recipe will be removed from the Crafting System. This action cannot be undone. Are you sure?"
+        }
+      },
       "tabs": {
         "recipes": {
+          "title": "Recipes",
           "optionTooltip": "{size} Components"
         },
         "essences": {
+          "title": "Essences",
           "description": "Essences are a quality of a component. They define something that the component has, not something that it is. Components might have multiple Essences in different quantities, all of which you can define for your system. Not every crafting system needs essences. However, if you do want to add them to components, you'll need to define them here first.",
           "create": "Add a new Essence",
           "tableHeadings": {
@@ -144,6 +165,15 @@
             "edit": "Edit essence",
             "delete": "Delete essence"
           }
+        },
+        "alchemy": {
+          "title": "Alchemy"
+        },
+        "checks": {
+          "title": "Checks"
+        },
+        "components": {
+          "title": "Components"
         }
       }
     },

src/scripts/common/Combination.ts

@@ -335,6 +335,12 @@ class Combination<T extends Identifiable> {
         return this;
     }
 
+    without(part: T) {
+        const combination: Map<string, Unit<T>> = new Map(this._amounts);
+        combination.delete(part.id);
+        return new Combination<T>(combination);
+    }
+
     public multiply(factor: number) {
         const modifiedAmounts: Map<string, Unit<T>> = new Map(this._amounts);
         this.members.forEach((member: T) => {

src/scripts/foundry/DocumentManager.ts

@@ -13,15 +13,38 @@ interface DocumentManager {
 
 }
 
+class ItemNotFoundError extends Error {
+
+    private static readonly formatErrorMessage = (id: string) => `Item with id ${id} was not found. `;
+    private readonly _itemUuid: string;
+
+    constructor(itemUuid: string) {
+        super(ItemNotFoundError.formatErrorMessage(itemUuid));
+        this._itemUuid = itemUuid;
+    }
+
+    get itemUuid(): string {
+        return this._itemUuid;
+    }
+
+}
+
 class DefaultDocumentManager implements DocumentManager {
 
     public async getDocumentByUuid(id: string): Promise<FabricateItemData> {
         const document = await fromUuid(id);
-        return this.formatItemData(document);
+        if (document) {
+            return this.formatItemData(document);
+        }
+        throw new ItemNotFoundError(id);
     }
 
     public async getDocumentsByUuid(ids: string[]): Promise<Map<string, FabricateItemData>> {
-        const itemData = await Promise.all(ids.map(id => this.getDocumentByUuid(id)));
+        const itemData = await Promise.all(ids.map(id => this.getDocumentByUuid(id).catch(e => e)));
+        const firstErrorResult = itemData.find(result => result instanceof Error);
+        if (firstErrorResult) {
+            throw firstErrorResult;
+        }
         return new Map(itemData.map(item => [item.uuid,item]));
     }
 
@@ -36,4 +59,4 @@ class DefaultDocumentManager implements DocumentManager {
 
 }
 
-export { FabricateItemData, DocumentManager, DefaultDocumentManager }
+export { FabricateItemData, DocumentManager, DefaultDocumentManager, ItemNotFoundError }