diff --git a/packages/actions/sources/kiosk.move b/packages/actions/sources/kiosk.move index b67a4c5..2785a94 100644 --- a/packages/actions/sources/kiosk.move +++ b/packages/actions/sources/kiosk.move @@ -142,13 +142,13 @@ public fun delist( account: &mut Account, kiosk: &mut Kiosk, name: String, - nft: ID, + nft_id: ID, ) { auth.verify_with_role(account.addr(), name); assert!(has_lock(account, name), ENoLock); let cap: &KioskOwnerCap = account.borrow_managed_object(KioskOwnerKey { name }, version::current()); - kiosk.delist(cap, nft); + kiosk.delist(cap, nft_id); } /// Members can withdraw the profits to the account diff --git a/packages/actions/tests/access_control_tests.move b/packages/actions/tests/access_control_tests.move index 7200023..7fa7441 100644 --- a/packages/actions/tests/access_control_tests.move +++ b/packages/actions/tests/access_control_tests.move @@ -72,8 +72,10 @@ fun role(action: vector, name: vector): String { let mut full_role = @account_actions.to_string(); full_role.append_utf8(b"::access_control::"); full_role.append_utf8(action); - full_role.append_utf8(b"::"); - full_role.append_utf8(name); + if (!name.is_empty()) { + full_role.append_utf8(b"::"); + full_role.append_utf8(name); + }; full_role } diff --git a/packages/actions/tests/config_tests.move b/packages/actions/tests/config_tests.move index 1d65267..03d8724 100644 --- a/packages/actions/tests/config_tests.move +++ b/packages/actions/tests/config_tests.move @@ -78,8 +78,10 @@ fun role(action: vector, name: vector): String { let mut full_role = @account_actions.to_string(); full_role.append_utf8(b"::config::"); full_role.append_utf8(action); - full_role.append_utf8(b"::"); - full_role.append_utf8(name); + if (!name.is_empty()) { + full_role.append_utf8(b"::"); + full_role.append_utf8(name); + }; full_role } @@ -87,8 +89,10 @@ fun upgrade_policies_role(action: vector, name: vector): String { let mut full_role = @account_actions.to_string(); full_role.append_utf8(b"::upgrade_policies::"); full_role.append_utf8(action); - full_role.append_utf8(b"::"); - full_role.append_utf8(name); + if (!name.is_empty()) { + full_role.append_utf8(b"::"); + full_role.append_utf8(name); + }; full_role } diff --git a/packages/actions/tests/currency_tests.move b/packages/actions/tests/currency_tests.move index 3417ac3..3f375d2 100644 --- a/packages/actions/tests/currency_tests.move +++ b/packages/actions/tests/currency_tests.move @@ -85,8 +85,10 @@ fun role(action: vector, name: vector): String { let mut full_role = @account_actions.to_string(); full_role.append_utf8(b"::currency::"); full_role.append_utf8(action); - full_role.append_utf8(b"::"); - full_role.append_utf8(name); + if (!name.is_empty()) { + full_role.append_utf8(b"::"); + full_role.append_utf8(name); + }; full_role } diff --git a/packages/actions/tests/kiosk_tests.move b/packages/actions/tests/kiosk_tests.move index 3835d75..f19b73c 100644 --- a/packages/actions/tests/kiosk_tests.move +++ b/packages/actions/tests/kiosk_tests.move @@ -102,8 +102,10 @@ fun role(action: vector, name: vector): String { let mut full_role = @account_actions.to_string(); full_role.append_utf8(b"::kiosk::"); full_role.append_utf8(action); - full_role.append_utf8(b"::"); - full_role.append_utf8(name); + if (!name.is_empty()) { + full_role.append_utf8(b"::"); + full_role.append_utf8(name); + }; full_role } diff --git a/packages/actions/tests/treasury_tests.move b/packages/actions/tests/treasury_tests.move index de84e79..abaf58e 100644 --- a/packages/actions/tests/treasury_tests.move +++ b/packages/actions/tests/treasury_tests.move @@ -90,8 +90,10 @@ fun role(action: vector, name: vector): String { let mut full_role = @account_actions.to_string(); full_role.append_utf8(b"::treasury::"); full_role.append_utf8(action); - full_role.append_utf8(b"::"); - full_role.append_utf8(name); + if (!name.is_empty()) { + full_role.append_utf8(b"::"); + full_role.append_utf8(name); + }; full_role } diff --git a/packages/actions/tests/upgrade_policies_tests.move b/packages/actions/tests/upgrade_policies_tests.move index 2156bea..9763d44 100644 --- a/packages/actions/tests/upgrade_policies_tests.move +++ b/packages/actions/tests/upgrade_policies_tests.move @@ -71,8 +71,10 @@ fun role(action: vector, name: vector): String { let mut full_role = @account_actions.to_string(); full_role.append_utf8(b"::upgrade_policies::"); full_role.append_utf8(action); - full_role.append_utf8(b"::"); - full_role.append_utf8(name); + if (!name.is_empty()) { + full_role.append_utf8(b"::"); + full_role.append_utf8(name); + }; full_role } diff --git a/packages/protocol/sources/auth.move b/packages/protocol/sources/auth.move index 83f3959..87d8b4d 100644 --- a/packages/protocol/sources/auth.move +++ b/packages/protocol/sources/auth.move @@ -72,8 +72,11 @@ public fun verify_with_role( name: String, ) { let mut r = type_name::get().into_string().to_string(); - r.append_utf8(b"::"); - r.append(name); + + if (!name.is_empty()) { + r.append_utf8(b"::"); + r.append(name); + }; let Auth { role, account_addr } = auth;