[r2r] disable_coin should fail if there are tokens dependent on the platform

[sergeyboyko0791]

The original PR - #1649

[DeckerSU]

LGTM

[shamardy]

Thank you for the fix! Just have one suggestion :)
Also a question about calling cancel on init_standalone_coin, it seems that this also removes dependant tokens. Is this the required behaviour for v2 coin initialization?

[sergeyboyko0791]

init_standalone_coin is used to initialize only one coin (like UTXO, so we can consider it as a platform) without tokens.
But later enable_eth_with_tokens and other similar RPCs should be refactored significantly: #1650 (comment)

Also a question about calling cancel on init_standalone_coin, it seems that this also removes dependant tokens. Is this the required behaviour for v2 coin initialization?

[shamardy]

One last question/suggestion that would require others input

[shamardy]

LGTM!

[smk762]

Expected response returned here

smk762@pig:~/mm2_777$ ./disable.sh BCH
{"error":"There are currently matching orders, active swaps, or tokens dependent on 'BCH'","orders":{"matching":[],"cancelled":[]},"active_swaps":[],"dependent_tokens":["ASLP-SLP"]}
smk762@pig:~/mm2_777$ ./disable.sh IRIS
{"error":"There are currently matching orders, active swaps, or tokens dependent on 'IRIS'","orders":{"matching":[],"cancelled":[]},"active_swaps":[],"dependent_tokens":["ATOM-IBC_IRIS"]}
smk762@pig:~/mm2_777$ ./disable.sh ETH
{"error":"There are currently matching orders, active swaps, or tokens dependent on 'ETH'","orders":{"matching":[],"cancelled":[]},"active_swaps":[],"dependent_tokens":["APE-ERC20","BCH-ERC20","BUSD-ERC20","MINDS-ERC20"]}
smk762@pig:~/mm2_777$ ./disable.sh MATIC
{"error":"There are currently matching orders, active swaps, or tokens dependent on 'MATIC'","orders":{"matching":[],"cancelled":[]},"active_swaps":[],"dependent_tokens":["AAVE-PLG20","PGX-PLG20"]}

[ca333]

Suggested change

	pub async fn get_dependent_tokens(&self, ticker: &str) -> HashSet<String> {
	pub async fn get_platform_coin_tokens(&self, ticker: &str) -> HashSet<String> {

[ca333]

Suggested change

	let dependent_tokens = coins_ctx.get_dependent_tokens(&ticker).await;
	if !dependent_tokens.is_empty() {
	return ERR!("There are tokens dependent on '{}': {:?}", ticker, dependent_tokens);
	}
	let platform_coin_tokens = coins_ctx.get_platform_coin_tokens(&ticker).await;
	if !platform_coin_tokens.is_empty() {
	return ERR!("There are tokens dependent on '{}': {:?}", ticker, platform_coin_tokens);
	}