Commits November 2018

commits@gajim.org

1 participants
84 discussions

[Git][gajim/gajim][master] 5 commits: Move htmltextview into gtk module
by Philipp Hörist 18 Nov '18

18 Nov '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: cee63aa0 by Philipp Hörist at 2018-11-18T09:35:13Z Move htmltextview into gtk module - - - - - 431d8075 by Philipp Hörist at 2018-11-18T09:35:13Z Use get_cursor from gtk.util - - - - - b5b38203 by Philipp Hörist at 2018-11-18T09:35:13Z Remove unused methods - - - - - f40f171e by Philipp Hörist at 2018-11-18T09:35:13Z Refactor parse_datetime and add unit test - - - - - d13baed8 by Philipp Hörist at 2018-11-18T09:35:13Z Improve Entity Time parsing - Use parse_datetime() - Improve tzo node validation - - - - - 7 changed files: - gajim/chat_control.py - gajim/common/modules/date_and_time.py - gajim/common/modules/entity_time.py - gajim/conversation_textview.py - gajim/htmltextview.py → gajim/gtk/htmltextview.py - gajim/gtkgui_helpers.py - + test/no_gui/unit/test_date_time.py Changes: ===================================== gajim/chat_control.py ===================================== @@ -55,6 +55,7 @@ from gajim import dialogs from gajim.gtk.dialogs import ConfirmationDialog from gajim.gtk.add_contact import AddNewContactWindow from gajim.gtk.util import get_icon_name +from gajim.gtk.util import get_cursor from gajim.command_system.implementation.hosts import ChatCommands from gajim.command_system.framework import CommandHost # pylint: disable=unused-import @@ -596,11 +597,11 @@ class ChatControl(ChatControlBase): """ Just moved the mouse so show the cursor """ - cursor = gtkgui_helpers.get_cursor('LEFT_PTR') + cursor = get_cursor('LEFT_PTR') self.parent_win.window.get_window().set_cursor(cursor) def on_location_eventbox_enter_notify_event(self, widget, event): - cursor = gtkgui_helpers.get_cursor('HAND2') + cursor = get_cursor('HAND2') self.parent_win.window.get_window().set_cursor(cursor) def update_ui(self): ===================================== gajim/common/modules/date_and_time.py ===================================== @@ -16,8 +16,13 @@ import re import time -from datetime import datetime, timedelta, timezone, tzinfo +import logging +from datetime import datetime +from datetime import timedelta +from datetime import timezone +from datetime import tzinfo +log = logging.getLogger('gajim.c.m.date_and_time') PATTERN_DATETIME = re.compile( r'([0-9]{4}-[0-9]{2}-[0-9]{2})' @@ -91,6 +96,33 @@ class LocalTimezone(tzinfo): return tt.tm_isdst > 0 +def create_tzinfo(hours=0, minutes=0, tz_string=None): + if tz_string is None: + return timezone(timedelta(hours=hours, minutes=minutes)) + + if tz_string.lower() == 'z': + return timezone.utc + + try: + hours, minutes = map(int, tz_string.split(':')) + except Exception: + log.warning('Wrong tz string: %s', tz_string) + return + + if hours not in range(-24, 24): + log.warning('Wrong tz string: %s', tz_string) + return + + if minutes not in range(0, 59): + log.warning('Wrong tz string: %s', tz_string) + return + + if hours in (24, -24) and minutes != 0: + log.warning('Wrong tz string: %s', tz_string) + return + return timezone(timedelta(hours=hours, minutes=minutes)) + + def parse_datetime(timestring, check_utc=False, convert='utc', epoch=False): ''' @@ -133,11 +165,30 @@ def parse_datetime(timestring, check_utc=False, except ValueError: pass else: - if not check_utc and convert == 'utc': + if check_utc: + if convert != 'utc': + raise ValueError( + 'check_utc can only be used with convert="utc"') + date_time.replace(tzinfo=timezone.utc) + if epoch: + return date_time.timestamp() + return date_time + + if convert == 'utc': date_time = date_time.astimezone(timezone.utc) + if epoch: + return date_time.timestamp() + return date_time + + if epoch: + # epoch is always UTC, use convert='utc' or check_utc=True + raise ValueError( + 'epoch not available while converting to local') + if convert == 'local': date_time = date_time.astimezone(LocalTimezone()) - if epoch: - return date_time.timestamp() + return date_time + + # convert=None return date_time return None ===================================== gajim/common/modules/entity_time.py ===================================== @@ -15,13 +15,14 @@ # XEP-0202: Entity Time import logging -import datetime import time import nbxmpp from gajim.common import app -from gajim.common.nec import NetworkIncomingEvent +from gajim.common.nec import NetworkEvent +from gajim.common.modules.date_and_time import parse_datetime +from gajim.common.modules.date_and_time import create_tzinfo log = logging.getLogger('gajim.c.m.entity_time') @@ -61,10 +62,10 @@ class EntityTime: log.info('Received: %s %s', stanza.getFrom(), time_info) - app.nec.push_incoming_event( - TimeResultReceivedEvent(None, conn=self._con, - jid=stanza.getFrom(), - time_info=time_info)) + app.nec.push_incoming_event(NetworkEvent('time-result-received', + conn=self._con, + jid=stanza.getFrom(), + time_info=time_info)) @staticmethod def _extract_info(stanza): @@ -74,42 +75,24 @@ class EntityTime: return tzo = time_.getTag('tzo').getData() - if tzo.lower() == 'z': - tzo = '0:0' - try: - tzoh, tzom = tzo.split(':') - except Exception: + if not tzo: log.warning('Wrong tzo node: %s', stanza) return - utc_time = time_.getTag('utc').getData() - if utc_time[-1:] == 'Z': - # Remove the trailing 'Z' - utc_time = utc_time[:-1] - elif utc_time[-6:] == "+00:00": - # Remove the trailing "+00:00" - utc_time = utc_time[:-6] - else: + remote_tz = create_tzinfo(tz_string=tzo) + if remote_tz is None: + log.warning('Wrong tzo node: %s', stanza) + return + + utc_time = time_.getTag('utc').getData() + date_time = parse_datetime(utc_time, check_utc=True) + if date_time is None: log.warning('Wrong timezone defintion: %s %s', utc_time, stanza.getFrom()) return - try: - dtime = datetime.datetime.strptime(utc_time, '%Y-%m-%dT%H:%M:%S') - except ValueError: - try: - dtime = datetime.datetime.strptime(utc_time, - '%Y-%m-%dT%H:%M:%S.%f') - except ValueError as error: - log.warning('Wrong time format: %s %s', - error, stanza.getFrom()) - return - - utc = datetime.timezone(datetime.timedelta(0)) - dtime = dtime.replace(tzinfo=utc) - utc_offset = datetime.timedelta(hours=int(tzoh), minutes=int(tzom)) - contact_tz = datetime.timezone(utc_offset, "remote timezone") - return dtime.astimezone(contact_tz).strftime('%c') + date_time = date_time.astimezone(remote_tz) + return date_time.strftime('%c %Z') def _answer_request(self, _con, stanza): log.info('%s asked for the time', stanza.getFrom()) @@ -132,9 +115,5 @@ class EntityTime: raise nbxmpp.NodeProcessed -class TimeResultReceivedEvent(NetworkIncomingEvent): - name = 'time-result-received' - - def get_instance(*args, **kwargs): return EntityTime(*args, **kwargs), 'EntityTime' ===================================== gajim/conversation_textview.py ===================================== @@ -42,7 +42,7 @@ from gajim.common.i18n import _ from gajim.common.fuzzyclock import FuzzyClock from gajim.common.const import StyleAttr -from gajim.htmltextview import HtmlTextView +from gajim.gtk.htmltextview import HtmlTextView from gajim.gtk import util from gajim.gtk.util import load_icon ===================================== gajim/htmltextview.py → gajim/gtk/htmltextview.py ===================================== ===================================== gajim/gtkgui_helpers.py ===================================== @@ -139,11 +139,6 @@ def get_pixbuf_from_data(file_data): return pixbuf -def get_cursor(attr): - display = Gdk.Display.get_default() - cursor = getattr(Gdk.CursorType, attr) - return Gdk.Cursor.new_for_display(display, cursor) - def file_is_locked(path_to_file): """ Return True if file is locked @@ -176,33 +171,6 @@ def file_is_locked(path_to_file): hfile.Close() return False -def get_fade_color(treeview, selected, focused): - """ - Get a gdk RGBA color that is between foreground and background in 0.3 - 0.7 respectively colors of the cell for the given treeview - """ - context = treeview.get_style_context() - if selected: - if focused: # is the window focused? - state = Gtk.StateFlags.SELECTED - else: # is it not? NOTE: many gtk themes change bg on this - state = Gtk.StateFlags.ACTIVE - else: - state = Gtk.StateFlags.NORMAL - - bg = context.get_property('background-color', state) - fg = context.get_color(state) - - p = 0.3 # background - q = 0.7 # foreground # p + q should do 1.0 - return Gdk.RGBA(bg.red*p + fg.red*q, bg.green*p + fg.green*q, - bg.blue*p + fg.blue*q) - -def make_pixbuf_grayscale(pixbuf): - pixbuf2 = pixbuf.copy() - pixbuf.saturate_and_pixelate(pixbuf2, 0.0, False) - return pixbuf2 - def get_possible_button_event(event): """ Mouse or keyboard caused the event? ===================================== test/no_gui/unit/test_date_time.py ===================================== @@ -0,0 +1,99 @@ +import unittest +from datetime import datetime +from datetime import timezone +from datetime import timedelta + +from gajim.common.modules.date_and_time import parse_datetime +from gajim.common.modules.date_and_time import LocalTimezone +from gajim.common.modules.date_and_time import create_tzinfo + + +class TestDateTime(unittest.TestCase): + + def test_convert_to_utc(self): + + strings = { + # Valid UTC strings and fractions + '2017-11-05T01:41:20Z': 1509846080.0, + '2017-11-05T01:41:20.123Z': 1509846080.123, + '2017-11-05T01:41:20.123123123+00:00': 1509846080.123123, + '2017-11-05T01:41:20.123123123123123-00:00': 1509846080.123123, + + # Invalid strings + '2017-11-05T01:41:20Z+05:00': None, + '2017-11-05T01:41:20+0000': None, + '2017-11-05T01:41:20-0000': None, + + # Valid strings with offset + '2017-11-05T01:41:20-05:00': 1509864080.0, + '2017-11-05T01:41:20+05:00': 1509828080.0, + } + + strings2 = { + # Valid strings with offset + '2017-11-05T01:41:20-05:00': datetime(2017, 11, 5, 1, 41, 20, 0, create_tzinfo(hours=-5)), + '2017-11-05T01:41:20+05:00': datetime(2017, 11, 5, 1, 41, 20, 0, create_tzinfo(hours=5)), + } + + for time_string, expected_value in strings.items(): + result = parse_datetime(time_string, convert='utc', epoch=True) + self.assertEqual(result, expected_value) + + for time_string, expected_value in strings2.items(): + result = parse_datetime(time_string, convert='utc') + self.assertEqual(result, expected_value.astimezone(timezone.utc)) + + def test_convert_to_local(self): + + strings = { + # Valid UTC strings and fractions + '2017-11-05T01:41:20Z': datetime(2017, 11, 5, 1, 41, 20, 0, timezone.utc), + '2017-11-05T01:41:20.123Z': datetime(2017, 11, 5, 1, 41, 20, 123000, timezone.utc), + '2017-11-05T01:41:20.123123123+00:00': datetime(2017, 11, 5, 1, 41, 20, 123123, timezone.utc), + '2017-11-05T01:41:20.123123123123123-00:00': datetime(2017, 11, 5, 1, 41, 20, 123123, timezone.utc), + + # Valid strings with offset + '2017-11-05T01:41:20-05:00': datetime(2017, 11, 5, 1, 41, 20, 0, create_tzinfo(hours=-5)), + '2017-11-05T01:41:20+05:00': datetime(2017, 11, 5, 1, 41, 20, 0, create_tzinfo(hours=5)), + } + + for time_string, expected_value in strings.items(): + result = parse_datetime(time_string, convert='local') + self.assertEqual(result, expected_value.astimezone(LocalTimezone())) + + def test_no_convert(self): + + strings = { + # Valid UTC strings and fractions + '2017-11-05T01:41:20Z': timedelta(0), + '2017-11-05T01:41:20.123Z': timedelta(0), + '2017-11-05T01:41:20.123123123+00:00': timedelta(0), + '2017-11-05T01:41:20.123123123123123-00:00': timedelta(0), + + # Valid strings with offset + '2017-11-05T01:41:20-05:00': timedelta(hours=-5), + '2017-11-05T01:41:20+05:00': timedelta(hours=5), + } + + for time_string, expected_value in strings.items(): + result = parse_datetime(time_string, convert=None) + self.assertEqual(result.utcoffset(), expected_value) + + def test_check_utc(self): + + strings = { + # Valid UTC strings and fractions + '2017-11-05T01:41:20Z': 1509846080.0, + '2017-11-05T01:41:20.123Z': 1509846080.123, + '2017-11-05T01:41:20.123123123+00:00': 1509846080.123123, + '2017-11-05T01:41:20.123123123123123-00:00': 1509846080.123123, + + # Valid strings with offset + '2017-11-05T01:41:20-05:00': None, + '2017-11-05T01:41:20+05:00': None, + } + + for time_string, expected_value in strings.items(): + result = parse_datetime( + time_string, check_utc=True, epoch=True) + self.assertEqual(result, expected_value) View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/a1b798b8fcbf8cae80a552de7728b74c0… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/a1b798b8fcbf8cae80a552de7728b74c0… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] 3 commits: Test: Fix KeyError
by Philipp Hörist 17 Nov '18

17 Nov '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: 5c0ff1de by André Apitzsch at 2018-11-17T10:23:38Z Test: Fix KeyError The added account causes an issue with test_gui_interface, so the account needs to be removed again - - - - - dd0b7260 by André Apitzsch at 2018-11-17T10:25:24Z CI: Make test verbose - - - - - a1b798b8 by André Apitzsch at 2018-11-17T10:25:24Z Remove useless comment - - - - - 3 changed files: - .gitlab-ci.yml - test/no_gui/unit/test_protocol_caps.py - test/unit/test_gui_interface.py Changes: ===================================== .gitlab-ci.yml ===================================== @@ -10,7 +10,7 @@ stages: run-test: stage: test script: - - python3 setup.py test -s test.no_gui -q + - python3 setup.py test -s test.no_gui run-mypy: stage: test ===================================== test/no_gui/unit/test_protocol_caps.py ===================================== @@ -31,6 +31,9 @@ class TestConnectionCaps(unittest.TestCase): self.module._account = 'account' self.module._capscache = MagicMock() + def tearDown(self): + app.contacts.remove_account('account') + def _nec_caps_presence_received(self, obj): self.assertTrue( isinstance(obj.client_caps, caps_cache.ClientCaps), ===================================== test/unit/test_gui_interface.py ===================================== @@ -53,5 +53,4 @@ class TestInterface(unittest.TestCase): if __name__ == "__main__": - #import sys;sys.argv = ['', 'Test.test'] unittest.main() View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/fb5cee69cbd11bf37ec53c4ee49a62288… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/fb5cee69cbd11bf37ec53c4ee49a62288… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] 4 commits: HtmlTextView: Remove some unnecessary methods
by Philipp Hörist 16 Nov '18

16 Nov '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: 97310a05 by Philipp Hörist at 2018-11-16T20:36:04Z HtmlTextView: Remove some unnecessary methods - - - - - eedd0c2a by Philipp Hörist at 2018-11-16T20:39:19Z HtmlTextView: Fix pylint errors and minor refactoring - - - - - 80954e95 by Philipp Hörist at 2018-11-16T20:39:45Z HtmlTextView: minor refactoring - Move updating tags into HtmlTextView - Rename some vars - Remove legacy GTK code - - - - - fb5cee69 by Philipp Hörist at 2018-11-16T20:44:20Z Dont use deprecated begins_tag() - - - - - 2 changed files: - gajim/conversation_textview.py - gajim/htmltextview.py Changes: ===================================== gajim/conversation_textview.py ===================================== @@ -172,7 +172,7 @@ class ConversationTextview(GObject.GObject): # no need to inherit TextView, use it as atrribute is safer self.tv = HtmlTextView(account) - self.tv.connect_tooltip(self.query_tooltip) + self.tv.connect('query-tooltip', self._query_tooltip) # set properties self.tv.set_border_width(1) @@ -199,7 +199,7 @@ class ConversationTextview(GObject.GObject): self.handlers[id_] = self.tv self.account = account - self.cursor_changed = False + self._cursor_changed = False self.last_time_printout = 0 self.encryption_enabled = False @@ -304,14 +304,12 @@ class ConversationTextview(GObject.GObject): self.just_cleared = False - def query_tooltip(self, widget, x_pos, y_pos, keyboard_mode, tooltip): + def _query_tooltip(self, widget, x_pos, y_pos, keyboard_mode, tooltip): window = widget.get_window(Gtk.TextWindowType.TEXT) x_pos, y_pos = self.tv.window_to_buffer_coords( Gtk.TextWindowType.TEXT, x_pos, y_pos) - if Gtk.MINOR_VERSION > 18: - iter_ = self.tv.get_iter_at_position(x_pos, y_pos)[1] - else: - iter_ = self.tv.get_iter_at_position(x_pos, y_pos)[0] + + iter_ = self.tv.get_iter_at_position(x_pos, y_pos)[1] for tag in iter_.get_tags(): tag_name = tag.get_property('name') if tag_name == 'focus-out-line': @@ -327,11 +325,11 @@ class ConversationTextview(GObject.GObject): text = text[:47] + '…' tooltip.set_text(text) window.set_cursor(get_cursor('HAND2')) - self.cursor_changed = True + self._cursor_changed = True return True if tag_name in ('url', 'mail', 'xmpp', 'sth_at_sth'): window.set_cursor(get_cursor('HAND2')) - self.cursor_changed = True + self._cursor_changed = True return False try: text = self.corrected_text_list[tag_name] @@ -339,9 +337,9 @@ class ConversationTextview(GObject.GObject): return True except KeyError: pass - if self.cursor_changed: + if self._cursor_changed: window.set_cursor(get_cursor('XTERM')) - self.cursor_changed = False + self._cursor_changed = False return False def del_handlers(self): @@ -358,11 +356,7 @@ class ConversationTextview(GObject.GObject): app.css_config.get_value('.gajim-status-message', StyleAttr.COLOR)) self.tagMarked.set_property('foreground', app.css_config.get_value('.gajim-highlight-message', StyleAttr.COLOR)) - color = app.css_config.get_value('.gajim-url', StyleAttr.COLOR) - self.tv.tagURL.set_property('foreground', color) - self.tv.tagMail.set_property('foreground', color) - self.tv.tagXMPP.set_property('foreground', color) - self.tv.tagSthAtSth.set_property('foreground', color) + self.tv.update_tags() def scroll_to_end(self, force=False): if self.autoscroll or force: ===================================== gajim/htmltextview.py ===================================== @@ -56,34 +56,32 @@ from gajim.gui_menu_builder import get_conv_context_menu log = logging.getLogger('gajim.htmlview') -__all__ = ['HtmlTextView'] - whitespace_rx = re.compile('\\s+') allwhitespace_rx = re.compile('^\\s*$') # embryo of CSS classes classes = { - #'system-message':';display: none', - 'problematic': ';color: red', + #'system-message':';display: none', + 'problematic': ';color: red', } # styles for elements _element_styles = { - 'u' : ';text-decoration: underline', - 'em' : ';font-style: oblique', - 'cite' : '; background-color:rgb(170,190,250);' - 'font-style: oblique', - 'li' : '; margin-left: 1em; margin-right: 10%', - 'strong' : ';font-weight: bold', - 'pre' : '; background-color:rgb(190,190,190);' - 'font-family: monospace; white-space: pre;' - 'margin-left: 1em; margin-right: 10%', - 'kbd' : ';background-color:rgb(210,210,210);' - 'font-family: monospace', - 'blockquote' : '; background-color:rgb(170,190,250);' - 'margin-left: 2em; margin-right: 10%', - 'dt' : ';font-weight: bold; font-style: oblique', - 'dd' : ';margin-left: 2em; font-style: oblique' + 'u' : ';text-decoration: underline', + 'em' : ';font-style: oblique', + 'cite' : '; background-color:rgb(170,190,250);' + 'font-style: oblique', + 'li' : '; margin-left: 1em; margin-right: 10%', + 'strong' : ';font-weight: bold', + 'pre' : '; background-color:rgb(190,190,190);' + 'font-family: monospace; white-space: pre;' + 'margin-left: 1em; margin-right: 10%', + 'kbd' : ';background-color:rgb(210,210,210);' + 'font-family: monospace', + 'blockquote' : '; background-color:rgb(170,190,250);' + 'margin-left: 2em; margin-right: 10%', + 'dt' : ';font-weight: bold; font-style: oblique', + 'dd' : ';margin-left: 2em; font-style: oblique' } # no difference for the moment _element_styles['dfn'] = _element_styles['em'] @@ -93,6 +91,12 @@ _element_styles['tt'] = _element_styles['kbd'] _element_styles['i'] = _element_styles['em'] _element_styles['b'] = _element_styles['strong'] +_supported_style_attrs = [ + 'background-color', 'color', 'font-family', 'font-size', 'font-style', + 'font-weight', 'margin-left', 'margin-right', 'text-align', + 'text-decoration', 'white-space', 'display', 'width', 'height' +] + # ========== # XEP-0071 # ========== @@ -182,8 +186,9 @@ for _name in BLOCK_HEAD: _num = int(_name[1]) _header_size = (_num - 1) // 2 _weight = (_num - 1) % 2 - _element_styles[_name] = '; font-size: %s; %s' % (('large', 'medium', 'small')[_header_size], - ('font-weight: bold', 'font-style: oblique')[_weight],) + _element_styles[_name] = '; font-size: %s; %s' % ( + ('large', 'medium', 'small')[_header_size], + ('font-weight: bold', 'font-style: oblique')[_weight]) def _parse_css_color(color): if color.startswith('rgb(') and color.endswith(')'): @@ -216,26 +221,35 @@ class HtmlHandler(xml.sax.handler.ContentHandler): self.list_counters = [] # stack (top at head) of list # counters, or None for unordered list + # build a dictionary mapping styles to methods + self.__style_methods = {} + for style in _supported_style_attrs: + method_names = '_parse_style_%s' % style.replace('-', '_') + self.__style_methods[style] = method_names + def _get_points_from_pixels(self, pixels): resolution = self.textview.get_screen().get_resolution() # points = pixels * 72 / resolution return pixels * 72 / resolution - def _parse_style_color(self, tag, value): + @staticmethod + def _parse_style_color(tag, value): color = _parse_css_color(value) tag.set_property('foreground-gdk', color) - def _parse_style_background_color(self, tag, value): + @staticmethod + def _parse_style_background_color(tag, value): color = _parse_css_color(value) tag.set_property('background-gdk', color) tag.set_property('paragraph-background-gdk', color) - def __parse_length_frac_size_allocate(self, textview, allocation, frac, - callback, args): + @staticmethod + def __parse_length_frac_size_allocate(_textview, allocation, frac, + callback, args): callback(allocation.width*frac, *args) def _parse_length(self, value, font_relative, block_relative, minl, maxl, - callback, *args): + callback, *args): """ Parse/calc length, converting to pixels, calls callback(length, *args) when the length is first computed or changes @@ -259,11 +273,11 @@ class HtmlHandler(xml.sax.handler.ContentHandler): # This is difficult/impossible to implement, so we use # textview width instead; a reasonable approximation.. alloc = self.textview.get_allocation() - self.__parse_length_frac_size_allocate(self.textview, alloc, - frac, callback, args) + self.__parse_length_frac_size_allocate( + self.textview, alloc, frac, callback, args) self.textview.connect('size-allocate', - self.__parse_length_frac_size_allocate, - frac, callback, args) + self.__parse_length_frac_size_allocate, + frac, callback, args) else: callback(frac, *args) return @@ -324,7 +338,8 @@ class HtmlHandler(xml.sax.handler.ContentHandler): elif type_ == 'px': tag.set_property('size-points', self._get_points_from_pixels(size)) - def _parse_style_display(self, tag, value): + @staticmethod + def _parse_style_display(tag, value): if value == 'none': tag.set_property('invisible', 'true') # FIXME: display: block, inline @@ -355,13 +370,14 @@ class HtmlHandler(xml.sax.handler.ContentHandler): self._parse_length( value, True, False, 5, 110, self.__parse_font_size_cb, tag) - def _parse_style_font_style(self, tag, value): + @staticmethod + def _parse_style_font_style(tag, value): try: style = { - 'normal': Pango.Style.NORMAL, - 'italic': Pango.Style.ITALIC, - 'oblique': Pango.Style.OBLIQUE, - }[value] + 'normal': Pango.Style.NORMAL, + 'italic': Pango.Style.ITALIC, + 'oblique': Pango.Style.OBLIQUE, + }[value] except KeyError: log.warning('unknown font-style %s', value) else: @@ -376,51 +392,55 @@ class HtmlHandler(xml.sax.handler.ContentHandler): def _parse_style_margin_left(self, tag, value): # block relative self._parse_length(value, False, True, 1, 1000, - self.__frac_length_tag_cb, tag, 'left-margin') + self.__frac_length_tag_cb, tag, 'left-margin') def _parse_style_margin_right(self, tag, value): # block relative self._parse_length(value, False, True, 1, 1000, - self.__frac_length_tag_cb, tag, 'right-margin') + self.__frac_length_tag_cb, tag, 'right-margin') - def _parse_style_font_weight(self, tag, value): + @staticmethod + def _parse_style_font_weight(tag, value): # TODO: missing 'bolder' and 'lighter' try: weight = { - '100': Pango.Weight.ULTRALIGHT, - '200': Pango.Weight.ULTRALIGHT, - '300': Pango.Weight.LIGHT, - '400': Pango.Weight.NORMAL, - '500': Pango.Weight.NORMAL, - '600': Pango.Weight.BOLD, - '700': Pango.Weight.BOLD, - '800': Pango.Weight.ULTRABOLD, - '900': Pango.Weight.HEAVY, - 'normal': Pango.Weight.NORMAL, - 'bold': Pango.Weight.BOLD, - }[value] + '100': Pango.Weight.ULTRALIGHT, + '200': Pango.Weight.ULTRALIGHT, + '300': Pango.Weight.LIGHT, + '400': Pango.Weight.NORMAL, + '500': Pango.Weight.NORMAL, + '600': Pango.Weight.BOLD, + '700': Pango.Weight.BOLD, + '800': Pango.Weight.ULTRABOLD, + '900': Pango.Weight.HEAVY, + 'normal': Pango.Weight.NORMAL, + 'bold': Pango.Weight.BOLD, + }[value] except KeyError: log.warning('unknown font-style %s', value) else: tag.set_property('weight', weight) - def _parse_style_font_family(self, tag, value): + @staticmethod + def _parse_style_font_family(tag, value): tag.set_property('family', value) - def _parse_style_text_align(self, tag, value): + @staticmethod + def _parse_style_text_align(tag, value): try: align = { - 'left': Gtk.Justification.LEFT, - 'right': Gtk.Justification.RIGHT, - 'center': Gtk.Justification.CENTER, - 'justify': Gtk.Justification.FILL, - }[value] + 'left': Gtk.Justification.LEFT, + 'right': Gtk.Justification.RIGHT, + 'center': Gtk.Justification.CENTER, + 'justify': Gtk.Justification.FILL, + }[value] except KeyError: log.warning('Invalid text-align: %s requested', value) else: tag.set_property('justification', align) - def _parse_style_text_decoration(self, tag, value): + @staticmethod + def _parse_style_text_decoration(tag, value): values = value.split(' ') if 'none' in values: tag.set_property('underline', Pango.Underline.NONE) @@ -438,7 +458,8 @@ class HtmlHandler(xml.sax.handler.ContentHandler): if 'overline' in values: log.warning('text-decoration:overline not implemented') - def _parse_style_white_space(self, tag, value): + @staticmethod + def _parse_style_white_space(tag, value): if value == 'pre': tag.set_property('wrap_mode', Gtk.WrapMode.NONE) elif value == 'normal': @@ -446,7 +467,8 @@ class HtmlHandler(xml.sax.handler.ContentHandler): elif value == 'nowrap': tag.set_property('wrap_mode', Gtk.WrapMode.NONE) - def __length_tag_cb(self, value, tag, propname): + @staticmethod + def __length_tag_cb(value, tag, propname): try: tag.set_property(propname, value) except Exception: @@ -455,29 +477,13 @@ class HtmlHandler(xml.sax.handler.ContentHandler): def _parse_style_width(self, tag, value): if value == 'auto': return - self._parse_length(value, False, False, 1, 1000, self.__length_tag_cb, - tag, "width") + self._parse_length(value, False, False, 1, 1000, + self.__length_tag_cb, tag, "width") def _parse_style_height(self, tag, value): if value == 'auto': return - self._parse_length(value, False, False, 1, 1000, self.__length_tag_cb, - tag, "height") - - - # build a dictionary mapping styles to methods, for greater speed - __style_methods = dict() - for style in ('background-color', 'color', 'font-family', 'font-size', - 'font-style', 'font-weight', 'margin-left', 'margin-right', - 'text-align', 'text-decoration', 'white-space', 'display', - 'width', 'height'): - try: - method = locals()['_parse_style_%s' % style.replace('-', '_')] - except KeyError: - log.warning('Style attribute "%s" not yet implemented', style) - else: - __style_methods[style] = method - del style # pylint: disable=undefined-loop-variable - # -- + self._parse_length(value, False, False, 1, 1000, + self.__length_tag_cb, tag, "height") def _get_style_tags(self): return [tag for tag in self.styles if tag is not None] @@ -523,8 +529,10 @@ class HtmlHandler(xml.sax.handler.ContentHandler): else: if self.conv_textview: img_mark = self.textbuf.create_mark(None, self.iter, True) - app.thread_interface(helpers.download_image, [ - self.conv_textview.account, attrs], self._update_img, + app.thread_interface( + helpers.download_image, + [self.conv_textview.account, attrs], + self._update_img, [attrs, img_mark, self._get_style_tags()]) alt = attrs.get('alt', '') if alt: @@ -542,34 +550,36 @@ class HtmlHandler(xml.sax.handler.ContentHandler): def width_cb(length): dims[0] = length # process width and height attributes - w = attrs.get('width') - h = attrs.get('height') + width = attrs.get('width') + height = attrs.get('height') # override with width and height styles for attr, val in style_iter(attrs.get('style', '')): if attr == 'width': - w = val + width = val elif attr == 'height': - h = val - if w: - self._parse_length(w, False, False, 1, 1000, width_cb) - if h: - self._parse_length(h, False, False, 1, 1000, height_cb) - def set_size(pixbuf, w, h, dims): + height = val + if width: + self._parse_length(width, False, False, 1, 1000, width_cb) + if height: + self._parse_length(height, False, False, 1, 1000, height_cb) + + def set_size(_pixbuf, width_, height_, dims): """ FIXME: Floats should be relative to the whole textview, and resize with it. This needs new pifbufs for every resize, GdkPixbuf.Pixbuf.scale_simple or similar. """ if isinstance(dims[0], float): - dims[0] = int(dims[0]*w) + dims[0] = int(dims[0] * width_) elif not dims[0]: - dims[0] = w + dims[0] = width_ if isinstance(dims[1], float): - dims[1] = int(dims[1]*h) + dims[1] = int(dims[1] * height_) if not dims[1]: - dims[1] = h + dims[1] = height_ loader.set_size(*dims) - if w or h: + + if width or height: loader.connect('size-prepared', set_size, dims) loader.write(mem) loader.close() @@ -621,12 +631,10 @@ class HtmlHandler(xml.sax.handler.ContentHandler): attr = attr.lower() val = val try: - method = self.__style_methods[attr] + getattr(self, self.__style_methods[attr])(tag, val) except KeyError: log.warning('Style attribute "%s" requested ' 'but not yet implemented', attr) - else: - method(self, tag, val) self.styles.append(tag) def _end_span(self): @@ -660,7 +668,7 @@ class HtmlHandler(xml.sax.handler.ContentHandler): else: self._insert_text(text.strip('\n')) - def _anchor_event(self, tag, textview, event, iter_, href, type_): + def _anchor_event(self, _tag, _textview, event, _iter, href, type_): if event.type == Gdk.EventType.BUTTON_PRESS: self.textview.emit('url-clicked', href, type_) return True @@ -668,8 +676,8 @@ class HtmlHandler(xml.sax.handler.ContentHandler): def handle_specials(self, text): if self.conv_textview: - self.iter = self.conv_textview.detect_and_print_special_text(text, - self._get_style_tags(), iter_=self.iter) + self.iter = self.conv_textview.detect_and_print_special_text( + text, self._get_style_tags(), iter_=self.iter) else: self._insert_text(text) @@ -682,7 +690,6 @@ class HtmlHandler(xml.sax.handler.ContentHandler): self.text += content self.starting = False - def startElement(self, name, attrs): self._flush_text() klass = [i for i in attrs.get('class', ' ').split(' ') if i] @@ -767,10 +774,10 @@ class HtmlHandler(xml.sax.handler.ContentHandler): log.warning('Unhandled element "%s"', name) def endElement(self, name): - endPreserving = False - newLine = False + end_preserving = False + newline = False if name == 'br': - newLine = True + newline = True elif name == 'hr': #FIXME: plenty of unused attributes (width, height,...) :) self._jump_line() @@ -779,7 +786,7 @@ class HtmlHandler(xml.sax.handler.ContentHandler): elif name in LIST_ELEMS: self.list_counters.pop() elif name == 'li': - newLine = True + newline = True elif name == 'img': pass elif name in ('body', 'html'): @@ -792,73 +799,62 @@ class HtmlHandler(xml.sax.handler.ContentHandler): pass elif name in BLOCK: if name == 'pre': - endPreserving = True + end_preserving = True elif name in BLOCK_STRUCT: - newLine = True + newline = True else: log.warning("Unhandled element '%s'", name) self._flush_text() - if endPreserving: + if end_preserving: self.preserve = False - if newLine: + if newline: self._jump_line() self._end_span() class HtmlTextView(Gtk.TextView): - def __init__(self, account=None): + _tags = ['url', 'mail', 'xmpp', 'sth_at_sth'] + + def __init__(self, account, standalone=False): Gtk.TextView.__init__(self) self.set_wrap_mode(Gtk.WrapMode.CHAR) self.set_editable(False) - self._changed_cursor = False self.set_has_tooltip(True) - self.connect('realize', self.on_html_text_view_realized) - self.connect('unrealize', self.on_html_text_view_unrealized) - self.connect('copy-clipboard', self.on_html_text_view_copy_clipboard) - self.id_ = self.connect('button-release-event', - self.on_left_mouse_button_release) + self.connect('copy-clipboard', self._on_copy_clipboard) self.get_buffer().eol_tag = self.get_buffer().create_tag('eol') - self.config = app.config - self.interface = app.interface + self.account = account self.plugin_modified = False + self._cursor_changed = False - def connect_tooltip(self, func=None): - self.connect('query-tooltip', func or self.__query_tooltip) + if standalone: + self.connect('query-tooltip', self._query_tooltip) + self.create_tags() def create_tags(self): - buffer_ = self.get_buffer() + color = app.css_config.get_value('.gajim-url', StyleAttr.COLOR) + + attrs = {'foreground': color, + 'underline': Pango.Underline.SINGLE} + + for tag in self._tags: + tag_ = self.get_buffer().create_tag(tag, **attrs) + tag_.connect('event', self.hyperlink_handler, tag) - self.tagURL = buffer_.create_tag('url') + def update_tags(self): + tag_table = self.get_buffer().get_tag_table() color = app.css_config.get_value('.gajim-url', StyleAttr.COLOR) - self.tagURL.set_property('foreground', color) - self.tagURL.set_property('underline', Pango.Underline.SINGLE) - self.tagURL.connect('event', self.hyperlink_handler, 'url') - - self.tagMail = buffer_.create_tag('mail') - self.tagMail.set_property('foreground', color) - self.tagMail.set_property('underline', Pango.Underline.SINGLE) - self.tagMail.connect('event', self.hyperlink_handler, 'mail') - - self.tagXMPP = buffer_.create_tag('xmpp') - self.tagXMPP.set_property('foreground', color) - self.tagXMPP.set_property('underline', Pango.Underline.SINGLE) - self.tagXMPP.connect('event', self.hyperlink_handler, 'xmpp') - - self.tagSthAtSth = buffer_.create_tag('sth_at_sth') - self.tagSthAtSth.set_property('foreground', color) - self.tagSthAtSth.set_property('underline', Pango.Underline.SINGLE) - self.tagSthAtSth.connect('event', self.hyperlink_handler, 'sth_at_sth') - - def __query_tooltip(self, widget, x_pos, y_pos, keyboard_mode, tooltip): + + for tag in self._tags: + tag_table.lookup(tag).set_property('foreground', color) + + def _query_tooltip(self, widget, x_pos, y_pos, _keyboard_mode, tooltip): window = widget.get_window(Gtk.TextWindowType.TEXT) x_pos, y_pos = self.window_to_buffer_coords( Gtk.TextWindowType.TEXT, x_pos, y_pos) - if Gtk.MINOR_VERSION > 18: - iter_ = self.get_iter_at_position(x_pos, y_pos)[1] - else: - iter_ = self.get_iter_at_position(x_pos, y_pos)[0] + iter_ = self.get_iter_at_position(x_pos, y_pos)[1] + for tag in iter_.get_tags(): if getattr(tag, 'is_anchor', False): text = getattr(tag, 'title', False) @@ -866,13 +862,19 @@ class HtmlTextView(Gtk.TextView): if len(text) > 50: text = text[:47] + '…' tooltip.set_text(text) - if not self._changed_cursor: window.set_cursor(get_cursor('HAND2')) - self._changed_cursor = True - return True - if self._changed_cursor: + self._cursor_changed = True + return True + + tag_name = tag.get_property('name') + if tag_name in ('url', 'mail', 'xmpp', 'sth_at_sth'): + window.set_cursor(get_cursor('HAND2')) + self._cursor_changed = True + return False + + if self._cursor_changed: window.set_cursor(get_cursor('XTERM')) - self._changed_cursor = False + self._cursor_changed = False return False def show_context_menu(self, _event, kind, text): @@ -880,7 +882,7 @@ class HtmlTextView(Gtk.TextView): if menu is None: return - def destroy(menu, pspec): + def destroy(menu, _pspec): visible = menu.get_property('visible') if not visible: GLib.idle_add(menu.destroy) @@ -893,7 +895,7 @@ class HtmlTextView(Gtk.TextView): if event.type == Gdk.EventType.BUTTON_PRESS: begin_iter = iter_.copy() # we get the beginning of the tag - while not begin_iter.begins_tag(texttag): + while not begin_iter.starts_tag(texttag): begin_iter.backward_char() end_iter = iter_.copy() # we get the end of the tag @@ -946,8 +948,8 @@ class HtmlTextView(Gtk.TextView): 'mlat=%(lat)s&mlon=%(lon)s&zoom=16' % \ {'lat': lat, 'lon': lon} helpers.launch_browser_mailer(kind, uri) - # other URIs else: + # other URIs helpers.launch_browser_mailer(kind, word) def display_html(self, html, textview, conv_textview, iter_=None): @@ -956,42 +958,16 @@ class HtmlTextView(Gtk.TextView): eob = iter_ else: eob = buffer_.get_end_iter() - ## this works too if libxml2 is not available - # parser = xml.sax.make_parser(['drv_libxml2']) - # parser.setFeature(xml.sax.handler.feature_validation, True) parser = xml.sax.make_parser() parser.setContentHandler(HtmlHandler(textview, conv_textview, eob)) parser.parse(StringIO(html)) - # too much space after :) - #if not eob.starts_line(): - # buffer_.insert(eob, '\n') - - def on_html_text_view_copy_clipboard(self, unused_data): - clipboard = self.get_clipboard(Gdk.SELECTION_CLIPBOARD) - selected = self.get_selected_text() + @staticmethod + def _on_copy_clipboard(textview): + clipboard = textview.get_clipboard(Gdk.SELECTION_CLIPBOARD) + selected = textview.get_selected_text() clipboard.set_text(selected, -1) - GObject.signal_stop_emission_by_name(self, 'copy-clipboard') - - def on_html_text_view_realized(self, unused_data): - self.get_buffer().remove_selection_clipboard(self.get_clipboard( - Gdk.SELECTION_PRIMARY)) - - def on_html_text_view_unrealized(self, unused_data): - self.get_buffer().add_selection_clipboard(self.get_clipboard( - Gdk.SELECTION_PRIMARY)) - - def on_left_mouse_button_release(self, widget, event): - if event.button != 1: - return - - bounds = self.get_buffer().get_selection_bounds() - if bounds: - # textview can be hidden while we add a new line in it. - if self.has_screen(): - clipboard = self.get_clipboard(Gdk.SELECTION_PRIMARY) - selected = self.get_selected_text() - clipboard.set_text(selected, -1) + GObject.signal_stop_emission_by_name(textview, 'copy-clipboard') def get_selected_text(self): bounds = self.get_buffer().get_selection_bounds() View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/63ff9478bd3eee0c5c150e8420af94a4f… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/63ff9478bd3eee0c5c150e8420af94a4f… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] XHTML: Fix parsing font-size
by Philipp Hörist 16 Nov '18

16 Nov '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: 63ff9478 by Philipp Hörist at 2018-11-16T15:23:42Z XHTML: Fix parsing font-size Remove checking for font_scale, Gtk.TextIter.get_attributes() seems buggy at the moment, and we dont need to check the current scale to apply a new scale Fixes #9067 - - - - - 2 changed files: - gajim/htmltextview.py - test/gtk/htmltextview.py Changes: ===================================== gajim/htmltextview.py ===================================== @@ -230,13 +230,6 @@ class HtmlHandler(xml.sax.handler.ContentHandler): tag.set_property('background-gdk', color) tag.set_property('paragraph-background-gdk', color) - def _get_current_attributes(self): - attrs = self.textview.get_default_attributes() - self.iter.backward_char() - attrs = (self.iter.get_attributes())[1] - self.iter.forward_char() - return attrs - def __parse_length_frac_size_allocate(self, textview, allocation, frac, callback, args): callback(allocation.width*frac, *args) @@ -338,30 +331,19 @@ class HtmlHandler(xml.sax.handler.ContentHandler): def _parse_style_font_size(self, tag, value): try: - # see http://developer.gnome.org/pango/stable/pango-Text-Attributes.html#PANGO-SC… - # http://consciouslyusing.blogspot.ru/2012/01/heads-up-missing-pango-text-sca… scale = { - #'xx-small': Pango.SCALE_XX_SMALL, - #'x-small': Pango.SCALE_X_SMALL, - #'small': Pango.SCALE_SMALL, - #'medium': Pango.SCALE_MEDIUM, - #'large': Pango.SCALE_LARGE, - #'x-large': Pango.SCALE_X_LARGE, - #'xx-large': Pango.SCALE_XX_LARGE, - 'xx-small': 0.5787037037037, - 'x-small': 0.6444444444444, - 'small': 0.8333333333333, - 'medium': 1.0, - 'large': 1.2, - 'x-large': 1.4399999999999, - 'xx-large': 1.728, - }[value] + 'xx-small': 0.5787037037037, + 'x-small': 0.6444444444444, + 'small': 0.8333333333333, + 'medium': 1.0, + 'large': 1.2, + 'x-large': 1.4399999999999, + 'xx-large': 1.728, + }[value] except KeyError: pass else: - attrs = self._get_current_attributes() - if attrs.font_scale == 0: - tag.set_property('scale', scale) + tag.set_property('scale', scale) return if value == 'smaller': tag.set_property('scale', 0.8333333333333) ===================================== test/gtk/htmltextview.py ===================================== @@ -19,6 +19,10 @@ app.interface = Interface() XHTML = [ + ''' + <body xmlns='http://www.w3.org/1999/xhtml'>rexxnor: i nimes chum a...</body> + ''', + ''' <div> Hello View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/63ff9478bd3eee0c5c150e8420af94a4f1… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/63ff9478bd3eee0c5c150e8420af94a4f1… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][gajim_1.1] 3 commits: Windows: Get objdump output as string
by Philipp Hörist 16 Nov '18

