tryton 6.8.12__tar.gz → 6.8.14__tar.gz

{tryton-6.8.12 → tryton-6.8.14}/CHANGELOG

@@ -1,4 +1,14 @@
 
+Version 6.8.14 - 2024-05-01
+---------------------------
+* Bug fixes (see mercurial logs for details)
+
+
+Version 6.8.13 - 2024-04-17
+---------------------------
+* Bug fixes (see mercurial logs for details)
+
+
 Version 6.8.12 - 2024-04-04
 ---------------------------
 * Bug fixes (see mercurial logs for details)

{tryton-6.8.12 → tryton-6.8.14}/COPYRIGHT

@@ -5,7 +5,7 @@ Copyright (C) 2007-2024 Cédric Krier.
 Copyright (C) 2008-2011 Udo Spallek.
 Copyright (C) 2008-2011 virtual things - Preisler & Spallek GbR.
 Copyright (C) 2008-2024 B2CK SPRL.
-Copyright (C) 2010-2023 Nicolas Évrard.
+Copyright (C) 2010-2024 Nicolas Évrard.
 Copyright (C) 2011-2012 Rodrigo Hübner.
 Copyright (C) 2012-2013 Antoine Smolders.
 Copyright (C) 2020-2021 Maxime Richez

{tryton-6.8.12 → tryton-6.8.14}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: tryton
-Version: 6.8.12
+Version: 6.8.14
 Summary: Tryton desktop client
 Home-page: http://www.tryton.org/
 Download-URL: http://downloads.tryton.org/6.8/

{tryton-6.8.12 → tryton-6.8.14}/setup-freeze.py

