From 263b01e7025a2aee2047b8186b8436a29928a7fe Mon Sep 17 00:00:00 2001
From: Susucre <34709007+Susucre@users.noreply.github.com>
Date: Sat, 29 Jul 2023 19:46:40 +0200
Subject: [PATCH] [CMM] rework Narci, Fable Singer (#10699)

* easy fix first

* use targetPointer in Narci effect.
---
 Mage.Sets/src/mage/cards/n/NarciFableSinger.java   | 14 +++++++++++++-
 ...inalChapterAbilityResolvesTriggeredAbility.java |  1 -
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/Mage.Sets/src/mage/cards/n/NarciFableSinger.java b/Mage.Sets/src/mage/cards/n/NarciFableSinger.java
index d526217430e..b395cc96f69 100644
--- a/Mage.Sets/src/mage/cards/n/NarciFableSinger.java
+++ b/Mage.Sets/src/mage/cards/n/NarciFableSinger.java
@@ -1,6 +1,7 @@
 package mage.cards.n;
 
 import mage.MageInt;
+import mage.MageObject;
 import mage.abilities.Ability;
 import mage.abilities.common.FinalChapterAbilityResolvesTriggeredAbility;
 import mage.abilities.common.SacrificePermanentTriggeredAbility;
@@ -17,6 +18,7 @@ import mage.constants.SubType;
 import mage.constants.SuperType;
 import mage.filter.StaticFilters;
 import mage.game.Game;
+import mage.target.targetpointer.FixedTarget;
 
 import java.util.UUID;
 
@@ -77,7 +79,17 @@ class NarciFableSingerEffect extends OneShotEffect {
 
     @Override
     public boolean apply(Game game, Ability source) {
-        int value = (int) getValue("SAGA_MANACOST");
+        FixedTarget fixedTarget = targetPointer.getFixedTarget(game, source);
+        if (fixedTarget == null) {
+            return false;
+        }
+
+        MageObject saga = game.getObject(fixedTarget.getTarget());
+        if (saga == null) {
+            return false;
+        }
+
+        int value = saga.getManaValue();
         if (value > 0) {
             new LoseLifeOpponentsEffect(value).apply(game, source);
             new GainLifeEffect(value).apply(game, source);
diff --git a/Mage/src/main/java/mage/abilities/common/FinalChapterAbilityResolvesTriggeredAbility.java b/Mage/src/main/java/mage/abilities/common/FinalChapterAbilityResolvesTriggeredAbility.java
index ef711a0a354..925988ae7ed 100644
--- a/Mage/src/main/java/mage/abilities/common/FinalChapterAbilityResolvesTriggeredAbility.java
+++ b/Mage/src/main/java/mage/abilities/common/FinalChapterAbilityResolvesTriggeredAbility.java
@@ -94,7 +94,6 @@ public class FinalChapterAbilityResolvesTriggeredAbility extends TriggeredAbilit
 
         if (rememberSaga) {
             getEffects().setTargetPointer(new FixedTarget(permanent.getId(), game));
-            getEffects().setValue("SAGA_MANACOST", permanent.getManaValue());
         }
         return true;
     }