Fix attacked_by being called after handled attacks (#27542)

code/_onclick/item_attack.dm

@@ -163,6 +163,8 @@
 	user.do_attack_animation(target)
 	add_fingerprint(user)
 
+	return TRUE
+
 /// The equivalent of the standard version of [/obj/item/proc/attack] but for non mob targets.
 /obj/item/proc/attack_obj(obj/attacked_obj, mob/living/user, params)
 	var/signal_return = SEND_SIGNAL(src, COMSIG_ATTACK_OBJ, attacked_obj, user) | SEND_SIGNAL(user, COMSIG_ATTACK_OBJ_LIVING, attacked_obj)

code/_onclick/item_attack_legacy.dm

@@ -25,10 +25,11 @@
 
 	SEND_SIGNAL(user, COMSIG_MOB_ITEM_ATTACK, M, user)
 
-	. = __attack_core(M, user)
+	if(!__attack_core(M, user))
+		return
 
 	if(!M.new_attack_chain)
-		M.attacked_by__legacy__attackchain(src, user, def_zone)
+		return M.attacked_by__legacy__attackchain(src, user, def_zone)
 
 /**
  * Called when `user` attacks us with item `W`.