[#4802] Add "display chat card" button to inventory item context menu (…

…#5021)
foundryvtt · Feb 4, 2025 · 8e2f6f0 · 8e2f6f0
1 parent 46c8287
commit 8e2f6f0
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 9 deletions.
diff --git a/lang/en.json b/lang/en.json
@@ -1740,6 +1740,7 @@
 "DND5E.Disadvantage": "Disadvantage",
 "DND5E.Discord": "Discord",
 "DND5E.Disclaimer": "Disclaimer",
+"DND5E.DisplayCard": "Display in Chat",
 "DND5E.DistAny": "Any",
 "DND5E.DistFt": "Feet",
 "DND5E.DistFtAbbr": "ft",

diff --git a/module/applications/components/inventory.mjs b/module/applications/components/inventory.mjs
@@ -212,6 +212,11 @@ export default class InventoryElement extends HTMLElement {
         condition: () => item.canDelete && item.isOwner && !item.compendium?.locked,
         callback: li => this._onAction(li[0], "delete")
       },
+      {
+        name: "DND5E.DisplayCard",
+        icon: '<i class="fas fa-message"></i>',
+        callback: () => item.displayCard()
+      },
       {
         name: "DND5E.Scroll.CreateScroll",
         icon: '<i class="fa-solid fa-scroll"></i>',

diff --git a/module/documents/chat-message.mjs b/module/documents/chat-message.mjs
@@ -669,16 +669,11 @@ export default class ChatMessage5e extends ChatMessage {
    * @protected
    */
   _enrichUsageEffects(html) {
+    if ( this.getFlag("dnd5e", "messageType") !== "usage" ) return;
     const item = this.getAssociatedItem();
-    let effects;
-    if ( this.getFlag("dnd5e", "messageType") === "usage" ) {
-      effects = this.getFlag("dnd5e", "use.effects")?.map(id => item?.effects.get(id));
-    } else {
-      if ( this.getFlag("dnd5e", "roll.type") ) return;
-      effects = item?.effects.filter(e => (e.type !== "enchantment")
-        && !item.getFlag("dnd5e", "riders.effect")?.includes(e.id));
-    }
-    effects = effects?.filter(e => e && (game.user.isGM || (e.transfer && (this.author.id === game.user.id))));
+    const effects = this.getFlag("dnd5e", "use.effects")
+      ?.map(id => item?.effects.get(id))
+      .filter(e => e && (game.user.isGM || (e.transfer && (this.author.id === game.user.id))));
     if ( !effects?.length ) return;
 
     const effectApplication = document.createElement("effect-application");