Commits

commits@gajim.org

17632 discussions

[Git][gajim/gajim][master] 2 commits: AvatarSelector: Add finalize check
by Daniel Brötzmann (＠wurstsalat) 03 Feb '22

03 Feb '22

Daniel Brötzmann pushed to branch master at gajim / gajim Commits: 84863f51 by wurstsalat at 2022-02-03T15:03:17+01:00 AvatarSelector: Add finalize check - - - - - 82f5e21c by wurstsalat at 2022-02-03T15:03:38+01:00 GroupchatManage: Fix leak - - - - - 2 changed files: - gajim/gtk/avatar_selector.py - gajim/gtk/groupchat_manage.py Changes: ===================================== gajim/gtk/avatar_selector.py ===================================== @@ -15,8 +15,8 @@ # You should have received a copy of the GNU General Public License # along with Gajim. If not, see <http://www.gnu.org/licenses/>. +from typing import Any from typing import Optional -from typing import Tuple import os import logging @@ -29,6 +29,7 @@ from gi.repository import Gtk import cairo +from gajim.common import app from gajim.common.const import AvatarSize from gajim.common.i18n import _ from gajim.common.helpers import get_file_path_from_dnd_dropped_uri @@ -78,6 +79,7 @@ def __init__(self) -> None: Gdk.DragAction.COPY | Gdk.DragAction.MOVE) self.drag_dest_set_target_list(dst_targets) self.connect('drag-data-received', self._on_drag_data_received) + self.connect('destroy', self._on_destroy) self._crop_area = CropArea() self._crop_area.set_vexpand(True) @@ -102,6 +104,9 @@ def __init__(self) -> None: self.show_all() + def _on_destroy(self, *args: Any) -> None: + app.check_finalize(self) + def prepare_crop_area(self, path: str) -> None: pixbuf = self._get_pixbuf_from_path(path) if pixbuf is None: @@ -154,7 +159,7 @@ def get_prepared(self) -> bool: @staticmethod def _scale_for_publish(pixbuf: GdkPixbuf.Pixbuf - ) -> Tuple[GdkPixbuf.Pixbuf, int, int]: + ) -> tuple[GdkPixbuf.Pixbuf, int, int]: width = pixbuf.get_width() height = pixbuf.get_height() if width > AvatarSize.PUBLISH or height > AvatarSize.PUBLISH: @@ -165,7 +170,7 @@ def _scale_for_publish(pixbuf: GdkPixbuf.Pixbuf GdkPixbuf.InterpType.BILINEAR) return pixbuf, width, height - def get_avatar_surface(self) -> Optional[Tuple[cairo.Surface, int, int]]: + def get_avatar_surface(self) -> Optional[tuple[cairo.Surface, int, int]]: pixbuf = self._crop_area.get_pixbuf() if pixbuf is None: return None @@ -174,7 +179,7 @@ def get_avatar_surface(self) -> Optional[Tuple[cairo.Surface, int, int]]: return Gdk.cairo_surface_create_from_pixbuf( scaled, self.get_scale_factor()), width, height - def get_avatar_bytes(self) -> Tuple[bool, Optional[bytes], int, int]: + def get_avatar_bytes(self) -> tuple[bool, Optional[bytes], int, int]: pixbuf = self._crop_area.get_pixbuf() if pixbuf is None: return False, None, 0, 0 ===================================== gajim/gtk/groupchat_manage.py ===================================== @@ -11,6 +11,7 @@ # You should have received a copy of the GNU General Public License # along with Gajim. If not, see <http://www.gnu.org/licenses/>. +from typing import Any from typing import Optional import logging @@ -63,9 +64,14 @@ def __init__(self, self._prepare_subject() self._prepare_manage() + self.connect('destroy', self._on_destroy) self.show_all() + def _on_destroy(self, *args: Any) -> None: + self._avatar_selector = None + app.check_finalize(self) + @property def disco_info(self): return app.storage.cache.get_last_disco_info(self._contact.jid) View it on GitLab: https://dev.gajim.org/gajim/gajim/-/compare/b4dcb38c0d6c0f2746f361e13c49cf2… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/-/compare/b4dcb38c0d6c0f2746f361e13c49cf2… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Only set urgency hint if we want to notify
by Daniel Brötzmann (＠wurstsalat) 03 Feb '22

03 Feb '22

