Commits

commits@gajim.org

17632 discussions

[Git][gajim/gajim][master] 2 commits: Fix deprecation warning
by Philipp Hörist 17 Aug '18

17 Aug '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: 73928836 by Philipp Hörist at 2018-08-17T22:10:10Z Fix deprecation warning - - - - - 042d8aa6 by Philipp Hörist at 2018-08-17T22:13:51Z Fix deprecation warning only keyword args should be used - - - - - 5 changed files: - gajim/accounts_window.py - gajim/chat_control_base.py - gajim/filetransfers_window.py - gajim/gtk/start_chat.py - gajim/gtk/themes.py Changes: ===================================== gajim/accounts_window.py ===================================== --- a/gajim/accounts_window.py +++ b/gajim/accounts_window.py @@ -313,7 +313,7 @@ class AddAccount(Gtk.Box): self.account = None - self.label = Gtk.Label(_('Add Account…')) + self.label = Gtk.Label(label=_('Add Account…')) self.label.set_halign(Gtk.Align.START) self.label.set_hexpand(True) @@ -342,7 +342,7 @@ class Preferences(Gtk.Box): self.parent = parent self.account = None - self.label = Gtk.Label(_('Preferences')) + self.label = Gtk.Label(label=_('Preferences')) self.label.set_halign(Gtk.Align.START) self.label.set_hexpand(True) ===================================== gajim/chat_control_base.py ===================================== --- a/gajim/chat_control_base.py +++ b/gajim/chat_control_base.py @@ -75,7 +75,7 @@ class ChatControlBase(MessageControl, ChatCommandProcessor, CommandTools): A base class containing a banner, ConversationTextview, MessageTextView """ - keymap = Gdk.Keymap.get_default() + keymap = Gdk.Keymap.get_for_display(Gdk.Display.get_default()) try: keycode_c = keymap.get_entries_for_keyval(Gdk.KEY_c)[1][0].keycode except TypeError: ===================================== gajim/filetransfers_window.py ===================================== --- a/gajim/filetransfers_window.py +++ b/gajim/filetransfers_window.py @@ -1071,7 +1071,7 @@ class FileRow(Gtk.ListBoxRow): def __init__(self, path): Gtk.ListBoxRow.__init__(self) self.path = Path(path) - label = Gtk.Label(self.path.name) + label = Gtk.Label(label=self.path.name) label.set_ellipsize(Pango.EllipsizeMode.END) label.set_xalign(0) self.add(label) ===================================== gajim/gtk/start_chat.py ===================================== --- a/gajim/gtk/start_chat.py +++ b/gajim/gtk/start_chat.py @@ -297,7 +297,7 @@ class ContactRow(Gtk.Grid): else: self.name = _('New Contact') - self.name_label = Gtk.Label(self.name) + self.name_label = Gtk.Label(label=self.name) self.name_label.set_ellipsize(Pango.EllipsizeMode.END) self.name_label.set_xalign(0) self.name_label.set_width_chars(25) @@ -310,7 +310,7 @@ class ContactRow(Gtk.Grid): self.name_label.get_style_context().add_class(css_class) middle_box.add(self.name_label) - self.jid_label = Gtk.Label(jid) + self.jid_label = Gtk.Label(label=jid) self.jid_label.set_ellipsize(Pango.EllipsizeMode.END) self.jid_label.set_xalign(0) self.jid_label.set_width_chars(25) @@ -320,7 +320,7 @@ class ContactRow(Gtk.Grid): self.add(middle_box) if show_account: - account_label = Gtk.Label(self.account_label) + account_label = Gtk.Label(label=self.account_label) account_label.set_halign(Gtk.Align.START) account_label.set_valign(Gtk.Align.START) ===================================== gajim/gtk/themes.py ===================================== --- a/gajim/gtk/themes.py +++ b/gajim/gtk/themes.py @@ -409,7 +409,7 @@ class ChooseOption(Gtk.ListBoxRow): def __init__(self, option): Gtk.ListBoxRow.__init__(self) self.option = option - label = Gtk.Label(option.label) + label = Gtk.Label(label=option.label) label.set_xalign(0) self.add(label) self.show_all() View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/3392f663dedfe38530ec81ee3b7b60917… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/3392f663dedfe38530ec81ee3b7b60917… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Update README.md
by Philipp Hörist 17 Aug '18

17 Aug '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: 3392f663 by Philipp Hörist at 2018-08-17T20:30:38Z Update README.md - - - - - 1 changed file: - README.md Changes: ===================================== README.md ===================================== --- a/README.md +++ b/README.md @@ -20,8 +20,7 @@ - For zeroconf (bonjour) you need python3-dbus - gir1.2-gspell-1 and hunspell-LANG where lang is your locale eg. en, fr etc - gir1.2-secret-1 for GNOME Keyring or KDE support as password storage -- D-Bus running to have gajim-remote working. Some distributions split dbus-x11, which is needed for dbus to work with Gajim. Version >= 0.80 is required. -- python3-dbus bindings (>=1.2.0) +- D-Bus running to have gajim-remote working - gir1.2-farstream-0.2, gir1.2-gstreamer-1.0 and gir1.2-gst-plugins-base-1.0 for audio and video calls - gir1.2-gupnpigd-1.0 for better NAT traversing - gir1.2-networkmanager-1.0 for network lose detection View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/3392f663dedfe38530ec81ee3b7b60917d… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/3392f663dedfe38530ec81ee3b7b60917d… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] 3 commits: Port remote control to GDBus
by Philipp Hörist 17 Aug '18