@@ -124,14 +124,13 @@ version = version.strip()
 
 setup(name='tryton',
     version=version,
-    packages=find_packages(),
     options={
         'build_exe': {
             'no_compress': True,
             'include_files': include_files,
             'excludes': ['tkinter'],
             'silent': True,
-            'packages': ['gi'],
+            'packages': find_packages() + ['gi'],
             'include_msvcr': True,
             },
         'bdist_mac': {

{tryton-6.8.12 → tryton-6.8.14}/tryton/__init__.py

@@ -1,6 +1,6 @@
 # This file is part of Tryton.  The COPYRIGHT file at the top level of
 # this repository contains the full copyright notices and license terms.
-__version__ = "6.8.12"
+__version__ = "6.8.14"
 import locale
 
 import gi

{tryton-6.8.12 → tryton-6.8.14}/tryton/common/common.py

@@ -229,7 +229,7 @@ class ModelAccess(object):
             self._models = rpc.execute('model', 'ir.model', 'list_models',
                 rpc.CONTEXT)
         except TrytonServerError:
-            pass
+            logger.error("Unable to get model list.")
 
     def __getitem__(self, model):
         if model in self._access:
@@ -243,7 +243,14 @@ class ModelAccess(object):
             access = rpc.execute('model', 'ir.model.access', 'get_access',
                 self._models[to_load], rpc.CONTEXT)
         except TrytonServerError:
-            access = {}
+            logger.error("Unable to get access for %s.", model)
+            access = {
+                model: {
+                    'read': True,
+                    'write': False,
+                    'create': False,
+                    'delete': False},
+                }
         self._access.update(access)
         return self._access[model]
 
@@ -1260,7 +1267,7 @@ class RPCProgress(object):
 
         def return_():
             if self.exception:
-                raise self.exception
+                raise RPCException(self.exception)
             else:
                 return self.res
 

{tryton-6.8.12 → tryton-6.8.14}/tryton/common/datetime_.py

@@ -9,7 +9,9 @@ from gi.repository import Gdk, GObject, Gtk
 
 from .common import IconFactory
 
-__all__ = ['Date', 'CellRendererDate', 'Time', 'CellRendererTime', 'DateTime']
+__all__ = [
+    'Date', 'CellRendererDate', 'Time', 'CellRendererTime', 'DateTime',
+    'date_parse']
 
 _ = gettext.gettext
 

{tryton-6.8.12 → tryton-6.8.14}/tryton/gui/window/view_form/screen/screen.py

@@ -114,11 +114,10 @@ class Screen:
                 return child
 
             # Remove first level Viewport and ScrolledWindow to fill the Vbox
-            for widget in [
-                    self.context_screen.screen_container.viewport,
-                    self.context_screen.current_view.widget.get_children()[0],
-                    ]:
-                remove_bin(widget)
+            remove_bin(self.context_screen.screen_container.viewport)
+            if self.context_screen.current_view:
+                remove_bin(
+                    self.context_screen.current_view.widget.get_children()[0])
 
             self.screen_container.filter_vbox.pack_start(
                 context_widget, expand=False, fill=True, padding=0)
@@ -515,7 +514,8 @@ class Screen:
         self.group.destroy()
 
     def default_row_activate(self):
-        if (self.current_view.view_type == 'tree'
+        if (self.current_view
+                and self.current_view.view_type == 'tree'
                 and int(self.current_view.attributes.get('keyword_open', 0))):
             return Action.exec_keyword('tree_open', {
                 'model': self.model_name,
@@ -637,7 +637,7 @@ class Screen:
 
     def new(self, default=True, rec_name=None):
         previous_view = self.current_view
-        if self.current_view.view_type == 'calendar':
+        if self.current_view and self.current_view.view_type == 'calendar':
             selected_date = self.current_view.get_selected_date()
         if self.current_view and not self.current_view.creatable:
             self.switch_view(creatable=True)
@@ -690,16 +690,19 @@ class Screen:
 
     def save_current(self):
         if not self.current_record:
-            if self.current_view.view_type == 'tree' and len(self.group):
+            if (self.current_view
+                    and self.current_view.view_type == 'tree'
+                    and len(self.group)):
                 self.current_record = self.group[0]
             else:
                 return True
-        self.current_view.set_value()
         saved = False
         record_id = None
-        fields = self.current_view.get_fields()
+        if self.current_view:
+            self.current_view.set_value()
+            fields = self.current_view.get_fields()
         path = self.current_record.get_path(self.group)
-        if self.current_view.view_type == 'tree':
+        if self.current_view and self.current_view.view_type == 'tree':
             # False value must be not saved
             saved = all((
                     x is not False and x >= 0
@@ -709,7 +712,7 @@ class Screen:
             record_id = self.current_record.save(force_reload=True)
             # False value must be not saved
             saved = record_id is not False and record_id >= 0
-        else:
+        elif self.current_view:
             self.set_cursor()
             self.current_view.display()
             return False
@@ -737,17 +740,19 @@ class Screen:
     def get(self):
         if not self.current_record:
             return None
-        self.current_view.set_value()
+        if self.current_view:
+            self.current_view.set_value()
         return self.current_record.get()
 
     def get_on_change_value(self):
         if not self.current_record:
             return None
-        self.current_view.set_value()
+        if self.current_view:
+            self.current_view.set_value()
         return self.current_record.get_on_change_value()
 
     def modified(self):
-        if self.current_view.view_type != 'tree':
+        if self.current_view and self.current_view.view_type != 'tree':
             if self.current_record:
                 if self.current_record.modified or self.current_record.id < 0:
                     return True
@@ -755,7 +760,7 @@ class Screen:
             for record in self.group:
                 if record.modified or record.id < 0:
                     return True
-        if self.current_view.modified:
+        if self.current_view and self.current_view.modified:
             return True
         return False
 
@@ -821,6 +826,8 @@ class Screen:
 
     def set_tree_state(self):
         view = self.current_view
+        if not view:
+            return
         if view.view_type not in {'tree', 'form', 'list-form'}:
             return
         if id(view) in self.tree_states_done:
@@ -930,7 +937,8 @@ class Screen:
 
     def load(self, ids, set_cursor=True, modified=False, position=-1):
         self.group.load(ids, modified=modified, position=position)
-        self.current_view.reset()
+        if self.current_view:
+            self.current_view.reset()
         if ids and self.current_view.view_type != 'calendar':
             self.display(ids[0])
         else:
@@ -950,7 +958,7 @@ class Screen:
                 self.current_record = self.group[0]
             else:
                 self.current_record = None
-        if self.views:
+        if self.views and self.current_view:
             self.search_active(self.current_view.view_type
                 in ('tree', 'graph', 'calendar'))
             for view in self.views:
@@ -982,7 +990,8 @@ class Screen:
 
     def _get_next_record(self, test=False):
         view = self.current_view
-        if (view.view_type in {'tree', 'form'}
+        if (view
+                and view.view_type in {'tree', 'form'}
                 and self.current_record
                 and self.current_record.group):
             group = self.current_record.group
@@ -1010,12 +1019,14 @@ class Screen:
                 record = next
                 break
             return record
-        elif (view.view_type == 'list-form' and len(self.group)
+        elif (view
+                and view.view_type == 'list-form'
+                and len(self.group)
                 and self.current_record in self.group):
             idx = self.group.index(self.current_record)
             if 0 <= idx < len(self.group) - 1:
                 return self.group[idx + 1]
-        elif view.view_type == 'calendar':
+        elif view and view.view_type == 'calendar':
             record = self.current_record
             goocalendar = view.widgets.get('goocalendar')
             if goocalendar:
@@ -1049,15 +1060,18 @@ class Screen:
 
     def display_next(self):
         view = self.current_view
-        view.set_value()
+        if view:
+            view.set_value()
         self.set_cursor(reset_view=False)
         self.current_record = self._get_next_record()
         self.set_cursor(reset_view=False)
-        view.display()
+        if view:
+            view.display()
 
     def _get_prev_record(self, test=False):
         view = self.current_view
-        if (view.view_type in {'tree', 'form'}
+        if (view
+                and view.view_type in {'tree', 'form'}
                 and self.current_record
                 and self.current_record.group):
             group = self.current_record.group
@@ -1075,7 +1089,7 @@ class Screen:
                 if parent and record.model_name == parent.model_name:
                     record = parent
             return record
-        elif view.view_type == 'calendar':
+        elif view and view.view_type == 'calendar':
             record = self.current_record
             goocalendar = view.widgets.get('goocalendar')
             if goocalendar:
@@ -1100,7 +1114,9 @@ class Screen:
                             if prev_id >= 0:
                                 return events[prev_id].record
                             break
-        elif (view.view_type == 'list-form' and len(self.group)
+        elif (view
+                and view.view_type == 'list-form'
+                and len(self.group)
                 and self.current_record in self.group):
             idx = self.group.index(self.current_record)
             if 0 < idx <= len(self.group) - 1:
@@ -1114,11 +1130,13 @@ class Screen:
 
     def display_prev(self):
         view = self.current_view
-        view.set_value()
+        if view:
+            view.set_value()
         self.set_cursor(reset_view=False)
         self.current_record = self._get_prev_record()
         self.set_cursor(reset_view=False)
-        view.display()
+        if view:
+            view.display()
 
     def invalid_message(self, record=None):
         if record is None:
@@ -1191,7 +1209,7 @@ class Screen:
         if not self.selected_records:
             return []
 
-        buttons = self.current_view.get_buttons()
+        buttons = self.current_view.get_buttons() if self.current_view else []
 
         for record in self.selected_records:
             buttons = [b for b in buttons if is_active(record, b)]
@@ -1201,8 +1219,9 @@ class Screen:
 
     def button(self, button):
         'Execute button on the selected records'
-        self.current_view.set_value()
-        fields = self.current_view.get_fields()
+        if self.current_view:
+            self.current_view.set_value()
+            fields = self.current_view.get_fields()
         for record in self.selected_records:
             domain = record.expr_eval(
                 button.get('states', {})).get('pre_validate', [])
@@ -1216,7 +1235,8 @@ class Screen:
         if button.get('confirm', False) and not sur(button['confirm']):
             return
         if button.get('type', 'class') == 'class':
-            if not self.current_record.save(force_reload=False):
+            record_id = self.current_record.save(force_reload=False)
+            if record_id is False or record_id < 0:
                 return
         if button.get('type', 'class') == 'class':
             self._button_class(button)
@@ -1247,6 +1267,7 @@ class Screen:
         except RPCException:
             action = None
         self.reload(ids, written=True)
+        self.record_saved()
         if isinstance(action, str):
             self.client_action(action)
         elif action:
@@ -1283,7 +1304,9 @@ class Screen:
         elif action.startswith('switch'):
             self.switch_view(*action.split(None, 2)[1:])
         elif action == 'reload':
-            if (self.current_view.view_type in ['tree', 'graph', 'calendar']
+            if (self.current_view
+                    and self.current_view.view_type in [
+                        'tree', 'graph', 'calendar']
                     and not self.parent):
                 self.search_filter()
         elif action == 'reload menu':
@@ -1313,7 +1336,7 @@ class Screen:
                         cls=JSONEncoder, separators=(',', ':'))))
         path = [CONFIG['login.db'], 'model', self.model_name]
         view_ids = [v.view_id for v in self.views] + self.view_ids
-        if self.current_view.view_type != 'form':
+        if self.current_view and self.current_view.view_type != 'form':
             if self.search_value:
                 search_value = self.search_value
             else:
@@ -1325,8 +1348,9 @@ class Screen:
                             separators=(',', ':'))))
         elif self.current_record and self.current_record.id > -1:
             path.append(str(self.current_record.id))
-            i = view_ids.index(self.current_view.view_id)
-            view_ids = view_ids[i:] + view_ids[:i]
+            if self.current_view:
+                i = view_ids.index(self.current_view.view_id)
+                view_ids = view_ids[i:] + view_ids[:i]
         if view_ids:
             query_string.append(('views', json.dumps(
                         view_ids, separators=(',', ':'))))

{tryton-6.8.12 → tryton-6.8.14}/tryton/gui/window/view_form/view/form_gtk/char.py

@@ -171,10 +171,9 @@ class Password(Char):
         self.widget.pack_start(
             self.visibility_checkbox, expand=False, fill=True, padding=0)
 
-    def _readonly_set(self, value):
-        super(Char, self)._readonly_set(value)
-        self.entry.set_editable(not value)
-        self.visibility_checkbox.props.visible = not value
-
     def toggle_visibility(self, button):
-        self.entry.props.visibility = not self.entry.props.visibility
+        if self.autocomplete:
+            entry = self.entry.get_child()
+        else:
+            entry = self.entry
+        entry.props.visibility = not self.entry.props.visibility

{tryton-6.8.12 → tryton-6.8.14}/tryton/gui/window/view_form/view/graph_gtk/graph.py

@@ -12,6 +12,7 @@ from dateutil.relativedelta import relativedelta
 from gi.repository import Gdk, Gtk
 
 import tryton.rpc as rpc
+from tryton import common
 from tryton.action import Action
 from tryton.common import COLOR_SCHEMES, generateColorscheme, hex2rgb
 from tryton.config import CONFIG
@@ -386,7 +387,8 @@ class Graph(Gtk.DrawingArea):
                     if isinstance(value, datetime.timedelta):
                         value = value.total_seconds()
                     self.datas[x][key] += float(value or 0)
-        date_format = self.view.screen.context.get('date_format', '%x')
+        date_format = common.date_format(
+            self.view.screen.context.get('date_format'))
         datetime_format = date_format + ' %X'
         if isinstance(minx, datetime.datetime):
             date = minx

{tryton-6.8.12 → tryton-6.8.14}/tryton/gui/window/view_form/view/list_gtk/editabletree.py

@@ -163,7 +163,8 @@ class EditableTreeView(TreeView):
         for renderer in column.get_cells():
             if renderer.props.editing:
                 widget = self.view.get_column_widget(column)
-                self.on_editing_done(widget.editable, renderer)
+                editable = widget.get_editable(renderer)
+                self.on_editing_done(editable, renderer)
         return True
 
     def on_keypressed(self, entry, event, renderer):

{tryton-6.8.12 → tryton-6.8.14}/tryton/gui/window/view_form/view/list_gtk/widget.py

@@ -23,7 +23,8 @@ from tryton.common.cellrenderertext import (
     CellRendererText, CellRendererTextCompletion)
 from tryton.common.cellrenderertoggle import CellRendererToggle
 from tryton.common.completion import get_completion, update_completion
-from tryton.common.datetime_ import CellRendererDate, CellRendererTime
+from tryton.common.datetime_ import (
+    CellRendererDate, CellRendererTime, date_parse)
 from tryton.common.domain_parser import quote
 from tryton.common.selection import (
     PopdownMixin, SelectionMixin, selection_shortcuts)
@@ -376,6 +377,14 @@ class GenericText(Cell):
         editable.connect('remove-widget', remove)
         return False
 
+    def get_editable(self, renderer):
+        if self.renderer == renderer:
+            return self.editable
+        for cell in self.prefixes + self.suffixes:
+            editable = cell.get_editable(renderer)
+            if editable:
+                return editable
+
 
 class Char(GenericText):
 
@@ -493,6 +502,17 @@ class Date(GenericText):
         else:
             return ''
 
+    def value_from_text(self, record, text, callback=None):
+        if isinstance(text, str):
+            field = record[self.attrs['name']]
+            try:
+                # Use a datetime instance and rely on field to convert to the
+                # proper type
+                text = date_parse(text, self.get_format(record, field))
+            except (ValueError, OverflowError):
+                text = None
+        return super().value_from_text(record, text, callback=callback)
+
 
 class Time(Date):
 

{tryton-6.8.12 → tryton-6.8.14}/tryton/gui/window/win_form.py

@@ -447,7 +447,7 @@ class WinForm(NoModal, InfoBar):
                 record.modified_fields.setdefault('id')
             result = False
         else:
-            result = response_id not in cancel_responses
+            result = (response_id not in cancel_responses) and not readonly
         if self.callback:
             self.callback(result)
         self.destroy()

{tryton-6.8.12 → tryton-6.8.14}/tryton/gui/window/win_import.py

@@ -2,6 +2,7 @@
 # this repository contains the full copyright notices and license terms.
 import base64
 import csv
+import datetime as dt
 import gettext
 import locale
 from decimal import Decimal
@@ -10,7 +11,6 @@ from gi.repository import Gtk
 
 import tryton.common as common
 from tryton.common import RPCException, RPCExecute
-from tryton.common.datetime_ import date_parse
 from tryton.gui.window.win_csv import WinCSV
 
 _ = gettext.gettext
@@ -211,13 +211,18 @@ class WinImport(WinCSV):
                             val = locale.atof(val)
                         elif type_ == 'numeric':
                             val = Decimal(locale.delocalize(val))
-                        elif type_ in ['date', 'datetime']:
-                            val = date_parse(val, common.date_format())
+                        elif type_ == 'date':
+                            val = dt.datetime.strptime(
+                                val, common.date_format()).date()
+                        elif type_ == 'datetime':
+                            val = dt.datetime.strptime(
+                                val, common.date_format() + ' %X')
                         elif type_ == 'binary':
                             val = base64.b64decode(val)
                     row.append(val)
                 data.append(row)
-        except (IOError, UnicodeDecodeError, csv.Error) as exception:
+        except (IOError, UnicodeDecodeError, csv.Error, ValueError) \
+                as exception:
             common.warning(str(exception), _("Import failed"))
             return
         try:

{tryton-6.8.12 → tryton-6.8.14}/tryton/gui/window/wizard.py

@@ -402,6 +402,8 @@ class WizardDialog(Wizard, NoModal):
         return True
 
     def show(self):
+        if not self.screen:
+            return
         view = self.screen.current_view
         if view.view_type == 'form':
             expand = False

{tryton-6.8.12 → tryton-6.8.14}/tryton/jsonrpc.py

@@ -138,7 +138,6 @@ class JSONUnmarshaller(object):
 class Transport(xmlrpc.client.SafeTransport):
 
     accept_gzip_encoding = True
-    encode_threshold = 1400  # common MTU
 
     def __init__(
             self, fingerprints=None, ca_certs=None, session=None):
@@ -241,6 +240,11 @@ class Transport(xmlrpc.client.SafeTransport):
         self._connection[1].sock.settimeout(DEFAULT_TIMEOUT)
         return self._connection[1]
 
+    @property
+    def encode_threshold(self):
+        if self.session:
+            return 1400  # common MTU
+
 
 class ServerProxy(xmlrpc.client.ServerProxy):
     __id = 0

{tryton-6.8.12 → tryton-6.8.14}/tryton/plugins/__init__.py

@@ -24,9 +24,11 @@ def register():
     imported = set()
     for path in paths:
         finder = importlib.machinery.FileFinder(
-            path, (
-                importlib.machinery.SourceFileLoader,
-                importlib.machinery.SOURCE_SUFFIXES))
+            path,
+            (importlib.machinery.SourceFileLoader,
+                importlib.machinery.SOURCE_SUFFIXES),
+            (importlib.machinery.SourcelessFileLoader,
+                importlib.machinery.BYTECODE_SUFFIXES))
         for plugin in os.listdir(path):
             module = os.path.splitext(plugin)[0]
             if (module.startswith('_') or module in imported):

{tryton-6.8.12 → tryton-6.8.14}/tryton/pyson.py

@@ -196,7 +196,7 @@ class Not(PYSON):
 
     @staticmethod
     def eval(dct, context):
-        return not dct['v']
+        return not Bool(dct['v']).eval(dct, context)
 
 
 class Bool(PYSON):

{tryton-6.8.12 → tryton-6.8.14}/tryton.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: tryton
-Version: 6.8.12
+Version: 6.8.14
 Summary: Tryton desktop client
 Home-page: http://www.tryton.org/
 Download-URL: http://downloads.tryton.org/6.8/