Feat: Actor crafting app salvage switch source actor (#232)

docs/_config.yml

@@ -1,4 +1,4 @@
-title: Fabricate 0.10.13
+title: Fabricate 0.10.14
 email: [email protected]
 description: >- 
   End user documentation for the Foundry Virtual Tabletop (VTT) Module, "Fabricate".

package.json

@@ -1,6 +1,6 @@
 {
   "name": "fabricate",
-  "version": "0.10.13",
+  "version": "0.10.14",
   "description": "A system-agnostic, flexible crafting module for FoundryVT",
   "main": "index.js",
   "type": "module",

src/applications/actorCraftingApp/ActorCraftingApp.svelte

@@ -86,6 +86,12 @@
     async function handleSourceActorChanged(event: CustomEvent<ActorDetails>) {
         sourceActorDetails = event.detail;
         await load();
+        if (salvageProcess.isReady) {
+            salvageProcess = await fabricateAPI.crafting.getSalvageProcess({
+                componentId: salvageProcess.componentId,
+                actorId: sourceActorDetails.id
+            });
+        }
     }
 
     async function salvageComponent(event: CustomEvent<SalvageProcess>) {

src/applications/actorCraftingApp/ComponentSalvageProcess.svelte

@@ -32,6 +32,9 @@
     const dispatch = createEventDispatcher();
     $: displayAmount = calculateDisplayAmount(batchSize, salvageProcess.ownedQuantity);
     let selectedSalvageOption: SalvageOption = salvageProcess.selectedOption;
+    $: {
+        selectedSalvageOption = salvageProcess.selectedOption;
+    }
     $: salvageProcessProducts = selectedSalvageOption.products;
     $: salvageProcessCatalysts = selectedSalvageOption.catalysts;
     $: selectedOptionName = selectedSalvageOption.name;

src/scripts/api/CraftingAPI.ts

@@ -36,7 +36,7 @@ import {
 } from "../../applications/actorCraftingApp/SalvageAssessment";
 import {
     DefaultSalvageOption,
-    DefaultSalvageProcess,
+    DefaultSalvageProcess, NoSalvageProcess,
     SalvageOption,
     SalvageProcess
 } from "../../applications/actorCraftingApp/SalvageProcess";
@@ -327,6 +327,7 @@ class DefaultCraftingAPI implements CraftingAPI {
     }
 
     async getSalvageProcess({ componentId, actorId }: { componentId: string; actorId: string }): Promise<SalvageProcess> {
+
         const componentToSalvage = await this.componentAPI.getById(componentId);
         await componentToSalvage.load();
         const actor = await this.gameProvider.loadActor(actorId);
@@ -336,14 +337,11 @@ class DefaultCraftingAPI implements CraftingAPI {
         const knownRecipesById = await this.recipeAPI.getAllByCraftingSystemId(componentToSalvage.craftingSystemId);
         const knownRecipes = Array.from(knownRecipesById.values());
         const inventory = this.inventoryFactory.make(gameSystemId, actor, knownComponents, knownRecipes);
+
         if (!inventory.getContents().has(componentToSalvage)) {
-            const message = this.localizationService.format(
-                `${DefaultCraftingAPI._LOCALIZATION_PATH}.salvage.componentNotOwned`,
-                { componentName: componentToSalvage.name }
-            );
-            this.notificationService.error(message);
-            throw new Error(message);
+            return new NoSalvageProcess();
         }
+
         const salvageOptions = componentToSalvage.salvageOptions.all
             .map(salvageOption => {
                 return new DefaultSalvageOption({
@@ -361,13 +359,15 @@ class DefaultCraftingAPI implements CraftingAPI {
                 name: salvageOption.name,
                 value: salvageOption,
             }));
+
         return new DefaultSalvageProcess({
             options: new DefaultSelectableOptions<SalvageOption>({options: salvageOptions}),
             componentName: componentToSalvage.name,
             componentId: componentToSalvage.id,
             componentImageUrl: componentToSalvage.imageUrl,
             ownedQuantity: inventory.getContents().amountFor(componentToSalvage),
         });
+
     }
 
     async assessCrafting({