pallet-uniques/nfts: Small optimizations (#2754)

Use `contains_key` to not require decoding the actual value.
paritytech · Dec 19, 2023 · 84d6342 · 84d6342
1 parent 7c79741
commit 84d6342
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/substrate/frame/nfts/src/features/transfer.rs b/substrate/frame/nfts/src/features/transfer.rs
@@ -173,8 +173,8 @@ impl<T: Config<I>, I: 'static> Pallet<T, I> {
 		who: T::AccountId,
 		maybe_collection: Option<T::CollectionId>,
 	) -> DispatchResult {
-		let old = OwnershipAcceptance::<T, I>::get(&who);
-		match (old.is_some(), maybe_collection.is_some()) {
+		let exists = OwnershipAcceptance::<T, I>::contains_key(&who);
+		match (exists, maybe_collection.is_some()) {
 			(false, true) => {
 				frame_system::Pallet::<T>::inc_consumers(&who)?;
 			},

diff --git a/substrate/frame/uniques/src/lib.rs b/substrate/frame/uniques/src/lib.rs
@@ -1433,8 +1433,8 @@ pub mod pallet {
 			maybe_collection: Option<T::CollectionId>,
 		) -> DispatchResult {
 			let who = ensure_signed(origin)?;
-			let old = OwnershipAcceptance::<T, I>::get(&who);
-			match (old.is_some(), maybe_collection.is_some()) {
+			let exists = OwnershipAcceptance::<T, I>::contains_key(&who);
+			match (exists, maybe_collection.is_some()) {
 				(false, true) => {
 					frame_system::Pallet::<T>::inc_consumers(&who)?;
 				},