lino 25.2.2__py3-none-any.whl → 25.2.3__py3-none-any.whl

lino/__init__.py

@@ -26,7 +26,7 @@ defines no models, some template files, a series of :term:`django-admin commands
 
 """
 
-__version__ = '25.2.2'
+__version__ = '25.2.3'
 
 # import setuptools  # avoid UserWarning "Distutils was imported before Setuptools"?
 
@@ -64,14 +64,19 @@ import warnings
 
 warnings.filterwarnings(
     "error",
-    "DateTimeField .* received a naive datetime (.*) while time zone support is active.",
+    r"DateTimeField .* received a naive datetime (.*) while time zone support is active.",
     RuntimeWarning,
     "django.db.models.fields",
 )
 
+# TODO: Is it okay to ignore the followgin warning? It's because e.g.
+# lino.modlib.excerpts has a pre_analyze receiver set_excerpts_actions(), which
+# accesses the database to set actions before Lino analyzes the models. It
+# catches OperationalError & Co because --of course-- it fails e.g. for
+# admin-commands like "pm prep".
 warnings.filterwarnings(
     "ignore",
-    "Accessing the database during app initialization is discouraged\. To fix this warning, avoid executing queries in AppConfig\.ready\(\) or when your app modules are imported\.",
+    r"Accessing the database during app initialization is discouraged\. To fix this warning, avoid executing queries in AppConfig\.ready\(\) or when your app modules are imported\.",
     RuntimeWarning,
     "django.db.backends.utils",
 )

lino/api/dd.py

@@ -209,6 +209,19 @@ from lino.modlib.linod.choicelists import Procedures
 
 
 def background_task(**kwargs):
+    """
+    Register the decorated function as a :term:`background task`.
+
+    Keyword arguments are used as default values when checkdata creates a
+    :class:`lino.modlib.linod.SystemTask` instance for this procedure.
+
+    Except for the special keyword ``class_name``, which defaults to
+    "linod.SystemTask". It is used by :mod:`lino_xl.lib.invoicing` to register a
+    procedure that will create an :term:`invoicing task` instead of a normal
+    :term:`background task`. :class:`lino_xl.lib.invoicing.InvoicingTask`
+    instead of :class:`lino.modlib.linod.SystemTask`.
+
+    """
     if "class_name" not in kwargs:
         kwargs["class_name"] = "linod.SystemTask"
 

lino/api/doctest.py

@@ -1,28 +1,45 @@
 # -*- coding: UTF-8 -*-
-# Copyright 2015-2024 Rumma & Ko Ltd
+# Copyright 2015-2025 Rumma & Ko Ltd
 # License: GNU Affero General Public License v3 (see file COPYING for details)
 """
-A selection of names to be used in tested documents.
+A selection of names to be used in tested documents as follows:
+
+>>> from lino.api.doctest import *
+
+This is by convention everything we want to have in the global namespace of a
+tested document. It includes
+
+- well-known Python standard modules like os, sys, datetime and collections.
+- A variable :data:`test_client`
+
 """
 
+import os
+import sys
+import datetime
+import collections
 import six  # TODO: remove here and then run all doctests
 import logging
 import sqlparse
+import json
+import textwrap
+
+from bs4 import BeautifulSoup
+from pprint import pprint, pformat
 from urllib.parse import urlencode
+
 import django
 django.setup()
+
 from lino.core.constants import *
 from lino.api.shell import *
 from django.utils import translation
 from django.utils.encoding import force_str
 from django.test import Client
 from django.db import connection, reset_queries as reset_sql_queries
-import json
-from bs4 import BeautifulSoup
-import textwrap
-from pprint import pprint, pformat
 
-from rstgen import table, ul
+import pytest
+# from rstgen import table, ul
 import rstgen
 from rstgen import attrtable
 from rstgen.utils import unindent, rmu, sixprint
@@ -49,10 +66,12 @@ from lino.core.actions import register_params
 from lino.core.layouts import BaseLayout
 
 test_client = Client()
-# naming it simply "client" caused conflict with a
-# `lino_welfare.pcsw.models.Client`
+"""An instance of :class:`django.test.Client`.
 
-import collections
+N.B. Naming it simply "client" caused conflict with a
+:class:`lino_welfare.pcsw.models.Client`
+
+"""
 
 HttpQuery = collections.namedtuple(
     "HttpQuery", ["username", "url_base", "json_fields", "expected_rows", "kwargs"]
@@ -293,7 +312,7 @@ def show_workflow(actions, all=False, language=None):
                         # required_roles
                     ]
                 )
-        print(table(cols, cells).strip())
+        print(rstgen.table(cols, cells).strip())
 
     if language:
         with translation.override(language):
@@ -363,7 +382,7 @@ def fields_help(model, fieldnames=None, columns=False, all=None):
 
     # return table(cols, cells).strip()
     items = ["{} ({}) : {}".format(row[1], row[0], row[2]) for row in cells]
-    return ul(items).strip()
+    return rstgen.ul(items).strip()
 
 
 def show_fields(*args, **kwargs):
@@ -552,7 +571,7 @@ def show_choicelist(cls):
     for i in cls.get_list_items():
         row = [i.value, i.name] + str2languages(i.text)
         rows.append(row)
-    print(table(headers, rows))
+    print(rstgen.table(headers, rows))
 
 
 def show_choicelists():
@@ -568,7 +587,7 @@ def show_choicelists():
             i.verbose_name_plural
         )
         rows.append(row)
-    print(table(headers, rows))
+    print(rstgen.table(headers, rows))
 
 
 def show_permissions(*args):
@@ -590,7 +609,7 @@ def show_translations(things, fmt, languages=None):
                 x, txt = fmt(thing)
                 cells.append(txt)
         rows.append(cells)
-    print(table(headers, rows))
+    print(rstgen.table(headers, rows))
 
 
 def show_model_translations(*models, **kwargs):
@@ -779,7 +798,7 @@ def show_change_watchers():
             rows.append(
                 [full_model_name(m), ws.master_key, " ".join(sorted(ws.ignored_fields))]
             )
-    print(table(headers, rows, max_width=40))
+    print(rstgen.table(headers, rows, max_width=40))
 
 def show_display_modes():
     """
@@ -796,4 +815,17 @@ def show_display_modes():
                     ("x" if dm in a.extra_display_modes else "")
                         for dm in dml]
             )
-    print(table(headers, rows))
+    print(rstgen.table(headers, rows))
+
+
+def checkdb(m, num):
+    """
+    Raise an exception if the database doesn't contain the specified number of
+    rows of the specified model.
+
+    This is for usage in :xfile:`startup.py` scripts.
+
+    """
+    if m.objects.count() != num:
+        raise Exception(
+            f"Model {m} should have {num} rows but has {m.objects.count()}")

lino/api/selenium.py

@@ -115,7 +115,7 @@ class Tour(object):
     error_message = None
     language = None
     languages = []
-    server_url = "http://127.0.0.1:8000/"
+    server_url = "http://127.0.0.1:8000"
     images_width = 90
 
     def __init__(

lino/core/actors.py

@@ -1948,17 +1948,26 @@ class Actor(actions.Parametrizable, Permittable, metaclass=ActorMetaClass):
         return html
 
     @classmethod
-    def get_table_summary(cls, obj, ar=None):
+    def get_slave_summary(cls, obj, ar=None):
+        """
+        :param cls: Slave table
+        :param obj: Master instance
+        :param ar: Action request on master table
+        """
+        if ar is None:
+            return ''
+        sar = cls.request(parent=ar, master_instance=obj, is_on_main_actor=False)
+        return cls.get_table_summary(sar)
+
+    @classmethod
+    def get_table_summary(cls, ar):
         """
         Return the HTML `<div>` to be displayed by
         :class:`lino.core.elems.TableSummaryPanel`.
         It basically just calls :meth:`table_as_summary`.
 
         """
-        if ar is None:
-            return ''
-        sar = cls.request(parent=ar, master_instance=obj, is_on_main_actor=False)
-        p = cls.table_as_summary(sar)
+        p = cls.table_as_summary(ar)
         # assert_safe(p)  # temporary 20240506
         # print("20240712", p)
         # return format_html(DIVTPL, p)
@@ -1981,9 +1990,9 @@ class Actor(actions.Parametrizable, Permittable, metaclass=ActorMetaClass):
         """
         p = qs2summary(
             ar,
-            ar.data_iterator,
+            ar.sliced_data_iterator,
             separator=cls.summary_sep,
-            max_items=cls.preview_limit,
+            max_items=ar.limit or cls.preview_limit,
             wraptpl=None,
         )
         # assert isinstance(p, str)

lino/core/elems.py

@@ -1830,7 +1830,7 @@ class SlaveSummaryPanel(LightWeightContainer):
     oui5_field_template = "openui5/elems/field/SlaveSummaryElement.xml"
 
     def __init__(self, lh, slave, name, **kw):
-        super().__init__(lh, slave, name, slave.get_table_summary, **kw)
+        super().__init__(lh, slave, name, slave.get_slave_summary, **kw)
 
 
 class StoryElement(LightWeightContainer):

lino/core/renderer.py

@@ -284,7 +284,7 @@ class HtmlRenderer(Renderer):
                 raise Exception("Both nosummary and display_mode were specified")
 
         if display_mode == constants.DISPLAY_MODE_SUMMARY:
-            yield ar.actor.get_table_summary(ar.master_instance, ar)
+            yield ar.actor.get_table_summary(ar)
             return
 
         if header_level is not None:
@@ -834,7 +834,7 @@ class TextRenderer(HtmlRenderer):
         # yield "20240506 {}".format(ar)
         if display_mode == constants.DISPLAY_MODE_SUMMARY:
             s = to_rst(
-                ar.actor.get_table_summary(ar.master_instance, ar),
+                ar.actor.get_table_summary(ar),
                 stripped=stripped,
             )
             if stripped:

lino/core/requests.py

@@ -24,6 +24,7 @@ from django.conf import settings
 from django.utils.translation import gettext_lazy as _
 from django.utils.translation import gettext
 from django.utils.translation import get_language, activate
+from django.utils.html import format_html
 from django.utils import translation
 from django.utils import timezone
 from django.core.mail import send_mail
@@ -103,6 +104,16 @@ def mi2bp(master_instance, bp):
             #     self.master_instance.__class__, self.master_instance))
 
 
+class PrintLogger(logging.Logger):
+    format = "%(message)s"
+
+    def __init__(self, level):
+        super().__init__("~", level)
+        h = logging.StreamHandler(stream=sys.stdout)
+        h.setFormatter(logging.Formatter(self.format))
+        self.addHandler(h)
+        # print("20231012", logging.getLevelName(self.level), self.__class__)
+
 class StringLogger(logging.Logger):
     # Instantiated by BaseRequest.capture_logger()
 
@@ -535,12 +546,26 @@ class BaseRequest:
         try:
             # We instantiate a temporary Logger object, which is not known by the
             # root logger.
+            # self.logger = PrintLogger(level)
             self.logger = StringLogger(old_logger, level)
             # self.logger.parent =
             yield self.logger
 
         finally:
             self.logger.streamer.flush()
+            # print(self.logger.getvalue())
+            self.logger = old_logger
+
+    @contextmanager
+    def print_logger(self, level=logging.INFO):
+        old_logger = self.logger
+        try:
+            # We instantiate a temporary Logger object, which is not known by the
+            # root logger.
+            self.logger = PrintLogger(level)
+            yield None
+
+        finally:
             self.logger = old_logger
 
     @contextmanager
@@ -1638,7 +1663,8 @@ class BaseRequest:
             # print("20190703", self.actor, self.actor.default_action)
             sar = self.spawn_request(actor=self.actor)
             list_title = tostring(sar.href_to_request(sar, list_title, icon_name=None))
-            return list_title + " » " + self.get_detail_title(elem)
+            # return list_title + " » " + self.get_detail_title(elem)
+            return format_html("{} » {}", list_title,  self.get_detail_title(elem))
 
     def form2obj_and_save(ar, data, elem, is_new):
         """

lino/core/site.py

@@ -356,7 +356,7 @@ class Site(object):
     verbose_client_info_message = False
 
     stopsignal = "SIGTERM"
-    help_url = "http://www.lino-framework.org"
+    help_url = "https://www.lino-framework.org"
 
     help_email = "users@lino-framework.org"
 
@@ -395,11 +395,11 @@ class Site(object):
 
     date_format_strftime = "%d.%m.%Y"
 
-    date_format_regex = "/^[0123]?\d\.[01]?\d\.-?\d+$/"
+    date_format_regex = r"/^[0123]?\d\.[01]?\d\.-?\d+$/"
 
     datetime_format_strftime = "%Y-%m-%dT%H:%M:%S"
 
-    datetime_format_extjs = "Y-m-d\TH:i:s"
+    datetime_format_extjs = r"Y-m-d\TH:i:s"
 
     quick_startup = False
 
@@ -973,6 +973,7 @@ class Site(object):
 
     def install_settings(self):
         assert not self.help_url.endswith("/")
+        assert not self.server_url.endswith("/")
 
         for p in self.installed_plugins:
             p.install_django_settings(self)
@@ -2136,13 +2137,12 @@ class Site(object):
 
         # yield "lino.modlib.lino_startup"
 
-    # server_url = None
     copyright_name = None
     """Name of copyright holder of the site's content."""
 
     copyright_url = None
 
-    server_url = "http://127.0.0.1:8000/"
+    server_url = "http://127.0.0.1:8000"
     """The "official" URL used by "normal" users when accessing this Lino
     site.
 

lino/help_texts.py

@@ -553,6 +553,10 @@ help_texts = {
     'lino.modlib.gkfs.GenericForeignKey' : _("""Add verbose_name and help_text to Django’s GFK."""),
     'lino.modlib.gkfs.GenericForeignKeyIdField' : _("""Use this instead of models.PositiveIntegerField for fields that are part of a GFK and you want Lino to render them using a Combobox."""),
     'lino.modlib.jinja.JinjaBuildMethod' : _("""Inherits from lino.modlib.printing.DjangoBuildMethod."""),
+    'lino.modlib.jinja.XMLMaker' : _("""Usage example in /topics/xml"""),
+    'lino.modlib.jinja.XMLMaker.xml_file_name' : _("""The name of the XML file to generate. This file will be overwritten without asking. The name formatted with one name self in the context."""),
+    'lino.modlib.jinja.XMLMaker.xml_file_template' : _("""The name of a Jinja template to render for generating the XML content."""),
+    'lino.modlib.jinja.XMLMaker.xml_validator_file' : _("""The name of a “validator” to use for validating the XML content."""),
     'lino.modlib.memo.Previewable' : _("""Adds three rich text fields (lino.core.fields.RichTextField):"""),
     'lino.modlib.memo.Previewable.body' : _("""An editable text body."""),
     'lino.modlib.memo.Previewable.body_short_preview' : _("""A read-only preview of the first paragraph of body."""),

lino/modlib/extjs/__init__.py

@@ -250,12 +250,12 @@ class Plugin(Plugin):
             ),
             url(
                 rx + r"choices/(?P<app_label>\w+)/(?P<rptname>\w+)/"
-                "(?P<fldname>\w+)$",
+                r"(?P<fldname>\w+)$",
                 views.Choices.as_view(),
             ),
             url(
                 rx + r"apchoices/(?P<app_label>\w+)/(?P<actor>\w+)/"
-                "(?P<an>\w+)/(?P<field>\w+)$",
+                r"(?P<an>\w+)/(?P<field>\w+)$",
                 views.ActionParamChoices.as_view(),
             ),
             # the thread_id can be a negative number:

lino/modlib/extjs/views.py

@@ -27,6 +27,7 @@ import os
 from pathlib import Path
 
 from django import http
+from django.contrib.messages import success
 from django.db import models
 from django.conf import settings
 from django.core.cache import cache
@@ -38,6 +39,7 @@ from django.utils.decorators import method_decorator
 from django.views.decorators.csrf import ensure_csrf_cookie, csrf_protect
 from django.utils.translation import gettext as _
 from django.utils.encoding import force_str
+from django.utils.html import mark_safe
 from lino.core import auth
 
 from lino.core.signals import pre_ui_delete
@@ -343,19 +345,44 @@ class ApiElement(View):
             ba = rpt.detail_action
             if ba is None:
                 raise http.Http404("%s has no detail_action" % rpt)
+
+        fmt = request.GET.get(constants.URL_PARAM_FORMAT, ba.action.default_format)
+
         try:
             if pk and pk != "-99999" and pk != "-99998":
                 sr = [pk]
-                # if issubclass(rpt.model, models.Model):
-                #     try:
-                #         ar = ba.request(request=request, selected_pks=sr)
-                #     # except ObjectDoesNotExist as e:  # 20250212
-                #     except rpt.model.DoesNotExist as e:
-                #         # print("20240911", e)
-                #         raise http.Http404(f"Object {sr} does not exist on {rpt}")
-                # else:
-                #     ar = ba.request(request=request, selected_pks=sr)
-                ar = ba.request(request=request, selected_pks=sr)
+                if issubclass(rpt.model, models.Model):
+                    try:
+                        ar = ba.request(request=request, selected_pks=sr)
+                    # except ObjectDoesNotExist as e:  # 20250212
+                    except rpt.model.DoesNotExist as e:
+                        if fmt == constants.URL_FORMAT_JSON:
+                            # rescue_ar: without sr and even request, to render a table request (grid view action) on breadcrumb
+                            rescue_ar = rpt.request(renderer=settings.SITE.kernel.default_renderer)
+                            default_table = rpt.model.get_default_table()
+
+                            title = tostring(rescue_ar.href_to_request(rescue_ar, icon_name=None))
+                            def get_response():
+                                msg = mark_safe(f'Record (pk={pk}) is no longer available on current table.')
+                                datarec = dict(success=False, message=msg, title=title)
+                                datarec.update(**vm)
+                                return datarec
+
+                            try:
+                                # take default table and try to show the row
+                                ar = default_table.detail_action.request(request=request, selected_pks=sr)
+                            except default_table.model.DoesNotExist as e:
+                                return json_response(get_response())
+
+                            url = ar.obj2url(ar.selected_rows[0])
+                            datarec = get_response()
+                            datarec['message'] += mark_safe(f' Reload in <a href="{url}">{default_table}</a>.')
+                            return json_response(datarec)
+                        # print("20240911", e)
+                        raise http.Http404(f"Object {sr} does not exist on {rpt}")
+                else:
+                    ar = ba.request(request=request, selected_pks=sr)
+                # ar = ba.request(request=request, selected_pks=sr)
                 elem = ar.selected_rows[0]
                 # print(
                 #     "20170116 views.ApiElement.get", ba,
@@ -398,8 +425,6 @@ class ApiElement(View):
 
         # print("20240402 permission", ar, "granted to", ar.get_user(), ar.bound_action.action.select_rows, ar.selected_rows)
 
-        fmt = request.GET.get(constants.URL_PARAM_FORMAT, ba.action.default_format)
-
         if ba.action.opens_a_window:
             if fmt == constants.URL_FORMAT_JSON:
                 if pk == "-99999":

lino/modlib/help/config/makehelp/conf.tpl.py

@@ -10,7 +10,7 @@ intersphinx_mapping = {}
 
 {% if makehelp.language.index == 0 -%}
 
-html_context = dict(public_url="{{settings.SITE.server_url}}media/cache/help")
+html_context = dict(public_url="{{settings.SITE.server_url}}/media/cache/help")
 
 from rstgen.sphinxconf import configure ; configure(globals())
 from lino.sphinxcontrib import configure ; configure(globals())

lino/modlib/jinja/mixins.py

@@ -0,0 +1,62 @@
+# -*- coding: UTF-8 -*-
+# Copyright 2022 Rumma & Ko Ltd
+# License: GNU Affero General Public License v3 (see file COPYING for details)
+
+import os
+from pathlib import Path
+from lxml import etree
+
+from django.conf import settings
+from django.utils import translation
+from django.utils.html import mark_safe, escape
+
+from lino.api import dd
+from lino.utils.xml import validate_xml
+
+def xml_element(name, value):
+    if value:
+        return f"<{name}>{escape(str(value))}</{name}>"
+    return ""
+
+
+class XMLMaker(dd.Model):
+
+    class Meta:
+        abstract = True
+
+    xml_validator_file = None
+    xml_file_template = None
+    xml_file_name = None
+
+    def make_xml_file(self, ar):
+        renderer = settings.SITE.plugins.jinja.renderer
+        tpl = renderer.jinja_env.get_template(self.xml_file_template)
+        context = self.get_printable_context(ar)
+        context.update(xml_element=xml_element)
+        xml = tpl.render(**context)
+        parts = [
+            dd.plugins.accounting.xml_media_dir,
+            self.xml_file_name.format(self=self)]
+        xmlfile = Path(settings.MEDIA_ROOT, *parts)
+        ar.logger.info("Make %s from %s ...", xmlfile, self)
+        xmlfile.parent.mkdir(exist_ok=True, parents=True)
+        xmlfile.write_text(xml)
+        # xmlfile.write_text(etree.tostring(xml))
+
+        if self.xml_validator_file:
+            # print("20250218 {xml[:100]}")
+            # doc = etree.fromstring(xml.encode("utf-8"))
+            ar.logger.info("Validate %s against %s ...", xmlfile.name, self.xml_validator_file)
+            if True:
+                validate_xml(xmlfile, self.xml_validator_file)
+            else:
+                try:
+                    validate_xml(xmlfile, self.xml_validator_file)
+                except Exception as e:
+                    msg = _("XML validation failed: {}").format(e)
+                    # print(msg)
+                    raise Warning(msg)
+
+        url = settings.SITE.build_media_url(*parts)
+        # return mark_safe(f"""<a href="{url}">{url}</a>""")
+        return (xmlfile, url)

lino/modlib/jinja/models.py

@@ -0,0 +1,6 @@
+# -*- coding: UTF-8 -*-
+# Copyright 2025 Rumma & Ko Ltd
+# License: GNU Affero General Public License v3 (see file COPYING for details)
+
+from .mixins import XMLMaker
+from .choicelists import JinjaBuildMethod

lino/modlib/linod/fixtures/linod.py

@@ -0,0 +1,32 @@
+# -*- coding: utf-8 -*-
+# Copyright 2025 Rumma & Ko Ltd
+# License: GNU Affero General Public License v3 (see file COPYING for details)
+from asgiref.sync import async_to_sync
+from lino.api import rt
+from lino.modlib.linod.mixins import start_task_runner
+
+# import logging
+# from lino import logger
+# def getHandlerByName(name):
+#     for l in logger.handlers:
+#         if l.name == name:
+#             return l
+
+def objects():
+    raise Exception("""
+
+    This fixture isn't used at the moment.  I wrote it because I thought it
+    would be nice to run the system task runner automatically when ``pm prep``
+    in order to cover the sync_ibanity system task. But (1) this would require
+    me to integrate also the ``checkdata`` and ``checksummaries`` fixtures into
+    it (otherwise they would run again as a system task) and (2) we don't want
+    to start `sync_ibanity` automatically on GitLab because it can't work
+    without credentials.
+
+        """)
+    ar = rt.login("robin")
+    # logger.setLevel(logging.DEBUG)
+    # getHandlerByName('console').setLevel(logging.DEBUG)
+    # ar.debug("Coucou")
+    async_to_sync(start_task_runner)(ar, max_count=1)
+    return []

lino/modlib/linod/mixins.py

@@ -81,9 +81,11 @@ class Runnable(Sequenced, RecurrenceSet):
 
     def full_clean(self, *args, **kwargs):
         super().full_clean(*args, **kwargs)
-        class_name = dd.full_model_name(self.__class__)
-        if self.procedure.class_name != class_name:
-            raise ValidationError(f"Invalid procedure for {class_name}")
+        # 20250213 The following caused 'Invalid procedure invoicing.Task for
+        # linod.SystemTask' during restore.py:
+        # class_name = dd.full_model_name(self.__class__)
+        # if self.procedure.class_name != class_name:
+        #     raise ValidationError(f"Invalid procedure {self.procedure.class_name} for {class_name}")
         if self.every_unit is None:
             self.every_unit = Recurrences.never
         if not self.name:

lino/modlib/memo/mixins.py

@@ -43,7 +43,8 @@ def rich_text_to_elems(ar, description):
 
     # After 20250213 #5929 (Links in the description of a ticket aren't rendered
     # correctly) we no longer try to automatically detect reSTructuredText
-    # markup in a RichTextField (anyway nobody has ever used this feature).
+    # markup in a RichTextField. Anyway nobody has ever used this feature
+    # (except for the furniture fixture of the products plugin).
 
     # if description.startswith("<"):
     if True:

lino/modlib/memo/parser.py

@@ -135,7 +135,7 @@ class Parser:
     def compile_suggester_regex(self):
         triggers = "".join(
             [
-                "\\" if key in "[\^$.|?*+(){}" else "" + key
+                r"\\" if key in r"[\^$.|?*+(){}" else "" + key
                 for key in self.suggesters.keys()
             ]
         )

lino/modlib/notify/models.py

@@ -386,7 +386,7 @@ class MyMessages(My, Messages):
     default_display_modes = {None: constants.DISPLAY_MODE_SUMMARY}
 
     @classmethod
-    def unused_get_table_summary(cls, mi, ar):
+    def unused_get_table_summary(cls, ar):
         # 20240710 Replaced by table_as_summary(), which is now more simple. But
         # I leave the old version here in case some unexpected regression
         # occurs.