lino 25.4.5__py3-none-any.whl → 25.5.1__py3-none-any.whl

lino/core/tables.py

@@ -323,6 +323,12 @@ class AbstractTable(actors.Actor):
     See :ref:`dg.table.default_display_modes`.
     """
 
+    parent_layout = None
+
+    @classmethod
+    def attach_to_parent_layout(cls, parent):
+        cls.parent_layout = parent
+
     @classmethod
     # def get_display_mode(cls, available_width=None):
     def get_display_mode(cls):
@@ -532,7 +538,7 @@ method in order to sort the rows of the queryset.
     @classmethod
     def get_column_names(self, ar):
         """Dynamic tables can subclass this method and return a value for
-        :attr:`column_names` which depends on the request.
+        :attr:`column_names` that depends on the request.
 
         """
         # if settings.SITE.mobile_view:
@@ -560,16 +566,14 @@ method in order to sort the rows of the queryset.
         For example, if you have two models :class:`Book` and :class:`Author`,
         and a foreign key :attr:`Book.author`, which points to the author of the
         book, and a table `BooksByAuthor` having `master_key` set to
-        ``'author'``, then `get_filter_kw` would return a dict `{'author':
-        <PK>}` where `<PK>` is the primary key of the action request's
-        :attr:`master_instance
+        ``'author'``, then :meth:`BooksByAuthor.get_filter_kw` would return a
+        dict `{'author': <PK>}` where `<PK>` is the primary key of the action
+        request's :attr:`master_instance
         <lino.core.requests.BaseRequest.master_instance>`.
 
-        Another example is
-        :class:`lino_xl.lib.tickets.EntriesBySession`, where blog
-        entries are not directly linked to a session, but in the
-        detail of a session we want to display a table of related blog
-        entries.
+        Another example is :class:`lino_xl.lib.tickets.EntriesBySession`, where
+        blog entries are not directly linked to a session, but in the detail of
+        a session we want to display a table of related blog entries.
 
         :class:`lino_xl.lib.households.SiblingsByPerson` Household
         members are not directly linked to a Person, but usually a

lino/core/utils.py

@@ -6,19 +6,19 @@ A collection of utilities which require Django settings to be
 importable.
 """
 
-from .exceptions import ChangedAPI
 from lino.utils import IncompleteDate
 import copy
 import sys
 import datetime
 # import yaml
-
+from importlib import import_module
 from django.utils.html import format_html, mark_safe, SafeString
 from django.db import models
 from django.db.models import Q
 from django.core.exceptions import FieldDoesNotExist
 # from django.utils.functional import lazy
-from importlib import import_module
+from django.core.validators import validate_email, ValidationError, URLValidator
+from django.apps import apps
 from django.utils.translation import gettext as _
 from django.conf import settings
 from django.core import exceptions
@@ -28,9 +28,7 @@ from lino.utils.html import E, assert_safe, tostring
 from lino.utils import capture_output
 from lino.utils.ranges import isrange
 
-from django.core.validators import validate_email, ValidationError, URLValidator
-
-from django.apps import apps
+from .exceptions import ChangedAPI
 
 get_models = apps.get_models
 
@@ -642,23 +640,6 @@ def navinfo(qs, elem, limit=None):
     )
 
 
