Você está usando variáveis de pacote (PKGYellowCount
e PKGRedCount
) que retêm seus valores entre as execuções do acionador.
Uma abordagem melhor seria usar uma tabela temporária para contar os cartões amarelos e vermelhos por jogador.
Por exemplo, este código:
CREATE OR REPLACE TRIGGER check_if_red_card_or_two_yellows_before_insert_new_summon
FOR INSERT OR UPDATE ON ACOES
COMPOUND TRIGGER
Temp_Counts TABLE (
NIF NUMBER,
ContagemAmarelos NUMBER := 0,
ContagemVermelhos NUMBER := 0
);
BEFORE EACH ROW IS
BEGIN
IF :NEW.TipoAcaoID = 7 THEN
FOR rec IN Temp_Counts.FIRST..Temp_Counts.LAST LOOP
IF Temp_Counts(rec).NIF = :NEW.NIF THEN
Temp_Counts(rec).ContagemAmarelos := Temp_Counts(rec).ContagemAmarelos + 1;
EXIT;
END IF;
END LOOP;
ELSIF :NEW.TipoAcaoID = 8 THEN
FOR rec IN Temp_Counts.FIRST..Temp_Counts.LAST LOOP
IF Temp_Counts(rec).NIF = :NEW.NIF THEN
Temp_Counts(rec).ContagemVermelhos := Temp_Counts(rec).ContagemVermelhos + 1;
EXIT;
END IF;
END LOOP;
END IF;
END BEFORE EACH ROW;
AFTER STATEMENT IS
BEGIN
FOR rec IN Temp_Counts.FIRST..Temp_Counts.LAST LOOP
IF Temp_Counts(rec).ContagemAmarelos = 2 THEN
UPDATE Convocatoria
SET MinutosJogados = (
SELECT Minuto
FROM Acoes
WHERE NIF = Temp_Counts(rec).NIF AND TipoAcaoID = 7
ORDER BY Minuto
FETCH FIRST ROW ONLY
)
WHERE NIF = Temp_Counts(rec).NIF;
ELSIF Temp_Counts(rec).ContagemVermelhos = 1 THEN
UPDATE Convocatoria
SET MinutosJogados = (
SELECT Minuto
FROM Acoes
WHERE NIF = Temp_Counts(rec).NIF AND TipoAcaoID = 8
ORDER BY Minuto
FETCH FIRST ROW ONLY
)
WHERE NIF = Temp_Counts(rec).NIF;
END IF;
END LOOP;
-- limpa a tabela temporária
Temp_Counts.DELETE;
END AFTER STATEMENT;
END check_if_red_card_or_two_yellows_before_insert_new_summon;
/