Fix logic
This commit is contained in:
+3
-3
@@ -12,13 +12,13 @@ async def disable_wifi_radios(client: SagemcomClient, radios: tuple[str] | list[
|
|||||||
active_radios = {radio['alias'] for radio in _radios \
|
active_radios = {radio['alias'] for radio in _radios \
|
||||||
if radio.keys() >= {'alias', 'enable'} and \
|
if radio.keys() >= {'alias', 'enable'} and \
|
||||||
radio['enable']}
|
radio['enable']}
|
||||||
if active_radios:
|
if not active_radios:
|
||||||
click.echo('No active radios')
|
click.echo('No active radios')
|
||||||
return
|
return
|
||||||
if radios:
|
if not radios:
|
||||||
radios = click.prompt('Choose radio', type=click.Choice(active_radios), show_choices=True),
|
radios = click.prompt('Choose radio', type=click.Choice(active_radios), show_choices=True),
|
||||||
invalid_radios = set(radios) - active_radios
|
invalid_radios = set(radios) - active_radios
|
||||||
if len(invalid_radios):
|
if invalid_radios:
|
||||||
raise click.BadParameter("Invalid radio(s): {0}".format(", ".join(invalid_radios)))
|
raise click.BadParameter("Invalid radio(s): {0}".format(", ".join(invalid_radios)))
|
||||||
disable_radios = set(radios) & active_radios
|
disable_radios = set(radios) & active_radios
|
||||||
for alias in disable_radios:
|
for alias in disable_radios:
|
||||||
|
|||||||
Reference in New Issue
Block a user