-# class Handle(object):
-#     """Base class for :class:`lino.core.tables.TableHandle`,
-#     :class:`lino.core.frames.FrameHandle` etc.
-
-#     The "handle" of an actor is responsible for expanding layouts into
-#     sets of (renderer-specific) widgets (called "elements"). This
-#     operation is done once per actor per renderer.
-
-#     """
-#     # def __init__(self):
-#     #     self.ui = settings.SITE.kernel.default_ui
-
-#     def setup(self, ar):
-#         settings.SITE.kernel.setup_handle(self, ar)
-#         # self.ui.setup_handle(self, ar)
-
-
 class Parametrizable(object):
     """
     Base class for both Actors and Actions.  See :doc:`/dev/parameters`.

lino/help_texts.py

@@ -32,7 +32,7 @@ help_texts = {
     'lino.mixins.dupable.SimilarObjects' : _("""Shows the other objects that are similar to this one."""),
     'lino.mixins.duplicable.Duplicate' : _("""Duplicate the selected row."""),
     'lino.mixins.duplicable.Duplicate.run_from_ui' : _("""This actually runs the action."""),
-    'lino.mixins.duplicable.Duplicable' : _("""Adds a row action “Duplicate” which duplicates (creates a clone of) the object it was called on."""),
+    'lino.mixins.duplicable.Duplicable' : _("""Adds a row action “Duplicate”, which duplicates (creates a clone of) the object it was called on."""),
     'lino.mixins.human.Human' : _("""Base class for models that represent a human."""),
     'lino.mixins.human.Human.title' : _("""Used to specify a professional position or academic qualification like “Dr.” or “PhD”."""),
     'lino.mixins.human.Human.first_name' : _("""The first name, also known as given name."""),
@@ -70,6 +70,7 @@ help_texts = {
     'lino.mixins.ref.Referrable.ref' : _("""The reference. This must be either empty or unique."""),
     'lino.mixins.ref.Referrable.ref_max_length' : _("""The preferred width of the ref field."""),
     'lino.mixins.ref.Referrable.on_duplicate' : _("""Before saving a duplicated object for the first time, we must change the ref in order to avoid an IntegrityError."""),
+    'lino.mixins.ref.Referrable.get_next_row' : _("""Return the next database row, or None if this is the last one."""),
     'lino.mixins.ref.Referrable.get_by_ref' : _("""Return the object identified by the given reference."""),
     'lino.mixins.ref.Referrable.quick_search_filter' : _("""Overrides the default behaviour defined in lino.core.model.Model.quick_search_filter(). For Referrable objects, when quick-searching for a text containing only digits, the user usually means the ref and not the primary key."""),
     'lino.mixins.ref.StructuredReferrable' : _("""A referrable whose ref field is used to define a hierarchical structure."""),
@@ -97,6 +98,7 @@ help_texts = {
     'lino.mixins.sequenced.Sequenced.get_siblings' : _("""Return a Django Queryset with all siblings of this, or None if this is a root element which cannot have any siblings."""),
     'lino.mixins.sequenced.Sequenced.set_seqno' : _("""Initialize seqno to the seqno of eldest sibling + 1."""),
     'lino.mixins.sequenced.Sequenced.seqno_changed' : _("""If the user manually assigns a seqno."""),
+    'lino.mixins.sequenced.Sequenced.dndreorder' : _("""A place holder column for drag and drop row reorder on React front end"""),
     'lino.mixins.sequenced.Hierarchical' : _("""Model mixin for things that have a “parent” and “siblings”."""),
     'lino.mixins.sequenced.Hierarchical.children_summary' : _("""A comma-separated list of the children."""),
     'lino.mixins.sequenced.Hierarchical.get_parental_line' : _("""Return an ordered list of all ancestors of this instance."""),
@@ -320,6 +322,11 @@ help_texts = {
     'lino.core.model.Model.set_widget_options' : _("""Set default values for the widget options of a given element."""),
     'lino.core.model.Model.get_overview_elems' : _("""Return a list of HTML elements to be shown in overview field."""),
     'lino.core.model.Model.merge_row' : _("""Merge this object into another object of same class."""),
+    'lino.utils.quantities.Quantity' : _("""The base class for all quantities."""),
+    'lino.utils.quantities.Quantity.limit_length' : _("""Reduce the number of decimal places so that the value fits into a field of the specified max_length, if possible. This will round the value (reducing precision) if needed."""),
+    'lino.utils.quantities.Duration' : _("""The class to represent a duration."""),
+    'lino.utils.quantities.Percentage' : _("""The class to represent a percentage."""),
+    'lino.utils.quantities.Fraction' : _("""The class to represent a fraction. (Not yet implemented)"""),
     'lino.core.model.Model.get_request_queryset' : _("""Return the Django queryset to be used by action request ar for any data table on this model."""),
     'lino.core.model.Model.before_ui_save' : _("""A hook for adding custom code to be executed each time an instance of this model gets updated via the user interface and before the changes are written to the database."""),
     'lino.core.model.Model.after_ui_save' : _("""Like before_ui_save(), but is called after the changes are written to the database."""),
@@ -709,4 +716,5 @@ help_texts = {
     'lino.core.model.Model.create_FOO_choice' : _("""For every field named “FOO” for which a chooser exists, if the model also has a method called “create_FOO_choice”, then this chooser will be a learning chooser. That is, users can enter text into the combobox, and Lino will create a new database object from it."""),
     'lino.core.model.Model.get_choices_text' : _("""Return the text to be displayed when an instance of this model is being used as a choice in a combobox of a ForeignKey field pointing to this model. request is the web request, actor is the requesting actor."""),
     'lino.core.model.Model.disable_delete' : _("""Decide whether this database object may be deleted. Return None when there is no veto against deleting this database row, otherwise a translatable message that explains to the user why they can’t delete this row."""),
+    'lino.core.model.Model.update_field' : _("""Shortcut to call lino.core.inject.update_field() for usage during lino.core.site.Site.do_site_startup() in a settings.py or similar place."""),
 }

lino/mixins/duplicable.py

@@ -1,5 +1,5 @@
 # -*- coding: UTF-8 -*-
-# Copyright 2012-2020 Rumma & Ko Ltd
+# Copyright 2012-2025 Rumma & Ko Ltd
 # License: GNU Affero General Public License v3 (see file COPYING for details)
 """Defines the model mixin :class:`Duplicable`.  "duplicable"
 [du'plikəblə] means "able to produce a duplicate
@@ -9,10 +9,11 @@
 
 from django.utils.translation import gettext_lazy as _
 
+from lino import logger
 from lino.core import actions
 from lino.core import model
 from lino.core.diff import ChangeWatcher
-from lino.core.roles import Expert
+# from lino.core.roles import Expert
 
 
 class Duplicate(actions.Action):
@@ -73,6 +74,7 @@ class Duplicate(actions.Action):
         new.full_clean()
         new.save(force_insert=True)
         cw = ChangeWatcher(new)
+        relcount = 0
 
         for fk, qs in related:
             for relobj in qs:
@@ -80,6 +82,7 @@ class Duplicate(actions.Action):
                 setattr(relobj, fk.name, new)
                 relobj.on_duplicate(ar, new)
                 relobj.save(force_insert=True)
+                relcount += 1
 
         new.after_duplicate(ar, obj)
 
@@ -87,6 +90,9 @@ class Duplicate(actions.Action):
             new.full_clean()
             new.save()
 
+        logger.info("%s has been duplicated to %s (%d related rows)",
+                    obj, new, relcount)
+
         return new
 
     def run_from_ui(self, ar, **kw):
@@ -115,7 +121,7 @@ class Duplicate(actions.Action):
 
 
 class Duplicable(model.Model):
-    """Adds a row action "Duplicate" which duplicates (creates a clone
+    """Adds a row action "Duplicate", which duplicates (creates a clone
     of) the object it was called on.
 
     Subclasses may override :meth:`lino.core.model.Model.on_duplicate`

lino/mixins/ref.py

@@ -10,6 +10,7 @@ from django.utils.translation import gettext_lazy as _
 from django.db.models.functions import Length
 
 from lino.utils.html import E
+from lino.utils import nextref
 from lino.core import model
 from lino.core.fields import displayfield
 
@@ -61,6 +62,14 @@ class Referrable(model.Model):
     def __str__(self):
         return self.ref or super().__str__()
 
+    def get_next_row(self):
+        """Return the next database row, or None if this is the last one.
+        """
+        ref = nextref(self.ref)
+        if ref is None:
+            return None
+        return self.__class__.get_by_ref(ref, None)
+
     @staticmethod
     def ref_prefix(obj, ar=None):
         return obj.as_ref_prefix(ar)

lino/mixins/registrable.py

@@ -37,7 +37,7 @@ class Registrable(model.Model):
     Base class to anything that may be "registered" and "deregistered", where
     "registered" means "this object has been taken account of".
 
-    For example, when a :term:`ledger voucher` is registered, its associated
+    For example, when a :term:`numbered voucher` is registered, its associated
     :term:`ledger movements <ledger movement>` have been generated.
     Deregistering a voucher will first delete these movements.
 
@@ -45,7 +45,7 @@ class Registrable(model.Model):
     usually limited to certain fields.
 
     :class:`lino_xl.lib.cal.Reservation` is an example of a registrable that is
-    not a ledger voucher.
+    not a numbered voucher.
 
     Subclasses must themselves define a field :attr:`state`.
 

lino/mixins/sequenced.py

@@ -279,15 +279,11 @@ class Sequenced(Duplicable):
         Initialize `seqno` to the `seqno` of eldest sibling + 1.
         """
         qs = self.get_siblings().order_by("seqno")
-        if qs is None:  # TODO: remove this as it is no longer used (?)
-            self.seqno = 0
+        if (count := qs.count()) == 0:
+            self.seqno = 1
         else:
-            n = qs.count()
-            if n == 0:
-                self.seqno = 1
-            else:
-                last = qs[n - 1]
-                self.seqno = last.seqno + 1
+            last = qs[count - 1]
+            self.seqno = last.seqno + 1
 
     def full_clean(self, *args, **kw):
         if not self.seqno:
@@ -329,6 +325,14 @@ class Sequenced(Duplicable):
             message=_("Renumbered {} of {} siblings.").format(n, qs.count()))
         ar.set_response(refresh_all=True)
 
+    @fields.displayfield("⇵")
+    def dndreorder(self, ar):
+        """A place holder column for drag and drop row reorder on :term:`React front end`
+
+        CAUTION: Do NOT rename this field, for react works on checking the name as dndreorder.
+        """
+        return None
+
     @fields.displayfield(_("Move"))
     def move_buttons(obj, ar):
         if ar is None:

lino/modlib/extjs/ext_renderer.py

@@ -96,7 +96,7 @@ class ExtRenderer(JsCacheRenderer):
         super().__init__(plugin)
         jsgen.register_converter(self.py2js_converter)
 
-        for s in "green blue red yellow".split():
+        for s in ('green', 'blue', 'red', 'yellow', 'lightgrey'):
             self.row_classes_map[s] = "x-grid3-row-%s" % s
 
         self.prepare_layouts()

lino/modlib/gfks/fields.py

@@ -5,7 +5,6 @@
 from django.db import models
 from django.conf import settings
 
-from lino.utils.choosers import chooser
 from lino.core.utils import full_model_name
 
 if settings.SITE.is_installed("contenttypes"):
@@ -31,6 +30,7 @@ if settings.SITE.is_installed("contenttypes"):
         def contribute_to_class(self, cls, name, **kwargs):
             # Automatically setup chooser and display field for ID field of
             # generic foreign key.
+            from lino.utils.choosers import chooser
 
             super().contribute_to_class(cls, name, **kwargs)
 

lino/modlib/help/models.py

@@ -6,7 +6,6 @@ from django.utils.translation import get_language
 from lino.core.actors import Actor
 from lino.api import dd, _
 from lino.modlib.memo.mixins import MemoReferrable
-from lino_xl.lib.contacts.mixins import ContactRelated
 
 use_contacts = dd.get_plugin_setting("help", "use_contacts")
 make_help_pages = dd.get_plugin_setting("help", "make_help_pages")
@@ -61,6 +60,8 @@ if make_help_pages:
 
 if use_contacts:
 
+    from lino_xl.lib.contacts.mixins import ContactRelated
+
     class SiteContactTypes(dd.ChoiceList):
         verbose_name = _("Site contact type")
         verbose_name_plural = _("Site contact types")

lino/modlib/jinja/renderer.py

@@ -71,6 +71,7 @@ class JinjaRenderer(HtmlRenderer):
         # ~ print 20130109, loaders
         self.jinja_env = jinja2.Environment(
             # ~ extensions=['jinja2.ext.i18n'],
+            extensions=['jinja2.ext.do'],
             loader=jinja2.ChoiceLoader(loaders)
         )
 

lino/modlib/linod/models.py

@@ -32,7 +32,7 @@ class SystemTasks(dd.Table):
     model = "linod.SystemTask"
     order_by = ['seqno']
     required_roles = dd.login_required(SiteStaff)
-    column_names = "seqno name log_level disabled status procedure *"
+    column_names = "seqno name log_level disabled status procedure dndreorder *"
     detail_layout = """
     seqno procedure
     name

lino/modlib/notify/fixtures/demo2.py

@@ -1,4 +1,4 @@
-# Copyright 2016-2018 Rumma & Ko Ltd
+# Copyright 2016-2025 Rumma & Ko Ltd
 # License: GNU Affero General Public License v3 (see file COPYING for details)
 """Emit a broadcast notification "The database has been initialized."
 
@@ -14,7 +14,12 @@ from django.utils.timezone import make_aware
 
 
 def objects():
-    now = datetime.datetime.combine(dd.today(), i2t(548))
+
+    # The messages are dated one day before today() because
+    # book/docs/specs/notify.rst has a code snippet that failed when pm prep had
+    # been run before 5:48am
+
+    now = datetime.datetime.combine(dd.today(-1), i2t(548))
     if settings.USE_TZ:
         now = make_aware(now)
     mt = rt.models.notify.MessageTypes.system

lino/modlib/periods/fixtures/std.py

@@ -8,16 +8,14 @@ from lino.api import dd, rt, _
 
 start_year = dd.get_plugin_setting("periods", "start_year", None)
 
+
 def objects():
     StoredYear = rt.models.periods.StoredYear
 
-    cfg = dd.plugins.periods
     site = settings.SITE
     if site.the_demo_date is not None:
         if start_year > site.the_demo_date.year:
             raise Exception("plugins.periods.start_year is after the_demo_date")
     today = site.the_demo_date or datetime.date.today()
     for y in range(start_year, today.year + 6):
-        # yield StoredYear.create_from_year(y)
         yield StoredYear.get_or_create_from_date(datetime.date(y, today.month, today.day))
-        # StoredYears.add_item(StoredYear.year2value(y), str(y))

lino/modlib/periods/models.py

@@ -19,7 +19,6 @@ NEXT_YEAR_SEP = "/"
 YEAR_PERIOD_SEP = "-"
 
 
-
 class StoredYear(DateRange, Referrable):
 
     class Meta:
@@ -30,7 +29,7 @@ class StoredYear(DateRange, Referrable):
 
     preferred_foreignkey_width = 10
 
-    state = PeriodStates.field(default='open')
+    state = PeriodStates.field(blank=True)
 
     @classmethod
     def get_simple_parameters(cls):
@@ -80,9 +79,23 @@ class StoredYear(DateRange, Referrable):
                 obj.save()
         return obj
 
+    def full_clean(self, *args, **kwargs):
+        if not self.state:
+            if self.start_date.year + 1 < dd.today().year:
+                self.state = PeriodStates.closed
+            else:
+                self.state = PeriodStates.open
+        super().full_clean(*args, **kwargs)
+
     def __str__(self):
         return self.ref
 
+    def get_next_row(self):
+        nextyear = self.start_date.replace(year=self.start_date.year+1)
+        ref = self.__class__.get_ref_for_date(nextyear)
+        return self.__class__.get_by_ref(ref, None)
+        # return self.__class__.get_or_create_from_date(nextyear)
+
 
 class StoredPeriod(DateRange, Referrable):
 
@@ -94,8 +107,9 @@ class StoredPeriod(DateRange, Referrable):
 
     preferred_foreignkey_width = 10
 
-    state = PeriodStates.field(default='open')
-    year = dd.ForeignKey('periods.StoredYear', blank=True, null=True, related_name="periods")
+    state = PeriodStates.field(blank=True)
+    year = dd.ForeignKey('periods.StoredYear', blank=True,
+                         null=True, related_name="periods")
     remark = models.CharField(_("Remark"), max_length=250, blank=True)
 
     @classmethod
@@ -107,7 +121,8 @@ class StoredPeriod(DateRange, Referrable):
     @classmethod
     def get_request_queryset(cls, ar):
         qs = super().get_request_queryset(ar)
-        if (pv := ar.param_values) is None: return qs
+        if (pv := ar.param_values) is None:
+            return qs
 
         # if pv.start_date is None or pv.end_date is None:
         #     period = None
@@ -206,6 +221,8 @@ class StoredPeriod(DateRange, Referrable):
             self.start_date = dd.today().replace(day=1)
         if not self.year_id:
             self.year = StoredYear.get_or_create_from_date(self.start_date)
+        if not self.state:
+            self.state = self.year.state
         super().full_clean(*args, **kwargs)
 
     def __str__(self):
@@ -230,11 +247,14 @@ class StoredPeriod(DateRange, Referrable):
                 return parts[1]
         return self.ref
 
+
 StoredPeriod.set_widget_options('ref', width=6)
 
+
 def date2ref(d):
     return StoredYear.get_ref_for_date(d) + YEAR_PERIOD_SEP + StoredPeriod.get_ref_for_date(d)
 
+
 class StoredYears(dd.Table):
     model = 'periods.StoredYear'
     required_roles = dd.login_required(OfficeStaff)
@@ -244,6 +264,7 @@ class StoredYears(dd.Table):
     # start_date end_date
     # """
 
+
 class StoredPeriods(dd.Table):
     required_roles = dd.login_required(OfficeStaff)
     model = 'periods.StoredPeriod'

lino/modlib/publisher/fixtures/synodalworld.py

@@ -15,8 +15,6 @@ home_children = [
 
 def objects():
     image = rt.models.uploads.Upload.objects.first()
-    print('='*80)
-    print(image)
     def iterate(iterable):
         try:
             for obj in iterable:

lino/modlib/weasyprint/__init__.py

@@ -38,6 +38,7 @@ class Plugin(ad.Plugin):
     margin = 10
     margin_left = 17
     margin_right = 10
+    space_before_recipient = 15
 
     def get_requirements(self, site):
         yield "imagesize"

lino/modlib/weasyprint/choicelists.py

@@ -2,9 +2,9 @@
 # Copyright 2016-2024 Rumma & Ko Ltd
 # License: GNU Affero General Public License v3 (see file COPYING for details)
 
-import os
 from pathlib import Path
-from copy import copy
+from lino.modlib.jinja.choicelists import JinjaBuildMethod
+from lino.modlib.printing.choicelists import BuildMethods
 
 try:
     from weasyprint import HTML
@@ -20,15 +20,6 @@ except ImportError:
     BULMA_CSS = None
 
 
-
-from django.conf import settings
-from django.utils import translation
-
-from lino.api import dd
-from lino.modlib.jinja.choicelists import JinjaBuildMethod
-from lino.modlib.printing.choicelists import BuildMethods
-
-
 class WeasyBuildMethod(JinjaBuildMethod):
     template_ext = ".weasy.html"
     templates_name = "weasy"

lino/modlib/weasyprint/config/weasyprint/base.weasy.html

@@ -77,7 +77,7 @@ div.recipient {
     size: {%- block pagesize %}landscape{%- endblock %};
     margin: {{dd.plugins.weasyprint.margin}}mm;
     margin-bottom: {{dd.plugins.weasyprint.margin+dd.plugins.weasyprint.footer_height}}mm;
-    {%- if dd.plugins.weasyprint.top_right_image -%}
+    {%- if True or dd.plugins.weasyprint.top_right_image -%}
     margin-top: {{dd.plugins.weasyprint.margin+dd.plugins.weasyprint.header_height}}mm;
     {%- endif -%}
     margin-left: {{dd.plugins.weasyprint.margin_left}}mm;

lino/utils/__init__.py

@@ -655,3 +655,23 @@ def logging_disabled(level):
         yield
     finally:
         logging.disable(logging.NOTSET)
+
+
+def nextref(ref):
+    """
+    Increment the first number found in the string, preserving everything
+    thereafter.
+
+    Tested examples in :doc:`/topics/utils`.
+    """
+    num = ""
+    suffix = ""
+    for i, c in enumerate(ref):
+        if c.isdigit():
+            num += c
+        else:
+            suffix = ref[i:]
+            break
+    if not num:
+        return None
+    return str(int(num)+1) + suffix

lino/utils/choosers.py

@@ -1,4 +1,4 @@
-# Copyright 2009-2021 Rumma & Ko Ltd
+# Copyright 2009-2025 Rumma & Ko Ltd
 # License: GNU Affero General Public License v3 (see file COPYING for details)
 """Extends the possibilities for defining choices for fields of a
 Django model.
@@ -13,19 +13,18 @@ Example values in :doc:`/topics/utils`.
 
 """
 
+import re
 import decimal
 import datetime
-from dateutil import parser as dateparser
-
-import re
 
-GFK_HACK = re.compile(
-    r'^<a href="javascript:Lino\.(\w+\.\w+)\.detail\.run\(.*,\{ &quot;record_id&quot;: (\w+) \}\)">.*</a>$'
-)
-
-from django.db import models
-from django.conf import settings
+from dateutil import parser as dateparser
 from django.core.exceptions import BadRequest
+from django.conf import settings
+from django.db import models
+
+from lino.core.utils import getrqdata
+from lino.core import fields
+from lino.core import constants
 
 try:
     from django.contrib.contenttypes.models import ContentType
@@ -33,10 +32,9 @@ except RuntimeError:
     pass
     # Happens when the site doesn't use contenttypes. Ignore silently.
 
-# from lino.api import rt
-from lino.core import constants
-from lino.core import fields
-from lino.core.utils import getrqdata
+GFK_HACK = re.compile(
+    r'^<a href="javascript:Lino\.(\w+\.\w+)\.detail\.run\(.*,\{ &quot;record_id&quot;: (\w+) \}\)">.*</a>$'
+)
 
 # class DataError(Exception):
 #     pass
@@ -257,6 +255,11 @@ class Chooser(FieldChooser):
         if isinstance(field, fields.VirtualField):
             selector = field.return_type
 
+        # TODO: move this to top and fix ImportError cannot import name
+        # 'chooser' from partially initialized module 'lino.utils.choosers'
+        # (most likely due to a circular import)
+        from lino.modlib.gfks.fields import GenericForeignKeyIdField
+
         if isinstance(selector, ChoiceListField):
             self.simple_values = getattr(meth, "simple_values", False)
             self.instance_values = getattr(meth, "instance_values", True)
@@ -265,6 +268,8 @@ class Chooser(FieldChooser):
             )
         elif is_foreignkey(selector):
             pass
+        elif isinstance(selector, GenericForeignKeyIdField):
+            pass
         # elif isinstance(field, fields.VirtualField) and isinstance(
         #     field.return_type, models.ForeignKey
         # ):
@@ -471,7 +476,7 @@ def _chooser(make, **options):
             return fn(*args)
 
         cp = options.pop(
-            "context_params", fn.__code__.co_varnames[1 : fn.__code__.co_argcount]
+            "context_params", fn.__code__.co_varnames[1: fn.__code__.co_argcount]
         )
         wrapped.context_params = cp
         for k, v in options.items():
@@ -518,6 +523,7 @@ def check_for_chooser(holder, field):
     methname = field.name + "_choices"
     m = getattr(holder, methname, None)
     if m is not None:
+        # print(f"20250511 chooser for {field.name} in {holder} is {m}")
         # if field.name.endswith('municipality'):
         #     print("20200524 check_for_chooser() found {} on {}", field.name, holder)
         # 20200425 fix theoretical bug