more item related stuff

module/actor/actor-sheet.js

@@ -273,6 +273,10 @@ export class DeltaGreenActorSheet extends ActorSheet {
     // Remove the type from the dataset since it's in the itemData.type prop.
     delete itemData.data["type"];
 
+    if(type == "weapon"){
+      itemData.data.skill = "firearms"; //default skill to firearms, since that will be most common
+    }
+    
     // Finally, create the item!
     return this.actor.createOwnedItem(itemData);
   }
@@ -307,6 +311,9 @@ export class DeltaGreenActorSheet extends ActorSheet {
         targetVal = this.actor.data.data.sanity.value;
         label += `, Target: ${targetVal}`;
       }
+      else if(dataset.target === "damage"){
+        // damage roll, not a skill check
+      }
 
       roll.roll().toMessage({
         speaker: ChatMessage.getSpeaker({ actor: this.actor }),

module/deltagreen.js

@@ -43,11 +43,19 @@ Hooks.once('init', async function() {
   });
 
   Handlebars.registerHelper('toLowerCase', function(str) {
-    return str.toLowerCase();
+    try {
+      return str.toLowerCase();
+    } catch (error) {
+      return "";
+    }
   });
 
   Handlebars.registerHelper('toUpperCase', function(str) {
-    return str.toUpperCase();
+    try {
+      return str.toUpperCase();
+    } catch (error) {
+      return "";
+    }
   });
 
   Handlebars.registerHelper('if_eq', function(a, b, opts) {
@@ -59,10 +67,15 @@ Hooks.once('init', async function() {
   });
 
   Handlebars.registerHelper('getActorSkillProp', function(actorData, skillName, prop) {
-    let skills = actorData.skills;
-    let skill = skills[skillName];
-    let propVal = skill[prop];
-    return propVal;
+    if(skillName != "" && prop != ""){
+      let skills = actorData.skills;
+      let skill = skills[skillName];
+      let propVal = skill[prop];
+      return propVal;
+    }
+    else{
+      return "";
+    }
   });
 });
 

template.json

@@ -377,7 +377,8 @@
             "templates": [
                 "base"
             ],
-            "armor": 3
+            "armor": 3,
+            "equipped": true
         }
     }
 }

templates/actor/actor-sheet.html

@@ -135,27 +135,43 @@
 
         {{!-- Owned Items Tab --}}
         <div class="tab items" data-group="primary" data-tab="gear">
-          <ol class="items-list">
+          <div>
+            <h3>WEAPONS</h3>
+            <ol class="items-list">
+              
               <li class="item flexrow item-header">
                 <div class="item-image"></div>
                 <div class="item-name">Name</div>
                 <div class="item-name">Skill</div>
+                <div class="item-name">Damage</div>
+                <div class="item-name">Lethality</div>
                 <div class="item-controls">
                   <a class="item-control item-create" title="Create item" data-type="weapon"><i class="fas fa-plus"></i> Add item</a>
                 </div>
               </li>
-          {{#each actor.items as |item id|}}
-              <li class="item flexrow" data-item-id="{{item._id}}">
-                  <div class="item-image"><img src="{{item.img}}" title="{{item.name}}" width="24" height="24"/></div>
-                  <h4 class="item-name">{{item.name}}</h4>
-                  <span class="item-name rollable" data-roll="d100" data-target="{{getActorSkillProp ../data item.data.skill 'proficiency'}}" data-label="{{getActorSkillProp ../data item.data.skill 'label'}}">{{getActorSkillProp ../data item.data.skill 'label'}}</span>
-                  <div class="item-controls">
-                      <a class="item-control item-edit" title="Edit Item"><i class="fas fa-edit"></i></a>
-                      <a class="item-control item-delete" title="Delete Item"><i class="fas fa-trash"></i></a>
-                  </div>
-                </li>
-          {{/each}}
-          </ol>
+
+              {{#each actor.items as |item id|}}
+                {{#if_eq item.type 'weapon'}}
+
+                  <li class="item flexrow" data-item-id="{{item._id}}">
+                    <div class="item-image"><img src="{{item.img}}" title="{{item.name}}" width="24" height="24"/></div>
+                    <h4 style="word-wrap: break-word;" class="item-name">{{item.name}}</h4>
+                    <span class="item-name rollable" data-roll="d100" data-target="{{getActorSkillProp ../data item.data.skill 'proficiency'}}" data-label="{{getActorSkillProp ../data item.data.skill 'label'}}">{{getActorSkillProp ../data item.data.skill 'label'}}</span>
+                    <span class="rollable" data-target="damage" data-label="{{toUpperCase item.name}}" data-roll="{{toUpperCase item.data.damage}}" >{{toUpperCase item.data.damage}}</span>
+                    <span>{{item.data.lethality}}%</span>
+                    <div class="item-controls">
+                        <a class="item-control item-edit" title="Edit Item"><i class="fas fa-edit"></i></a>
+                        <a class="item-control item-delete" title="Delete Item"><i class="fas fa-trash"></i></a>
+                    </div>
+                  </li>
+
+                {{/if_eq}}
+              {{/each}}
+            </ol>
+          </div>
+          <div>
+            <h3>ARMOR</h3>
+          </div>
         </div>
 
         {{!-- Biography Tab --}}

templates/item/item-weapon-sheet.html

@@ -14,7 +14,11 @@
               </div>
               <div class="resource">
                 <label class="resource-label">Damage</label>
-                  <input type="text" name="data.damage" value="{{data.damage}}" data-dtype="String"/>
+                <input type="text" name="data.damage" value="{{data.damage}}" data-dtype="String"/>
+              </div>
+              <div class="resource">
+                <label class="resource-label">Lethality %</label>
+                <input type="text" name="data.lethality" value="{{data.lethality}}" data-dtype="Number"/>
               </div>
             </div>
         </div>