Problem in setting PNFT primary sale happened switch to 1

[isiik]

private async UniTask StartMinting(NetworkConnection conn, string metaName, string metaSymbol, string nftNumber, string collectionAddressPubKey, string clientWalletAddress)
{
    PublicKey treasuryAccount = new PublicKey(treasuryAccountHash);

    // Create mint account and associated token account
    var mintAccount = new Account();
    var associatedTokenAccount = AssociatedTokenAccountProgram.DeriveAssociatedTokenAccount(Web3.Account, mintAccount.PublicKey);

    // Metadata URL
    string webAddress = "ipfs://" + collectionStorageHash + "/" + nftNumber.ToString() + ".json";
    PublicKey metaplexPnftRuleset = new PublicKey("xxx");

    // Metadata creation
    var metadata = new Metadata()
    {   
        name = metaName,
        symbol = metaSymbol,
        uri = webAddress,
        sellerFeeBasisPoints = royaltiesValue,
        creators = new List<Creator> { new(Web3.Account.PublicKey, 0, true), new(treasuryAccount, 100, false) },
        collection = new Collection(new PublicKey(collectionAddressPubKey), verified: false)
        programmableConfig = new ProgrammableConfig(metaplexPnftRuleset),
    };

    // Prepare the transaction
    var blockHash = await Web3.Rpc.GetLatestBlockHashAsync();
    var minimumRent = await Web3.Rpc.GetMinimumBalanceForRentExemptionAsync(TokenProgram.MintAccountDataSize);
    var transaction = new TransactionBuilder()
        .SetRecentBlockHash(blockHash.Result.Value.Blockhash)
        .SetFeePayer(Web3.Account)
        .AddInstruction(SystemProgram.CreateAccount(
            Web3.Account,
            mintAccount,
            minimumRent.Result,
            TokenProgram.MintAccountDataSize,
            TokenProgram.ProgramIdKey))
        .AddInstruction(TokenProgram.InitializeMint(
            mintAccount,
            0,
            Web3.Account,
            Web3.Account))
        .AddInstruction(AssociatedTokenAccountProgram.CreateAssociatedTokenAccount(
            Web3.Account,
            Web3.Account,
            mintAccount))
        .AddInstruction(MetadataProgram.CreateMetadataAccount(
            PDALookup.FindMetadataPDA(mintAccount),
            mintAccount,
            Web3.Account,
            Web3.Account,
            Web3.Account,
            metadata,
            TokenStandard.ProgrammableNonFungible,
            true,
            true,
            maxSupply: 0,
            masterEditionKey: PDALookup.FindMasterEditionPDA(mintAccount),
            metadataVersion: MetadataVersion.V4))
        .AddInstruction(MetadataProgram.Mint(
            associatedTokenAccount, 
            PDALookup.FindMetadataPDA(mintAccount),
            PDALookup.FindMasterEditionPDA(mintAccount),
            mintAccount,
            Web3.Account,
            Web3.Account,
            tokenRecord: PDALookup.FindTokenRecordPDA(Web3.Account, mintAccount)
            ))
        // This results in error with ProgrammableNonFungible but works with NonFungible
        .AddInstruction(MetadataProgram.UpdatePrimarySaleHappendViaToken(
             metadataKey: PDALookup.FindMetadataPDA(mintAccount),
             owner: Web3.Account.PublicKey,
             tokenAccount: associatedTokenAccount))
        .Build(new List<Account> { Web3.Account, mintAccount });

        // Execute the transaction

        var result = await Web3.Rpc.SendTransactionAsync(transaction);

Commented the instruction that results in an error. Not sure how to fix this one with PNFT.

Program log: Instruction not supported for ProgrammableNonFungible assets.

[GabrielePicco]

I looked up the source code of how Metaplex define the UpdatePrimarySaleHappendViaToken (which seems to be the cause of the issue) and the exception could be potentially triggered at this line: https://github.com/metaplex-foundation/mpl-token-metadata/blob/main/programs/token-metadata/program/src/processor/mod.rs#L175 . There is a check for pNFT before forwarding to the process_legacy.

The check is defined here: https://github.com/metaplex-foundation/mpl-token-metadata/blob/main/programs/token-metadata/program/src/processor/mod.rs#L350.

Not entirely sure, but seems that this instruction is not supported with pNFT. Perhaps the update rules does not allow this kind of update. Does using UpdateMetadata lead to the same issue?

Also, I would probably ask this question on Metaplex Github/Discord