support DynamicNFT XLS-46

CHANGELOG.md

@@ -43,6 +43,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Add `nfts_by_issuer` clio-only API definition
 - Included `ctid` field in the `tx` request.
 - `from_xrpl` method accepts input dictionary keys exclusively in the proper XRPL format.
+- Add `NFTokenModify` transaction and add `TF_MUTABLE` flag in `NFTokenMintFlag`
 
 ### Fixed
 - Added support for `XChainModifyBridge` flag maps (fixing an issue with `NFTokenCreateOffer` flag names)

tests/unit/models/transactions/test_better_transaction_flags.py

@@ -87,6 +87,7 @@ def test_nftoken_mint_flags(self):
                 TF_ONLY_XRP=True,
                 TF_TRANSFERABLE=True,
                 TF_TRUSTLINE=True,
+                TF_MUTABLE=True,
             ),
         )
         self.assertTrue(actual.has_flag(flag=0x00000001))

tests/unit/models/transactions/test_nftoken_modify.py

@@ -0,0 +1,55 @@
+from unittest import TestCase
+
+from xrpl.models.exceptions import XRPLModelException
+from xrpl.models.transactions.nftoken_modify import NFTokenModify
+
+_ACCOUNT = "r9LqNeG6qHxjeUocjvVki2XR35weJ9mZgQ"
+_SEQUENCE = 19048
+_FEE = "0.00001"
+_URI = "abc"
+_NFTOKEN_ID = "00090032B5F762798A53D543A014CAF8B297CFF8F2F937E844B17C9E00000003"
+
+
+class TestNFTokenModify(TestCase):
+    def test_nftoken_miss(self):
+        with self.assertRaises(XRPLModelException):
+            NFTokenModify(
+                account=_ACCOUNT,
+                owner=_ACCOUNT,
+                sequence=_SEQUENCE,
+                fee=_FEE,
+                uri=_ACCOUNT,
+            )
+
+    def test_uri_empty(self):
+        with self.assertRaises(XRPLModelException):
+            NFTokenModify(
+                account=_ACCOUNT,
+                owner=_ACCOUNT,
+                sequence=_SEQUENCE,
+                fee=_FEE,
+                nftoken_id=_NFTOKEN_ID,
+                uri="",
+            )
+
+    def test_uri_too_long(self):
+        with self.assertRaises(XRPLModelException):
+            NFTokenModify(
+                account=_ACCOUNT,
+                owner=_ACCOUNT,
+                sequence=_SEQUENCE,
+                fee=_FEE,
+                nftoken_id=_NFTOKEN_ID,
+                uri=_ACCOUNT * 1000,
+            )
+
+    def test_valid(self):
+        obj = NFTokenModify(
+            account=_ACCOUNT,
+            owner=_ACCOUNT,
+            sequence=_SEQUENCE,
+            fee=_FEE,
+            uri=_URI,
+            nftoken_id=_NFTOKEN_ID,
+        )
+        self.assertTrue(obj.is_valid())

xrpl/core/binarycodec/definitions/definitions.json

@@ -3085,6 +3085,7 @@
     "NFTokenCancelOffer": 28,
     "NFTokenCreateOffer": 27,
     "NFTokenMint": 25,
+    "NFTokenModify": 61,
     "OfferCancel": 8,
     "OfferCreate": 7,
     "OracleDelete": 52,
@@ -3138,4 +3139,4 @@
     "Vector256": 19,
     "XChainBridge": 25
   }
-}
+}

xrpl/models/transactions/__init__.py

@@ -67,6 +67,7 @@
     NFTokenMintFlag,
     NFTokenMintFlagInterface,
 )
