fix: 🐛 Handle NFT holders in case of nft.controller_transfer

In case of `ControllerTransfer` as the reason for `NFTPortfolioUpdate`
event, asset transaction entry was being made but NFTHolder set of
nft_ids was not getting updated. This handles the logic to  update the
NFTHolders

db/migrations/4_add_extrinsic_for_issued_txs.sql

@@ -0,0 +1,20 @@
+-- for older blocks, the extrinsic_idx in asset_transactions was not being populated. We use the events table to update the value for `Issued` event
+update 
+  asset_transactions ast
+set 
+  extrinsic_idx = e.extrinsic_idx
+from 
+  events e
+where 
+  e.block_id = ast.created_block_id
+  and e.event_idx = ast.event_idx
+  and ast.extrinsic_idx is null
+  and e.extrinsic_idx is not null;
+
+-- extrinsic_id mapping in events was not being populated correctly. We use the block_id and extrinsic_idx to update the missing values
+update 
+  events
+set 
+  extrinsic_id = block_id || '/' || extrinsic_idx
+where 
+  extrinsic_id is null;

src/mappings/entities/mapAsset.ts

@@ -278,7 +278,13 @@ const handleDivisibilityChanged = async (blockId: string, params: Codec[]): Prom
   await asset.save();
 };
 
-const handleIssued = async ({ blockId, params, eventIdx, block }: HandlerArgs): Promise<void> => {
+const handleIssued = async ({
+  blockId,
+  params,
+  eventIdx,
+  block,
+  extrinsic,
+}: HandlerArgs): Promise<void> => {
   const [, rawTicker, rawBeneficiaryDid, rawAmount, rawFundingRound, rawTotalFundingAmount] =
     params;
 
@@ -304,6 +310,7 @@ const handleIssued = async ({ blockId, params, eventIdx, block }: HandlerArgs):
     eventIdx,
     amount: issuedAmount,
     fundingRound,
+    extrinsicIdx: extrinsic?.idx,
     datetime: block.timestamp,
     createdBlockId: blockId,
     updatedBlockId: blockId,

src/mappings/entities/mapEvent.ts

@@ -65,6 +65,11 @@ export function handleToolingEvent(
 
   const { claimExpiry, claimIssuer, claimScope, claimType } = extractClaimInfo(harvesterLikeArgs);
 
+  let extrinsicId: string;
+  if (extrinsic) {
+    extrinsicId = `${blockId}/${extrinsic?.idx}`;
+  }
+
   return Event.create({
     id: `${blockId}/${eventIdx}`,
     blockId,
@@ -85,6 +90,7 @@ export function handleToolingEvent(
     corporateActionTicker: extractCorporateActionTicker(harvesterLikeArgs),
     fundraiserOfferingAsset: extractOfferingAsset(harvesterLikeArgs),
     transferTo: extractTransferTo(harvesterLikeArgs),
+    extrinsicId,
   });
 }
 

src/mappings/entities/mapNfts.ts

@@ -92,8 +92,15 @@ const handleNftPortfolioUpdates = async (
     const nftHolder = await getNftHolder(ticker, did, blockId);
     nftHolder.nftIds.push(...ids);
     promises.push(nftHolder.save());
-  } else if (reason === 'transferred') {
-    eventId = EventIdEnum.Transfer;
+  } else if (reason === 'redeemed') {
+    eventId = EventIdEnum.RedeemedNFT;
+    asset.totalSupply -= BigInt(ids.length);
+
+    const nftHolder = await getNftHolder(ticker, did, blockId);
+    nftHolder.nftIds = nftHolder.nftIds.filter(heldId => !ids.includes(heldId));
+    nftHolder.updatedBlockId = blockId;
+    promises.push(nftHolder.save());
+  } else if (reason === 'transferred' || reason === 'controllerTransfer') {
     const [fromHolder, toHolder] = await Promise.all([
       getNftHolder(ticker, fromDid, blockId),
       getNftHolder(ticker, toDid, blockId),
@@ -105,21 +112,18 @@ const handleNftPortfolioUpdates = async (
 
     asset.totalTransfers += BigInt(1);
 
-    const details = value as unknown as {
-      readonly instructionId: Option<u64>;
-      readonly instructionMemo: Option<U8aFixed>;
-    };
-
-    instructionId = getTextValue(details.instructionId);
-    instructionMemo = bytesToString(details.instructionMemo);
-  } else if (reason === 'redeemed') {
-    eventId = EventIdEnum.RedeemedNFT;
-    asset.totalSupply -= BigInt(ids.length);
-
-    const nftHolder = await getNftHolder(ticker, did, blockId);
-    nftHolder.nftIds = nftHolder.nftIds.filter(heldId => !ids.includes(heldId));
-    nftHolder.updatedBlockId = blockId;
-    promises.push(nftHolder.save());
+    if (reason === 'transferred') {
+      eventId = EventIdEnum.Transfer;
+      const details = value as unknown as {
+        readonly instructionId: Option<u64>;
+        readonly instructionMemo: Option<U8aFixed>;
+      };
+
+      instructionId = getTextValue(details.instructionId);
+      instructionMemo = bytesToString(details.instructionMemo);
+    } else {
+      eventId = EventIdEnum.ControllerTransfer;
+    }
   }
 
   promises.push(