From 2b06b09aefa4106ce6c3b13d97a8a70817c47207 Mon Sep 17 00:00:00 2001
From: Yaroslav Bolyukin <iam@lach.pw>
Date: Thu, 04 Mar 2021 11:42:48 +0000
Subject: [PATCH] fix: don't read storage in Default impl


Instead assign sponsored_data_size at collection creation time

---
--- a/pallets/nft/src/lib.rs
+++ b/pallets/nft/src/lib.rs
@@ -194,7 +194,7 @@
         CollectionLimits { 
             account_token_ownership_limit: 10_000_000, 
             token_limit: u32::max_value(),
-            sponsored_data_size: ChainLimit::get().custom_data_limit, 
+            sponsored_data_size: u32::MAX,
             sponsor_transfer_timeout: 14400,
             owner_can_transfer: true,
             owner_can_destroy: true
@@ -576,8 +576,10 @@
                 _ => 0
             };
 
+            let chain_limit = ChainLimit::get();
+
             // bound Total number of collections
-            ensure!(CollectionCount::get() < ChainLimit::get().collection_numbers_limit, Error::<T>::TotalCollectionsLimitExceeded);
+            ensure!(CollectionCount::get() < chain_limit.collection_numbers_limit, Error::<T>::TotalCollectionsLimitExceeded);
 
             // check params
             ensure!(decimal_points <= MAX_DECIMAL_POINTS, Error::<T>::CollectionDecimalPointLimitExceeded);
@@ -598,6 +600,11 @@
             CreatedCollectionCount::put(next_id);
             CollectionCount::put(total);
 
+            let limits = CollectionLimits {
+                sponsored_data_size: chain_limit.custom_data_limit,
+                ..Default::default()
+            };
+
             // Create new collection
             let new_collection = CollectionType {
                 owner: who.clone(),
@@ -614,7 +621,7 @@
                 sponsor_confirmed: false,
                 variable_on_chain_schema: Vec::new(),
                 const_on_chain_schema: Vec::new(),
-                limits: CollectionLimits::default()
+                limits,
             };
 
             // Add new collection to map

-- 
gitstuff