tryton 7.0.9__tar.gz → 7.0.11__tar.gz

{tryton-7.0.9 → tryton-7.0.11}/CHANGELOG

@@ -1,4 +1,14 @@
 
+Version 7.0.11 - 2024-07-01
+---------------------------
+* Bug fixes (see mercurial logs for details)
+
+
+Version 7.0.10 - 2024-05-01
+---------------------------
+* Bug fixes (see mercurial logs for details)
+
+
 Version 7.0.9 - 2024-04-17
 --------------------------
 * Bug fixes (see mercurial logs for details)

{tryton-7.0.9 → tryton-7.0.11}/PKG-INFO

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

{tryton-7.0.9 → tryton-7.0.11}/bin/tryton

@@ -15,7 +15,8 @@ if hasattr(sys, 'frozen'):
     share = os.path.join(prefix, 'share')
     os.environ['GTK_EXE_PREFIX'] = prefix
     os.environ['GTK_DATA_PREFIX'] = prefix
-    os.environ['EV_BACKENDS_DIR'] = prefix
+    os.environ['EV_BACKENDS_DIR'] = os.path.join(
+        prefix, 'lib', 'evince', '4', 'backends')
     os.environ['XDG_DATA_DIRS'] = share
     os.environ['GDK_PIXBUF_MODULE_FILE'] = os.path.join(
             share, 'gtk-3.0', 'gdk-pixbuf.loaders')

{tryton-7.0.9 → tryton-7.0.11}/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-7.0.9 → tryton-7.0.11}/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__ = "7.0.9"
+__version__ = "7.0.11"
 import locale
 
 import gi

{tryton-7.0.9 → tryton-7.0.11}/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]
 
@@ -1254,7 +1261,7 @@ class RPCProgress(object):
 
         def return_():
             if self.exception:
-                raise self.exception
+                raise RPCException(self.exception)
             else:
                 return self.res
 

{tryton-7.0.9 → tryton-7.0.11}/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-7.0.9 → tryton-7.0.11}/tryton/gui/window/view_form/model/field.py

@@ -9,6 +9,7 @@ import os
 import tempfile
 from decimal import Decimal
 from itertools import chain
+from pathlib import Path
 
 import tryton.common as common
 from tryton.common import (
@@ -1048,26 +1049,56 @@ class ReferenceField(Field):
 
 
 class _FileCache(object):
-    def __init__(self, path):
-        self.path = path
+    def __init__(self, data=None):
+        _, filename = tempfile.mkstemp(prefix='tryton_')
+        self.path = Path(filename)
+        self.suffixes = {}
+        if data:
+            with open(self.path, 'wb') as fp:
+                fp.write(data)
+
+    @property
+    def data(self):
+        with open(self.path, 'rb') as fp:
+            return fp.read()
 
     def __del__(self):
         try:
             os.remove(self.path)
         except IOError:
             pass
+        for path in self.suffixes.values():
+            try:
+                os.remove(path)
+            except IOError:
+                pass
+
+    def with_suffix(self, suffix):
+        if suffix in self.suffixes:
+            return self.suffixes[suffix]
+        _, filename = tempfile.mkstemp(prefix='tryton_', suffix=suffix)
+        self.suffixes[suffix] = path = Path(filename)
+        with open(path, 'wb') as fp:
+            fp.write(self.data)
+        return path
 
 
 class BinaryField(Field):
 
     _default = None
 
+    def _set_file_cache(self, record, data):
+        if isinstance(data, str):
+            data = data.encode('utf-8')
+        file_cache = _FileCache(data)
+        self.set(record, file_cache)
+        return file_cache
+
     def get(self, record):
         result = record.value.get(self.name, self._default)
         if isinstance(result, _FileCache):
             try:
-                with open(result.path, 'rb') as fp:
-                    result = fp.read()
+                result = result.data
             except IOError:
                 result = self.get_data(record)
         return result
@@ -1076,13 +1107,7 @@ class BinaryField(Field):
         return self.get(record)
 
     def set_client(self, record, value, force_change=False):
-        _, filename = tempfile.mkstemp(prefix='tryton_')
-        data = value or b''
-        if isinstance(data, str):
-            data = data.encode('utf-8')
-        with open(filename, 'wb') as fp:
-            fp.write(data)
-        self.set(record, _FileCache(filename))
+        self._set_file_cache(record, value or b'')
         self.sig_changed(record)
         record.validate(softvalidation=True)
         record.set_modified(self.name)
@@ -1106,15 +1131,20 @@ class BinaryField(Field):
                     [record.id], [self.name], context=context)
             except RPCException:
                 return b''
