Apply suggestion

- Remove default value for CharacterAbility
- Put back accidentally removed comment

common/src/comp/ability.rs

@@ -17,7 +17,6 @@ use serde::{Deserialize, Serialize};
 use specs::{Component, FlaggedStorage};
 use specs_idvs::IdvStorage;
 use std::time::Duration;
-use tracing::error;
 use vek::Vec3;
 
 #[derive(Copy, Clone, Hash, Eq, PartialEq, Debug, Serialize, Deserialize)]
@@ -242,16 +241,6 @@ impl Asset for CharacterAbility {
     type Loader = assets::RonLoader;
 
     const EXTENSION: &'static str = "ron";
-
-    fn default_value(specifier: &str, err: assets::Error) -> Result<Self, assets::Error> {
-        error!(
-            ?err,
-            "Error loading CharacterAbility: {} for the ability map: replacing with default",
-            specifier
-        );
-
-        Ok(CharacterAbility::default())
-    }
 }
 
 impl CharacterAbility {

common/src/comp/inventory/item/mod.rs

@@ -171,6 +171,11 @@ impl assets::Compound for ItemDef {
             kind,
             quality,
         } = raw;
+
+        // Some commands like /give_item provide the asset specifier separated with \
+        // instead of .
+        //
+        // TODO: This probably does not belong here
         let item_definition_id = specifier.replace('\\', ".");
 
         Ok(ItemDef {