17 Aug '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: 7447f33c by André Apitzsch at 2018-08-17T20:06:02Z Port remote control to GDBus - - - - - f5a52c52 by André Apitzsch at 2018-08-17T20:06:02Z Remote control: remove undefined variable - - - - - 36efb61a by André Apitzsch at 2018-08-17T20:06:02Z Remote control: fix some errors - - - - - 3 changed files: - gajim/gajim_remote.py - gajim/gui_interface.py - gajim/remote_control.py Changes: ===================================== gajim/gajim_remote.py ===================================== --- a/gajim/gajim_remote.py +++ b/gajim/gajim_remote.py @@ -60,7 +60,7 @@ except Exception: OBJ_PATH = '/org/gajim/dbus/RemoteObject' INTERFACE = 'org.gajim.dbus.RemoteInterface' -SERVICE = 'org.gajim.dbus' +SERVICE = 'org.gajim.Gajim' BASENAME = 'gajim-remote' class GajimRemote: ===================================== gajim/gui_interface.py ===================================== --- a/gajim/gui_interface.py +++ b/gajim/gui_interface.py @@ -2583,7 +2583,7 @@ class Interface: if app.config.get('remote_control'): try: from gajim import remote_control - self.remote_ctrl = remote_control.Remote() + remote_control.GajimRemote() except Exception: pass GLib.timeout_add_seconds(5, remote_init) ===================================== gajim/remote_control.py ===================================== --- a/gajim/remote_control.py +++ b/gajim/remote_control.py @@ -26,11 +26,13 @@ ## along with Gajim. If not, see <http://www.gnu.org/licenses/>. ## -from gi.repository import GLib import os import base64 import mimetypes +from gi.repository import GLib +from gi.repository import Gio + from gajim.common import app from gajim.common import helpers from gajim.gtk import AddNewContactWindow @@ -39,71 +41,264 @@ from gajim.common.connection_handlers_events import MessageOutgoingEvent from gajim.common.connection_handlers_events import GcMessageOutgoingEvent -from gajim.common import dbus_support -if dbus_support.supported: - import dbus - if dbus_support: - import dbus.service - -INTERFACE = 'org.gajim.dbus.RemoteInterface' -OBJ_PATH = '/org/gajim/dbus/RemoteObject' -SERVICE = 'org.gajim.dbus' - -# type mapping - -# in most cases it is a utf-8 string -DBUS_STRING = dbus.String - -# general type (for use in dicts, where all values should have the same type) -DBUS_BOOLEAN = dbus.Boolean -DBUS_DOUBLE = dbus.Double -DBUS_INT32 = dbus.Int32 -# dictionary with string key and binary value -DBUS_DICT_SV = lambda : dbus.Dictionary({}, signature="sv") -# dictionary with string key and value -DBUS_DICT_SS = lambda : dbus.Dictionary({}, signature="ss") -# empty type (there is no equivalent of None on D-Bus, but historically gajim -# used 0 instead) -DBUS_NONE = lambda : dbus.Int32(0) - def get_dbus_struct(obj): """ Recursively go through all the items and replace them with their casted dbus equivalents """ if obj is None: - return DBUS_NONE() + return None if isinstance(obj, str): - return DBUS_STRING(obj) + return GLib.Variant('s', obj) if isinstance(obj, int): - return DBUS_INT32(obj) + return GLib.Variant('i', obj) if isinstance(obj, float): - return DBUS_DOUBLE(obj) + return GLib.Variant('d', obj) if isinstance(obj, bool): - return DBUS_BOOLEAN(obj) + return GLib.Variant('b', obj) if isinstance(obj, (list, tuple)): - result = dbus.Array([get_dbus_struct(i) for i in obj], - signature='v') - if result == []: - return DBUS_NONE() + lst = [get_dbus_struct(i) for i in obj if i is not None] + result = GLib.Variant('av', lst) return result if isinstance(obj, dict): - result = DBUS_DICT_SV() + result = GLib.VariantDict() for key, value in obj.items(): - result[DBUS_STRING(key)] = get_dbus_struct(value) - if result == {}: - return DBUS_NONE() - return result + result.insert_value(key, get_dbus_struct(value)) + return result.end() # unknown type - return DBUS_NONE() + return None -class Remote: - def __init__(self): - self.signal_object = None - session_bus = dbus_support.session_bus.SessionBus() - bus_name = dbus.service.BusName(SERVICE, bus=session_bus) - self.signal_object = SignalObject(bus_name) +class Server: + def __init__(self, con, path): + method_outargs = {} + method_inargs = {} + for interface in Gio.DBusNodeInfo.new_for_xml(self.__doc__).interfaces: + + for method in interface.methods: + method_outargs[method.name] = '(' + ''.join( + [arg.signature for arg in method.out_args]) + ')' + method_inargs[method.name] = tuple( + arg.signature for arg in method.in_args) + + con.register_object( + object_path=path, + interface_info=interface, + method_call_closure=self.on_method_call) + + self.method_inargs = method_inargs + self.method_outargs = method_outargs + + def on_method_call(self, connection, sender, object_path, interface_name, + method_name, parameters, invocation): + + args = list(parameters.unpack()) + for i, sig in enumerate(self.method_inargs[method_name]): + if sig is 'h': + msg = invocation.get_message() + fd_list = msg.get_unix_fd_list() + args[i] = fd_list.get(args[i]) + + result = getattr(self, method_name)(*args) + + # out_args is atleast (signature1). We therefore always wrap the result + # as a tuple. Refer to https://bugzilla.gnome.org/show_bug.cgi?id=765603 + result = (result, ) + + out_args = self.method_outargs[method_name] + if out_args != '()': + variant = GLib.Variant(out_args, result) + invocation.return_value(variant) + else: + invocation.return_value(None) + + +class GajimRemote(Server): + ''' + <!DOCTYPE node PUBLIC '-//freedesktop//DTD D-BUS Object Introspection 1.0//EN' + 'http://www.freedesktop.org/standards/dbus/1.0/introspect.dtd'> + <node> + <interface name='org.freedesktop.DBus.Introspectable'> + <method name='Introspect'> + <arg name='data' direction='out' type='s'/> + </method> + </interface> + <interface name='org.gajim.dbus.RemoteInterface'> + <method name='account_info'> + <arg name='account' type='s' /> + <arg direction='out' type='a{ss}' /> + </method> + <method name='add_contact'> + <arg name='jid' type='s' /> + <arg name='account' type='s' /> + <arg direction='out' type='b' /> + </method> + <method name='change_avatar'> + <arg name='picture' type='s' /> + <arg name='account' type='s' /> + </method> + <method name='change_status'> + <arg name='status' type='s' /> + <arg name='message' type='s' /> + <arg name='account' type='s' /> + <arg direction='out' type='b' /> + </method> + <method name='get_status'> + <arg name='account' type='s' /> + <arg direction='out' type='s' /> + </method> + <method name='get_status_message'> + <arg name='account' type='s' /> + <arg direction='out' type='s' /> + </method> + <method name='get_unread_msgs_number'> + <arg direction='out' type='s' /> + </method> + <method name='join_room'> + <arg name='room_jid' type='s' /> + <arg name='nick' type='s' /> + <arg name='password' type='s' /> + <arg name='account' type='s' /> + </method> + <method name='list_accounts'> + <arg direction='out' type='as' /> + </method> + <method name='list_contacts'> + <arg name='account' type='s' /> + <arg direction='out' type='aa{sv}' /> + </method> + <method name='open_chat'> + <arg name='jid' type='s' /> + <arg name='account' type='s' /> + <arg name='message' type='s' /> + <arg direction='out' type='b' /> + </method> + <method name='prefs_del'> + <arg name='key' type='s' /> + <arg direction='out' type='b' /> + </method> + <method name='prefs_list'> + <arg direction='out' type='a{ss}' /> + </method> + <method name='prefs_put'> + <arg name='key' type='s' /> + <arg direction='out' type='b' /> + </method> + <method name='prefs_store'> + <arg direction='out' type='b' /> + </method> + <method name='remove_contact'> + <arg name='jid' type='s' /> + <arg name='account' type='s' /> + <arg direction='out' type='b' /> + </method> + <method name='send_chat_message'> + <arg name='jid' type='s' /> + <arg name='message' type='s' /> + <arg name='keyID' type='s' /> + <arg name='account' type='s' /> + <arg direction='out' type='b' /> + </method> + <method name='send_file'> + <arg name='file_path' type='s' /> + <arg name='jid' type='s' /> + <arg name='account' type='s' /> + <arg direction='out' type='b' /> + </method> + <method name='send_groupchat_message'> + <arg name='room_jid' type='s' /> + <arg name='message' type='s' /> + <arg name='account' type='s' /> + <arg direction='out' type='b' /> + </method> + <method name='send_single_message'> + <arg name='jid' type='s' /> + <arg name='subject' type='s' /> + <arg name='message' type='s' /> + <arg name='keyID' type='s' /> + <arg name='account' type='s' /> + <arg direction='out' type='b' /> + </method> + <method name='send_xml'> + <arg name='xml' type='s' /> + <arg name='account' type='s' /> + </method> + <method name='set_priority'> + <arg name='prio' type='s' /> + <arg name='account' type='s' /> + </method> + <method name='show_next_pending_event' /> + <method name='show_roster' /> + <method name='start_chat'> + <arg name='account' type='s' /> + <arg direction='out' type='b' /> + </method> + <method name='toggle_ipython' /> + <method name='toggle_roster_appearance' /> + <signal name='AccountPresence'> + <arg type='av' /> + </signal> + <signal name='ChatState'> + <arg type='av' /> + </signal> + <signal name='ContactAbsence'> + <arg type='av' /> + </signal> + <signal name='ContactPresence'> + <arg type='av' /> + </signal> + <signal name='ContactStatus'> + <arg type='av' /> + </signal> + <signal name='EntityTime'> + <arg type='av' /> + </signal> + <signal name='GCMessage'> + <arg type='av' /> + </signal> + <signal name='GCPresence'> + <arg type='av' /> + </signal> + <signal name='MessageSent'> + <arg type='av' /> + </signal> + <signal name='NewAccount'> + <arg type='av' /> + </signal> + <signal name='NewMessage'> + <arg type='av' /> + </signal> + <signal name='OsInfo'> + <arg type='av' /> + </signal> + <signal name='Roster'> + <arg type='av' /> + </signal> + <signal name='RosterInfo'> + <arg type='av' /> + </signal> + <signal name='Subscribe'> + <arg type='av' /> + </signal> + <signal name='Subscribed'> + <arg type='av' /> + </signal> + <signal name='Unsubscribed'> + <arg type='av' /> + </signal> + <signal name='VcardInfo'> + <arg type='av' /> + </signal> + </interface> + </node> + ''' + + def __init__(self): + self.con = Gio.bus_get_sync(Gio.BusType.SESSION, None) + Gio.bus_own_name_on_connection(self.con, 'org.gajim.Gajim', + Gio.BusNameOwnerFlags.NONE, None, None) + super().__init__(self.con, '/org/gajim/dbus/RemoteObject') + self.first_show = True app.ged.register_event_handler('version-result-received', ged.POSTGUI, self.on_os_info) @@ -140,7 +335,7 @@ class Remote: try: chatstate = obj.chatstate except AttributeError: - chatstate = "" + chatstate = '' self.raise_signal('MessageSent', (obj.conn.name, [ obj.jid, obj.message, obj.keyID, chatstate])) @@ -160,15 +355,15 @@ class Remote: obj.sub, obj.ask, obj.groups])) def on_presence_received(self, obj): - event = None if obj.old_show < 2 and obj.new_show > 1: event = 'ContactPresence' elif obj.old_show > 1 and obj.new_show < 2: event = 'ContactAbsence' elif obj.new_show > 1: event = 'ContactStatus' - if event: - self.raise_signal(event, (obj.conn.name, [obj.jid, obj.show, + else: + return + self.raise_signal(event, (obj.conn.name, [obj.jid, obj.show, obj.status, obj.resource, obj.prio, obj.keyID, obj.timestamp, obj.contact_nickname])) @@ -200,132 +395,35 @@ class Remote: def on_vcard_received(self, obj): self.raise_signal('VcardInfo', (obj.conn.name, obj.vcard_dict)) - def raise_signal(self, signal, arg): - if self.signal_object: - try: - getattr(self.signal_object, signal)(get_dbus_struct(arg)) - except UnicodeDecodeError: - pass # ignore error when we fail to announce on dbus - - -class SignalObject(dbus.service.Object): - """ - Local object definition for /org/gajim/dbus/RemoteObject - - This docstring is not be visible, because the clients can access only the - remote object. - """ - - def __init__(self, bus_name): - self.first_show = True - self.vcard_account = None - - # register our dbus API - dbus.service.Object.__init__(self, bus_name, OBJ_PATH) - - @dbus.service.signal(INTERFACE, signature='av') - def Roster(self, account_and_data): - pass + def raise_signal(self, event_name, data): + self.con.emit_signal(None, + '/org/gajim/dbus/RemoteObject', + 'org.gajim.dbus.RemoteInterface', + event_name, + GLib.Variant.new_tuple(get_dbus_struct(data))) - @dbus.service.signal(INTERFACE, signature='av') - def AccountPresence(self, status_and_account): - pass - - @dbus.service.signal(INTERFACE, signature='av') - def ContactPresence(self, account_and_array): - pass - - @dbus.service.signal(INTERFACE, signature='av') - def ContactAbsence(self, account_and_array): - pass - - @dbus.service.signal(INTERFACE, signature='av') - def ContactStatus(self, account_and_array): - pass - - @dbus.service.signal(INTERFACE, signature='av') - def NewMessage(self, account_and_array): - pass - - @dbus.service.signal(INTERFACE, signature='av') - def Subscribe(self, account_and_array): - pass - - @dbus.service.signal(INTERFACE, signature='av') - def Subscribed(self, account_and_array): - pass - - @dbus.service.signal(INTERFACE, signature='av') - def Unsubscribed(self, account_and_jid): - pass - - @dbus.service.signal(INTERFACE, signature='av') - def NewAccount(self, account_and_array): - pass - - @dbus.service.signal(INTERFACE, signature='av') - def VcardInfo(self, account_and_vcard): - pass - - @dbus.service.signal(INTERFACE, signature='av') - def OsInfo(self, account_and_array): - pass - - @dbus.service.signal(INTERFACE, signature='av') - def EntityTime(self, account_and_array): - pass - - @dbus.service.signal(INTERFACE, signature='av') - def GCPresence(self, account_and_array): - pass - - @dbus.service.signal(INTERFACE, signature='av') - def GCMessage(self, account_and_array): - pass - - @dbus.service.signal(INTERFACE, signature='av') - def RosterInfo(self, account_and_array): - pass - - @dbus.service.signal(INTERFACE, signature='av') - def ChatState(self, account_and_array): - pass - - @dbus.service.signal(INTERFACE, signature='av') - def MessageSent(self, account_and_array): - pass - - def raise_signal(self, signal, arg): - """ - Raise a signal, with a single argument of unspecified type Instead of - obj.raise_signal("Foo", bar), use obj.Foo(bar) - """ - getattr(self, signal)(arg) - - @dbus.service.method(INTERFACE, in_signature='s', out_signature='s') def get_status(self, account): """ - Return status (show to be exact) which is the global one unless account is - given + Return status (show to be exact) which is the global one unless + account is given """ if not account: # If user did not ask for account, returns the global status - return DBUS_STRING(helpers.get_global_show()) + return helpers.get_global_show() # return show for the given account index = app.connections[account].connected - return DBUS_STRING(app.SHOW_LIST[index]) + return app.SHOW_LIST[index] - @dbus.service.method(INTERFACE, in_signature='s', out_signature='s') def get_status_message(self, account): """ Return status which is the global one unless account is given """ if not account: # If user did not ask for account, returns the global status - return DBUS_STRING(str(helpers.get_global_status())) + return str(helpers.get_global_status()) # return show for the given account status = app.connections[account].status - return DBUS_STRING(status) + return status def _get_account_and_contact(self, account, jid): """ @@ -339,14 +437,14 @@ class SignalObject(dbus.service.Object): if not account and len(accounts) == 1: account = accounts[0] if account: - if app.connections[account].connected > 1: # account is connected + if app.connections[account].connected > 1: # account is connected connected_account = account - contact = app.contacts.get_contact_with_highest_priority(account, - jid) + contact = app.contacts.get_contact_with_highest_priority( + account, jid) else: for account in accounts: - contact = app.contacts.get_contact_with_highest_priority(account, - jid) + contact = app.contacts.get_contact_with_highest_priority( + account, jid) if contact and app.connections[account].connected > 1: # account is connected connected_account = account @@ -369,50 +467,56 @@ class SignalObject(dbus.service.Object): account = accounts[0] if account: if app.connections[account].connected > 1 and \ - room_jid in app.gc_connected[account] and \ - app.gc_connected[account][room_jid]: + room_jid in app.gc_connected[account] and \ + app.gc_connected[account][room_jid]: # account and groupchat are connected connected_account = account else: for account in accounts: if app.connections[account].connected > 1 and \ - room_jid in app.gc_connected[account] and \ - app.gc_connected[account][room_jid]: + room_jid in app.gc_connected[account] and \ + app.gc_connected[account][room_jid]: # account and groupchat are connected connected_account = account break return connected_account - @dbus.service.method(INTERFACE, in_signature='sss', out_signature='b') def send_file(self, file_path, jid, account): """ Send file, located at 'file_path' to 'jid', using account (optional) 'account' """ jid = self._get_real_jid(jid, account) - connected_account, contact = self._get_account_and_contact(account, jid) + connected_account, contact = self._get_account_and_contact( + account, jid) if connected_account: if file_path.startswith('file://'): - file_path=file_path[7:] - if os.path.isfile(file_path): # is it file? + file_path = file_path[7:] + if os.path.isfile(file_path): # is it file? app.interface.instances['file_transfers'].send_file( - connected_account, contact, file_path) - return DBUS_BOOLEAN(True) - return DBUS_BOOLEAN(False) + connected_account, contact, file_path) + return True + return False - def _send_message(self, jid, message, keyID, account, type_ = 'chat', - subject = None): + def _send_message(self, + jid, + message, + keyID, + account, + type_='chat', + subject=None): """ Can be called from send_chat_message (default when send_message) or send_single_message """ if not jid or not message: - return DBUS_BOOLEAN(False) + return False if not keyID: keyID = '' - connected_account, contact = self._get_account_and_contact(account, jid) + connected_account, contact = self._get_account_and_contact( + account, jid) if connected_account: connection = app.connections[connected_account] sessions = connection.get_sessions(jid) @@ -420,17 +524,24 @@ class SignalObject(dbus.service.Object): session = sessions[0] else: session = connection.make_new_session(jid) - ctrl = app.interface.msg_win_mgr.search_control(jid, - connected_account) + ctrl = app.interface.msg_win_mgr.search_control( + jid, connected_account) if ctrl: ctrl.send_message(message) else: - app.nec.push_outgoing_event(MessageOutgoingEvent(None, account=connected_account, jid=jid, message=message, keyID=keyID, type_=type_, control=ctrl)) + app.nec.push_outgoing_event( + MessageOutgoingEvent( + None, + account=connected_account, + jid=jid, + message=message, + keyID=keyID, + type_=type_, + control=ctrl)) - return DBUS_BOOLEAN(True) - return DBUS_BOOLEAN(False) + return True + return False - @dbus.service.method(INTERFACE, in_signature='ssss', out_signature='b') def send_chat_message(self, jid, message, keyID, account): """ Send chat 'message' to 'jid', using account (optional) 'account'. If keyID @@ -439,44 +550,46 @@ class SignalObject(dbus.service.Object): jid = self._get_real_jid(jid, account) return self._send_message(jid, message, keyID, account) - @dbus.service.method(INTERFACE, in_signature='sssss', out_signature='b') def send_single_message(self, jid, subject, message, keyID, account): """ Send single 'message' to 'jid', using account (optional) 'account'. If keyID is specified, encrypt the message with the pgp key """ jid = self._get_real_jid(jid, account) - return self._send_message(jid, message, keyID, account, 'normal', subject) + return self._send_message(jid, message, keyID, account, 'normal', + subject) - @dbus.service.method(INTERFACE, in_signature='sss', out_signature='b') def send_groupchat_message(self, room_jid, message, account): """ Send 'message' to groupchat 'room_jid', using account (optional) 'account' """ if not room_jid or not message: - return DBUS_BOOLEAN(False) + return False connected_account = self._get_account_for_groupchat(account, room_jid) if connected_account: connection = app.connections[connected_account] - app.nec.push_outgoing_event(GcMessageOutgoingEvent(None, - account=connected_account, jid=room_jid, message=message)) - return DBUS_BOOLEAN(True) - return DBUS_BOOLEAN(False) + app.nec.push_outgoing_event( + GcMessageOutgoingEvent( + None, + account=connected_account, + jid=room_jid, + message=message)) + return True + return False - @dbus.service.method(INTERFACE, in_signature='sss', out_signature='b') def open_chat(self, jid, account, message): """ Shows the tabbed window for new message to 'jid', using account (optional) 'account' """ if not jid: - raise dbus_support.MissingArgument() + raise ValueError('jid is missing') jid = self._get_real_jid(jid, account) try: jid = helpers.parse_jid(jid) except Exception: # Jid is not conform, ignore it - return DBUS_BOOLEAN(False) + return False minimized_control = None if account: @@ -488,7 +601,7 @@ class SignalObject(dbus.service.Object): connected_account = None first_connected_acct = None for acct in accounts: - if app.connections[acct].connected > 1: # account is online + if app.connections[acct].connected > 1: # account is online contact = app.contacts.get_first_contact_from_jid(acct, jid) if app.interface.msg_win_mgr.has_window(jid, acct): connected_account = acct @@ -511,40 +624,39 @@ class SignalObject(dbus.service.Object): connected_account = first_connected_acct if minimized_control: - app.interface.roster.on_groupchat_maximized(None, jid, - connected_account) + app.interface.roster.on_groupchat_maximized( + None, jid, connected_account) if connected_account: app.interface.new_chat_from_jid(connected_account, jid, message) # preserve the 'steal focus preservation' - win = app.interface.msg_win_mgr.get_window(jid, - connected_account).window + win = app.interface.msg_win_mgr.get_window( + jid, connected_account).window if win.get_property('visible'): win.window.present() - return DBUS_BOOLEAN(True) - return DBUS_BOOLEAN(False) + return True + return False - @dbus.service.method(INTERFACE, in_signature='sss', out_signature='b') def change_status(self, status, message, account): """ change_status(status, message, account). Account is optional - if not specified status is changed for all accounts """ - if status not in ('offline', 'online', 'chat', - 'away', 'xa', 'dnd', 'invisible'): + if status not in ('offline', 'online', 'chat', 'away', 'xa', 'dnd', + 'invisible'): status = '' if account: if not status: if account not in app.connections: - return DBUS_BOOLEAN(False) + return False status = app.SHOW_LIST[app.connections[account].connected] GLib.idle_add(app.interface.roster.send_status, account, status, - message) + message) else: # account not specified, so change the status of all accounts for acc in app.contacts.get_accounts(): if not app.config.get_per('accounts', acc, - 'sync_with_global_status'): + 'sync_with_global_status'): continue if status: status_ = status @@ -553,10 +665,9 @@ class SignalObject(dbus.service.Object): continue status_ = app.SHOW_LIST[app.connections[acc].connected] GLib.idle_add(app.interface.roster.send_status, acc, status_, - message) - return DBUS_BOOLEAN(False) + message) + return False - @dbus.service.method(INTERFACE, in_signature='ss', out_signature='') def set_priority(self, prio, account): """ set_priority(prio, account). Account is optional - if not specified @@ -566,23 +677,20 @@ class SignalObject(dbus.service.Object): app.config.set_per('accounts', account, 'priority', prio) show = app.SHOW_LIST[app.connections[account].connected] status = app.connections[account].status - GLib.idle_add(app.connections[account].change_status, show, - status) + GLib.idle_add(app.connections[account].change_status, show, status) else: # account not specified, so change prio of all accounts for acc in app.contacts.get_accounts(): if not app.account_is_connected(acc): continue if not app.config.get_per('accounts', acc, - 'sync_with_global_status'): + 'sync_with_global_status'): continue app.config.set_per('accounts', acc, 'priority', prio) show = app.SHOW_LIST[app.connections[acc].connected] status = app.connections[acc].status - GLib.idle_add(app.connections[acc].change_status, show, - status) + GLib.idle_add(app.connections[acc].change_status, show, status) - @dbus.service.method(INTERFACE, in_signature='', out_signature='') def show_next_pending_event(self): """ Show the window(s) with next pending event in tabbed/group chats @@ -593,46 +701,43 @@ class SignalObject(dbus.service.Object): return app.interface.handle_event(account, jid, event.type_) - @dbus.service.method(INTERFACE, in_signature='', out_signature='as') def list_accounts(self): """ List register accounts """ result = app.contacts.get_accounts() - result_array = dbus.Array([], signature='s') - if result and len(result) > 0: + result_array = [] + if result: for account in result: - result_array.append(DBUS_STRING(account)) + result_array.append(account) return result_array - @dbus.service.method(INTERFACE, in_signature='s', out_signature='a{ss}') def account_info(self, account): """ Show info on account: resource, jid, nick, prio, message """ - result = DBUS_DICT_SS() + result = {} if account in app.connections: # account is valid con = app.connections[account] index = con.connected - result['status'] = DBUS_STRING(app.SHOW_LIST[index]) - result['name'] = DBUS_STRING(con.name) - result['jid'] = DBUS_STRING(app.get_jid_from_account(con.name)) - result['message'] = DBUS_STRING(con.status) - result['priority'] = DBUS_STRING(str(con.priority)) - result['resource'] = DBUS_STRING(app.config.get_per('accounts', - con.name, 'resource')) + result['status'] = app.SHOW_LIST[index] + result['name'] = con.name + result['jid'] = app.get_jid_from_account(con.name) + result['message'] = con.status + result['priority'] = str(con.priority) + result['resource'] = app.config.get_per('accounts', con.name, + 'resource') return result - @dbus.service.method(INTERFACE, in_signature='s', out_signature='aa{sv}') def list_contacts(self, account): """ - List all contacts in the roster. If the first argument is specified, then - return the contacts for the specified account + List all contacts in the roster. If the first argument is specified, + then return the contacts for the specified account """ - result = dbus.Array([], signature='aa{sv}') + result = [] accounts = app.contacts.get_accounts() - if len(accounts) == 0: + if not accounts: return result if account: accounts_to_search = [account] @@ -647,9 +752,9 @@ class SignalObject(dbus.service.Object): result.append(item) return result - @dbus.service.method(INTERFACE, in_signature='', out_signature='a{ss}') def prefs_list(self): - prefs_dict = DBUS_DICT_SS() + prefs_dict = {} + def get_prefs(data, name, path, value): if value is None: return @@ -658,60 +763,56 @@ class SignalObject(dbus.service.Object): for node in path: key += node + '#' key += name - prefs_dict[DBUS_STRING(key)] = DBUS_STRING(value) + prefs_dict[key] = str(value) + app.config.foreach(get_prefs) return prefs_dict - @dbus.service.method(INTERFACE, in_signature='', out_signature='b') def prefs_store(self): try: app.interface.save_config() except Exception: - return DBUS_BOOLEAN(False) - return DBUS_BOOLEAN(True) + return False + return True - @dbus.service.method(INTERFACE, in_signature='s', out_signature='b') def prefs_del(self, key): if not key: - return DBUS_BOOLEAN(False) + return False key_path = key.split('#', 2) if len(key_path) != 3: - return DBUS_BOOLEAN(False) + return False if key_path[2] == '*': app.config.del_per(key_path[0], key_path[1]) else: app.config.del_per(key_path[0], key_path[1], key_path[2]) - return DBUS_BOOLEAN(True) + return True - @dbus.service.method(INTERFACE, in_signature='s', out_signature='b') def prefs_put(self, key): if not key: - return DBUS_BOOLEAN(False) + return False key_path = key.split('#', 2) if len(key_path) < 3: subname, value = key.split('=', 1) app.config.set(subname, value) - return DBUS_BOOLEAN(True) + return True subname, value = key_path[2].split('=', 1) app.config.set_per(key_path[0], key_path[1], subname, value) - return DBUS_BOOLEAN(True) + return True - @dbus.service.method(INTERFACE, in_signature='ss', out_signature='b') def add_contact(self, jid, account): if account: if account in app.connections and \ app.connections[account].connected > 1: # if given account is active, use it - AddNewContactWindow(account = account, jid = jid) + AddNewContactWindow(account=account, jid=jid) else: # wrong account - return DBUS_BOOLEAN(False) + return False else: # if account is not given, show account combobox - AddNewContactWindow(account = None, jid = jid) - return DBUS_BOOLEAN(True) + AddNewContactWindow(account=None, jid=jid) + return True - @dbus.service.method(INTERFACE, in_signature='ss', out_signature='b') def remove_contact(self, jid, account): jid = self._get_real_jid(jid, account) accounts = app.contacts.get_accounts() @@ -728,7 +829,7 @@ class SignalObject(dbus.service.Object): app.interface.roster.remove_contact(contact, account) app.contacts.remove_jid(account, jid) contact_exists = True - return DBUS_BOOLEAN(contact_exists) + return contact_exists def _is_first(self): if self.first_show: @@ -736,7 +837,7 @@ class SignalObject(dbus.service.Object): return True return False - def _get_real_jid(self, jid, account = None): + def _get_real_jid(self, jid, account=None): """ Get the real jid from the given one: removes xmpp: or get jid from nick if account is specified, search only in this account @@ -746,8 +847,8 @@ class SignalObject(dbus.service.Object): else: accounts = app.connections.keys() if jid.startswith('xmpp:'): - return jid[5:] # len('xmpp:') = 5 - nick_in_roster = None # Is jid a nick ? + return jid[5:] # len('xmpp:') = 5 + nick_in_roster = None # Is jid a nick ? for account in accounts: # Does jid exists in roster of one account ? if app.contacts.get_contacts(account, jid): @@ -771,14 +872,15 @@ class SignalObject(dbus.service.Object): """ if not contacts: return None - prim_contact = None # primary contact + prim_contact = None # primary contact for contact in contacts: if prim_contact is None or contact.priority > prim_contact.priority: prim_contact = contact - contact_dict = DBUS_DICT_SV() - contact_dict['name'] = DBUS_STRING(prim_contact.name) - contact_dict['show'] = DBUS_STRING(prim_contact.show) - contact_dict['jid'] = DBUS_STRING(prim_contact.jid) + contact_dict = {} + name = prim_contact.name if prim_contact.name is not None else '' + contact_dict['name'] = GLib.Variant('s', name) + contact_dict['show'] = GLib.Variant('s', prim_contact.show) + contact_dict['jid'] = GLib.Variant('s', prim_contact.jid) if prim_contact.keyID: keyID = None if len(prim_contact.keyID) == 8: @@ -786,30 +888,30 @@ class SignalObject(dbus.service.Object): elif len(prim_contact.keyID) == 16: keyID = prim_contact.keyID[8:] if keyID: - contact_dict['openpgp'] = keyID - contact_dict['resources'] = dbus.Array([], signature='(sis)') + contact_dict['openpgp'] = GLib.Variant('s', keyID) + resources = GLib.VariantBuilder(GLib.VariantType('a(sis)')) for contact in contacts: - resource_props = dbus.Struct((DBUS_STRING(contact.resource), - dbus.Int32(contact.priority), DBUS_STRING(contact.status))) - contact_dict['resources'].append(resource_props) - contact_dict['groups'] = dbus.Array([], signature='(s)') + resource_props = (contact.resource, int(contact.priority), + contact.status) + resources.add_value(GLib.Variant('(sis)', resource_props)) + contact_dict['resources'] = resources.end() + + groups = GLib.VariantBuilder(GLib.VariantType('as')) for group in prim_contact.groups: - contact_dict['groups'].append((DBUS_STRING(group),)) + groups.add_value(GLib.Variant('s', group)) + contact_dict['groups'] = groups.end() return contact_dict - @dbus.service.method(INTERFACE, in_signature='', out_signature='s') def get_unread_msgs_number(self): - return DBUS_STRING(str(app.events.get_nb_events())) + return str(app.events.get_nb_events()) - @dbus.service.method(INTERFACE, in_signature='s', out_signature='b') def start_chat(self, account): if not account: # error is shown in gajim-remote check_arguments(..) - return DBUS_BOOLEAN(False) + return False app.app.activate_action('start-chat') - return DBUS_BOOLEAN(True) + return True - @dbus.service.method(INTERFACE, in_signature='ss', out_signature='') def send_xml(self, xml, account): if account: app.connections[account].send_stanza(str(xml)) @@ -817,7 +919,6 @@ class SignalObject(dbus.service.Object): for acc in app.contacts.get_accounts(): app.connections[acc].send_stanza(str(xml)) - @dbus.service.method(INTERFACE, in_signature='ss', out_signature='') def change_avatar(self, picture, account): filesize = os.path.getsize(picture) invalid_file = False @@ -847,7 +948,6 @@ class SignalObject(dbus.service.Object): app.connections[acc].get_module('VCardTemp').send_vcard( vcard, sha) - @dbus.service.method(INTERFACE, in_signature='ssss', out_signature='') def join_room(self, room_jid, nick, password, account): if not account: # get the first connected account @@ -868,3 +968,6 @@ class SignalObject(dbus.service.Object): app.interface.join_gc_minimal(account, room_jid) else: app.interface.join_gc_room(account, room_jid, nick, password) + + def Introspect(self): + return self.__doc__ View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/923ce01c479107a2c3ca185be762a2a8e… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/923ce01c479107a2c3ca185be762a2a8e… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Flatpak: update dependencies
by Philipp Hörist 17 Aug '18

17 Aug '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: 923ce01c by André Apitzsch at 2018-08-17T19:59:35Z Flatpak: update dependencies - - - - - 1 changed file: - org.gajim.Gajim.json Changes: ===================================== org.gajim.Gajim.json ===================================== --- a/org.gajim.Gajim.json +++ b/org.gajim.Gajim.json @@ -149,8 +149,8 @@ "sources": [ { "type": "archive", - "url": "https://files.pythonhosted.org/packages/1a/04/d6f1159feaccdfc508517dba1929e…", - "sha256": "f7cddbb5f5c640311eb00eab6e849f7701fa70bf6a183fc8a2c33dd1d1672fb2" + "url": "https://files.pythonhosted.org/packages/d3/3e/1d74cdcb393b68ab9ee18d78c11ae…", + "sha256": "012adb8e25fbfd64c652e99e7bab58799a3aaf05d39ab38561f69190a909015f" } ] }, @@ -194,8 +194,8 @@ "sources": [ { "type": "archive", - "url": "https://files.pythonhosted.org/packages/ec/b2/faa78c1ab928d2b2c634c8b41ff11…", - "sha256": "9fc295bf69130a342e7a19a39d7bbeb15c0bcaabc7382ec33ef3b2b7d18d2f63" + "url": "https://files.pythonhosted.org/packages/22/21/233e38f74188db94e8451ef638575…", + "sha256": "8d10113ca826a4c29d5b85b2c4e045ffa8bad74fb525ee0eceb1d38d4c70dfd6" } ] }, @@ -274,13 +274,27 @@ "name": "python3-keyring", "buildsystem": "simple", "build-commands": [ - "pip3 install --prefix=/app keyring-13.0.0-py2.py3-none-any.whl" + "pip3 install --prefix=/app keyring-13.2.1-py2.py3-none-any.whl" ], "sources": [ { "type": "file", - "url": "https://files.pythonhosted.org/packages/a6/2a/f262337da5acd435e1013189e1cab…", - "sha256": "6849b41e69f77cdb122f026f78797675248252775a1db6ebc47610b769772b23" + "url": "https://files.pythonhosted.org/packages/88/a8/341fa4686e98bf10289eb4eaea89e…", + "sha256": "6e01954fd3e404820e1fade262ee661974051551ed08c899ffc5e88bb9df288e" + } + ] + }, + { + "name": "python3-cssutils", + "buildsystem": "simple", + "build-commands": [ + "pip3 install --prefix=/app cssutils-1.0.2-py3-none-any.whl" + ], + "sources": [ + { + "type": "file", + "url": "https://files.pythonhosted.org/packages/6b/15/a9fb9010f58d1c55dd0b7779db233…", + "sha256": "c74dbe19c92f5052774eadb15136263548dd013250f1ed1027988e7fef125c8d" } ] }, View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/923ce01c479107a2c3ca185be762a2a8eb… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/923ce01c479107a2c3ca185be762a2a8eb… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] 3 commits: Rework Emoji implementation
by Philipp Hörist 17 Aug '18

17 Aug '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: 5feb4bec by Philipp Hörist at 2018-08-17T19:55:42Z Rework Emoji implementation - Use emoji data from a generated dict based on the offical unicode docs, this makes it easier to update in the future - Rewrite the emoji chooser - Add a search field to the emoji chooser - The emoji chooser is loaded async - Update to current Unicode 11 Noto theme - - - - - 1577bcba by Philipp Hörist at 2018-08-17T19:55:48Z Move the noto atlas generator code into a new project - - - - - d97c726a by Philipp Hörist at 2018-08-17T19:55:49Z Add twitter emojis - - - - - 30 changed files: - gajim/chat_control_base.py - gajim/common/configpaths.py - gajim/common/helpers.py - gajim/conversation_textview.py - − gajim/data/emoticons/font-emoticons/emoticons_theme.py - − gajim/data/emoticons/noto-emoticons/README.md - − gajim/data/emoticons/noto-emoticons/emoticons.png - − gajim/data/emoticons/noto-emoticons/emoticons_theme.py - − gajim/data/emoticons/noto-emoticons/png/u0023.png - − gajim/data/emoticons/noto-emoticons/png/u0023_20e3.png - − gajim/data/emoticons/noto-emoticons/png/u002a.png - − gajim/data/emoticons/noto-emoticons/png/u002a_20e3.png - − gajim/data/emoticons/noto-emoticons/png/u0030.png - − gajim/data/emoticons/noto-emoticons/png/u0030_20e3.png - − gajim/data/emoticons/noto-emoticons/png/u0031.png - − gajim/data/emoticons/noto-emoticons/png/u0031_20e3.png - − gajim/data/emoticons/noto-emoticons/png/u0032.png - − gajim/data/emoticons/noto-emoticons/png/u0032_20e3.png - − gajim/data/emoticons/noto-emoticons/png/u0033.png - − gajim/data/emoticons/noto-emoticons/png/u0033_20e3.png - − gajim/data/emoticons/noto-emoticons/png/u0034.png - − gajim/data/emoticons/noto-emoticons/png/u0034_20e3.png - − gajim/data/emoticons/noto-emoticons/png/u0035.png - − gajim/data/emoticons/noto-emoticons/png/u0035_20e3.png - − gajim/data/emoticons/noto-emoticons/png/u0036.png - − gajim/data/emoticons/noto-emoticons/png/u0036_20e3.png - − gajim/data/emoticons/noto-emoticons/png/u0037.png - − gajim/data/emoticons/noto-emoticons/png/u0037_20e3.png - − gajim/data/emoticons/noto-emoticons/png/u0038.png - − gajim/data/emoticons/noto-emoticons/png/u0038_20e3.png The diff was not included because it is too large. View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/e37ab6b59ac2b8ddaf5f98f3f1fc17398… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/e37ab6b59ac2b8ddaf5f98f3f1fc17398… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Fix typo
by Philipp Hörist 17 Aug '18

17 Aug '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: e37ab6b5 by Philipp Hörist at 2018-08-17T07:37:43Z Fix typo - - - - - 1 changed file: - gajim/gui_interface.py Changes: ===================================== gajim/gui_interface.py ===================================== --- a/gajim/gui_interface.py +++ b/gajim/gui_interface.py @@ -658,7 +658,7 @@ class Interface: if helpers.allow_showing_notification(obj.account): event_type = _('Groupchat Invitation') - text = _('Your are invited to {room} by {user}').format( + text = _('You are invited to {room} by {user}').format( room=obj.room_jid, user=str(obj.from_)) app.notification.popup( event_type, obj.room_jid, obj.account, 'gc-invitation', View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/e37ab6b59ac2b8ddaf5f98f3f1fc173981… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/e37ab6b59ac2b8ddaf5f98f3f1fc173981… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Update README.md
by Philipp Hörist 17 Aug '18

17 Aug '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: 34cf5a59 by Philipp Hörist at 2018-08-17T06:35:13Z Update README.md - - - - - 1 changed file: - README.md Changes: ===================================== README.md ===================================== --- a/README.md +++ b/README.md @@ -9,6 +9,7 @@ - gir1.2-gtk-3.0 (>=3.22) - python3-nbxmpp - python3-openssl (>=0.14) +- python3-cssutils (>=1.0.2) ### Optional Runtime Requirements View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/34cf5a5996ffda4280e8043c669cae8d92… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/34cf5a5996ffda4280e8043c669cae8d92… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Update setup.py
by Philipp Hörist 17 Aug '18

17 Aug '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: 160bc3f2 by Philipp Hörist at 2018-08-17T06:26:56Z Update setup.py - - - - - 1 changed file: - setup.py Changes: ===================================== setup.py ===================================== --- a/setup.py +++ b/setup.py @@ -226,7 +226,7 @@ package_data_iconsets = ['data/iconsets/*/*/*.gif', package_data_moods = ['data/moods/*/*.png'] package_data_other = ['data/other/*'] package_data_sounds = ['data/sounds/*.wav'] -package_data_style = ['data/style/gajim.css'] +package_data_style = ['data/style/*.css'] package_plugins_data = ['data/plugins/*/*'] package_data = (package_data_activities + package_data_emoticons @@ -288,5 +288,6 @@ setup( 'keyring', 'nbxmpp>=0.6.6', 'pyOpenSSL>=0.12', + 'cssutils>=1.0.2' ] ) View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/160bc3f2f26730950867d62f9d67924ffd… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/160bc3f2f26730950867d62f9d67924ffd… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Rework Gajim Theming
by Philipp Hörist 16 Aug '18

16 Aug '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: 9d5131b8 by Philipp Hörist at 2018-08-16T20:53:33Z Rework Gajim Theming - Save all Theme settings to .css instead of the config file - Add a gajim-dark.css - Refactor the ThemesWindow - - - - - 27 changed files: - gajim/application.py - gajim/chat_control.py - gajim/chat_control_base.py - gajim/common/app.py - gajim/common/config.py - gajim/common/configpaths.py - gajim/common/connection_handlers_events.py - gajim/common/const.py - + gajim/common/css_config.py - gajim/conversation_textview.py - gajim/data/gui/chat_control.ui - − gajim/data/gui/gajim_themes_window.ui - gajim/data/gui/groupchat_control.ui - gajim/data/gui/preferences_window.ui - gajim/data/gui/service_discovery_window.ui - + gajim/data/gui/themes_window.ui - + gajim/data/style/default-dark.css - + gajim/data/style/default.css - + gajim/data/style/gajim-dark.css - gajim/data/style/gajim.css - gajim/dialogs.py - gajim/disco.py - − gajim/gajim_themes_window.py - gajim/groupchat_control.py - gajim/gtk/__init__.py - gajim/gtk/dialogs.py - gajim/gtk/preferences.py The diff was not included because it is too large. View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/9d5131b8b4e3f4265c15def9db65b712bb… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/9d5131b8b4e3f4265c15def9db65b712bb… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Reduce number of notifications kept in message tray
by Philipp Hörist 16 Aug '18

16 Aug '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: ee3cc9cb by Sophie Herold at 2018-08-16T20:48:40Z Reduce number of notifications kept in message tray - - - - - 3 changed files: - gajim/common/connection_handlers_events.py - gajim/gui_interface.py - gajim/notify.py Changes: ===================================== gajim/common/connection_handlers_events.py ===================================== --- a/gajim/common/connection_handlers_events.py +++ b/gajim/common/connection_handlers_events.py @@ -1114,27 +1114,23 @@ class NotificationEvent(nec.NetworkIncomingEvent): else: # We don't want message preview, do_preview = False self.popup_text = '' + if msg_obj.mtype == 'normal': # single message self.popup_msg_type = 'normal' self.popup_event_type = _('New Single Message') - self.popup_title = _('New Single Message from %(nickname)s') % \ - {'nickname': nick} elif msg_obj.mtype == 'pm': self.popup_msg_type = 'pm' self.popup_event_type = _('New Private Message') - self.popup_title = _('New Private Message from group chat %s') % \ - msg_obj.jid - if self.popup_text: - self.popup_text = _('%(nickname)s: %(message)s') % \ - {'nickname': nick, 'message': self.popup_text} - else: - self.popup_text = _('Messaged by %(nickname)s') % \ - {'nickname': nick} else: # chat message self.popup_msg_type = 'chat' self.popup_event_type = _('New Message') - self.popup_title = _('New Message from %(nickname)s') % \ - {'nickname': nick} + + num_unread = len(app.events.get_events(self.conn.name, self.jid, + ['printed_' + self.popup_msg_type, self.popup_msg_type])) + self.popup_title = i18n.ngettext( + 'New message from %(nickname)s', + '%(n_msgs)i unread messages from %(nickname)s', + num_unread) % {'nickname': nick, 'n_msgs': num_unread} if app.config.get('notify_on_new_message'): if self.first_unread or (app.config.get('autopopup_chat_opened') \ ===================================== gajim/gui_interface.py ===================================== --- a/gajim/gui_interface.py +++ b/gajim/gui_interface.py @@ -658,9 +658,11 @@ class Interface: if helpers.allow_showing_notification(obj.account): event_type = _('Groupchat Invitation') + text = _('Your are invited to {room} by {user}').format( + room=obj.room_jid, user=str(obj.from_)) app.notification.popup( - event_type, str(obj.from_), obj.account, 'gc-invitation', - 'gajim-gc_invitation', event_type, obj.room_jid) + event_type, obj.room_jid, obj.account, 'gc-invitation', + 'gajim-gc_invitation', event_type, text) def forget_gpg_passphrase(self, keyid): if keyid in self.gpg_passphrase: ===================================== gajim/notify.py ===================================== --- a/gajim/notify.py +++ b/gajim/notify.py @@ -74,6 +74,9 @@ class Notification: def __init__(self): app.ged.register_event_handler( 'notification', ged.GUI2, self._nec_notification) + app.ged.register_event_handler( + 'our-show', ged.GUI2, self._nec_our_status) + app.events.event_removed_subscribe(self._on_event_removed) def _nec_notification(self, obj): if obj.do_popup: @@ -95,6 +98,18 @@ class Notification: except Exception: pass + def _on_event_removed(self, event_list): + for event in event_list: + if event.type_ == 'gc-invitation': + self.withdraw('gc-invitation', event.account, event.room_jid) + if event.type_ in ('normal', 'printed_chat', 'chat', + 'printed_pm', 'pm'): + self.withdraw('new-message', event.account, event.jid) + + def _nec_our_status(self, obj): + if app.account_is_connected(obj.conn.name): + self.withdraw('connection-failed', obj.conn.name) + def _get_icon_name(self, obj): if obj.notif_type == 'msg': if obj.base_event.mtype == 'pm': @@ -153,13 +168,28 @@ class Notification: #Button in notification notification.add_button_with_target(_('Open'), action, variant_dict) notification.set_default_action_and_target(action, variant_dict) - if event_type in (_('New Message'), _('New Single Message'), + + # Only one notification per JID + if event_type in (_('Contact Signed In'), _('Contact Signed Out'), + _('Contact Changed Status')): + notif_id = self._id('contact-status-changed', account, jid) + elif event_type == _('Groupchat Invitation'): + notif_id = self._id('gc-invitation', account, jid) + elif event_type == _('Connection Failed'): + notif_id = self._id('connection-failed', account) + elif event_type in (_('New Message'), _('New Single Message'), _('New Private Message')): - # Only one notification per JID - notif_id = jid + notif_id = self._id('new-message', account, jid) + notification.set_priority(Gio.NotificationPriority.NORMAL) app.app.send_notification(notif_id, notification) + def withdraw(self, *args): + if sys.platform != 'win32': + app.app.withdraw_notification(self._id(*args)) + + def _id(self, *args): + return ','.join(args) class PopupNotificationWindow: def __init__(self, event_type, jid, account, msg_type='', View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/ee3cc9cbab7fe2f2f6130fb0eeca812dee… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/ee3cc9cbab7fe2f2f6130fb0eeca812dee… You're receiving this email because of your account on dev.gajim.org.

1 0

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

Commits