fix: reorder storage operations in dnsAlloc function

massalabs · Dec 26, 2024 · 863ef47 · 863ef47
1 parent 96b5733
commit 863ef47
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/smart-contract/assembly/contracts/main.ts b/smart-contract/assembly/contracts/main.ts
@@ -188,8 +188,6 @@ export function dnsAlloc(binaryArgs: StaticArray<u8>): StaticArray<u8> {
   assert(!Storage.has(domainToTargetKey(domain)), 'Domain already registered');
 
   const counter = bytesToU256(Storage.get(COUNTER_KEY));
-  // @ts-ignore (fix for IDE)
-  Storage.set(COUNTER_KEY, u256ToBytes(counter + u256.One));
 
   // Mint the token
   _update(owner, counter, '');
@@ -199,6 +197,8 @@ export function dnsAlloc(binaryArgs: StaticArray<u8>): StaticArray<u8> {
   Storage.set(targetToDomainKey(target, domain), []);
   Storage.set(tokenIdToDomainKey(counter), stringToBytes(domain));
   Storage.set(domainToTokenIdKey(domain), u256ToBytes(counter));
+  // @ts-ignore (fix for IDE)
+  Storage.set(COUNTER_KEY, u256ToBytes(counter + u256.One));
 
   const storageCosts = initialBalance - balance();
   const totalCost = calculateCreationCost(domain.length) + storageCosts;