refactor: remove unused player inventory (#20)

Author: ssouthcity

src/simulation/item/inventory.rs

@@ -2,14 +2,8 @@ use bevy::prelude::*;
 
 mod assets;
 mod compendium;
-mod inventory;
-mod stack;
 
-pub use self::{
-    assets::{ItemAssets, ItemDef},
-    inventory::Inventory,
-    stack::Stack,
-};
+pub use self::assets::{ItemAssets, ItemDef};
 
 pub fn plugin(app: &mut App) {
     app.register_type::<Item>();
@@ -19,9 +13,6 @@ pub fn plugin(app: &mut App) {
     app.add_systems(Update, mark_full);
 
     app.add_plugins((assets::plugin, compendium::plugin));
-
-    app.register_type::<PlayerInventory>()
-        .add_systems(Startup, spawn_player_inventory);
 }
 
 #[derive(Component, Reflect)]
@@ -55,14 +46,3 @@ fn mark_full(
         }
     }
 }
-
-#[derive(Component, Reflect, Deref, DerefMut)]
-#[reflect(Component)]
-pub struct PlayerInventory(Inventory);
-
-fn spawn_player_inventory(mut commands: Commands) {
-    commands.spawn((
-        Name::new("Player Inventory"),
-        PlayerInventory(Inventory::sized(9)),
-    ));
-}

src/simulation/item/mod.rs

@@ -6,16 +6,11 @@ use serde::Deserialize;
 use crate::{
     assets::{
         LoadResource,
-        indexing::IndexMap,
         manifest::{Id, Manifest, ManifestPlugin},
     },
     screens::Screen,
-    simulation::{
-        item::{ItemDef, PlayerInventory, Stack},
-        recipe::RecipeDef,
-        world::{MAP_SIZE, Terrain, WorldSpawnSystems},
-    },
-    ui::{Interact, Interactable, YSort},
+    simulation::world::{MAP_SIZE, Terrain, WorldSpawnSystems},
+    ui::{Interactable, YSort},
 };
 
 pub fn plugin(app: &mut App) {
@@ -27,8 +22,6 @@ pub fn plugin(app: &mut App) {
         OnEnter(Screen::Gameplay),
         spawn_deposits.in_set(WorldSpawnSystems::SpawnDeposits),
     );
-
-    app.add_observer(on_mine_deposit);
 }
 
 #[derive(Debug, Deserialize, TypePath)]
@@ -103,37 +96,3 @@ fn spawn_deposits(
         }
     }
 }
-
-fn on_mine_deposit(
-    trigger: Trigger<Interact>,
-    deposits: Query<&DepositRecipe>,
-    mut inventory: Single<&mut PlayerInventory>,
-    recipes: Res<Assets<RecipeDef>>,
-    recipe_index: Res<IndexMap<RecipeDef>>,
-    items: Res<Assets<ItemDef>>,
-    item_index: Res<IndexMap<ItemDef>>,
-) {
-    let Ok(deposit_recipe) = deposits.get(trigger.target()) else {
-        return;
-    };
-
-    let recipe_def = recipe_index
-        .get(&deposit_recipe.0)
-        .and_then(|asset_id| recipes.get(*asset_id))
-        .expect("Deposit refers to non-existent recipe");
-
-    for (item_id, quantity) in recipe_def.output.iter() {
-        let item_def = item_index
-            .get(item_id)
-            .and_then(|asset_id| items.get(*asset_id))
-            .expect("Recipe refers to invalid item id");
-
-        let mut stack = Stack {
-            item_id: item_def.id.to_owned(),
-            quantity: *quantity,
-            max_quantity: item_def.stack_size,
-        };
-
-        let _ = inventory.add_stack(&mut stack);
-    }
-}