git.delta.rocks / unique-network / refs/commits / e137eba0a7ea

difftreelog

fix small fixes

Trubnikov Sergey2022-08-25parent: #9d78c86.patch.diff
in: master

7 files changed

modifiedpallets/common/src/erc.rsdiffbeforeafterboth
--- a/pallets/common/src/erc.rs
+++ b/pallets/common/src/erc.rs
@@ -179,15 +179,19 @@
 	///
 	/// @return Tuble with sponsor address and his substrate mirror. If there is no confirmed sponsor error "Contract has no sponsor" throw.
 	fn get_collection_sponsor(&self) -> Result<(address, uint256)> {
-		let sponsor = match self.collection.sponsorship {
-			SponsorshipState::Disabled | SponsorshipState::Unconfirmed(_) => {
-				return Ok(Default::default())
-			}
-			SponsorshipState::Confirmed(ref sponsor) => sponsor,
+		let sponsor = match self.collection.sponsorship.sponsor() {
+			Some(sponsor) => sponsor,
+			None => return Ok(Default::default()),
 		};
 		let sponsor = T::CrossAccountId::from_sub(sponsor.clone());
-		let sponsor_sub = convert_cross_account_to_uint256::<T>(&sponsor);
-		Ok((*sponsor.as_eth(), sponsor_sub))
+		let result: (address, uint256) = if sponsor.is_canonical_substrate() {
+			let sponsor = convert_cross_account_to_uint256::<T>(&sponsor);
+			(Default::default(), sponsor)
+		} else {
+			let sponsor = *sponsor.as_eth();
+			(sponsor, Default::default())
+		};
+		Ok(result)
 	}
 
 	/// Set limits for the collection.
modifiedpallets/common/src/eth.rsdiffbeforeafterboth
--- a/pallets/common/src/eth.rs
+++ b/pallets/common/src/eth.rs
@@ -64,7 +64,7 @@
 where
 	T::AccountId: From<[u8; 32]>,
 {
-	let mut new_admin_arr: [u8; 32] = Default::default();
+	let mut new_admin_arr = [0_u8; 32];
 	from.to_big_endian(&mut new_admin_arr);
 	let account_id = T::AccountId::from(new_admin_arr);
 	T::CrossAccountId::from_sub(account_id)
modifiedpallets/fungible/CHANGELOG.mddiffbeforeafterboth
--- a/pallets/fungible/CHANGELOG.md
+++ b/pallets/fungible/CHANGELOG.md
@@ -5,7 +5,7 @@
 ## [v0.1.4] - 2022-08-24
 
 ### Change
- - Add bound `AsRef<[u8]>` to `T::CrossAccountId`.
+ - Add bound `AsRef<[u8; 32]>` to `T::CrossAccountId`.
 
 <!-- bureaucrate goes here -->
 ## [v0.1.3] 2022-08-16
modifiedpallets/nonfungible/CHANGELOG.mddiffbeforeafterboth
--- a/pallets/nonfungible/CHANGELOG.md
+++ b/pallets/nonfungible/CHANGELOG.md
@@ -5,7 +5,7 @@
 ## [v0.1.5] - 2022-08-24
 
 ### Change
- - Add bound `AsRef<[u8]>` to `T::CrossAccountId`.
+ - Add bound `AsRef<[u8; 32]>` to `T::CrossAccountId`.
 
 <!-- bureaucrate goes here -->
 ## [v0.1.4] 2022-08-16
modifiedpallets/refungible/CHANGELOG.mddiffbeforeafterboth
before · pallets/refungible/CHANGELOG.md
1# Change Log23All notable changes to this project will be documented in this file.45## [v0.2.4] - 2022-08-2467### Change8 - Add bound `AsRef<[u8]>` to `T::CrossAccountId`.910<!-- bureaucrate goes here -->11## [v0.2.3] 2022-08-161213### Other changes1415- build: Upgrade polkadot to v0.9.27 2c498572636f2b34d53b1c51b7283a761a7dc90a1617- build: Upgrade polkadot to v0.9.26 85515e54c4ca1b82a2630034e55dcc804c643bf81819- refactor: Switch to new prefix removal methods 26734e9567589d75cdd99e404eabf11d5a97d9752021New methods allows to call `remove_prefix` with limit multiple times22in the same block23However, we don't use prefix removal limits, so upgrade is24straightforward2526Upstream-Change: https://github.com/paritytech/substrate/pull/114902728- build: Upgrade polkadot to v0.9.25 cdfb9bdc7b205ff1b5134f034ef9973d769e5e6b2930## [v0.2.2] 2022-08-043132### Product changes3334- Now RefungibleMultipleItems may only receive single user on type level.3536### Added features3738- Implement property RPC 7bf45b532e32daa91f03c157b58874d21b42ae1f3940### Other changes4142- refactor: Disallow invalid bulk mints 53fec71cf728dddd012257b407ea30441e699f884344`create_multiple_items_ex` was allowing invalid (that will be always45rejected at runtime level) refungible mint extrinsics, by passing46multiple users into `RefungibleMultipleItems` call.4748## [v0.2.1] - 2022-07-274950### New features5152Implementation of ERC-721 EVM API ([#452](https://github.com/UniqueNetwork/unique-chain/pull/452))5354## [v0.2.0] - 2022-08-015556### Deprecated5758`const_data` field is removed5960- `ItemData`61- `TokenData`6263## [v0.1.2] - 2022-07-146465### Other changes6667feat(refungible-pallet): add ERC-20 EVM API for RFT token pieces ([#413](https://github.com/UniqueNetwork/unique-chain/pull/413))68test(refungible-pallet): add tests for ERC-20 EVM API for RFT token pieces ([#413](https://github.com/UniqueNetwork/unique-chain/pull/413))6970## [v0.1.1] - 2022-07-147172### Added features7374- Support for properties for RFT collections and tokens.7576### Other changes7778- feat: RPC method `token_owners` returning 10 owners in no particular order.7980This was an internal request to improve the web interface and support fractionalization event.
after · pallets/refungible/CHANGELOG.md
1# Change Log23All notable changes to this project will be documented in this file.45## [v0.2.4] - 2022-08-2467### Change8 - Add bound `AsRef<[u8; 32]>` to `T::CrossAccountId`.910<!-- bureaucrate goes here -->11## [v0.2.3] 2022-08-161213### Other changes1415- build: Upgrade polkadot to v0.9.27 2c498572636f2b34d53b1c51b7283a761a7dc90a1617- build: Upgrade polkadot to v0.9.26 85515e54c4ca1b82a2630034e55dcc804c643bf81819- refactor: Switch to new prefix removal methods 26734e9567589d75cdd99e404eabf11d5a97d9752021New methods allows to call `remove_prefix` with limit multiple times22in the same block23However, we don't use prefix removal limits, so upgrade is24straightforward2526Upstream-Change: https://github.com/paritytech/substrate/pull/114902728- build: Upgrade polkadot to v0.9.25 cdfb9bdc7b205ff1b5134f034ef9973d769e5e6b2930## [v0.2.2] 2022-08-043132### Product changes3334- Now RefungibleMultipleItems may only receive single user on type level.3536### Added features3738- Implement property RPC 7bf45b532e32daa91f03c157b58874d21b42ae1f3940### Other changes4142- refactor: Disallow invalid bulk mints 53fec71cf728dddd012257b407ea30441e699f884344`create_multiple_items_ex` was allowing invalid (that will be always45rejected at runtime level) refungible mint extrinsics, by passing46multiple users into `RefungibleMultipleItems` call.4748## [v0.2.1] - 2022-07-274950### New features5152Implementation of ERC-721 EVM API ([#452](https://github.com/UniqueNetwork/unique-chain/pull/452))5354## [v0.2.0] - 2022-08-015556### Deprecated5758`const_data` field is removed5960- `ItemData`61- `TokenData`6263## [v0.1.2] - 2022-07-146465### Other changes6667feat(refungible-pallet): add ERC-20 EVM API for RFT token pieces ([#413](https://github.com/UniqueNetwork/unique-chain/pull/413))68test(refungible-pallet): add tests for ERC-20 EVM API for RFT token pieces ([#413](https://github.com/UniqueNetwork/unique-chain/pull/413))6970## [v0.1.1] - 2022-07-147172### Added features7374- Support for properties for RFT collections and tokens.7576### Other changes7778- feat: RPC method `token_owners` returning 10 owners in no particular order.7980This was an internal request to improve the web interface and support fractionalization event.
modifiedtests/src/eth/collectionSponsoring.test.tsdiffbeforeafterboth
--- a/tests/src/eth/collectionSponsoring.test.ts
+++ b/tests/src/eth/collectionSponsoring.test.ts
@@ -1,4 +1,4 @@
-import {addToAllowListExpectSuccess, confirmSponsorshipExpectSuccess, createCollectionExpectSuccess, enablePublicMintingExpectSuccess, getDetailedCollectionInfo, setCollectionSponsorExpectSuccess} from '../util/helpers';
+import {addToAllowListExpectSuccess, bigIntToSub, confirmSponsorshipExpectSuccess, createCollectionExpectSuccess, enablePublicMintingExpectSuccess, getDetailedCollectionInfo, setCollectionSponsorExpectSuccess} from '../util/helpers';
 import {itWeb3, createEthAccount, collectionIdToAddress, GAS_ARGS, normalizeEvents, createEthAccountWithBalance, evmCollectionHelpers, getCollectionAddressFromResult, evmCollection, ethBalanceViaSub, subToEth} from './util/helpers';
 import nonFungibleAbi from './nonFungibleAbi.json';
 import {expect} from 'chai';
@@ -57,7 +57,7 @@
     expect(await collectionEvm.methods.hasCollectionPendingSponsor().call({from: owner})).to.be.false;
     
     const sponsorTuple = await collectionEvm.methods.getCollectionSponsor().call({from: owner});
-    expect(sponsorTuple.field_0).to.be.eq(subToEth(sponsor.address));
+    expect(bigIntToSub(api, BigInt(sponsorTuple[1]))).to.be.eq(sponsor.address);
   });
 
   itWeb3('Remove sponsor', async ({api, web3, privateKeyWrapper}) => {
modifiedtests/src/util/helpers.tsdiffbeforeafterboth
--- a/tests/src/util/helpers.ts
+++ b/tests/src/util/helpers.ts
@@ -99,6 +99,10 @@
   }
 }
 
+export function bigIntToSub(api: ApiPromise, number: bigint) {
+  return api.registry.createType('AccountId', '0x' + number.toString(16).padStart(64, '0')).toJSON();
+}
+
 export function normalizeAccountId(input: string | AccountId | CrossAccountId | IKeyringPair): CrossAccountId {
   if (typeof input === 'string') {
     if (input.length >= 47) {