[omemo] Don’t use GajimPluginConfigDialog

It will be removed in the future

omemo/gtk/config.py

@@ -18,43 +18,58 @@
 
 import logging
 
+from gi.repository import Gtk
+from gi.repository import Gdk
+
 from gajim.common import app
-from gajim.plugins.gui import GajimPluginConfigDialog
 from gajim.plugins.helpers import get_builder
+from gajim.plugins.plugins_i18n import _
 
 from omemo.backend.util import get_fingerprint
 
 log = logging.getLogger('gajim.p.omemo')
 
 
-class OMEMOConfigDialog(GajimPluginConfigDialog):
-    def init(self):
-        # pylint: disable=attribute-defined-outside-init
-        path = self.plugin.local_file_path('gtk/config.ui')
+class OMEMOConfigDialog(Gtk.ApplicationWindow):
+    def __init__(self, plugin, transient):
+        Gtk.ApplicationWindow.__init__(self)
+        self.set_application(app.app)
+        self.set_show_menubar(False)
+        self.set_title(_('OMEMO Settings'))
+        self.set_transient_for(transient)
+        self.set_default_size(400, 400)
+        self.set_type_hint(Gdk.WindowTypeHint.DIALOG)
+        self.set_destroy_with_parent(True)
+
+        self._plugin = plugin
+
+        path = self._plugin.local_file_path('gtk/config.ui')
         self._ui = get_builder(path)
 
-        image_path = self.plugin.local_file_path('omemo.png')
+        image_path = self._plugin.local_file_path('omemo.png')
         self._ui.image.set_from_file(image_path)
 
         try:
-            self.disabled_accounts = self.plugin.config['DISABLED_ACCOUNTS']
+            self.disabled_accounts = self._plugin.config['DISABLED_ACCOUNTS']
         except KeyError:
-            self.plugin.config['DISABLED_ACCOUNTS'] = []
-            self.disabled_accounts = self.plugin.config['DISABLED_ACCOUNTS']
+            self._plugin.config['DISABLED_ACCOUNTS'] = []
+            self.disabled_accounts = self._plugin.config['DISABLED_ACCOUNTS']
 
-        box = self.get_content_area()
+        box = Gtk.Box()
         box.pack_start(self._ui.notebook1, True, True, 0)
 
+        self.add(box)
+
         self._ui.connect_signals(self)
+        self.show_all()
 
         self.plugin_active = False
-
-    def on_run(self):
         for plugin in app.plugin_manager.active_plugins:
             log.debug(type(plugin))
             if type(plugin).__name__ == 'OmemoPlugin':
                 self.plugin_active = True
                 break
+
         self.update_account_store()
         self.update_account_combobox()
         self.update_disabled_account_view()
@@ -102,7 +117,7 @@ class OMEMOConfigDialog(GajimPluginConfigDialog):
                 self._ui.account_store.append(row=(account[0],))
                 self.disabled_accounts.remove(account[0])
         self.update_disabled_account_view()
-        self.plugin.config['DISABLED_ACCOUNTS'] = self.disabled_accounts
+        self._plugin.config['DISABLED_ACCOUNTS'] = self.disabled_accounts
         self.update_account_combobox()
 
     def disable_accounts_btn_clicked(self, _button, *args):
@@ -116,7 +131,7 @@ class OMEMOConfigDialog(GajimPluginConfigDialog):
                 self.disabled_accounts.append(account[0])
                 self._ui.account_store.remove(it)
         self.update_disabled_account_view()
-        self.plugin.config['DISABLED_ACCOUNTS'] = self.disabled_accounts
+        self._plugin.config['DISABLED_ACCOUNTS'] = self.disabled_accounts
         self.update_account_combobox()
 
     def cleardevice_button_clicked_cb(self, button, *args):
@@ -129,7 +144,7 @@ class OMEMOConfigDialog(GajimPluginConfigDialog):
         self.update_context_list()
 
     def _on_blind_trust(self, button):
-        self.plugin.config['BLIND_TRUST'] = button.get_active()
+        self._plugin.config['BLIND_TRUST'] = button.get_active()
 
     def update_context_list(self):
         self._ui.deviceid_store.clear()
@@ -151,7 +166,7 @@ class OMEMOConfigDialog(GajimPluginConfigDialog):
             self._ui.cleardevice_button.set_sensitive(True)
 
         # Set FPR Label and DeviceID
-        omemo = self.plugin.get_omemo(account)
+        omemo = self._plugin.get_omemo(account)
         self._ui.ID.set_markup('<tt>%s</tt>' % omemo.backend.own_device)
 
         identity_key = omemo.backend.storage.getIdentityKeyPair()
@@ -165,4 +180,4 @@ class OMEMOConfigDialog(GajimPluginConfigDialog):
 
     def update_settings(self):
         self._ui.blind_trust_checkbutton.set_active(
-            self.plugin.config['BLIND_TRUST'])
\ No newline at end of file
+            self._plugin.config['BLIND_TRUST'])

omemo/plugin.py

@@ -21,6 +21,7 @@ import binascii
 import threading
 from enum import IntEnum, unique
 from pathlib import Path
+from functools import partial
 
 from gi.repository import GLib
 from gi.repository import Gtk
@@ -95,7 +96,8 @@ class OmemoPlugin(GajimPlugin):
             'room-joined': (ged.GUI1, self._on_muc_joined),
         }
         self.modules = [omemo]
-        self.config_dialog = OMEMOConfigDialog(self)
+
+        self.config_dialog = partial(OMEMOConfigDialog, self)
         self.gui_extension_points = {
             'encrypt' + self.encryption_name: (self._encrypt_message, None),
             'gc_encrypt' + self.encryption_name: (