16 Nov '18

Philipp Hörist pushed to branch gajim_1.1 at gajim / gajim Commits: 9ba97c4b by Philipp Hörist at 2018-11-16T14:28:28Z Windows: Get objdump output as string Otherwise we have to guess the encoding of the host system - - - - - adfcfb1b by Philipp Hörist at 2018-11-16T14:28:28Z Fix incorrect contact status on tabs - - - - - 5271f192 by Philipp Hörist at 2018-11-16T14:28:40Z Windows: Fix build data is now a str - - - - - 2 changed files: - gajim/common/connection_handlers.py - win/misc/depcheck.py Changes: ===================================== gajim/common/connection_handlers.py ===================================== @@ -146,6 +146,7 @@ class ConnectionHandlersBase: obj.contact_list.append(obj.contact) obj.contact.resource = resource + obj.need_redraw = True obj.need_add_in_roster = True if not app.jid_is_transport(jid) and len(obj.contact_list) == 1: ===================================== win/misc/depcheck.py ===================================== @@ -47,11 +47,11 @@ SYSTEM_LIBS = ['advapi32.dll', def get_dependencies(filename): deps = [] try: - data = subprocess.check_output(["objdump", "-p", filename]) + data = subprocess.getoutput("objdump -p %s" % filename) except Exception as error: print(error) return deps - data = data.decode("utf-8") + for line in data.splitlines(): line = line.strip() if line.startswith("DLL Name:"): View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/df62671217256defc2b7e51b1e5ac76da… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/df62671217256defc2b7e51b1e5ac76da… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Windows: Fix build
by Philipp Hörist 16 Nov '18

16 Nov '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: 77d64267 by Philipp Hörist at 2018-11-16T14:26:38Z Windows: Fix build data is now a str - - - - - 1 changed file: - win/misc/depcheck.py Changes: ===================================== win/misc/depcheck.py ===================================== @@ -51,7 +51,7 @@ def get_dependencies(filename): except Exception as error: print(error) return deps - data = data.decode("utf-8") + for line in data.splitlines(): line = line.strip() if line.startswith("DLL Name:"): View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/77d64267c72e11d687041272a5f693847e… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/77d64267c72e11d687041272a5f693847e… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] New DataFormWidget: Add Captcha support
by Philipp Hörist 15 Nov '18

15 Nov '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: 7f1dfe5a by Philipp Hörist at 2018-11-15T20:02:09Z New DataFormWidget: Add Captcha support - - - - - 8 changed files: - gajim/common/app.py - gajim/common/configpaths.py - gajim/common/modules/bits_of_binary.py - gajim/common/modules/message.py - gajim/common/modules/register.py - gajim/data/style/gajim.css - gajim/gtk/dataform.py - test/gtk/dataform.py Changes: ===================================== gajim/common/app.py ===================================== @@ -57,6 +57,7 @@ config = c_config.Config() version = gajim.__version__ connections = {} # type: Dict[str, ConnectionT] avatar_cache = {} # type: Dict[str, Dict[str, Any]] +bob_cache = {} # type: Dict[str, bytes] ipython_window = None app = None # Gtk.Application @@ -668,3 +669,14 @@ def get_win_debug_mode() -> bool: debug_folder = Path(configpaths.get('DEBUG')) debug_enabled = debug_folder / 'debug-enabled' return debug_enabled.exists() + +def get_stored_bob_data(algo_hash: str) -> Optional[bytes]: + try: + return bob_cache[algo_hash] + except KeyError: + filepath = Path(configpaths.get('BOB')) / algo_hash + if filepath.exists(): + with open(str(filepath), 'r+b') as file: + data = file.read() + return data + return None ===================================== gajim/common/configpaths.py ===================================== @@ -229,6 +229,7 @@ class ConfigPaths: # Cache paths ('CACHE_DB', 'cache.db', PathLocation.CACHE, PathType.FILE), ('AVATAR', 'avatars', PathLocation.CACHE, PathType.FOLDER), + ('BOB', 'bob', PathLocation.CACHE, PathType.FOLDER), # Config paths ('MY_THEME', 'theme', PathLocation.CONFIG, PathType.FOLDER), ===================================== gajim/common/modules/bits_of_binary.py ===================================== @@ -15,9 +15,15 @@ # along with Gajim. If not, see <http://www.gnu.org/licenses/>. import logging +import hashlib +from base64 import b64decode +from pathlib import Path import nbxmpp +from gajim.common import app +from gajim.common import configpaths + log = logging.getLogger('gajim.c.m.bob') @@ -91,5 +97,78 @@ class BitsOfBinary: iq, self._on_bob_received, {'cid': cid}) +def parse_bob_data(stanza): + data_node = stanza.getTag('data', namespace=nbxmpp.NS_BOB) + if data_node is None: + return + + cid = data_node.getAttr('cid') + type_ = data_node.getAttr('type') + max_age = data_node.getAttr('max-age') + if max_age is not None: + try: + max_age = int(max_age) + except Exception: + log.exception(stanza) + return + + if cid is None or type_ is None: + log.warning('Invalid data node (no cid or type attr): %s', stanza) + return + + try: + algo_hash = cid.split('@')[0] + algo, hash_ = algo_hash.split('+') + except Exception: + log.exception('Invalid cid: %s', stanza) + return + + bob_data = data_node.getData() + if not bob_data: + log.warning('No data found: %s', stanza) + return + + filepath = Path(configpaths.get('BOB')) / algo_hash + if algo_hash in app.bob_cache or filepath.exists(): + log.info('BoB data already cached') + return + + try: + bob_data = b64decode(bob_data) + except Exception: + log.warning('Unable to decode data') + log.exception(stanza) + return + + if len(bob_data) > 10000: + log.warning('%s: data > 10000 bytes', stanza.getFrom()) + return + + try: + sha = hashlib.new(algo) + except ValueError as error: + log.warning(stanza) + log.warning(error) + return + + sha.update(bob_data) + if sha.hexdigest() != hash_: + log.warning('Invalid hash: %s', stanza) + return + + if max_age == 0: + app.bob_cache[algo_hash] = bob_data + else: + try: + with open(str(filepath), 'w+b') as file: + file.write(bob_data) + except Exception: + log.warning('Unable to save data') + log.exception(stanza) + return + log.info('BoB data stored: %s', algo_hash) + return filepath + + def get_instance(*args, **kwargs): return BitsOfBinary(*args, **kwargs), 'BitsOfBinary' ===================================== gajim/common/modules/message.py ===================================== @@ -26,6 +26,7 @@ from gajim.common.nec import NetworkIncomingEvent, NetworkEvent from gajim.common.modules.security_labels import parse_securitylabel from gajim.common.modules.user_nickname import parse_nickname from gajim.common.modules.carbons import parse_carbon +from gajim.common.modules.bits_of_binary import parse_bob_data from gajim.common.modules.misc import parse_delay from gajim.common.modules.misc import parse_eme from gajim.common.modules.misc import parse_correction @@ -228,6 +229,8 @@ class Message: timestamp = time.time() delayed = False + parse_bob_data(event.stanza) + event_attr = { 'popup': False, 'msg_log_id': None, ===================================== gajim/common/modules/register.py ===================================== @@ -20,6 +20,7 @@ import weakref import nbxmpp from gajim.common import app +from gajim.common.modules.bits_of_binary import parse_bob_data log = logging.getLogger('gajim.c.m.register') @@ -125,6 +126,7 @@ class Register: error_cb()(error) else: log.info('Register form received') + parse_bob_data(stanza.getQuery()) form = stanza.getQuery().getTag('x', namespace=nbxmpp.NS_DATA) is_form = form is not None if not is_form: ===================================== gajim/data/style/gajim.css ===================================== @@ -250,3 +250,6 @@ button.flat.link { padding: 0; border: 0; } .data-form-widget grid { margin: 18px; } .data-form-widget treeview { padding: 5px; } .data-form-widget scrolledwindow { border: 1px solid; border-color:@unfocused_borders; } + +/*Image Preview*/ +.preview-image { box-shadow: 0px 0px 3px 0px alpha(@theme_text_color, 0.2); } ===================================== gajim/gtk/dataform.py ===================================== @@ -17,6 +17,9 @@ from gi.repository import GLib from gi.repository import GObject from gi.repository import Pango +from gajim.gtkgui_helpers import scale_pixbuf_from_data + +from gajim.common import app from gajim.common.i18n import _ @@ -82,21 +85,21 @@ class FormGrid(Gtk.Grid): if form_node.title is not None: self.title = form_node.title - self.add_row(Title(form_node.title)) + self._add_row(Title(form_node.title)) if form_node.instructions is not None: self.instructions = form_node.instructions - self.add_row(Instructions(form_node.instructions)) + self._add_row(Instructions(form_node.instructions)) - self.analyse_fields(form_node, options) - self.parse_form(form_node, options) + self._analyse_fields(form_node, options) + self._parse_form(form_node, options) - def add_row(self, field): + def _add_row(self, field): field.add(self, self.row_count) self.row_count += 1 self.rows.append(field) @staticmethod - def analyse_fields(form_node, options): + def _analyse_fields(form_node, options): if 'right_align' in options: # Dont overwrite option return @@ -113,13 +116,33 @@ class FormGrid(Gtk.Grid): options['right_align'] = max(label_lengths) < 30 - def parse_form(self, form_node, options): + def _parse_form(self, form_node, options): for field in form_node.iter_fields(): if field.type_ == 'hidden': continue + if field.media: + if not self._add_media_field(field, options): + # We dont understand this media element, ignore it + continue + widget = self._fields[field.type_] - self.add_row(widget(field, self, options)) + self._add_row(widget(field, self, options)) + + def _add_media_field(self, field, options): + if not field.type_ in ('text-single', 'text-private', 'text-multi'): + return False + + for uri in field.media.uris: + if not uri.type_.startswith('image/'): + continue + + if not uri.uri_data.startswith('cid'): + continue + + self._add_row(ImageMediaField(uri, self, options)) + return True + return False def validate(self, is_valid): value = self._data_form.is_valid() if is_valid else False @@ -460,3 +483,23 @@ class TextMultiField(Field): def _changed(self, widget): self._field.value = widget.get_text(*widget.get_bounds(), False) self._validate() + + +class ImageMediaField(): + def __init__(self, uri, form_grid, options): + self._uri = uri + self._form_grid = form_grid + + filename = uri.uri_data.split(':')[1].split('@')[0] + data = app.bob_cache.get(filename) + if data is None: + self._image = Gtk.Image() + return + + pixbuf = scale_pixbuf_from_data(data, 170) + self._image = Gtk.Image.new_from_pixbuf(pixbuf) + self._image.set_halign(Gtk.Align.CENTER) + self._image.get_style_context().add_class('preview-image') + + def add(self, form_grid, row_number): + form_grid.attach(self._image, 1, row_number, 1, 1) ===================================== test/gtk/dataform.py ===================================== @@ -1,13 +1,45 @@ +from base64 import b64decode + from gi.repository import Gtk import nbxmpp from gajim.gtk.dataform import DataFormWidget from gajim.common.modules.dataforms import extend_form from gajim.common.const import CSSPriority +from gajim.common import app from test.gtk import util util.load_style('gajim.css', CSSPriority.APPLICATION) +image = '''iVBORw0KGgoAAAANSUhEUgAAAIwAAAA8CAAAAACRYQ2XAAAABGdBTUEAALGPC/xhBQAAACBjSFJNAAB6JgAAgIQAAPoAAACA6AAAdTAAAOpgAAA6mAAAF3CculE8AAAAAmJLR0QA/4ePzL8AAAAHdElNRQfiCwQXMiypK +zsIAAAM4ElEQVRo3u1ZeVxUR7b+moZmE1AWFVDQNoIiEBFFxUHEXaJGQX0TnWhi0DhjYlQwbviEbOKOE9eIOjFGR8e44C4qcYkIssmiIMiO7Es3vXffe94ftxsaaUh0Zt689/ul/ulb59a5/VXVWb5TxSP832lG+B3M/wMwxr9plFpjWlFtZm9n+p8Gwxrll2vuXXDyf +F5erZw5yde1G+/fBYb3K97UoErf3U/8y/xRrNREaFqVnFDuPmKm/38AjIaflVF32mKWzYCxSp4ZwJIRT1GRefflkNnuFv+LYIiHUsWLWNPelWs8BD3a20/Dkx+q5v/J9N8OhshI09SgMOled6nW5orZ+y5CD9aQv52Py4hY3ckkdJhN/jkwz1NLNeVHrUcNKsnkBziqZ +gyFdWdqqow9Rit921kym/pUxhbJl/pwvVtPhMNd3xyMbCtrIjwU5lNcZTTaVt7Lqks9pvBk9WJv8zZ0zx9W8potnMXx7vssATQ6fWotsQ8Y9XpoqLUF7X1B9NFhIiK2TfqSOmtHBeFMa+d2+NIrxURUVf6+cTMRZbqJM09ti1yXQK/R2sA02xIRhR1u91pxLyLmfme6z +AbEcU8VMwKv6aTKFb4qovMTiaguYdfmVdd/Oxh+lG6JEhUhAC71Gaa/bvHzhqvLkkydDW+xf0h8el9bQLln/I63dFL+tJ/Uw/CT+TTAQuhrJi28iAG819wmZkxZh5VRxAeubWQT1m+o72wuqQs+KlaLAqOU+kKNjZhmn9B22KRd/73yREaxiDGkr2hKZwxsU/p4ehWMJ +n5gtIKI6IIp0xka9Wbe6u3fERGRiEQVnHDCNerzgoNCJWmPPuo7dqyvh8PITTfK24EmWf3ZyWPTO4BRJY982AFM/FvbtU8uDZ1vtGyB1TdFRKTaE+T6eSYREZ2areITEYlF18PWjHP89Lu1c0P+69vvFnR3mbQps9W2xDX7/ec/+sPVV8Fori3Io1fAaB4HhTfpwJS2K +rAallTiekmrQDQsIXx9qniL6fo7mVPncv9j2WDNUP6Vg2+P+jL6oqKphSj7+PgnZRWpcR+Fxb1QapTS5trLy51W5xFNv9QGxhgA6rfnrXV/1ZieTl0Z2cHCFKUCgUrDmFFlvZOVnRkA/PjZxL7HhvszN72s+QcGAwAE0rweP+/uVS8eeYcfCZgC8PSs3H3Bx9dtQvm+4 +3PGVD7PTPKeHWtsMOgpzHT9JSPDtOE4vM9Xdjqp630XAFCJzLsBGhjBCKRUSR34PKiHplgCrHCo0SC/t+1tc3mmNhX3o6ecGxRiP9T3A3fpsdURACtvqjp9YVBzBWPEMAqV2m/8iEBtdJ+8YZw+nxF9fMy8I3XYItjWFoNNuQQlcIBKoKVACpM6Jc+BD8UxSwBXxm7Jv +hnWwOfHOBSns1ONcwLiTOyBF3PHfXqwV8SK3OuPCow1Qg9zSVn6rYmj1Olw131c4tPOtUu/1zPHJZzNMMuC9Q3/4xadxeib7pOvTqt1AxKJiORxYw9s3puSG+fzwQEiInK9+5gk2UHGth/+Le1gj3y1VNQslp8JmbRywZaXTMcPgqiFISJqriMiougfuFDvn/wbIqbG5 +zvFBO5RSkQkmnKPiNXQrFi3LCIiWhEYtfzDgYvObBKuyvnSNl+n11JYKu0kHdwtIKKCE1F60mehadxDdVqrTCpWdNBOnvPMmwsNRESSoG84sWWJmZx7Klw3adrGVJWkdn+A04QH+qqSB5HniOL36ct4hL2Jm4sCexjkJaya41DPe1vjZOouMCqpREZWfEY2SDtQeTkUQ +JlfFQ+AlvtMDVrmJAUAaf61wpHdr7qst4Rcni8+Eu5qLeDzQIys/kUhuh1cEPJOgm37dHA7ZNwdooYcA+G1TlpFRET9y6hJuCrUwRgWfb2dHIVvBZ/htjxPLSGqS1z3I5HiciOntdOzpdiZiDT1RzyXPiOiA2YnGSIimUgpKc1Nunxk+5boDT2Jrvv5r2tnM8ZAYG8PA +HVzngFArY0pSGnKA4DaS3JBwXYAYKoLH6t4Yxb2t+cnz19kIzZNmP/JV2TJw/WSsMFGtt39J6Nw2FfBAIDGockWVWMA5k604JA/ACxbdlTUA4A5qirLihpYEys7xx1bgSljeiUxaxwMJMo8d87kfqT6YdrAO9vrxY5wImJETgt8FnbPaVSxxL59iojy7wZ48VbmlFKCX +eitPCKi6r885pRu3GOI1JeKs7aHJ7PPOjP9bB8VEf2wsyF82j9qOiZKLZiww3T8fc67Z3Y/xewIJ2JuBfNXPNeOVAk4l0/0rtos+ODsbeOdX+QSEb3gck72nIgWIqLGZkX73Pp1pX5PNf82EVEOQ0QamZrVTwev8FvOaJX7Lh7a59obN79UfZYT7qKLUcYCLiY39t4QG +RMxo5fNagBN9yYLAbB3jk6d1Q3igkye0KWbpSXL8GpSHEcBgNdnf+e3/UW+33gAGAIAfPOuam0tKzZ3Xnp1l3Xzwo1Lr8wDAM7f+Fr0MkcIjCOLRxtlCNeWKfaZmANA3b3xoRbVj49/n1nUyLcqTXwGvvPMmxMBYIbDfj3ujo8BYGuj1iNvHr1dLGO6LPx5JubO3krPu +FB7YwB4/ywAWPuqAQBDkgAA77ksOcgb4XWtf25euazSIWJeXtyq0OujE/Fkd2Smr4cJD/x1k2MAYP+xsrbpmpkBSCjQurVpwB8qVzl4SrustSW5CVXBE3XWPmfnHAD4VlsQaRf9ATB69gW51whfmxLrSZO7jeo26GthReMf5dVVxaeXAIDg8+DPjQB88tcdOizHRgPA5 +pO6PzJ3c1vIii1hwIBzbHYdPHE1rbzy8LjlWnfoFy4iNZetZNdqGEVx9FGtmpyI1cgZNvdUcNDW809FLGlkTc3diUiW65PNjbkZSkQk9lRpde7OICKq3trBx3ikJRD57+ZpKQSbdopnbMx/nmwVFaKFLo07v2SuzNoIAJKXl1Cfpct1k5LTz172AKrr+1NNUna112hPi ++wnM+wBnEk8wHmEtQIAyrROQHmuFgAgtTREyCNLKxV6K0PESH4Os/WYt2nriWwdXWeJZVLkHeZSM5gy5hARtZSQSq5KTvilQi+D9ZZxvz0NhZqHmg4iYwDyfgunOKh0jqQSl2fEZwXfHKzJevyk+P6ftU7Hxs5NOnD01amkj0DUNgCY6828O9TMT8sHv/cdDgAV5f0AA +DXtdA6PGwjg+MvRBstbBS/2iE/3xGl9vFytI9w8Smrh6ie04wN4ud9ulW7kjBUBQYExr6hHOQRsuAxgz7mlC/RcN97Wrbd+LcxZe6HQCMCTPz4DgEGXBnZaN92PGhL9J38XCxgNWByXKmpl222canq8RDb3Cx0zl64hIqKpyUG5RERZhqM+2yImlpoecyHWRkJE7Lw9R +EQVFmwX5W3tvMrC4irRkrj27znKteuBkqaP3P0kKXJjloghVe2DaE8iInKscDaAQbWGLSrhirQz04+c/XASJ7ZuJqIacyIi2vStAb3WOONwGoCm3qL9Tl7MWwsAsR+mDJJY1C+yc796/O0+/RtqG5s8AUDSWBioN1pZVlUnkmt4txx5TjMHxwIwnRsSc+hdrakpTAGcn +wAAOHm/qyMRUY2bgdexFTsAwHVnGv9cv6vIepTWtLikqJe796plkwD8lCaZNV5nKLn55XKlSsMQBk93gmytRXjPdt8q8JbwoR5w1RNA+bhCXhdgSqctdrG2d3ZoH5JPpu/QlirJ7MICABhy2hNAw5AiCwABf5vwlDvda8jIkUjh3t/JrpuA44ctsWZr2n3ri6yzwPOAG +gDY1fB1V4W/YtvmfxR22MUSxxRdRakRqImItqxjiZTbNhIRka9SQERElRe2Ra4+XdBet2HtsHP6/aHniGjHAiIi8kvv+nxGYtAfEqJik7Tl7Z+5k5aeKSS9MqqZiGjxiZYeRFR+NmZ9TJIB5cfLF91urQi+8WsiUg9JISJiuxmsDtp2xdJgsgxS3mp+5AEA+HTJFAA4M +uulhdcJGwA53+9ttkLFo0KR4wwPQ8q+tkl30we/A0AjPb9nT3egrnwEAJS7WrzJCTn/neAbN2orrAAMNK/tCWB6pS6URT4yr2APNYmcQwd2QkKEfZMuSQ4GvHfF4XLGlukAUrwBAEff+7Uzvc5bYpWGiOihTzvpjU9YEo3fuLegS1025eIm89W2X9xvISJatI2IiMb+8 +itner+hRS7W65wJLCJi6kp+XU1MORru7EvVkztO6ln3z4Nh123WLUNDjFUxvXaTcc6nMFcbfP1a9028z9gj2WIWqpqHu9QF/V7/CNxkeFYjA/llf+M3ulV5pdUduNZjoEtZtdLtL65vctPzc0SjmDE3uzL4XwEGaE59Wir0GODyhncVCiWBZ2GCfw2Y3y9MfwfzO5hO2 +v8AIg7mWYx8/rwAAAAldEVYdGRhdGU6Y3JlYXRlADIwMTgtMTEtMDRUMjM6NTA6NDQrMDE6MDBAxMf7AAAAJXRFWHRkYXRlOm1vZGlmeQAyMDE4LTExLTA0VDIzOjUwOjQ0KzAxOjAwMZl/RwAAAABJRU5ErkJggolQTkcNChoKAAAADUlIRFIAAAAoAAAAPAEAAAAAP +MLFTQAAAARnQU1BAACxjwv8YQUAAAAgY0hSTQAAeiYAAICEAAD6AAAAgOgAAHUwAADqYAAAOpgAABdwnLpRPAAAAAJiS0dEAAHdihOkAAAAB3RJTUUH4gsEFzIsqSs7CAAAABBJREFUGNNj+A8CDKMkjUkAKsYq5D2hXoMAAAAldEVYdGRhdGU6Y3JlYXRlADIwMTgtM +TEtMDRUMjM6NTA6NDQrMDE6MDBAxMf7AAAAJXRFWHRkYXRlOm1vZGlmeQAyMDE4LTExLTA0VDIzOjUwOjQ0KzAxOjAwMZl/RwAAAABJRU5ErkJggg==''' + +app.bob_cache['sha1+8f35fef110ffc5df08d579a50083ff9308fb6242'] = b64decode(image) + FORM = ''' <x xmlns='jabber:x:data' type='form'> <title>Bot Configuration</title> @@ -66,9 +98,16 @@ FORM = ''' <desc>Tell all your friends about your new bot!</desc> <required/> </field> + <field var='ocr' type='text-single' label='Fill in what you see'> + <media xmlns='urn:xmpp:media-element'> + <uri type='image/png'>cid:sha1+8f35fef110ffc5df08d579a50083ff9308fb6242@bob.xmpp.org</uri> + </media> + <required/> + </field> </x> ''' + class DataFormWindow(Gtk.Window): def __init__(self): Gtk.Window.__init__(self, title="Data Form Test") View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/7f1dfe5a8caddf0f23bb13ac8d40ea0714… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/7f1dfe5a8caddf0f23bb13ac8d40ea0714… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] fix typos in config.py
by Yann Leboulanger 14 Nov '18

