Propagate change from the moulinette (no more msignals madness)

This commit is contained in:
Alexandre Aubin 2021-07-09 20:43:52 +02:00
parent c3e7587766
commit 5e2478d309
6 changed files with 18 additions and 17 deletions

View file

@ -33,6 +33,7 @@
# Global parameters # # Global parameters #
############################# #############################
_global: _global:
name: yunohost.admin
authentication: authentication:
api: ldap_admin api: ldap_admin
cli: null cli: null

View file

@ -36,7 +36,7 @@ import urllib.parse
import tempfile import tempfile
from collections import OrderedDict from collections import OrderedDict
from moulinette import msignals, m18n, msettings from moulinette import prompt, m18n, msettings
from moulinette.core import MoulinetteError from moulinette.core import MoulinetteError
from moulinette.utils.log import getActionLogger from moulinette.utils.log import getActionLogger
from moulinette.utils.network import download_json from moulinette.utils.network import download_json
@ -825,7 +825,7 @@ def app_install(
return return
if confirm in ["danger", "thirdparty"]: if confirm in ["danger", "thirdparty"]:
answer = msignals.prompt( answer = prompt(
m18n.n("confirm_app_install_" + confirm, answers="Yes, I understand"), m18n.n("confirm_app_install_" + confirm, answers="Yes, I understand"),
color="red", color="red",
) )
@ -833,7 +833,7 @@ def app_install(
raise YunohostError("aborting") raise YunohostError("aborting")
else: else:
answer = msignals.prompt( answer = prompt(
m18n.n("confirm_app_install_" + confirm, answers="Y/N"), color="yellow" m18n.n("confirm_app_install_" + confirm, answers="Y/N"), color="yellow"
) )
if answer.upper() != "Y": if answer.upper() != "Y":
@ -2729,7 +2729,7 @@ class YunoHostArgumentFormatParser(object):
) )
try: try:
question.value = msignals.prompt( question.value = prompt(
text_for_user_input_in_cli, self.hide_user_input_in_prompt text_for_user_input_in_cli, self.hide_user_input_in_prompt
) )
except NotImplementedError: except NotImplementedError:

View file

@ -38,7 +38,7 @@ from collections import OrderedDict
from functools import reduce from functools import reduce
from packaging import version from packaging import version
from moulinette import msignals, m18n, msettings from moulinette import prompt, m18n, msettings
from moulinette.utils import filesystem from moulinette.utils import filesystem
from moulinette.utils.log import getActionLogger from moulinette.utils.log import getActionLogger
from moulinette.utils.filesystem import read_file, mkdir, write_to_yaml, read_yaml from moulinette.utils.filesystem import read_file, mkdir, write_to_yaml, read_yaml
@ -1839,7 +1839,7 @@ class BackupMethod(object):
# Ask confirmation for copying # Ask confirmation for copying
if size > MB_ALLOWED_TO_ORGANIZE: if size > MB_ALLOWED_TO_ORGANIZE:
try: try:
i = msignals.prompt( i = prompt(
m18n.n( m18n.n(
"backup_ask_for_copying_if_needed", "backup_ask_for_copying_if_needed",
answers="y/N", answers="y/N",
@ -2343,7 +2343,7 @@ def backup_restore(name, system=[], apps=[], force=False):
if not force: if not force:
try: try:
# Ask confirmation for restoring # Ask confirmation for restoring
i = msignals.prompt( i = prompt(
m18n.n("restore_confirm_yunohost_installed", answers="y/N") m18n.n("restore_confirm_yunohost_installed", answers="y/N")
) )
except NotImplemented: except NotImplemented:

View file

@ -26,7 +26,7 @@
import os import os
import re import re
from moulinette import m18n, msettings, msignals from moulinette import m18n, msettings, prompt
from moulinette.core import MoulinetteError from moulinette.core import MoulinetteError
from yunohost.utils.error import YunohostError, YunohostValidationError from yunohost.utils.error import YunohostError, YunohostValidationError
from moulinette.utils.log import getActionLogger from moulinette.utils.log import getActionLogger
@ -237,7 +237,7 @@ def domain_remove(operation_logger, domain, remove_apps=False, force=False):
if apps_on_that_domain: if apps_on_that_domain:
if remove_apps: if remove_apps:
if msettings.get("interface") == "cli" and not force: if msettings.get("interface") == "cli" and not force:
answer = msignals.prompt( answer = prompt(
m18n.n( m18n.n(
"domain_remove_confirm_apps_removal", "domain_remove_confirm_apps_removal",
apps="\n".join([x[1] for x in apps_on_that_domain]), apps="\n".join([x[1] for x in apps_on_that_domain]),

View file

@ -30,7 +30,7 @@ import time
from importlib import import_module from importlib import import_module
from packaging import version from packaging import version
from moulinette import msignals, m18n from moulinette import prompt, m18n
from moulinette.utils.log import getActionLogger from moulinette.utils.log import getActionLogger
from moulinette.utils.process import check_output, call_async_output from moulinette.utils.process import check_output, call_async_output
from moulinette.utils.filesystem import read_yaml, write_to_yaml from moulinette.utils.filesystem import read_yaml, write_to_yaml
@ -692,7 +692,7 @@ def tools_shutdown(operation_logger, force=False):
if not shutdown: if not shutdown:
try: try:
# Ask confirmation for server shutdown # Ask confirmation for server shutdown
i = msignals.prompt(m18n.n("server_shutdown_confirm", answers="y/N")) i = prompt(m18n.n("server_shutdown_confirm", answers="y/N"))
except NotImplemented: except NotImplemented:
pass pass
else: else:
@ -711,7 +711,7 @@ def tools_reboot(operation_logger, force=False):
if not reboot: if not reboot:
try: try:
# Ask confirmation for restoring # Ask confirmation for restoring
i = msignals.prompt(m18n.n("server_reboot_confirm", answers="y/N")) i = prompt(m18n.n("server_reboot_confirm", answers="y/N"))
except NotImplemented: except NotImplemented:
pass pass
else: else:

View file

@ -33,7 +33,7 @@ import string
import subprocess import subprocess
import copy import copy
from moulinette import msignals, msettings, m18n from moulinette import prompt, display, msettings, m18n
from moulinette.utils.log import getActionLogger from moulinette.utils.log import getActionLogger
from moulinette.utils.process import check_output from moulinette.utils.process import check_output
@ -123,12 +123,12 @@ def user_create(
) )
else: else:
# On affiche les differents domaines possibles # On affiche les differents domaines possibles
msignals.display(m18n.n("domains_available")) display(m18n.n("domains_available"))
for domain in domain_list()["domains"]: for domain in domain_list()["domains"]:
msignals.display("- {}".format(domain)) display("- {}".format(domain))
maindomain = _get_maindomain() maindomain = _get_maindomain()
domain = msignals.prompt( domain = prompt(
m18n.n("ask_user_domain") + " (default: %s)" % maindomain m18n.n("ask_user_domain") + " (default: %s)" % maindomain
) )
if not domain: if not domain:
@ -380,7 +380,7 @@ def user_update(
# without a specified value, change_password will be set to the const 0. # without a specified value, change_password will be set to the const 0.
# In this case we prompt for the new password. # In this case we prompt for the new password.
if msettings.get("interface") == "cli" and not change_password: if msettings.get("interface") == "cli" and not change_password:
change_password = msignals.prompt(m18n.n("ask_password"), True, True) change_password = prompt(m18n.n("ask_password"), True, True)
# Ensure sufficiently complex password # Ensure sufficiently complex password
assert_password_is_strong_enough("user", change_password) assert_password_is_strong_enough("user", change_password)