closes #930

Muttley · Nov 2, 2024 · 3106c3d · 3106c3d
1 parent ce502da
commit 3106c3d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/RELEASE_NOTES.md b/RELEASE_NOTES.md
@@ -16,6 +16,7 @@
 * [#906] There's always an extra space between special attack name and effect
 * [#918] Actor and Item type names missing from i18n data
 * [#919] Class item template has incorrect default spellcasting class value
+* [#930] Class Abilities can't be used if they have no associated skill roll
 
 ## Chores
 * [#911] Replaced compendium art mapping with default Foundry method.

diff --git a/system/src/documents/ActorSD.mjs b/system/src/documents/ActorSD.mjs
@@ -1414,7 +1414,7 @@ export default class ActorSD extends Actor {
 			title = game.i18n.localize("SHADOWDARK.chat.use_ability.title");
 
 			// does ability use on a roll check?
-			if (typeof item.system.ability !== "undefined") {
+			if (item.system.ability !== "") {
 				options = foundry.utils.mergeObject({target: item.system.dc}, options);
 				const result = await this.rollAbility(
 					item.system.ability,