lino 25.6.1__py3-none-any.whl → 25.7.1__py3-none-any.whl

lino/core/tables.py

@@ -403,7 +403,7 @@ class AbstractTable(actors.Actor):
     order_by = None
     """If specified, this must be a tuple or list of field names that
 will be passed to Django's `order_by
-<https://docs.djangoproject.com/en/5.0/ref/models/querysets/#order-by>`__
+<https://docs.djangoproject.com/en/5.2/ref/models/querysets/#order-by>`__
 method in order to sort the rows of the queryset.
 
     """
@@ -412,7 +412,7 @@ method in order to sort the rows of the queryset.
     """
     If specified, this must be a :class:`django.db.models.Q` object that will be
     passed to Django's `filter
-    <https://docs.djangoproject.com/en/5.0/ref/models/querysets/#filter>`__
+    <https://docs.djangoproject.com/en/5.2/ref/models/querysets/#filter>`__
     method.
 
     If you allow a user to insert rows into a filtered table, you should make
@@ -427,7 +427,7 @@ method in order to sort the rows of the queryset.
     One advantage of :attr:`filter` over
     :attr:`known_values  <lino.core.actors.Actor.known_values>`
     is that this can use the full range of Django's `field lookup methods
-    <https://docs.djangoproject.com/en/5.0/topics/db/queries/#field-lookups>`_
+    <https://docs.djangoproject.com/en/5.2/topics/db/queries/#field-lookups>`_
 
     """
 
@@ -435,7 +435,7 @@ method in order to sort the rows of the queryset.
     """
     If specified, this must be a :class:`django.db.models.Q` object that will be
     passed to Django's `exclude
-    <https://docs.djangoproject.com/en/5.0/ref/models/querysets/#exclude>`__
+    <https://docs.djangoproject.com/en/5.2/ref/models/querysets/#exclude>`__
     method.
 
     This is the logical opposite of :attr:`filter`.
@@ -485,6 +485,8 @@ method in order to sort the rows of the queryset.
         resolve_fields_list(cls, "mobile_columns", set, {})
         resolve_fields_list(cls, "popin_columns", set, {})
         if cls.model is not None:
+            if not isinstance(cls.model, type):
+                raise Exception(f"{cls}.model is {repr(cls.model)}")
             if not issubclass(cls.model, models.Model):
                 if cls.model._lino_default_table is None:
                     cls.model._lino_default_table = cls
@@ -526,13 +528,24 @@ method in order to sort the rows of the queryset.
     @classmethod
     def get_default_action(cls):
         if cls.default_record_id is not None:
+            # return cls._detail_action_class(
+            #     label=cls.label, help_text=cls.help_text,
+            #     default_record_id = cls.default_record_id)
             # assert cls.display_mode == ((None, constants.DISPLAY_MODE_DETAIL))
             assert cls.detail_action is not None
-            cls.detail_action.action.label = cls.label
-            cls.detail_action.action.help_text = cls.help_text
-            cls.detail_action.action.default_record_id = cls.default_record_id
-            return cls.detail_action
-        return actions.ShowTable()
+            # if cls.detail_action.action.defining_actor is not cls:
+            #     raise Exception(
+            #         f"{cls.detail_action.action.defining_actor} is not {cls}")
+            a = cls.detail_action.action
+            return a.__class__(
+                a.owner,
+                label=cls.label, help_text=cls.help_text,
+                default_record_id=cls.default_record_id)
+            # cls.detail_action.action.label = cls.label
+            # cls.detail_action.help_text = cls.help_text
+            # cls.detail_action.action.default_record_id = cls.default_record_id
+            # return cls.detail_action
+        return actions.SHOW_TABLE
 
     @classmethod
     def get_actor_editable(self):
@@ -611,7 +624,7 @@ method in order to sort the rows of the queryset.
                 if not isinstance(master_instance, self.master):
                     # e.g. a ByUser table descendant called by AnonymousUser
                     msg = "20240731 %r is not a %s (%s.master_key = '%s')" % (
-                        master_instance.__class__,
+                        master_instance,
                         self.master,
                         self,
                         self.master_key,

lino/core/utils.py

@@ -13,6 +13,7 @@ import datetime
 # import yaml
 from importlib import import_module
 from django.utils.html import format_html, mark_safe, SafeString
+from django.utils.functional import Promise
 from django.db import models
 from django.db.models import Q
 from django.core.exceptions import FieldDoesNotExist
@@ -90,7 +91,7 @@ def getrqdata(request):
     """Return the request data.
 
     Unlike the now defunct `REQUEST
-    <https://docs.djangoproject.com/en/5.0/ref/request-response/#django.http.HttpRequest.REQUEST>`_
+    <https://docs.djangoproject.com/en/5.2/ref/request-response/#django.http.HttpRequest.REQUEST>`_
     attribute, this inspects the request's `method` in order to decide
     what to return.
 