-            _, filename = tempfile.mkstemp(prefix='tryton_')
-            data = values[self.name] or b''
-            if isinstance(data, str):
-                data = data.encode('utf-8')
-            with open(filename, 'wb') as fp:
-                fp.write(data)
-            self.set(record, _FileCache(filename))
+            self._set_file_cache(record, values[self.name] or b'')
         return self.get(record)
 
+    def get_filename(self, record, suffix=None):
+        data = self.get_data(record)
+        file_cache = record.value.get(self.name)
+        if not isinstance(file_cache, _FileCache):
+            file_cache = self._set_file_cache(record, data)
+        if suffix:
+            filename = file_cache.with_suffix(suffix)
+        else:
+            filename = file_cache.path
+        return filename
+
 
 class DictField(Field):
 

{tryton-7.0.9 → tryton-7.0.11}/tryton/gui/window/view_form/screen/screen.py

@@ -116,11 +116,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)
@@ -520,7 +519,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,
@@ -642,7 +642,7 @@ class Screen:
 
     def new(self, default=True, defaults=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)
@@ -695,16 +695,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
@@ -714,7 +717,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
@@ -742,17 +745,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
@@ -760,7 +765,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
 
@@ -829,6 +834,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:
@@ -938,12 +945,13 @@ 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()
         self.current_record = None
         self.display(set_cursor=set_cursor)
 
     def display(self, set_cursor=False):
-        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:
@@ -975,7 +983,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
@@ -1003,12 +1012,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:
@@ -1042,15 +1053,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
@@ -1068,7 +1082,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:
@@ -1093,7 +1107,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:
@@ -1107,11 +1123,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:
@@ -1184,7 +1202,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)]
@@ -1194,8 +1212,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', [])
@@ -1209,7 +1228,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)
@@ -1240,6 +1260,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:
@@ -1276,7 +1297,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':
@@ -1306,7 +1329,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:
@@ -1318,8 +1341,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-7.0.9 → tryton-7.0.11}/tryton/gui/window/view_form/view/calendar_gtk/calendar_.py

@@ -67,13 +67,18 @@ class Calendar_(goocalendar.Calendar):
             self.current_domain_period.get_dates(True)
         dtstart = self.attrs['dtstart']
         dtend = self.attrs.get('dtend') or dtstart
-        domain = ['OR',
-            ['AND', (dtstart, '>=', first_datetime),
-                (dtstart, '<', last_datetime)],
-            ['AND', (dtend, '>=', first_datetime),
-                (dtend, '<', last_datetime)],
-            ['AND', (dtstart, '<', first_datetime),
-                (dtend, '>', last_datetime)]]
+        domain = [
+            (dtstart, '!=', None),
+            (dtend, '!=', None),
+            ['OR',
+                ['AND', (dtstart, '>=', first_datetime),
+                    (dtstart, '<', last_datetime)],
+                ['AND', (dtend, '>=', first_datetime),
+                    (dtend, '<', last_datetime)],
+                ['AND', (dtstart, '<', first_datetime),
+                    (dtend, '>', last_datetime)],
+                ],
+            ]
         return domain
 
     def get_colors(self, record):

{tryton-7.0.9 → tryton-7.0.11}/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-7.0.9 → tryton-7.0.11}/tryton/gui/window/view_form/view/form_gtk/document.py

@@ -1,7 +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.
 from pathlib import Path
-from tempfile import NamedTemporaryFile
 
 from gi.repository import Gdk, GLib, Gtk
 
@@ -76,17 +75,17 @@ class Document(BinaryMixin, Widget):
             self.image.hide()
             if self.evince_view:
                 self.evince_scroll.show()
+                suffix = None
                 if self.filename_field:
-                    suffix = self.filename_field.get(self.record)
-                else:
-                    suffix = None
+                    filename = self.filename_field.get(self.record)
+                    if filename:
+                        suffix = Path(filename).suffix
+                filename = Path(self.field.get_filename(self.record, suffix))
                 try:
-                    with NamedTemporaryFile(suffix=suffix) as fp:
-                        fp.write(data)
-                        path = Path(fp.name)
-                        document = (
-                            EvinceDocument.Document.factory_get_document(
-                                path.as_uri()))
+                    document = (
+                        EvinceDocument.Document.factory_get_document_full(
+                            filename.as_uri(),
+                            EvinceDocument.DocumentLoadFlags.NONE))
                     model = EvinceView.DocumentModel()
                     model.set_document(document)
                     self.evince_view.set_model(model)

{tryton-7.0.9 → tryton-7.0.11}/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):