+from xrpl.models.transactions.nftoken_modify import NFTokenModify
 from xrpl.models.transactions.offer_cancel import OfferCancel
 from xrpl.models.transactions.offer_create import (
     OfferCreate,
@@ -161,6 +162,7 @@
     "NFTokenMint",
     "NFTokenMintFlag",
     "NFTokenMintFlagInterface",
+    "NFTokenModify",
     "OfferCancel",
     "OfferCreate",
     "OfferCreateFlag",

xrpl/models/transactions/nftoken_mint.py

@@ -46,6 +46,11 @@ class NFTokenMintFlag(int, Enum):
     issuer.
     """
 
+    TF_MUTABLE = 0x00000010
+    """
+    If set, indicates that this NFT's URI can be modified.
+    """
+
 
 class NFTokenMintFlagInterface(FlagInterface):
     """Transaction Flags for an NFTokenMint Transaction."""
@@ -54,6 +59,7 @@ class NFTokenMintFlagInterface(FlagInterface):
     TF_ONLY_XRP: bool
     TF_TRUSTLINE: bool
     TF_TRANSFERABLE: bool
+    TF_MUTABLE: bool
 
 
 @require_kwargs_on_init

xrpl/models/transactions/nftoken_modify.py

@@ -0,0 +1,69 @@
+"""Model for NFTokenModify transaction type."""
+
+from __future__ import annotations
+
+from dataclasses import dataclass, field
+from typing import Dict, Optional
+
+from typing_extensions import Final, Self
+
+from xrpl.models.required import REQUIRED
+from xrpl.models.transactions.transaction import Transaction
+from xrpl.models.transactions.types import TransactionType
+from xrpl.models.utils import KW_ONLY_DATACLASS, require_kwargs_on_init
+
+_MAX_URI_LENGTH: Final[int] = 512
+
+
+@require_kwargs_on_init
+@dataclass(frozen=True, **KW_ONLY_DATACLASS)
+class NFTokenModify(Transaction):
+    """
+    The NFTokenModify transaction modifies an NFToken's URI
+    if its tfMutable is set to true.
+    """
+
+    nftoken_id: str = REQUIRED  # type: ignore
+    """
+    Identifies the TokenID of the NFToken object that the
+    offer references. This field is required.
+    """
+
+    owner: Optional[str] = None
+    """
+    Indicates the AccountID of the account that owns the
+    corresponding NFToken.
+    """
+
+    uri: Optional[str] = None
+    """
+    URI that points to the data and/or metadata associated with the NFT.
+    This field need not be an HTTP or HTTPS URL; it could be an IPFS URI, a
+    magnet link, immediate data encoded as an RFC2379 "data" URL, or even an
+    opaque issuer-specific encoding. The URI is not checked for validity.
+
+    This field must be hex-encoded. You can use `xrpl.utils.str_to_hex` to
+    convert a UTF-8 string to hex.
+    """
+
+    transaction_type: TransactionType = field(
+        default=TransactionType.NFTOKEN_MODIFY,
+        init=False,
+    )
+
+    def _get_errors(self: Self) -> Dict[str, str]:
+        return {
+            key: value
+            for key, value in {
+                **super()._get_errors(),
+                "uri": self._get_uri_error(),
+            }.items()
+            if value is not None
+        }
+
+    def _get_uri_error(self: Self) -> Optional[str]:
+        if not self.uri:
+            return "URI must not be empty string"
+        elif len(self.uri) > _MAX_URI_LENGTH:
+            return f"Must not be longer than {_MAX_URI_LENGTH} characters"
+        return None

xrpl/models/transactions/types/transaction_type.py

@@ -36,6 +36,7 @@ class TransactionType(str, Enum):
     NFTOKEN_CANCEL_OFFER = "NFTokenCancelOffer"
     NFTOKEN_CREATE_OFFER = "NFTokenCreateOffer"
     NFTOKEN_MINT = "NFTokenMint"
+    NFTOKEN_MODIFY = "NFTokenModify"
     OFFER_CANCEL = "OfferCancel"
     OFFER_CREATE = "OfferCreate"
     ORACLE_SET = "OracleSet"