Fixes

README.ru.md

@@ -72,4 +72,16 @@ sudo dnf install _packages/ispmanager-plugin-nginx_mod_rewrite_plugin-0.0.1-1.el
 
 Если вдруг установка провалилась, лог установки можно изучить здесь - `/usr/local/mgr5/var/nginx-mod-rewrite-plugin.log`
 
-Если все прошло успешно, то необходимо перейти
+Если все прошло успешно, то необходимо перейти в раздел "Модули":
+
+![Модули](img/isp_mod_rewrite1.png)
+
+Найти модуль с названием "Модуль nginx mod_rewrite" и нажать на значек настройки модуля. На изображении выше обведены синим контуром.
+
+Далее откроется окно сборки модуля:
+
+![Сборка](img/isp_mod_rewrite2.png)
+
+Для отмены сборки, нажмите кнопку "Отменить сборку". Для сборки и установки хука пересборки при обновлении nginx необходимо нажать "Собать модуль и установить хук на пересборку".
+
+Процесс сборки можно будет изучить в логе: `/usr/local/mgr5/var/nginx-mod-rewrite-plugin.log.

usr/local/mgr5/addon/nginx_mod_rewrite_plugin.py

@@ -68,22 +68,42 @@ def Handle(root):
     result = True
     result_error = CheckModuleExists()
     if result_error is None:
+        for parent in root.iter():
+            for child in list(parent):
+                if child.tag == 'textdata' and child.get('name') == 'nginx_mod_rewrite_plugin_msg_info':
+                    parent.remove(child)
         result_error = InstallModule()
-    result = result_error is None
+        result = result_error is None
 
-    if not result:
+        if not result:
-        # Формирование сообщения об ошибке согласно структуре в metadata
+            # Формирование сообщения об ошибке согласно структуре в metadata
-        error = etree.SubElement(root, "error")
+            error = etree.SubElement(root, "error")
-        error.set('code', '1')
+            error.set('code', '1')
-        error.set('type', 'nginx_mod_rewrite_plugin')
+            error.set('type', 'nginx_mod_rewrite_plugin')
-        error.set('object', 'wrong_value')
+            error.set('object', 'wrong_value')
-        param = etree.SubElement(error, 'param')
+            param = etree.SubElement(error, 'param')
-        param.text = 'wrong_value'
+            param.text = 'wrong_value'
-        param.set('name', 'object')
+            param.set('name', 'object')
-        param.set('type', 'msg')
+            param.set('type', 'msg')
-        param = etree.SubElement(error, 'param')
+            param = etree.SubElement(error, 'param')
-        param.set('name', 'value')
+            param.set('name', 'value')
-        param.text = result_error
+            param.text = result_error
+        else:
+            # Add <ok/> element to root
+            etree.SubElement(root, 'ok')
+    else:
+        # Remove textdata element with name="nginx_mod_rewrite_plugin_msg"
+        for parent in root.iter():
+            for child in list(parent):
+                if child.tag == 'textdata' and child.get('name') == 'nginx_mod_rewrite_plugin_msg':
+                    parent.remove(child)
+        # Find the messages element with name "nginx_mod_rewrite_plugin.settings"
+        messages_elem = root.find(".//messages[@name='nginx_mod_rewrite_plugin.settings']")
+        if messages_elem is not None:
+            # Find the msg element with name "msg_create" and change its text
+            msg_create_elem = messages_elem.find(".//msg[@name='msg_create']")
+            if msg_create_elem is not None:
+                msg_create_elem.text = 'Ничего не делать'
 
 
 if __name__ == "__main__":
@@ -100,6 +120,28 @@ if __name__ == "__main__":
     if correct_function:
         if os.getenv("PARAM_clicked_button") == "ok":
             Handle(root)
+        else:
+            result_error = CheckModuleExists()
+            if result_error is None:
+                for parent in root.iter():
+                    for child in list(parent):
+                        if child.tag == 'textdata' and child.get('name') == 'nginx_mod_rewrite_plugin_msg_info':
+                            parent.remove(child)
+            else:
+                # Remove textdata element with name="nginx_mod_rewrite_plugin_msg"
+                for parent in root.iter():
+                    for child in list(parent):
+                        if child.tag == 'textdata' and child.get('name') == 'nginx_mod_rewrite_plugin_msg':
+                            parent.remove(child)
+
+                # Find the messages element with name "nginx_mod_rewrite_plugin.settings"
+                messages_elem = root.find(".//messages[@name='nginx_mod_rewrite_plugin.settings']")
+                if messages_elem is not None:
+                    # Find the msg element with name "msg_create" and change its text
+                    msg_create_elem = messages_elem.find(".//msg[@name='msg_create']")
+                    if msg_create_elem is not None:
+                        msg_create_elem.text = 'Ничего не делать'
+
 
     etree.dump(root)
     quit(0)

usr/local/mgr5/etc/xml/ispmgr_mod_nginx_mod_rewrite_plugin.xml

@@ -9,15 +9,16 @@
         <func name="nginx_mod_rewrite_plugin.settings" />
     </handler>
     <metadata name="nginx_mod_rewrite_plugin.settings" type="form" mgr="ispmgr">
-    <form>
+        <form>
-       	<field name="nginx_mod_rewrite_plugin_field" noname="yes" fullwidth="yes">
+           	<field name="nginx_mod_rewrite_plugin_field" noname="yes" fullwidth="yes">
-           	<textdata name="nginx_mod_rewrite_plugin_msg" type="msg" />
+               	<textdata name="nginx_mod_rewrite_plugin_msg" type="banner" status="warning"/>
-       	</field>
+                <textdata name="nginx_mod_rewrite_plugin_msg_info" type="banner" status="info"/>
-    </form>
+           	</field>
-    <buttons>
+        </form>
-        <button name="ok" type="ok" />
+        <buttons>
-       	<button name="cancel" type="cancel" />
+            <button name="ok" type="ok" />
-    </buttons>
+           	<button name="cancel" type="cancel" />
+        </buttons>
     </metadata>
     <metadata name="site.edit" type="form">
         <form>
@@ -66,6 +67,7 @@
     	<messages name="nginx_mod_rewrite_plugin.settings">
             <msg name="title">Установка модуля для nginx</msg>
             <msg name="nginx_mod_rewrite_plugin_msg">Данный плагин установит ряд пакетов для сборки модуля nginx</msg>
+            <msg name="nginx_mod_rewrite_plugin_msg_info">Модуль уже установлен</msg>
             <msg name="msg_create">Собрать модуль и установить хук на пересборку</msg>
             <msg name="msg_back">&lt;&lt; Отменить сборку</msg>
     	</messages>