@@ -1163,6 +1164,8 @@ class Panel:
 
 def resolve_layout(cls, k, spec, layout_class, **options):
     # k: just for naming the culprit in error messages
+    if isinstance(spec, Promise):
+        spec = str(spec)
     if isinstance(spec, str):
         if "\n" in spec or "." not in spec:
             return layout_class(spec, cls, **options)

lino/core/workflows.py

@@ -255,7 +255,8 @@ class ChangeStateAction(actions.Action):
             self.button_text = target_state.button_text
 
         if self.icon_name:
-            self.help_text = format_lazy("{}. {}", self.label, self.help_text)
+            self.help_text = format_lazy(
+                "{}. {}", self.label, self.help_text)
 
     # def get_action_permission(self, ar, obj, state):
     #     if not super(ChangeStateAction, self).get_action_permission(ar, obj, state):

lino/help_texts.py

@@ -110,7 +110,6 @@ help_texts = {
     'lino.modlib.bootstrap3.views.Element' : _("""Render a single record."""),
     'lino.modlib.bootstrap3.views.Index' : _("""Render the main page."""),
     'lino.modlib.checkdata.Plugin' : _("""The config descriptor for this plugin."""),
-    'lino.modlib.checkdata.Plugin.responsible_user' : _("""The username of the main checkdata responsible, i.e. a designated user who will be attributed to checkdata messages for which no specific responible could be designated (returned by the checker’s get_responsible_user method)."""),
     'lino.modlib.checkdata.Plugin.on_plugins_loaded' : _("""Set responsible_user to "'robin' if this is a demo site (is_demo_site)."""),
     'lino.modlib.checkdata.roles.CheckdataUser' : _("""Can see checkdata messages."""),
     'lino.modlib.comments.Plugin' : _("""See /dev/plugins."""),
@@ -560,7 +559,7 @@ help_texts = {
     'lino.modlib.jinja.XMLMaker.xml_validator_file' : _("""The name of a “validator” to use for validating the XML content."""),
     'lino.modlib.jinja.XMLMaker.get_xml_file' : _("""Get the name of the XML file to be generated for this database row."""),
     'lino.modlib.jinja.XMLMaker.make_xml_file' : _("""Make the XML file for this database row."""),
-    'lino.modlib.memo.Previewable' : _("""Adds three rich text fields (lino.core.fields.RichTextField):"""),
+    'lino.modlib.memo.Previewable' : _("""See dg.memo.Previewable."""),
     '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.memo.Previewable.body_full_preview' : _("""A read-only full preview of body."""),

lino/management/commands/prep.py

@@ -5,7 +5,7 @@
 from django.core.management import call_command
 from django.core.management.base import BaseCommand, CommandError
 from django.conf import settings
-from lino.utils import dbhash
+from lino.core.signals import database_prepared
 
 
 class Command(BaseCommand):
@@ -62,4 +62,4 @@ class Command(BaseCommand):
             kwargs["removemedia"] = True
         call_command("initdb", *args, **kwargs)
 
-        dbhash.mark_virgin()
+        database_prepared.send(self)

lino/management/commands/show.py

@@ -2,25 +2,23 @@
 # Copyright 2013-2021 Rumma & Ko Ltd
 # License: GNU Affero General Public License v3 (see file COPYING for details)
 
-import argparse
 from lino.api import rt
 from django.core.management.base import BaseCommand, CommandError
 
+
 class Command(BaseCommand):
     help = "Show the content of a specified table to standard output."
 
     # args = "action_spec [options] [args ...]"
 
     def add_arguments(self, parser):
-        (
-            parser.add_argument(
-                "-u",
-                "--username",
-                action="store",
-                dest="username",
-                default=None,
-                help="The username to act as. Default is `None`.",
-            ),
+        parser.add_argument(
+            "-u",
+            "--username",
+            action="store",
+            dest="username",
+            default=None,
+            help="The username to act as. Default is `None`.",
         )
         parser.add_argument(
             "-l",

lino/mixins/__init__.py

@@ -61,7 +61,7 @@ class Contactable(model.Model):
     email.
     """
 
-    class Meta(object):
+    class Meta:
         abstract = True
 
     email = models.EmailField(_("e-mail address"), blank=True)
@@ -78,7 +78,7 @@ class Phonable(model.Model):
     phone.
     """
 
-    class Meta(object):
+    class Meta:
         abstract = True
 
     url = models.URLField(_("URL"), blank=True)
@@ -105,7 +105,7 @@ class Modified(model.Model):
     you explicitly call :meth:`touch`.
     """
 
-    class Meta(object):
+    class Meta:
         abstract = True
 
     modified = models.DateTimeField(_("Modified"), editable=False, null=True)
@@ -132,7 +132,7 @@ class Created(model.Model):
         because their deserialization would be problematic.
     """
 
-    class Meta(object):
+    class Meta:
         abstract = True
 
     created = models.DateTimeField(_("Created"), editable=False)
@@ -153,7 +153,7 @@ class CreatedModified(Created, Modified):
 
     """
 
-    class Meta(object):
+    class Meta:
         abstract = True
 
 
@@ -175,7 +175,7 @@ class ProjectRelated(model.Model):
         <lino.core.fields.DummyField>`.
     """
 
-    class Meta(object):
+    class Meta:
         abstract = True
 
     project = fields.ForeignKey(
@@ -186,11 +186,12 @@ class ProjectRelated(model.Model):
     )
 
     def get_related_project(self):
-        if settings.SITE.project_model:
-            return self.project
+        # if settings.SITE.project_model:
+        # When project_model is None, project is a dummy field which always returns None
+        return self.project
 
     # def on_create(self, ar):
-    #     super(ProjectRelated, self).on_create(ar)
+    #     super().on_create(ar)
     #     print(20200327, ar.actor.master_key, ar.master_instance)
     #     if ar.actor.master_key and ar.actor.master_key == "project":
     #         self.project = ar.master_instance
@@ -213,24 +214,24 @@ class ProjectRelated(model.Model):
         """
         if isinstance(controllable, ProjectRelated):
             controllable.project = self.project
-        super(ProjectRelated, self).update_owned_instance(controllable)
+        super().update_owned_instance(controllable)
 
     def get_mailable_recipients(self):
         if isinstance(self.project, settings.SITE.models.contacts.Partner):
             if self.project.email:
                 yield ("to", self.project)
-        for r in super(ProjectRelated, self).get_mailable_recipients():
+        for r in super().get_mailable_recipients():
             yield r
 
     def get_postable_recipients(self):
         if isinstance(self.project, settings.SITE.models.contacts.Partner):
             yield self.project
-        for p in super(ProjectRelated, self).get_postable_recipients():
+        for p in super().get_postable_recipients():
             yield p
 
     @classmethod
     def get_simple_parameters(cls):
-        for p in super(ProjectRelated, cls).get_simple_parameters():
+        for p in super().get_simple_parameters():
             yield p
         # if settings.SITE.project_model:
         yield "project"

lino/mixins/periods.py

@@ -18,6 +18,7 @@ from django.utils.translation import gettext_lazy as _
 from django.utils.translation import pgettext_lazy as pgettext
 from django.core.exceptions import ValidationError
 from django.utils.timezone import is_aware
+from asgiref.sync import sync_to_async
 
 from lino.utils import last_day_of_month
 from lino.api import dd
@@ -276,6 +277,7 @@ class DateRange(DateRangeObservable):
             return False
         return True
 
+
 DateRange.set_widget_options("start_date", width=10)
 DateRange.set_widget_options("end_date", width=10)
 

lino/mixins/sequenced.py

@@ -325,7 +325,7 @@ class Sequenced(Duplicable):
             message=_("Renumbered {} of {} siblings.").format(n, qs.count()))
         ar.set_response(refresh_all=True)
 
-    @fields.displayfield("⇵")
+    @fields.displayfield("⇵", wildcard_data_elem=True)
     def dndreorder(self, ar):
         """A place holder column for drag and drop row reorder on :term:`React front end`
 

lino/modlib/about/models.py

@@ -8,7 +8,7 @@ import datetime
 # from django.contrib.humanize.templatetags.humanize import naturaltime
 from django.utils.translation import gettext_lazy as _
 from django.utils.translation import gettext
-from django.utils.html import mark_safe
+from django.utils.html import mark_safe, format_html
 from django.conf import settings
 
 from lino.utils.report import EmptyTable
@@ -64,7 +64,7 @@ class About(EmptyTable):
         body += "".join([tostring(e) for e in site.welcome_html()])
 
         for p in site.sorted_plugins:
-            for i in p.get_site_info():
+            for i in p.get_site_info(ar):
                 body += i
 
         if site.languages:
@@ -142,7 +142,8 @@ class About(EmptyTable):
                 )
             )
         )
-        body = "<div>{}</div>".format(body)
+        body = mark_safe(body)
+        body = format_html("<div>{}</div>", body)
         # return js_code(rt.html_text(body))
         # return rt.html_text(body)
         # return mark_safe(body)

lino/modlib/checkdata/__init__.py

@@ -14,6 +14,7 @@ See :doc:`/plugins/checkdata`.
 """
 
 from lino.api import ad, _
+from lino.core.exceptions import ChangedAPI
 
 
 class Plugin(ad.Plugin):
@@ -21,42 +22,44 @@ class Plugin(ad.Plugin):
 
     verbose_name = _("Checkdata")
     needs_plugins = ["lino.modlib.users", "lino.modlib.gfks",
-        "lino.modlib.office", "lino.modlib.linod"]
+                     "lino.modlib.office", "lino.modlib.linod"]
+
+    fix_in_background = True
 
     # plugin settings
-    responsible_user = None  # the username (a string)
-    """
-
-    The :attr:`username <lino.modlib.users.User.username>`
-    of the **main checkdata responsible**, i.e. a designated
-    user who will be attributed to checkdata messages for which
-    no *specific responible* could be designated (returned by the
-    checker's :meth:`get_responsible_user
-    <lino.modlib.checkdata.Checker.get_responsible_user>`
-    method).
-
-    The default value for this is `None`, except on a demo site
-    (i.e. which has :attr:`is_demo_site
-    <lino.core.site.Site.is_demo_site>` set to `True`) where it is
-    ``"'robin'``.
-
-    """
-
-    _responsible_user = None  # the cached User object
-
-    def get_responsible_user(self, checker, obj):
-        if self.responsible_user is None:
-            return None
-        if self._responsible_user is None:
-            User = self.site.models.users.User
-            try:
-                self._responsible_user = User.objects.get(
-                    username=self.responsible_user)
-            except User.DoesNotExist:
-                msg = "Invalid username '{0}' in `responsible_user` "
-                msg = msg.format(self.responsible_user)
-                raise Exception(msg)
-        return self._responsible_user
+    # responsible_user = None  # the username (a string)
+    # """
+    #
+    # The :attr:`username <lino.modlib.users.User.username>`
+    # of the **main checkdata responsible**, i.e. a designated
+    # user who will be attributed to checkdata messages for which
+    # no *specific responible* could be designated (returned by the
+    # checker's :meth:`get_responsible_user
+    # <lino.modlib.checkdata.Checker.get_responsible_user>`
+    # method).
+    #
+    # The default value for this is `None`, except on a demo site
+    # (i.e. which has :attr:`is_demo_site
+    # <lino.core.site.Site.is_demo_site>` set to `True`) where it is
+    # ``"'robin'``.
+    #
+    # """
+
+    # _responsible_user = None  # the cached User object
+    #
+    # def get_responsible_user(self, checker, obj):
+    #     if self.responsible_user is None:
+    #         return None
+    #     if self._responsible_user is None:
+    #         User = self.site.models.users.User
+    #         try:
+    #             self._responsible_user = User.objects.get(
+    #                 username=self.responsible_user)
+    #         except User.DoesNotExist:
+    #             msg = "Invalid username '{0}' in `responsible_user` "
+    #             msg = msg.format(self.responsible_user)
+    #             raise Exception(msg)
+    #     return self._responsible_user
 
     def on_plugins_loaded(self, site):
         """Set :attr:`responsible_user` to ``"'robin'`` if this is a demo site
@@ -64,8 +67,11 @@ class Plugin(ad.Plugin):
 
         """
         super().on_plugins_loaded(site)
-        if site.is_demo_site and self.responsible_user is None:
-            self.configure(responsible_user="robin")
+        if hasattr(self, 'responsible_user'):
+            raise ChangedAPI(
+                "20250703 checkdata.responsible_user is replaced by users.demo_username")
+        # if site.is_demo_site and self.responsible_user is None:
+        #     self.configure(responsible_user="robin")
 
     def post_site_startup(self, site):
         super().post_site_startup(site)

lino/modlib/checkdata/choicelists.py

@@ -49,6 +49,9 @@ class Checker(dd.Choice):
             text = self.verbose_name
         super().__init__(value, text, None, **kwargs)
 
+    def __str__(self):
+        return self.value
+
     @classmethod
     def activate(cls, **kwargs):
         if cls.self is not None:
@@ -56,6 +59,11 @@ class Checker(dd.Choice):
         cls.self = cls(**kwargs)
         Checkers.add_item_instance(cls.self)
 
+    # @classmethod
+    # def fix_problems_silently(cls, ar, obj=None):
+    #     for fixable, msg in cls.self.get_checkdata_problems(ar, obj, True):
+    #         pass
+
     @classmethod
     def update_unbound_problems(cls, ar, **kwargs):
         assert cls.self.model is None
@@ -130,7 +138,7 @@ class Checker(dd.Choice):
         return []
 
     def get_responsible_user(self, obj):
-        return dd.plugins.checkdata.get_responsible_user(self, obj)
+        return dd.plugins.users.get_demo_user(self, obj)
 
 
 class Checkers(dd.ChoiceList):

lino/modlib/checkdata/fixtures/checkdata.py

@@ -1,5 +1,5 @@
 # -*- coding: UTF-8 -*-
-# Copyright 2015 Rumma & Ko Ltd
+# Copyright 2015-2025 Rumma & Ko Ltd
 # License: GNU Affero General Public License v3 (see file COPYING for details)
 """Runs the :manage:`checkdata` management command with `--fix`
 option.
@@ -7,8 +7,10 @@ option.
 """
 
 from django.core.management import call_command
+from lino.api import dd
 
 
 def objects():
-    call_command("checkdata", fix=True)
+    if not dd.is_installed("linod"):
+        call_command("checkdata", fix=True)
     return []

lino/modlib/checkdata/management/commands/checkdata.py

@@ -1,5 +1,5 @@
 # -*- coding: UTF-8 -*-
-# Copyright 2015-2023 Rumma & Ko Ltd
+# Copyright 2015-2025 Rumma & Ko Ltd
 # License: GNU Affero General Public License v3 (see file COPYING for details)
 
 from django.core.management.base import BaseCommand, CommandError
@@ -7,7 +7,7 @@ from django.core.management.base import BaseCommand, CommandError
 from lino.modlib.checkdata.choicelists import Checkers
 from lino.modlib.checkdata.models import check_data
 
-from lino.api import rt
+from lino.api import dd, rt
 
 
 class Command(BaseCommand):
@@ -56,7 +56,7 @@ class Command(BaseCommand):
         app = options.get("checkers", args)
         if app:
             args += tuple(app)
-        ar = rt.login()
+        ar = rt.login(dd.plugins.users.demo_username)
         if options["list"]:
             ar.show(Checkers, column_names="value text")
         else:

lino/modlib/checkdata/models.py

@@ -317,6 +317,13 @@ def check_instance(ar, obj, **kwargs):
             print(msg)
 
 
+def fix_instance(ar, obj, **kwargs):
+    kwargs['fix'] = True
+    for chk in get_checkers_for(obj.__class__):
+        for fixable, msg in chk.check_instance(ar, obj, **kwargs):
+            pass
+
+
 def get_checkable_models(*args, only_auto=False):
     checkable_models = OrderedDict()
     for chk in Checkers.get_list_items():
@@ -407,8 +414,8 @@ def check_data(ar, args=[], fix=True, prune=False):
     ar.logger.info(msg, done, what, found, fixed)
 
 
-@background_task(every_unit="daily", every=1)
+@background_task(every_unit="daily", every=1, run_after='generate_calendar_entries')
 def checkdata(ar):
     """Run all data checkers."""
-    check_data(ar, fix=False)
+    check_data(ar, fix=dd.plugins.checkdata.fix_in_background)
     # rt.login().run(settings.SITE.site_config.run_checkdata)

lino/modlib/comments/models.py

@@ -3,7 +3,6 @@
 # License: GNU Affero General Public License v3 (see file COPYING for details)
 
 # from html import escape
-from .ui import *
 from lino.modlib.checkdata.choicelists import Checker
 from django.contrib.humanize.templatetags.humanize import naturaltime
 from django.db import models
@@ -14,7 +13,7 @@ from django.contrib.contenttypes.models import ContentType
 from django.conf import settings
 # from lino.utils.html import E, tostring, fromstring
 
-from lino.api import dd, rt, _
+from lino.api import dd, rt, gettext, _
 
 from lino.core.requests import BaseRequest
 from lino.mixins import CreatedModified, BabelNamed
@@ -30,6 +29,8 @@ from .mixins import Commentable, MyEmotionField
 from .roles import CommentsReader, CommentsStaff
 # from .choicelists import PublishAllComments, PublishComment
 
+from .ui import *
+
 
 class CommentType(BabelNamed):
     class Meta(object):
@@ -287,7 +288,7 @@ class Comment(
 
     def after_ui_save(self, ar, cw):
         super().after_ui_save(ar, cw)
-        if self.owner_id:
+        if self.owner is not None:
             self.owner.on_commented(self, ar, cw)
 
     def full_clean(self):

lino/modlib/extjs/ext_renderer.py

@@ -210,7 +210,7 @@ class ExtRenderer(JsCacheRenderer):
         label = str(label or ba.get_button_label())
         uri = self.js2url(self.action_call(ar, ba, status or {}))
         return self.href_button_action(
-            ba, uri, label, title or ba.action.help_text, **kw
+            ba, uri, label, title or ba.get_help_text(), **kw
         )
 
     def quick_manage_toolbar(self, ar, obj):
@@ -866,7 +866,7 @@ class ExtRenderer(JsCacheRenderer):
             # ~ text=unicode(a.label),
         )
 
-        if a.help_text:
+        if (help_text := ba.get_help_text()) is not None:
             # if a.__class__.__name__ in ('ChangePassword', 'SubmitDetail'):
             #     logger.info("20160829 a2btn() %r %r", a, str(a.help_text))
 
@@ -874,7 +874,7 @@ class ExtRenderer(JsCacheRenderer):
             # iconCls. On a button which has only text we must use
             # Lino.quicktip_renderer. But I didn't find out why this
             # doesn't seem to work.
-            kw.update(tooltip=a.help_text, tooltipType="title")
+            kw.update(tooltip=help_text, tooltipType="title")
             # if not a.icon_name:
             #     kw.update(tooltipType='title')
             #    kw.update(listen   ers=dict(render=js_code(
@@ -1496,7 +1496,7 @@ class ExtRenderer(JsCacheRenderer):
         either `data_record` or a `record_id`.
 
         Usually `data_record`, except if it is a `file upload
-        <https://docs.djangoproject.com/en/5.0/topics/http/file-uploads/>`_
+        <https://docs.djangoproject.com/en/5.2/topics/http/file-uploads/>`_
         where some mysterious decoding problems (:blogref:`20120209`)
         force us to return a `record_id` which has the same visible
         result but using an additional GET.

lino/modlib/extjs/views.py

@@ -330,6 +330,7 @@ class ApiElement(View):
         vm = test_version_mismatch(request)
         if vm and settings.SITE.kernel.editing_front_end.app_label == "react":
             return json_response(vm)
+
         # this is also used by the react front end
         rpt = requested_actor(app_label, actor)
         # if not rpt.get_view_permission(request.user.user_type):
@@ -341,12 +342,17 @@ class ApiElement(View):
         if action_name:
             ba = rpt.get_url_action(action_name)
             if ba is None:
-                raise http.Http404("%s has no action %r" % (rpt, action_name))
+                msg = f"{rpt} has no action {action_name}"
+                print(msg)
+                raise http.Http404(msg)
         else:
             ba = rpt.detail_action
             if ba is None:
-                raise http.Http404("%s has no detail_action" % rpt)
+                msg = f"{rpt} has no detail_action"
+                print(msg)
+                raise http.Http404(msg)
 
+        # print(f"20250608 {rpt} {action_name}")
         fmt = request.GET.get(constants.URL_PARAM_FORMAT,
                               ba.action.default_format)
 

lino/modlib/gfks/fields.py

@@ -75,7 +75,7 @@ if settings.SITE.is_installed("contenttypes"):
 
         def deconstruct(self):
             # needed for Django 1.7
-            # https://docs.djangoproject.com/en/5.0/howto/custom-model-fields/#custom-field-deconstruct-method
+            # https://docs.djangoproject.com/en/5.2/howto/custom-model-fields/#custom-field-deconstruct-method
 
             name, path, args, kwargs = super().deconstruct()
             args = [self.type_field]