Daniel Brötzmann pushed to branch master at gajim / gajim Commits: b4dcb38c by wurstsalat at 2022-02-03T14:24:12+01:00 Only set urgency hint if we want to notify - - - - - 1 changed file: - gajim/gtk/controls/base.py Changes: ===================================== gajim/gtk/controls/base.py ===================================== @@ -1250,7 +1250,7 @@ def add_message(self, if notify: self._notify(name, text, tim, additional_data) - if not chat_active: + if not chat_active and notify: if self.is_groupchat: needs_highlight = message_needs_highlight( text, View it on GitLab: https://dev.gajim.org/gajim/gajim/-/commit/b4dcb38c0d6c0f2746f361e13c49cf2e… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/-/commit/b4dcb38c0d6c0f2746f361e13c49cf2e… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] 2 commits: GroupchatDetails: Fix finalizing
by Daniel Brötzmann (＠wurstsalat) 03 Feb '22

03 Feb '22

Daniel Brötzmann pushed to branch master at gajim / gajim Commits: 26b53576 by wurstsalat at 2022-02-03T14:02:52+01:00 GroupchatDetails: Fix finalizing - - - - - e9a56ab2 by wurstsalat at 2022-02-03T14:15:28+01:00 XMLConsole: Fix leak - - - - - 2 changed files: - gajim/gtk/groupchat_details.py - gajim/gtk/xml_console.py Changes: ===================================== gajim/gtk/groupchat_details.py ===================================== @@ -174,4 +174,7 @@ def _on_key_press(self, self.destroy() def _on_destroy(self, _widget: GroupchatDetails) -> None: + app.ged.remove_event_handler('muc-disco-update', + ged.GUI1, + self._on_muc_disco_update) app.check_finalize(self) ===================================== gajim/gtk/xml_console.py ===================================== @@ -99,6 +99,7 @@ def __init__(self) -> None: self.show_all() self.connect('key_press_event', self._on_key_press) + self.connect('destroy', self._on_destroy) self._ui.connect_signals(self) self.register_events([ @@ -107,6 +108,10 @@ def __init__(self) -> None: ('style-changed', ged.GUI1, self._on_style_changed) ]) + def _on_destroy(self, *args: Any) -> None: + self._ui.popover.destroy() + app.check_finalize(self) + def _get_style_scheme(self) -> Optional[GtkSource.StyleScheme]: if app.css_config.prefer_dark: style_scheme = self._style_scheme_manager.get_scheme( @@ -227,7 +232,8 @@ def _on_send(self, *args: Any) -> None: # stream management node = nbxmpp.Protocol(node=stanza, attrs={'xmlns': 'jabber:client'}) - app.connections[self._selected_send_account].connection.send(node) + client = app.get_client(self._selected_send_account) + client.connection.send(node) self.last_stanza = stanza buffer_.set_text('') View it on GitLab: https://dev.gajim.org/gajim/gajim/-/compare/49b4a0673c4a9d5cf7f309d1786f9ff… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/-/compare/49b4a0673c4a9d5cf7f309d1786f9ff… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Remove unused code
by Philipp Hörist (＠lovetox) 02 Feb '22

02 Feb '22

Philipp Hörist pushed to branch master at gajim / gajim Commits: 49b4a067 by lovetox at 2022-02-02T23:43:16+01:00 Remove unused code - - - - - 1 changed file: - gajim/common/client.py Changes: ===================================== gajim/common/client.py ===================================== @@ -473,11 +473,6 @@ def send_message(self, message: OutgoingMessage) -> None: stanza = self.get_module('Message').build_message_stanza(message) message.stanza = stanza - if message.contact is None: - # Only Single Message should have no contact - self._send_message(message) - return - method = message.contact.settings.get('encryption') if not method: self._send_message(message) @@ -513,17 +508,6 @@ def _send_message(self, message: OutgoingMessage) -> None: self.get_module('Message').log_message(message) - def send_messages(self, jids: list[JID], message: OutgoingMessage) -> None: - if not self._state.is_available: - log.warning('Trying to send message while offline') - return - - # for jid in jids: - # message = message.copy() - # stanza = self.get_module('Message').build_message_stanza(message) - # message.stanza = stanza - # self._send_message(message) - def _prepare_for_connect(self) -> None: custom_host = get_custom_host(self._account) if custom_host is not None: View it on GitLab: https://dev.gajim.org/gajim/gajim/-/commit/49b4a0673c4a9d5cf7f309d1786f9ffe… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/-/commit/49b4a0673c4a9d5cf7f309d1786f9ffe… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Remove Single Message Window
by Philipp Hörist (＠lovetox) 02 Feb '22

02 Feb '22

Philipp Hörist pushed to branch master at gajim / gajim Commits: 9feb0a0f by lovetox at 2022-02-02T23:41:11+01:00 Remove Single Message Window - - - - - 6 changed files: - − gajim/data/gui/single_message_window.ui - gajim/gtk/application.py - gajim/gtk/builder.pyi - gajim/gtk/const.py - gajim/gtk/menus.py - − gajim/gtk/single_message.py Changes: ===================================== gajim/data/gui/single_message_window.ui deleted ===================================== @@ -1,166 +0,0 @@ -<?xml version="1.0" encoding="UTF-8"?> - -<interface> - <requires lib="gtk+" version="3.24"/> - <object class="GtkBox" id="box"> - <property name="visible">True</property> - <property name="can-focus">False</property> - <property name="orientation">vertical</property> - <property name="spacing">12</property> - <child> -  - <object class="GtkGrid" id="message_grid"> - <property name="visible">True</property> - <property name="can-focus">False</property> - <property name="row-spacing">6</property> - <property name="column-spacing">12</property> - <child> - <object class="GtkLabel"> - <property name="visible">True</property> - <property name="can-focus">False</property> - <property name="halign">end</property> - <property name="label" translatable="yes">_To</property> - <property name="use-underline">True</property> - <property name="mnemonic-widget">recipients_entry</property> - <style> - <class name="dim-label"/> - </style> - </object> - <packing> - <property name="left-attach">0</property> - <property name="top-attach">0</property> - </packing> - </child> - <child> - <object class="GtkEntry" id="recipients_entry"> - <property name="visible">True</property> - <property name="can-focus">True</property> - <property name="hexpand">True</property> - <property name="placeholder-text" translatable="yes">user(a)example.org</property> - <signal name="changed" handler="_on_changed" swapped="no"/> - </object> - <packing> - <property name="left-attach">1</property> - <property name="top-attach">0</property> - </packing> - </child> - <child> - <object class="GtkLabel"> - <property name="visible">True</property> - <property name="can-focus">False</property> - <property name="halign">end</property> - <property name="label" translatable="yes">Su_bject</property> - <property name="use-underline">True</property> - <property name="mnemonic-widget">subject_entry</property> - <property name="single-line-mode">True</property> - <style> - <class name="dim-label"/> - </style> - </object> - <packing> - <property name="left-attach">0</property> - <property name="top-attach">1</property> - </packing> - </child> - <child> - <object class="GtkEntry" id="subject_entry"> - <property name="visible">True</property> - <property name="can-focus">True</property> - <property name="placeholder-text" translatable="yes">Type the subject here...</property> - </object> - <packing> - <property name="left-attach">1</property> - <property name="top-attach">1</property> - </packing> - </child> - <child> - <object class="GtkLabel"> - <property name="visible">True</property> - <property name="can-focus">False</property> - <property name="halign">end</property> - <property name="label" translatable="yes">_Message</property> - <property name="use-underline">True</property> - <property name="mnemonic-widget">message_textview</property> - <style> - <class name="dim-label"/> - </style> - </object> - <packing> - <property name="left-attach">0</property> - <property name="top-attach">2</property> - </packing> - </child> - <child> - <object class="GtkScrolledWindow" id="message_scrolledwindow"> - <property name="height-request">100</property> - <property name="visible">True</property> - <property name="can-focus">True</property> - <property name="vexpand">True</property> - <property name="shadow-type">in</property> - <child> - <object class="GtkTextView" id="message_textview"> - <property name="visible">True</property> - <property name="can-focus">True</property> - <property name="wrap-mode">word-char</property> - <property name="left-margin">6</property> - <property name="right-margin">6</property> - <property name="top-margin">6</property> - <property name="bottom-margin">6</property> - </object> - </child> - </object> - <packing> - <property name="left-attach">1</property> - <property name="top-attach">2</property> - </packing> - </child> - <child> - <object class="GtkLabel" id="count_chars_label"> - <property name="visible">True</property> - <property name="can-focus">False</property> - <property name="halign">end</property> - <property name="label" translatable="yes">Characters typed: 0</property> - <style> - <class name="dim-label"/> - </style> - </object> - <packing> - <property name="left-attach">0</property> - <property name="top-attach">3</property> - <property name="width">2</property> - </packing> - </child> - </object> - <packing> - <property name="expand">True</property> - <property name="fill">True</property> - <property name="position">0</property> - </packing> - </child> - <child> - <object class="GtkButton" id="send_button"> - <property name="label" translatable="yes">_Send</property> - <property name="visible">True</property> - <property name="sensitive">False</property> - <property name="can-focus">True</property> - <property name="can-default">True</property> - <property name="receives-default">False</property> - <property name="tooltip-text" translatable="yes">Send message and close window</property> - <property name="halign">end</property> - <property name="use-underline">True</property> - <signal name="clicked" handler="_on_send_clicked" swapped="no"/> - <style> - <class name="suggested-action"/> - </style> - </object> - <packing> - <property name="expand">False</property> - <property name="fill">True</property> - <property name="position">1</property> - </packing> - </child> - <style> - <class name="padding-18"/> - </style> - </object> -</interface> ===================================== gajim/gtk/application.py ===================================== @@ -435,7 +435,6 @@ def _get_account_actions(self, # pylint: disable=line-too-long return [ ('-bookmarks', self._on_bookmarks_action, 'online', 's'), - ('-start-single-chat', self._on_start_single_message_action, 'online', 's'), ('-open-chat', self._on_open_chat_action, 'online', 'as'), ('-add-contact', self._on_add_contact_account_action, 'online', 'as'), ('-services', self._on_services_action, 'online', 's'), @@ -602,12 +601,6 @@ def _on_add_contact_account_action(_action: Gio.SimpleAction, jid = JID.from_string(jid) open_window('AddContact', account=account or None, jid=jid or None) - @staticmethod - def _on_start_single_message_action(_action: Gio.SimpleAction, - param: GLib.Variant) -> None: - account = param.get_string() - open_window('SingleMessageWindow', account=account) - @staticmethod def _on_add_account_action(_action: Gio.SimpleAction, _param: Optional[GLib.Variant]) -> None: ===================================== gajim/gtk/builder.pyi ===================================== @@ -848,17 +848,6 @@ class ShortcutsWindowBuilder(Builder): shortcuts_window: Gtk.ShortcutsWindow -class SingleMessageWindowBuilder(Builder): - box: Gtk.Box - message_grid: Gtk.Grid - recipients_entry: Gtk.Entry - subject_entry: Gtk.Entry - message_scrolledwindow: Gtk.ScrolledWindow - message_textview: Gtk.TextView - count_chars_label: Gtk.Label - send_button: Gtk.Button - - class SslErrorDialogBuilder(Builder): ssl_error_box: Gtk.Box intro_text: Gtk.Label @@ -1074,8 +1063,6 @@ def get_builder(file_name: Literal['service_discovery_window.ui'], widgets: list @overload def get_builder(file_name: Literal['shortcuts_window.ui'], widgets: list[str] = ...) -> ShortcutsWindowBuilder: ... @overload -def get_builder(file_name: Literal['single_message_window.ui'], widgets: list[str] = ...) -> SingleMessageWindowBuilder: ... -@overload def get_builder(file_name: Literal['ssl_error_dialog.ui'], widgets: list[str] = ...) -> SslErrorDialogBuilder: ... @overload def get_builder(file_name: Literal['start_chat_dialog.ui'], widgets: list[str] = ...) -> StartChatDialogBuilder: ... ===================================== gajim/gtk/const.py ===================================== @@ -162,7 +162,6 @@ def __str__(self): 'ServerInfo': 'gajim.gui.server_info', 'ServiceDiscoveryWindow': 'gajim.gui.discovery', 'ServiceRegistration': 'gajim.gui.service_registration', - 'SingleMessageWindow': 'gajim.gui.single_message', 'SSLErrorDialog': 'gajim.gui.ssl_error_dialog', 'StartChatDialog': 'gajim.gui.start_chat', 'SynchronizeAccounts': 'gajim.gui.synchronize_accounts', ===================================== gajim/gtk/menus.py ===================================== @@ -171,7 +171,6 @@ def get_account_menu(account: str) -> Gio.Menu: menuitems: MenuItemListT = [ (_('Profile'), f'app.{account}-profile', val), - (_('Send Single Message…'), f'app.{account}-start-single-chat', val), (_('Discover Services…'), f'app.{account}-services', val), (_('Server Info'), f'app.{account}-server-info', val), ] ===================================== gajim/gtk/single_message.py deleted ===================================== @@ -1,150 +0,0 @@ -# This file is part of Gajim. -# -# Gajim is free software; you can redistribute it and/or modify -# it under the terms of the GNU General Public License as published -# by the Free Software Foundation; version 3 only. -# -# Gajim is distributed in the hope that it will be useful, -# but WITHOUT ANY WARRANTY; without even the implied warranty of -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -# GNU General Public License for more details. -# -# You should have received a copy of the GNU General Public License -# along with Gajim. If not, see <http://www.gnu.org/licenses/>. - -from typing import Any -from typing import Optional - -from gi.repository import Gdk -from gi.repository import Gtk - -from nbxmpp.protocol import JID - -from gajim.common import app -from gajim.common.helpers import get_contact_dict_for_account -from gajim.common.helpers import validate_jid -from gajim.common.i18n import _ -from gajim.common.structs import OutgoingMessage - -from .dialogs import ErrorDialog -from .builder import get_builder -from .util import get_completion_liststore -from .util import get_icon_name - -if app.is_installed('GSPELL'): - from gi.repository import Gspell # pylint: disable=ungrouped-imports - - -class SingleMessageWindow(Gtk.ApplicationWindow): - def __init__(self, - account: str, - recipients: Optional[str] = None - ) -> None: - Gtk.ApplicationWindow.__init__(self) - self.set_application(app.app) - self.set_position(Gtk.WindowPosition.CENTER) - self.set_show_menubar(False) - self.set_size_request(500, -1) - self.set_name('SingleMessageWindow') - - self.account = account - self._client = app.get_client(account) - self._recipients = recipients - self._completion_dict = {} - - if len(app.connections) > 1: - title = _('Single Message (%s)') % self.account - else: - title = _('Single Message') - self.set_title(title) - - self._ui = get_builder('single_message_window.ui') - self._message_buffer = self._ui.message_textview.get_buffer() - self._message_buffer.connect('changed', self._update_char_counter) - - if recipients is not None: - self._ui.recipients_entry.set_text(recipients) - - if (app.settings.get('use_speller') and - app.is_installed('GSPELL')): - lang = app.settings.get('speller_language') - gspell_lang = Gspell.language_lookup(lang) - if gspell_lang is None: - gspell_lang = Gspell.language_get_default() - spell_buffer = Gspell.TextBuffer.get_from_gtk_text_buffer( - self._ui.message_textview.get_buffer()) - spell_buffer.set_spell_checker(Gspell.Checker.new(gspell_lang)) - spell_view = Gspell.TextView.get_from_gtk_text_view( - self._ui.message_textview) - spell_view.set_inline_spell_checking(True) - spell_view.set_enable_language_menu(True) - - liststore = get_completion_liststore(self._ui.recipients_entry) - self._completion_dict = get_contact_dict_for_account(account) - for jid, contact in self._completion_dict.items(): - icon_name = get_icon_name(contact.show.value) - liststore.append((icon_name, jid)) - - self.add(self._ui.box) - - if self._recipients is not None: - self._ui.message_textview.grab_focus() - - self.connect('key-press-event', self._on_key_press) - self._ui.connect_signals(self) - - self.show_all() - - def _on_key_press(self, _widget: Gtk.Widget, event: Gdk.EventKey) -> None: - if event.keyval == Gdk.KEY_Escape: - self.destroy() - - def _on_changed(self, *args: Any) -> None: - recipients = self._ui.recipients_entry.get_text() - begin, end = self._message_buffer.get_bounds() - message = self._message_buffer.get_text(begin, end, True) - self._ui.send_button.set_sensitive(bool(recipients and message)) - - def _update_char_counter(self, _widget: Gtk.TextView) -> None: - characters_no = self._message_buffer.get_char_count() - self._ui.count_chars_label.set_text( - _('Characters typed: %s') % str(characters_no)) - self._on_changed() - - def _on_send_clicked(self, _widget: Gtk.Button) -> None: - if not app.account_is_available(self.account): - ErrorDialog(_('Not Connected'), - _('Please make sure you are connected with "%s".') % - self.account) - return - - jids = self._ui.recipients_entry.get_text() - subject = self._ui.subject_entry.get_text() - begin, end = self._message_buffer.get_bounds() - message = self._message_buffer.get_text(begin, end, True) - - recipient_list: list[JID] = [] - for jid in jids.split(','): - try: - jid = validate_jid(jid.strip()) - except ValueError as err: - ErrorDialog( - _('Cannot Send Message'), - _('XMPP Address "%(jid)s" is invalid.\n%(error)s') % { - 'jid': jid, 'error': err}) - return - - if '/announce/' in str(jid): - self._client.get_module('Announce').set_announce( - jid, subject, message) - continue - - recipient_list.append(jid) - - message = OutgoingMessage(account=self.account, - contact=None, - message=message, - type_='normal', - subject=subject) - self._client.send_messages(recipient_list, message) - self.destroy() View it on GitLab: https://dev.gajim.org/gajim/gajim/-/commit/9feb0a0f92464179d04ac34e20c986ac… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/-/commit/9feb0a0f92464179d04ac34e20c986ac… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Add SimpleClientState.CONNECTING
by Philipp Hörist (＠lovetox) 02 Feb '22

02 Feb '22

Philipp Hörist pushed to branch master at gajim / gajim Commits: bab39049 by lovetox at 2022-02-02T23:34:41+01:00 Add SimpleClientState.CONNECTING - - - - - 2 changed files: - gajim/common/client.py - gajim/common/const.py Changes: ===================================== gajim/common/client.py ===================================== @@ -561,6 +561,7 @@ def connect(self, ignored_tls_errors: IgnoredTlsErrorsT = None) -> None: self._reconnect = True self._disable_reconnect_timer() self._set_state(ClientState.CONNECTING) + self.notify('state-changed', SimpleClientState.CONNECTING) if warn_about_plain_connection(self._account, self._client.connection_types): ===================================== gajim/common/const.py ===================================== @@ -304,18 +304,23 @@ def is_available(self): class SimpleClientState(Enum): + CONNECTING = 'connecting' DISCONNECTED = 'disconnected' CONNECTED = 'connected' RESUME_IN_PROGRESS = 'resume-in-progress' @property - def is_disconnected(self): - return self == SimpleClientState.DISCONNECTED + def is_connecting(self): + return self == SimpleClientState.CONNECTING @property def is_connected(self): return self == SimpleClientState.CONNECTED + @property + def is_disconnected(self): + return self == SimpleClientState.DISCONNECTED + @property def is_resume_in_progress(self): return self == SimpleClientState.RESUME_IN_PROGRESS View it on GitLab: https://dev.gajim.org/gajim/gajim/-/commit/bab39049531ea50ff488035b134ef30c… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/-/commit/bab39049531ea50ff488035b134ef30c… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Remove support for server announce
by Philipp Hörist (＠lovetox) 02 Feb '22

02 Feb '22

Philipp Hörist pushed to branch master at gajim / gajim Commits: 318001e5 by lovetox at 2022-02-02T22:53:40+01:00 Remove support for server announce The functionality is available via adhoc commands - - - - - 3 changed files: - − gajim/common/modules/announce.py - gajim/gtk/application.py - gajim/gtk/menus.py Changes: ===================================== gajim/common/modules/announce.py deleted ===================================== @@ -1,33 +0,0 @@ -# This file is part of Gajim. -# -# Gajim is free software; you can redistribute it and/or modify -# it under the terms of the GNU General Public License as published -# by the Free Software Foundation; version 3 only. -# -# Gajim is distributed in the hope that it will be useful, -# but WITHOUT ANY WARRANTY; without even the implied warranty of -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -# GNU General Public License for more details. -# -# You should have received a copy of the GNU General Public License -# along with Gajim. If not, see <http://www.gnu.org/licenses/>. - -# Server MOTD and Announce - -import nbxmpp - -from gajim.common.modules.base import BaseModule - - -class Announce(BaseModule): - def __init__(self, con): - BaseModule.__init__(self, con) - - def delete_motd(self): - server = self._con.get_own_jid().domain - jid = '%s/announce/motd/delete' % server - self.set_announce(jid) - - def set_announce(self, jid, subject=None, body=None): - message = nbxmpp.Message(to=jid, body=body, subject=subject) - self._nbxmpp().send(message) ===================================== gajim/gtk/application.py ===================================== @@ -445,10 +445,6 @@ def _get_account_actions(self, ('-pep-config', self._on_pep_config_action, 'online', 's'), ('-sync-history', self._on_sync_history_action, 'online', 's'), ('-blocking', self._on_blocking_action, 'feature', 's'), - ('-send-server-message', self._on_send_server_message_action, 'online', 's'), - ('-set-motd', self._on_set_motd_action, 'online', 's'), - ('-update-motd', self._on_update_motd_action, 'online', 's'), - ('-delete-motd', self._on_delete_motd_action, 'online', 's'), ('-open-event', self._on_open_event_action, 'always', 'a{sv}'), ('-mark-as-read', self._on_mark_as_read_action, 'always', 'a{sv}'), ('-import-contacts', self._on_import_contacts_action, 'online', 's'), @@ -577,14 +573,6 @@ def _on_profile_action(_action: Gio.SimpleAction, account = param.get_string() open_window('ProfileWindow', account=account) - @staticmethod - def _on_send_server_message_action(_action: Gio.SimpleAction, - param: GLib.Variant) -> None: - account = param.get_string() - server = app.settings.get_account_setting(account, 'hostname') - server += '/announce/online' - open_window('SingleMessageWindow', account=account, recipients=server) - @staticmethod def _on_services_action(_action: Gio.SimpleAction, param: GLib.Variant) -> None: @@ -676,28 +664,6 @@ def _on_manage_proxies_action(_action: Gio.SimpleAction, _param: Optional[GLib.Variant]) -> None: open_window('ManageProxies') - @staticmethod - def _on_set_motd_action(_action: Gio.SimpleAction, - param: GLib.Variant) -> None: - account = param.get_string() - server = app.settings.get_account_setting(account, 'hostname') - server += '/announce/motd' - open_window('SingleMessageWindow', account=account, recipients=server) - - @staticmethod - def _on_update_motd_action(_action: Gio.SimpleAction, - param: GLib.Variant) -> None: - account = param.get_string() - server = app.settings.get_account_setting(account, 'hostname') - server += '/announce/motd/update' - open_window('SingleMessageWindow', account=account, recipients=server) - - @staticmethod - def _on_delete_motd_action(_action: Gio.SimpleAction, - param: GLib.Variant) -> None: - account = param.get_string() - app.connections[account].get_module('Announce').delete_motd() - @staticmethod def _on_content_action(_action: Gio.SimpleAction, _param: Optional[GLib.Variant]) -> None: ===================================== gajim/gtk/menus.py ===================================== @@ -191,15 +191,6 @@ def get_account_menu(account: str) -> Gio.Menu: menu.append_submenu(_('Advanced'), make_menu(advanced_menuitems)) - admin_menuitems: MenuItemListT = [ - (_('Send Server Message…'), f'app.{account}-send-server-message', val), - (_('Set MOTD…'), f'app.{account}-set-motd', val), - (_('Update MOTD…'), f'app.{account}-update-motd', val), - (_('Delete MOTD…'), f'app.{account}-delete-motd', val) - ] - - menu.append_submenu(_('Admin'), make_menu(admin_menuitems)) - return menu View it on GitLab: https://dev.gajim.org/gajim/gajim/-/commit/318001e558b14863e0ce7ca2298fcc85… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/-/commit/318001e558b14863e0ce7ca2298fcc85… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Update pyrightconfig.json
by Philipp Hörist (＠lovetox) 02 Feb '22

02 Feb '22

Philipp Hörist pushed to branch master at gajim / gajim Commits: a6615913 by lovetox at 2022-02-02T22:49:02+01:00 Update pyrightconfig.json - - - - - 1 changed file: - pyrightconfig.json Changes: ===================================== pyrightconfig.json ===================================== @@ -71,6 +71,7 @@ "gajim/gtk/structs.py", "gajim/gtk/themes.py", "gajim/gtk/tooltips.py", + "gajim/gtk/util.py", "gajim/gtk/video_preview.py", "gajim/gtk/workspace_dialog.py", ], View it on GitLab: https://dev.gajim.org/gajim/gajim/-/commit/a6615913fa489ba56b6f38e33d65ccb6… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/-/commit/a6615913fa489ba56b6f38e33d65ccb6… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] 2 commits: util: Add type annotations
by Philipp Hörist (＠lovetox) 02 Feb '22

02 Feb '22

Philipp Hörist pushed to branch master at gajim / gajim Commits: 5c91d145 by lovetox at 2022-02-02T22:45:07+01:00 util: Add type annotations - - - - - 974b8dd3 by lovetox at 2022-02-02T22:45:17+01:00 Remove unused method - - - - - 1 changed file: - gajim/gtk/util.py Changes: ===================================== gajim/gtk/util.py ===================================== @@ -275,9 +275,11 @@ def get_total_screen_geometry() -> tuple[int, int]: total_width = 0 total_height = 0 display = Gdk.Display.get_default() + assert display is not None monitors = display.get_n_monitors() for num in range(0, monitors): monitor = display.get_monitor(num) + assert monitor is not None geometry = monitor.get_geometry() total_width += geometry.width total_height = max(total_height, geometry.height) @@ -358,6 +360,7 @@ def get_completion_liststore(entry: Gtk.Entry) -> Gtk.ListStore: def get_cursor(name: str) -> Gdk.Cursor: display = Gdk.Display.get_default() + assert display is not None cursor = Gdk.Cursor.new_from_name(display, name) if cursor is not None: return cursor @@ -412,13 +415,6 @@ def get_image_button(icon_name: str, tooltip: str, return button -def get_image_from_icon_name(icon_name: str, scale: int) -> Any: - icon_theme = Gtk.IconTheme.get_default() - icon = get_icon_name(icon_name) - surface = icon_theme.load_surface(icon, 16, scale, None, 0) - return Gtk.Image.new_from_surface(surface) - - def python_month(month: int) -> int: return month + 1 @@ -460,6 +456,7 @@ def make_rgba(color_string: str) -> Gdk.RGBA: def get_monitor_scale_factor() -> int: display = Gdk.Display.get_default() + assert display is not None monitor = display.get_primary_monitor() if monitor is None: log.warning('Could not determine scale factor') @@ -477,6 +474,7 @@ def get_primary_accel_mod() -> Optional[Gdk.ModifierType]: def get_hardware_key_codes(keyval: int) -> list[int]: display = Gdk.Display.get_default() + assert display is not None keymap = Gdk.Keymap.get_for_display(display) valid, key_map_keys = keymap.get_entries_for_keyval(keyval) @@ -485,9 +483,9 @@ def get_hardware_key_codes(keyval: int) -> list[int]: return [key.keycode for key in key_map_keys] -def ensure_not_destroyed(func): +def ensure_not_destroyed(func: Any) -> Any: @wraps(func) - def func_wrapper(self, *args, **kwargs): + def func_wrapper(self: Any, *args: Any, **kwargs: Any): if self._destroyed: # pylint: disable=protected-access return None return func(self, *args, **kwargs) @@ -513,9 +511,9 @@ def get_account_tune_icon_name(account: str) -> Optional[str]: def format_location(location: LocationData) -> str: - location = location._asdict() + location_dict = location._asdict() location_string = '' - for attr, value in location.items(): + for attr, value in location_dict.items(): if value is None: continue text = GLib.markup_escape_text(value) @@ -599,7 +597,7 @@ def text_to_color(text: str) -> tuple[float, float, float]: background = (0, 0, 0) # RGB (0, 0, 0) black else: background = (1, 1, 1) # RGB (255, 255, 255) white - return nbxmpp.util.text_to_color(text, background) + return nbxmpp.util.text_to_color(text, background) # type: ignore def get_color_for_account(account: str) -> str: @@ -646,7 +644,7 @@ def get_pixbuf_from_data(file_data: bytes) -> Optional[GdkPixbuf.Pixbuf]: 'convert avatar image using pillow') try: avatar = Image.open(BytesIO(file_data)).convert("RGBA") - array = GLib.Bytes.new(avatar.tobytes()) + array = GLib.Bytes.new(avatar.tobytes()) # type: ignore width, height = avatar.size pixbuf = GdkPixbuf.Pixbuf.new_from_bytes( array, GdkPixbuf.Colorspace.RGB, @@ -683,6 +681,7 @@ def scale_pixbuf_from_data(data: bytes, size: int ) -> Optional[GdkPixbuf.Pixbuf]: pixbuf = get_pixbuf_from_data(data) + assert pixbuf is not None return scale_pixbuf(pixbuf, size) @@ -705,7 +704,7 @@ def load_pixbuf(path: Union[str, Path], log.debug('Error', exc_info=True) return None - array = GLib.Bytes.new(avatar.tobytes()) + array = GLib.Bytes.new(avatar.tobytes()) # type: ignore width, height = avatar.size pixbuf = GdkPixbuf.Pixbuf.new_from_bytes( array, GdkPixbuf.Colorspace.RGB, True, @@ -746,7 +745,7 @@ def make_href_markup(string: str) -> str: assert isinstance(url_color, str) color = convert_rgb_to_hex(url_color) - def _to_href(match: Match) -> str: + def _to_href(match: Match[str]) -> str: url = match.group() if '://' not in url: url = 'https://' + url @@ -760,7 +759,7 @@ def get_app_windows(account: str) -> list[Gtk.Window]: windows: list[Gtk.Window] = [] for win in app.app.get_windows(): if hasattr(win, 'account'): - if win.account == account: + if win.account == account: # type: ignore windows.append(win) return windows @@ -774,11 +773,11 @@ def get_app_window(name: str, continue if account is not None: - if account != win.account: + if account != win.account: # type: ignore continue if jid is not None: - if jid != win.jid: + if jid != win.jid: # type: ignore continue return win return None @@ -800,7 +799,7 @@ def open_window(name: str, **kwargs: Any) -> Any: class EventHelper(CommonEventHelper): def __init__(self): CommonEventHelper.__init__(self) - self.connect('destroy', self.__on_destroy) # pylint: disable=no-member + self.connect('destroy', self.__on_destroy) # type: ignore def __on_destroy(self, *args: Any) -> None: self.unregister_events() @@ -812,7 +811,12 @@ def _destroy(*args: Any) -> None: widget.connect('destroy', _destroy) -def _connect_destroy(sender, func, detailed_signal, handler, *args, **kwargs): +def _connect_destroy(sender: Any, + func: Any, + detailed_signal: str, + handler: Any, + *args: Any, + **kwargs: Any) -> int: """Connect a bound method to a foreign object signal and disconnect if the object the method is bound to emits destroy (Gtk.Widget subclass). Also works if the handler is a nested function in a method and @@ -834,25 +838,27 @@ def _connect_destroy(sender, func, detailed_signal, handler, *args, **kwargs): handler_id = func(detailed_signal, handler, *args, **kwargs) - def disconnect_cb(*args): + def disconnect_cb(*args: Any) -> None: sender.disconnect(handler_id) obj.connect('destroy', disconnect_cb) return handler_id -def connect_destroy(sender, *args, **kwargs): +def connect_destroy(sender: Any, *args: Any, **kwargs: Any) -> int: return _connect_destroy(sender, sender.connect, *args, **kwargs) -def wrap_with_event_box(klass): +def wrap_with_event_box(klass: Any) -> Any: @wraps(klass) - def klass_wrapper(*args, **kwargs): + def klass_wrapper(*args: Any, **kwargs: Any) -> Gtk.EventBox: widget = klass(*args, **kwargs) event_box = Gtk.EventBox() - def _on_realize(*args): - event_box.get_window().set_cursor(get_cursor('pointer')) + def _on_realize(*args: Any) -> None: + window = event_box.get_window() + assert window is not None + window.set_cursor(get_cursor('pointer')) event_box.connect_after('realize', _on_realize) event_box.add(widget) View it on GitLab: https://dev.gajim.org/gajim/gajim/-/compare/2d25dd85f2d3042d6dbcdf2fa9b0c92… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/-/compare/2d25dd85f2d3042d6dbcdf2fa9b0c92… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] 3 commits: StatusMessageSelector: Cleanup correctly when it gets destroyed
by Philipp Hörist (＠lovetox) 02 Feb '22

02 Feb '22

Philipp Hörist pushed to branch master at gajim / gajim Commits: 34c06500 by lovetox at 2022-02-02T21:40:36+01:00 StatusMessageSelector: Cleanup correctly when it gets destroyed - - - - - 20831fc8 by lovetox at 2022-02-02T21:41:05+01:00 StatusSelector: Cleanup correctly when it gets destroyed - - - - - 2d25dd85 by lovetox at 2022-02-02T21:44:01+01:00 AccountSideBar: Support all client states - Simplify code - - - - - 3 changed files: - gajim/gtk/account_side_bar.py - gajim/gtk/status_message_selector.py - gajim/gtk/status_selector.py Changes: ===================================== gajim/gtk/account_side_bar.py ===================================== @@ -14,14 +14,15 @@ from __future__ import annotations +from typing import Any from typing import Optional from typing import cast from gi.repository import Gtk -from gajim.common.const import AvatarSize from gajim.common import app -from gajim.common import types +from gajim.common.const import AvatarSize +from gajim.common.helpers import get_client_status from gajim.common.i18n import _ @@ -105,33 +106,34 @@ def update(self) -> None: class AccountAvatar(Gtk.Image): def __init__(self, account: str) -> None: Gtk.Image.__init__(self) + self._account = account jid = app.get_jid_from_account(self._account) - client = app.get_client(self._account) - self._contact = client.get_module('Contacts').get_contact(jid) - self._contact.connect('avatar-update', self._on_avatar_update) - self._contact.connect('presence-update', self._on_presence_update) + self._client = app.get_client(self._account) - self.connect('destroy', self._on_destroy) - self._update_image() + self._client.connect_signal('state-changed', self._on_event) + + self._contact = self._client.get_module('Contacts').get_contact(jid) + self._contact.connect('avatar-update', self._on_event) + self._contact.connect('presence-update', self._on_event) - def _on_presence_update(self, - _contact: types.BareContact, - _signal_name: str) -> None: + self.connect('destroy', self._on_destroy) self._update_image() - def _on_avatar_update(self, - _contact: types.BareContact, - _signal_name: str) -> None: + def _on_event(self, *args: Any) -> None: self._update_image() def _update_image(self) -> None: - surface = self._contact.get_avatar(AvatarSize.ACCOUNT_SIDE_BAR, - self.get_scale_factor(), - style='circle') + status = get_client_status(self._account) + surface = app.app.avatar_storage.get_surface( + self._contact, + AvatarSize.ACCOUNT_SIDE_BAR, + self.get_scale_factor(), + status) self.set_from_surface(surface) def _on_destroy(self, _widget: Gtk.Image) -> None: self._contact.disconnect_all_from_obj(self) + self._client.disconnect_all_from_obj(self) app.check_finalize(self) ===================================== gajim/gtk/status_message_selector.py ===================================== @@ -12,6 +12,8 @@ # You should have received a copy of the GNU General Public License # along with Gajim. If not, see <http://www.gnu.org/licenses/>. +from __future__ import annotations + from typing import Any from typing import Optional @@ -26,10 +28,13 @@ from gajim.common.helpers import to_one_line from gajim.common.i18n import _ +from .util import EventHelper + -class StatusMessageSelector(Gtk.Box): +class StatusMessageSelector(Gtk.Box, EventHelper): def __init__(self, account: Optional[str] = None) -> None: Gtk.Box.__init__(self) + EventHelper.__init__(self) self.get_style_context().add_class('linked') self._account = account @@ -46,14 +51,15 @@ def __init__(self, account: Optional[str] = None) -> None: self._button.connect('clicked', self._set_status_message) self.add(self._entry) self.add(self._button) + + self.connect('destroy', self._on_destroy) + self.show_all() - app.ged.register_event_handler('our-show', - ged.POSTGUI, - self._on_our_show) - app.ged.register_event_handler('account-enabled', - ged.POSTGUI, - self._on_account_enabled) + self.register_event('our-show', ged.GUI1, self._on_our_show) + self.register_event('account-enabled', + ged.GUI1, + self._on_account_enabled) for client in app.get_clients(): client.connect_signal('state-changed', @@ -100,3 +106,6 @@ def update(self) -> None: message = client.status_message self._entry.set_text(message) + + def _on_destroy(self, widget: StatusMessageSelector) -> None: + app.check_finalize(self) ===================================== gajim/gtk/status_selector.py ===================================== @@ -32,11 +32,13 @@ from gajim.common.i18n import _ from .avatar import get_show_circle +from .util import EventHelper -class StatusSelector(Gtk.MenuButton): +class StatusSelector(Gtk.MenuButton, EventHelper): def __init__(self, account: Optional[str] = None, compact: bool = False): Gtk.MenuButton.__init__(self) + EventHelper.__init__(self) self.set_direction(Gtk.ArrowType.UP) self._account = account self._compact = compact @@ -60,12 +62,12 @@ def __init__(self, account: Optional[str] = None, compact: bool = False): box.show_all() self.add(box) - app.ged.register_event_handler('our-show', - ged.POSTGUI, - self._on_our_show) - app.ged.register_event_handler('account-enabled', - ged.POSTGUI, - self._on_account_enabled) + self.connect('destroy', self._on_destroy) + + self.register_event('our-show', ged.GUI1, self._on_our_show) + self.register_event('account-enabled', + ged.GUI1, + self._on_account_enabled) for client in app.get_clients(): client.connect_signal('state-changed', @@ -153,3 +155,7 @@ def update(self) -> None: _('Status: %s') % show_label) if not self._compact: self._current_show_label.set_text(show_label) + + def _on_destroy(self, widget: StatusSelector) -> None: + self._status_popover.destroy() + app.check_finalize(self) View it on GitLab: https://dev.gajim.org/gajim/gajim/-/compare/ba910b98c4d8f5546e8af8ac65bd76e… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/-/compare/ba910b98c4d8f5546e8af8ac65bd76e… 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