14 Nov '18

Yann Leboulanger pushed to branch master at gajim / gajim Commits: 4be49987 by Bronko at 2018-11-14T21:44:41Z fix typos in config.py - - - - - 1 changed file: - gajim/common/config.py Changes: ===================================== gajim/common/config.py ===================================== @@ -166,7 +166,7 @@ class Config: 'history_window_x-position': [opt_int, 0], 'history_window_y-position': [opt_int, 0], 'latest_disco_addresses': [opt_str, ''], - 'time_stamp': [opt_str, '[%X] ', _('This option let you customize timestamp that is printed in conversation. For exemple "[%H:%M] " will show "[hour:minute] ". See python doc on strftime for full documentation: http://docs.python.org/lib/module-time.html')], + 'time_stamp': [opt_str, '[%X] ', _('This option let you customize timestamp that is printed in conversation. For example "[%H:%M] " will show "[hour:minute] ". See python doc on strftime for full documentation: http://docs.python.org/lib/module-time.html')], 'before_nickname': [opt_str, '', _('Characters that are printed before the nickname in conversations')], 'after_nickname': [opt_str, ':', _('Characters that are printed after the nickname in conversations')], 'use_gpg_agent': [opt_bool, False], @@ -284,7 +284,7 @@ class Config: 'stun_server': [opt_str, '', _('STUN server to use when using Jingle')], 'show_affiliation_in_groupchat': [opt_bool, True, _('If true, Gajim will show affiliation of groupchat occupants by adding a colored square to the status icon')], 'global_proxy': [opt_str, '', _('Proxy used for all outgoing connections if the account does not have a specific proxy configured')], - 'ignore_incoming_attention': [opt_bool, False, _('If true, Gajim will ignore incoming attention requestd ("wizz").')], + 'ignore_incoming_attention': [opt_bool, False, _('If true, Gajim will ignore incoming attention requests ("wizz").')], 'remember_opened_chat_controls': [opt_bool, True, _('If enabled, Gajim will reopen chat windows that were opened last time Gajim was closed.')], 'positive_184_ack': [opt_bool, False, _('If enabled, Gajim will show an icon to show that sent message has been received by your contact')], 'show_avatar_in_tabs': [opt_bool, False, _('Show a mini avatar in chat window tabs and in window icon')], View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/4be49987045c74707a482ebbb5919788b3… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/4be49987045c74707a482ebbb5919788b3… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Fix incorrect contact status on tabs
by Philipp Hörist 13 Nov '18

