"Number C79: Battlin' Boxer General Kaiser" fix

Fixed an issue where it would attach your battling monster to itself if the opponent controlled it on resolution.

Author: pyrQ

official/c10300821.lua

@@ -4,51 +4,48 @@
 local s,id=GetID()
 function s.initial_effect(c)
 	c:EnableReviveLimit()
-	--Xyz Summon Procedure
+	--Xyz Summon procedure: 3 Level 5 monsters
 	Xyz.AddProcedure(c,nil,5,3)
-	--Gain 200 ATK per each material attached
+	--Gains 200 ATK for each material attached to it
 	local e1=Effect.CreateEffect(c)
 	e1:SetType(EFFECT_TYPE_SINGLE)
 	e1:SetProperty(EFFECT_FLAG_SINGLE_RANGE)
 	e1:SetCode(EFFECT_UPDATE_ATTACK)
 	e1:SetRange(LOCATION_MZONE)
-	e1:SetValue(function(_,c) return c:GetOverlayCount()*200 end)
+	e1:SetValue(function(e,c) return c:GetOverlayCount()*200 end)
 	c:RegisterEffect(e1)
-	--Negate Special Summons and destroy that monster(s)
+	--Detach 2 materials from this card, and if you do, negate your opponent's Special Summon, and if you do that, destroy that monster(s)
 	local e2=Effect.CreateEffect(c)
 	e2:SetDescription(aux.Stringid(id,0))
 	e2:SetCategory(CATEGORY_DISABLE_SUMMON+CATEGORY_DESTROY)
 	e2:SetType(EFFECT_TYPE_QUICK_O)
 	e2:SetCode(EVENT_SPSUMMON)
 	e2:SetRange(LOCATION_MZONE)
 	e2:SetCountLimit(1)
-	e2:SetCondition(s.negsumcond)
+	e2:SetCondition(function(e,tp,eg,ep,ev,re,r,rp) return tp==1-ep and Duel.GetCurrentChain(true)==0 end)
 	e2:SetTarget(s.negsumtg)
 	e2:SetOperation(s.negsumop)
 	c:RegisterEffect(e2)
-	--Attach monsters when an attack is declared involving a "Battlin' Boxer" monster
+	--Send 1 "Battlin' Boxer" monster from your hand or Deck to the GY, and if you do, attach that opponent's battling monster to this card as material
 	local e3=Effect.CreateEffect(c)
 	e3:SetDescription(aux.Stringid(id,1))
 	e3:SetCategory(CATEGORY_TOGRAVE)
 	e3:SetType(EFFECT_TYPE_FIELD+EFFECT_TYPE_TRIGGER_O)
 	e3:SetCode(EVENT_ATTACK_ANNOUNCE)
 	e3:SetRange(LOCATION_MZONE)
 	e3:SetCountLimit(1)
-	e3:SetCondition(s.atchcond)
-	e3:SetTarget(s.atchtg)
-	e3:SetOperation(s.atchop)
+	e3:SetCondition(s.tgattachcon)
+	e3:SetTarget(s.tgattachtg)
+	e3:SetOperation(s.tgattachop)
 	c:RegisterEffect(e3)
 end
 s.xyz_number=79
-s.listed_names={71921856} --Number 79: Battlin' Boxer Nova Kaiser
+s.listed_names={71921856} --"Number 79: Battlin' Boxer Nova Kaiser"
 s.listed_series={SET_BATTLIN_BOXER}
-function s.negsumcond(e,tp,eg,ep,ev,re,r,rp)
-	return tp==1-ep and Duel.GetCurrentChain(true)==0
-end
 function s.negsumtg(e,tp,eg,ep,ev,re,r,rp,chk)
 	if chk==0 then return e:GetHandler():CheckRemoveOverlayCard(tp,2,REASON_EFFECT) end
-	Duel.SetOperationInfo(0,CATEGORY_DISABLE_SUMMON,eg,#eg,0,0)
-	Duel.SetOperationInfo(0,CATEGORY_DESTROY,eg,#eg,0,0)
+	Duel.SetOperationInfo(0,CATEGORY_DISABLE_SUMMON,eg,#eg,tp,0)
+	Duel.SetOperationInfo(0,CATEGORY_DESTROY,eg,#eg,tp,0)
 end
 function s.negsumop(e,tp,eg,ep,ev,re,r,rp)
 	local c=e:GetHandler()
@@ -57,29 +54,30 @@ function s.negsumop(e,tp,eg,ep,ev,re,r,rp)
 		Duel.Destroy(eg,REASON_EFFECT)
 	end
 end
-function s.atchcond(e,tp,eg,ep,ev,re,r,rp)
+function s.tgattachcon(e,tp,eg,ep,ev,re,r,rp)
 	if not e:GetHandler():GetOverlayGroup():IsExists(Card.IsCode,1,nil,71921856) then return false end
 	local a,b=Duel.GetBattleMonster(tp)
 	return a and b and a:IsFaceup() and a:IsSetCard(SET_BATTLIN_BOXER)
 end
 function s.tgfilter(c)
 	return c:IsSetCard(SET_BATTLIN_BOXER) and c:IsMonster() and c:IsAbleToGrave()
 end
-function s.atchtg(e,tp,eg,ep,ev,re,r,rp,chk)
+function s.tgattachtg(e,tp,eg,ep,ev,re,r,rp,chk)
 	local _,b=Duel.GetBattleMonster(tp)
 	if chk==0 then return b:IsAbleToChangeControler() and b:IsCanBeXyzMaterial(e:GetHandler(),tp,REASON_EFFECT)
 		and Duel.IsExistingMatchingCard(s.tgfilter,tp,LOCATION_HAND|LOCATION_DECK,0,1,nil) end
+	Duel.SetTargetCard(b)
 	Duel.SetOperationInfo(0,CATEGORY_TOGRAVE,nil,1,tp,LOCATION_HAND|LOCATION_DECK)
 end
-function s.atchop(e,tp,eg,ep,ev,re,r,rp)
+function s.tgattachop(e,tp,eg,ep,ev,re,r,rp)
 	Duel.Hint(HINT_SELECTMSG,tp,HINTMSG_TOGRAVE)
 	local g=Duel.SelectMatchingCard(tp,s.tgfilter,tp,LOCATION_HAND|LOCATION_DECK,0,1,1,nil)
 	if #g>0 and Duel.SendtoGrave(g,REASON_EFFECT)>0 and g:GetFirst():IsLocation(LOCATION_GRAVE) then
 		local c=e:GetHandler()
-		local _,tc=Duel.GetBattleMonster(tp)
-		if tc and tc:IsRelateToBattle() and tc:IsControler(1-tp)
-			and not tc:IsImmuneToEffect(e) and c:IsRelateToEffect(e) then
+		local tc=Duel.GetFirstTarget()
+		if tc:IsRelateToEffect(e) and tc:IsControler(1-tp) and not tc:IsImmuneToEffect(e)
+			and c:IsRelateToEffect(e) then
 			Duel.Overlay(c,tc,true)
 		end
 	end
-end
+end