Fixes

code/modules/spellbook/_spellbook.dm

@@ -227,9 +227,9 @@ GLOBAL_LIST_EMPTY(spells_by_categories)
 		dat += "<A href='byond://?src=\ref[src];purchase=[S]'>Purchase ([initial(S.spell_cost)] points)</a><br>"
 	else if(OS.level_max[UPGRADE_TOTAL] > 0)
 		var/up_count = 0
-		for(var/up_type in spell_levels)
-			up_count += spell_levels[up_type]
-		dat += "Maximum amount of upgrades: [up_count]/[OS.level_max[UPGRADE_TOTAL]]"
+		for(var/up_type in OS.spell_levels)
+			up_count += OS.spell_levels[up_type]
+		dat += "Maximum amount of upgrades: [up_count]/[OS.level_max[UPGRADE_TOTAL]].<br>"
 		for(var/upgrade_type in OS.spell_levels)
 			if(OS.level_max[upgrade_type] <= 0)
 				continue

code/modules/spells/_spell.dm

@@ -365,8 +365,7 @@
 	if(!user.mind)
 		return FALSE
 	var/mob/living/L = user
-	L.mind.mana.UseMana(L, amount, FALSE)
-	return TRUE
+	return L.mind.mana.UseMana(L, amount, FALSE)
 
 /datum/spell/proc/invocation(mob/user = usr, var/list/targets) //spelling the spell out and setting it on recharge/reducing charges amount
 

code/modules/spells/hand/_hand.dm

@@ -66,8 +66,6 @@
 /datum/spell/hand/proc/cast_hand(atom/a, mob/user) //same for casting.
 	if(!TakeMana(user, mana_cost_per_cast))
 		return FALSE
-	SEND_SIGNAL(user, COMSIG_SPELL_CAST_HAND, src, a)
-	SEND_GLOBAL_SIGNAL(COMSIG_GLOB_SPELL_CAST_HAND, user, src, a)
 	return TRUE
 
 /datum/spell/hand/charges

code/modules/spells/hand/_hand_item.dm

@@ -47,6 +47,8 @@ Basically: I can use it to target things where I click. I can then pass these ta
 		user.visible_message("\The [user][hand_spell.show_message]")
 	if(hand_spell.cast_hand(A,user))
 		next_spell_time = world.time + hand_spell.spell_delay
+		SEND_SIGNAL(user, COMSIG_SPELL_CAST_HAND, hand_spell, A)
+		SEND_GLOBAL_SIGNAL(COMSIG_GLOB_SPELL_CAST_HAND, user, hand_spell, A)
 		if(hand_spell.move_delay)
 			user.ExtraMoveCooldown(hand_spell.move_delay)
 		if(hand_spell.click_delay)