13 Nov '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: 9e24c72b by Philipp Hörist at 2018-11-13T22:47:54Z Fix incorrect contact status on tabs - - - - - 1 changed file: - gajim/common/connection_handlers.py Changes: ===================================== gajim/common/connection_handlers.py ===================================== @@ -146,6 +146,7 @@ class ConnectionHandlersBase: obj.contact_list.append(obj.contact) obj.contact.resource = resource + obj.need_redraw = True obj.need_add_in_roster = True if not app.jid_is_transport(jid) and len(obj.contact_list) == 1: View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/9e24c72ba5f747f8f6f9718a0413959c10… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/9e24c72ba5f747f8f6f9718a0413959c10… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][gajim_1.1] 19 commits: Flatpak: update dependencies and use pip to install modules
by Philipp Hörist 13 Nov '18

13 Nov '18

Philipp Hörist pushed to branch gajim_1.1 at gajim / gajim Commits: 8bbf3392 by André Apitzsch at 2018-11-13T20:56:19Z Flatpak: update dependencies and use pip to install modules - - - - - 213da73b by André Apitzsch at 2018-11-13T20:56:19Z Flatpak: use X11 access only if a X11 session is running Requires flatpak 1.0 or newer - - - - - 298fb153 by Philipp Hörist at 2018-11-13T20:56:20Z Better determine soundplayer on unix systems - sys.platform == 'linux', excludes many other unix systems - use shutil.which() to test if a command is available - - - - - 8061146d by Philipp Hörist at 2018-11-13T20:56:20Z Remove unused code Code is also not cross-platform compatible - - - - - 959a51c5 by Philipp Hörist at 2018-11-13T20:56:20Z Dont test if winsound is available Its a standard library which will be always available in our builds - - - - - 67786dfd by Philipp Hörist at 2018-11-13T20:56:20Z Try to play sound on all unix systems sys.platform == 'linux' excludeds other unix systems, for example FreeBSD - - - - - 584c9ff6 by Philipp Hörist at 2018-11-13T20:56:20Z Move imports into function - - - - - 08bde952 by Philipp Hörist at 2018-11-13T20:56:20Z Dont test if precis_i18n is available Its listed as dependency - - - - - 57dad43d by Philipp Hörist at 2018-11-13T20:56:21Z Dont test for sys.platform == 'linux' This excludes other unix systems which was not intended - - - - - 510985d7 by Philipp Hörist at 2018-11-13T20:56:21Z Enforce min and max version for plugin manifest This reduces problems with old plugins without max version - - - - - 5981396a by Philipp Hörist at 2018-11-13T20:56:21Z Pass account to plugins when encrypting a file - - - - - 2c2943de by Philipp Hörist at 2018-11-13T20:56:21Z Determine absolut path for custom config folder When a relative custom config path is passed via commandline, the logic which decides if a plugin can be deleted from the plugin manager fails - - - - - 19a46070 by Philipp Hörist at 2018-11-13T20:58:32Z Windows: Remove old ssl fix - - - - - e95a7fcc by André Apitzsch at 2018-11-13T20:59:58Z Test: Remove unused xmpp_mocks - - - - - e8941a4f by André Apitzsch at 2018-11-13T21:00:40Z Test: remove unused imports - - - - - 8946af33 by André Apitzsch at 2018-11-13T21:00:40Z Test: caps cache: use mock from unittest - - - - - 958e1390 by André Apitzsch at 2018-11-13T21:00:40Z Restructure test - - - - - cd65b591 by Andrey Gursky at 2018-11-13T21:00:41Z Fix clipboard paste in inactive window Fixes #9427 - - - - - df626712 by Philipp Hörist at 2018-11-13T21:01:18Z Correctly remove placeholder on focus in This is a regression from e281fb68 Fixes #9440 - - - - - 30 changed files: - .gitlab-ci.yml - flatpak/org.gajim.Gajim.yaml - gajim/common/configpaths.py - gajim/common/dbus_support.py - gajim/common/helpers.py - gajim/common/modules/httpupload.py - gajim/gui_interface.py - gajim/message_textview.py - gajim/notify.py - gajim/plugins/pluginmanager.py - gajim/roster_window.py - setup.cfg - setup.py - test/integration/test_gui_event_integration.py → test/broken/integration/test_gui_event_integration.py - test/integration/test_roster.py → test/broken/integration/test_roster.py - test/test_pluginmanager.py → test/broken/test_pluginmanager.py - test/unit/test_jingle.py → test/broken/unit/test_jingle.py - test/unit/test_sessions.py → test/broken/unit/test_sessions.py - test/unit/test_socks5.py → test/broken/unit/test_socks5.py - test/gtk/htmltextview.py - test/integration/test_resolver.py - test/lib/__init__.py - test/lib/gajim_mocks.py - − test/lib/xmpp_mocks.py - + test/no_gui/__init__.py - + test/no_gui/unit/__init__.py - test/unit/test_account.py → test/no_gui/unit/test_account.py - test/unit/test_caps_cache.py → test/no_gui/unit/test_caps_cache.py - test/unit/test_contacts.py → test/no_gui/unit/test_contacts.py - test/unit/test_protocol_caps.py → test/no_gui/unit/test_protocol_caps.py The diff was not included because it is too large. View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/335c3e2d3df71fb07a6d96e9c0b6ae3db… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/335c3e2d3df71fb07a6d96e9c0b6ae3db… You're receiving this email because of your account on dev.gajim.org.

1 0

← Newer
1
2
3
4
5
6
7
8
9
Older →

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

Commits November 2018