lino 25.4.1__py3-none-any.whl → 25.4.3__py3-none-any.whl

lino/management/commands/demotest.py

@@ -68,34 +68,32 @@ class TestCase(DemoTestCase):
 
         # For this test we reduce max_blacklist_time because we are going to
         # simulate a hacker who patiently waits:
-        ipdict.max_blacklist_time = timedelta(seconds=1)
+        ipdict.max_blacklist_time = timedelta(seconds=4)
 
         self.assertEqual(ipdict.ip_records, {})
 
-        def login(pwd):
+        def login(pwd, expected):
             d = self.login("robin", pwd)
-            return d.message
+            if d.message != expected:
+                self.fail(f"Expected {expected} but got {d.message} ({d})")
 
-        self.assertEqual(login("bad"), "Failed to sign in as robin.")
+        login("bad", "Failed to sign in as robin.")
         rec = ipdict.ip_records["127.0.0.1"]
         self.assertEqual(rec.login_failures, 1)
-        self.assertEqual(login("bad"), "Failed to sign in as robin.")
+        login("bad", "Failed to sign in as robin.")
         self.assertEqual(rec.login_failures, 2)
-        self.assertEqual(login("bad"), "Failed to sign in as robin.")
+        login("bad", "Failed to sign in as robin.")
         self.assertEqual(rec.login_failures, 3)
-        self.assertEqual(login("bad"), "Failed to sign in as robin.")
+        login("bad", "Failed to sign in as robin.")
         self.assertEqual(rec.login_failures, 4)
-        self.assertEqual(
-            login("bad"), "Too many authentication failures from 127.0.0.1"
-        )
+        login("bad", "Too many authentication failures from 127.0.0.1")
 
         # login_failures doesn't continue to increase when the ip is blacklisted:
         self.assertEqual(rec.login_failures, 4)
 
         # Even with the right password you cannot unlock a blacklisted ip
-        self.assertEqual(
-            login(dd.plugins.users.demo_password), "Too many authentication failures from 127.0.0.1"
-        )
+        login(dd.plugins.users.demo_password,
+              "Too many authentication failures from 127.0.0.1")
 
         # After max_blacklist_time, the IP gets removed from the blacklist, but
         # every new failure will now blacklist it again, the
@@ -103,24 +101,20 @@ class TestCase(DemoTestCase):
 
         # time.sleep(1.5)
         time.sleep(5)
-        self.assertEqual(login("bad"), "Failed to sign in as robin.")
+        login("bad", "Failed to sign in as robin.")
         self.assertEqual(rec.login_failures, 5)
-        self.assertEqual(
-            login("bad"), "Too many authentication failures from 127.0.0.1"
-        )
+        login("bad", "Too many authentication failures from 127.0.0.1")
         self.assertEqual(rec.login_failures, 5)
 
-        time.sleep(1.5)
-        self.assertEqual(
-            login(dd.plugins.users.demo_password),
-            "Now signed in as Robin Rood")
+        time.sleep(5)
+        login(dd.plugins.users.demo_password, "Now signed in as Robin Rood")
 
         # Once you manage to authenticate, your ip address gets removed from the
         # blacklist, i.e. when you log out and in for some reason, you get again
         # max_failed_auth_per_ip attempts
 
         self.assertEqual(ipdict.ip_records, {})
-        self.assertEqual(login("bad"), "Failed to sign in as robin.")
+        login("bad", "Failed to sign in as robin.")
         rec = ipdict.ip_records["127.0.0.1"]
         self.assertEqual(rec.login_failures, 1)
 

lino/mixins/__init__.py

@@ -1,5 +1,5 @@
 # -*- coding: UTF-8 -*-
-# Copyright 2010-2021 Rumma & Ko Ltd
+# Copyright 2010-2025 Rumma & Ko Ltd
 # License: GNU Affero General Public License v3 (see file COPYING for details)
 """
 This package contains model mixins, some of which are heavily used
@@ -22,18 +22,37 @@ by applications and the :ref:`xl`. But none of them is mandatory.
 from django.db import models
 from django.conf import settings
 from django.utils.translation import gettext_lazy as _
-from django.utils.html import format_html
-from django.utils.text import format_lazy
-from django.utils import timezone
+# from django.utils.html import format_html
+# from django.utils.text import format_lazy
+# from django.utils import timezone
 from django.contrib.humanize.templatetags.humanize import naturaltime
 
+# Note that reordering the imports here can cause the field ordering to change
+# in models like lino_voga.lib.courses.TeacherType, which inherits from
+# `(Referrable, BabelNamed, Printable)`. This can cause doctests like
+# docs/specs/voga/courses.rst to fail because the `ref` field then came after
+# the name field. The TeacherTypes table has no explicit `column_names`, so it
+# uses the "natural" field ordering, which is, as this observation shows, quite
+# unpredictable.
+
 from lino.core import actions
 from lino.core import fields
 from lino.core import model
-from lino.core.workflows import ChangeStateAction
+# from lino.core.workflows import ChangeStateAction
+# from lino.core.exceptions import ChangedAPI
 from lino.utils.mldbc.fields import LanguageField
-from lino.core.exceptions import ChangedAPI
 from lino.utils.html import E
+from lino.utils.mldbc.mixins import BabelNamed, BabelDesignated
+from lino.utils.mldbc.fields import BabelCharField, BabelTextField
+
+from .human import Human
+from .polymorphic import Polymorphic
+from .periods import ObservedDateRange, Yearly, Monthly, Today
+from .periods import DateRange
+from .sequenced import Sequenced, Hierarchical
+from .duplicable import Duplicable, Duplicate
+from .registrable import Registrable, RegistrableState
+from .ref import Referrable, StructuredReferrable
 
 
 class Contactable(model.Model):
@@ -97,19 +116,19 @@ class Modified(model.Model):
         super().save(*args, **kwargs)
 
     def touch(self):
-        self.modified = timezone.now()
+        self.modified = settings.SITE.now()
 
 
 class Created(model.Model):
     """
-    Adds a timestamp field which holds the creation time of every
-    individual database object.
+    Adds a timestamp field that holds the creation time of every
+    individual :term:`database row`.
 
     .. attribute:: created
 
-        The time when this object was created.
+        The time when this :term:`database row` was created.
 
-        Does nut use Django's `auto_now` and `auto_now_add` features
+        Does not use Django's `auto_now` and `auto_now_add` features
         because their deserialization would be problematic.
     """
 
@@ -123,8 +142,8 @@ class Created(model.Model):
         return naturaltime(self.created)
 
     def save(self, *args, **kwargs):
-        if self.created is None and not settings.SITE.loading_from_dump:
-            self.created = timezone.now()
+        if self.created is None:  # and not settings.SITE.loading_from_dump:
+            self.created = settings.SITE.now()
         super().save(*args, **kwargs)
 
 
@@ -264,25 +283,3 @@ class Draggable(model.Model):
 
     def on_dropped(self, ar, **kwargs):
         pass
-
-
-from .ref import Referrable, StructuredReferrable
-from .registrable import Registrable, RegistrableState
-from lino.mixins.duplicable import Duplicable, Duplicate
-from lino.mixins.sequenced import Sequenced, Hierarchical
-from lino.mixins.periods import DateRange
-from lino.mixins.periods import ObservedDateRange, Yearly, Monthly, Today
-from lino.mixins.polymorphic import Polymorphic
-
-# Observation: moving the following two lines to the top (to be together with
-# the import of LanguageField) caused the field ordering to change in models
-# like `lino_voga.lib.courses.TeacherType` which inherits from `(Referrable,
-# BabelNamed, Printable)`. Which caused docs/specs/voga/courses.rst to fail
-# because the `ref` field then came after the name field. The TeacherTypes table
-# has no explicit `column_names`, so it uses the "natural" field ordering, which
-# is, as this observation shows, quite unpredictable.
-
-from lino.utils.mldbc.fields import BabelCharField, BabelTextField
-from lino.utils.mldbc.mixins import BabelNamed, BabelDesignated
-
-from lino.mixins.human import Human

lino/mixins/dupable.py

@@ -39,6 +39,7 @@ from lino.core.actions import SubmitInsert
 from lino.utils import join_elems
 from lino.utils.html import E, tostring, mark_safe
 from lino.core import constants
+from lino.modlib.checkdata.choicelists import Checker
 
 
 class CheckedSubmitInsert(SubmitInsert):
@@ -242,9 +243,6 @@ class Dupable(dd.Model):
         return qs[:limit]
 
 
-from lino.modlib.checkdata.choicelists import Checker
-
-
 class DupableChecker(Checker):
     """Checks for the following repairable problem:
 
@@ -255,7 +253,7 @@ class DupableChecker(Checker):
     verbose_name = _("Check for missing phonetic words")
     model = Dupable
 
-    def get_checkdata_problems(self, obj, fix=False):
+    def get_checkdata_problems(self, ar, obj, fix=False):
         msg = obj.update_dupable_words(fix)
         if msg:
             yield (True, msg)

lino/mixins/registrable.py

@@ -119,8 +119,11 @@ class Registrable(model.Model):
 
     def disabled_fields(self, ar):
         if not self.state.is_editable:
-            return self._registrable_fields
-        return super(Registrable, self).disabled_fields(ar)
+            # return self._registrable_fields
+            # Copy _registrable_fields otherwise _registrable_fields get
+            # modified as more disabled fields are added to the set.
+            return self._registrable_fields.copy()
+        return super().disabled_fields(ar)
 
     def get_row_permission(self, ar, state, ba):
         """Only rows in an editable state may be edited.

lino/modlib/about/models.py

@@ -119,8 +119,8 @@ class About(EmptyTable):
         packages = set(["django"])
 
         items.append(
-            E.li(gettext("Server timestamp"), " : ", E.b(dtfmt(site.lino_version)))
-        )
+            E.li(gettext("Server timestamp"), " : ",
+                 E.b(dtfmt(site.kernel.lino_version))))
 
         for p in site.installed_plugins:
             packages.add(p.app_name.split(".")[0])

lino/modlib/changes/models.py

@@ -159,7 +159,7 @@ class ChangesByMaster(Changes):
 def log_change(type, request, master, obj, msg="", changed_fields=""):
     Change(
         type=type,
-        time=timezone.now(),
+        time=dd.now(),
         master=master,
         user=request.user,
         object=obj,
@@ -177,7 +177,7 @@ if remove_after:
     def delete_older_changes(ar):
         days = datetime.timedelta(days=remove_after)
         # django.core.exceptions.FieldError: Cannot resolve keyword 'time_lt' into field. Choices are: changed_fields, diff, id, list_item, master, master_id, master_type, master_type_id, name_column, navigation_panel, object, object_id, object_type, object_type_id, overview, time, type, user, user_id, workflow_buttons
-        qs = Change.objects.filter(time__lt=timezone.now() - days)
+        qs = Change.objects.filter(time__lt=dd.now() - days)
         if qs.count() > 0:
             ar.logger.info(
                 "Removing %d changes older than %d days.", qs.count(), remove_after

lino/modlib/checkdata/choicelists.py

@@ -64,8 +64,8 @@ class Checker(dd.Choice):
         ar.logger.info(msg.format(len(todo), len(done), cls.self))
 
     @classmethod
-    def check_instance(cls, *args, **kwargs):
-        return cls.self.get_checkdata_problems(*args, **kwargs)
+    def check_instance(cls, ar, *args, **kwargs):
+        return cls.self.get_checkdata_problems(ar, *args, **kwargs)
 
     def get_checkable_models(self):
         if self.model is None:
@@ -92,7 +92,7 @@ class Checker(dd.Choice):
 
         done = []
         todo = []
-        for fixable, msg in self.get_checkdata_problems(obj, fix):
+        for fixable, msg in self.get_checkdata_problems(ar, obj, fix):
             if fixable:
                 # attn: do not yet translate
                 # msg = string_concat(u"(\u2605) ", msg)
@@ -126,7 +126,7 @@ class Checker(dd.Choice):
                     prb.save()
         return (todo, done)
 
-    def get_checkdata_problems(self, obj, fix=False):
+    def get_checkdata_problems(self, ar, obj, fix=False):
         return []
 
     def get_responsible_user(self, obj):

lino/modlib/checkdata/models.py

@@ -301,9 +301,9 @@ def get_checkers_for(model):
     return checkers
 
 
-def check_instance(obj, **kwargs):
+def check_instance(ar, obj, **kwargs):
     for chk in get_checkers_for(obj.__class__):
-        for fixable, msg in chk.check_instance(obj, **kwargs):
+        for fixable, msg in chk.check_instance(ar, obj, **kwargs):
             if fixable:
                 msg = f"(\u2605) {msg}"
             print(msg)

lino/modlib/comments/fixtures/demo2.py

@@ -42,12 +42,16 @@ BODIES.items.insert(0, "")
 
 
 def objects():
+
     Comment = rt.models.comments.Comment
     User = rt.models.users.User
     Comment.auto_touch = False
     # use_linod = settings.SITE.use_linod
     # settings.SITE.use_linod = False
 
+    # avoid channels.exceptions.ChannelFull:
+    settings.SITE.loading_from_dump = True
+
     MENTIONED = Cycler()
     for model in rt.models_by_base(Commentable):
         if model.memo_command is not None:

lino/modlib/comments/models.py

@@ -429,7 +429,7 @@ class CommentChecker(Checker):
     model = Comment
     msg_missing = _("Missing owner in reply to comment.")
 
-    def get_checkdata_problems(self, obj, fix=False):
+    def get_checkdata_problems(self, ar, obj, fix=False):
         if obj.reply_to_id and not obj.owner_id and obj.reply_to.owner_id:
             yield (True, self.msg_missing)
             if fix:

lino/modlib/dupable/mixins.py

@@ -13,6 +13,7 @@ from lino.api import dd, rt, _
 from lino.core.actions import SubmitInsert
 from lino.core.gfks import gfk2lookup
 from lino.core.gfks import ContentType
+from lino.modlib.checkdata.choicelists import Checker
 
 
 class CheckedSubmitInsert(SubmitInsert):
@@ -154,9 +155,6 @@ class Dupable(dd.Model):
         return qs[:limit]
 
 
-from lino.modlib.checkdata.choicelists import Checker
-
-
 class DupableChecker(Checker):
     """Checks for the following repairable problem:
 
@@ -167,7 +165,7 @@ class DupableChecker(Checker):
     verbose_name = _("Check for missing phonetic words")
     model = Dupable
 
-    def get_checkdata_problems(self, obj, fix=False):
+    def get_checkdata_problems(self, ar, obj, fix=False):
         msg = obj.update_dupable_words(fix)
         if msg:
             yield (True, msg)
@@ -180,7 +178,7 @@ class SimilarObjectsChecker(Checker):
     model = Dupable
     verbose_name = _("Check for similar objects")
 
-    def get_checkdata_problems(self, obj, fix=False):
+    def get_checkdata_problems(self, ar, obj, fix=False):
         lst = list(obj.find_similar_instances(1))
         if len(lst):
             msg = _("Similar clients: {clients}").format(

lino/modlib/export_excel/__init__.py

@@ -1,12 +1,7 @@
-# Copyright 2014-2020 Rumma & Ko Ltd
+# Copyright 2014-2025 Rumma & Ko Ltd
 # License: GNU Affero General Public License v3 (see file COPYING for details)
-"""This plugin installs a button to export any table to excel xls format.
-
-To use it, simply add the following line to your
-:meth:`lino.core.site.Site.get_installed_plugins`::
-
-    yield 'lino.modlib.export_excel'
-
+"""
+See :doc:`/plugins/export_excel`.
 """
 
 from lino import ad, _
@@ -16,3 +11,6 @@ class Plugin(ad.Plugin):
     "See :doc:`/dev/plugins`."
 
     verbose_name = _("Export to Excel xls format")
+
+    def get_requirements(self, site):
+        yield "openpyxl"

lino/modlib/export_excel/models.py

@@ -1,9 +1,7 @@
 # -*- coding: UTF-8 -*-
-# Copyright 2014-2024 Rumma & Ko Ltd
+# Copyright 2014-2025 Rumma & Ko Ltd
 # License: GNU Affero General Public License v3 (see file COPYING for details)
-"""Database models for `lino.modlib.export_excel`.
 
-"""
 import os
 
 from django.conf import settings

lino/modlib/extjs/ext_renderer.py

@@ -1481,7 +1481,7 @@ class ExtRenderer(JsCacheRenderer):
             # if settings.SITE.never_build_site_cache:
             #     yield "GEN_TIMESTAMP = '%s';" % settings.SITE.kernel.lino_version
             # else:
-            yield "GEN_TIMESTAMP = %s;" % py2js(settings.SITE.lino_version)
+            yield "GEN_TIMESTAMP = %s;" % py2js(settings.SITE.kernel.lino_version)
 
         return "\n".join(fn())
 

lino/modlib/extjs/views.py

@@ -138,7 +138,7 @@ def test_version_mismatch(request):
     if os.environ.get("PYCHARM_HOSTED", False):
         return {}
     lv = request.GET.get(constants.URL_PARAM_LINO_VERSION)
-    if lv is None or float(lv) == settings.SITE.lino_version:
+    if lv is None or float(lv) == settings.SITE.kernel.lino_version:
         return {}
     # print("20201217", lv, settings.SITE.kernel.code_mtime)
     cache.clear()

lino/modlib/help/fixtures/demo2.py

@@ -8,15 +8,16 @@ from lino.api import dd, rt, _
 if dd.get_plugin_setting("help", "use_contacts"):
 
     from lino.api.shell import help, contacts
+    from lino_xl.lib.contacts.models import PARTNER_NUMBERS_START_AT as PS
 
     def site_contact(type, company=None, **kwargs):
         return help.SiteContact(site_contact_type=type, company=company, **kwargs)
 
     def objects():
         yield site_contact("owner", settings.SITE.site_config.site_company)
-        yield site_contact("serveradmin", contacts.Company.objects.get(pk=106))
+        yield site_contact("serveradmin", contacts.Company.objects.get(pk=PS+6))
         yield site_contact(
             "hotline",
-            contact_person=contacts.Person.objects.get(pk=113),
+            contact_person=contacts.Person.objects.get(pk=PS+13),
             **dd.babelkw("remark", _("Mon and Fri from 11:30 to 12:00")),
         )

lino/modlib/jinja/mixins.py

@@ -8,10 +8,9 @@ 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.api import dd, _
 from lino.utils.xml import validate_xml
 from lino.utils.media import MediaFile
 
@@ -31,10 +30,21 @@ class XMLMaker(dd.Model):
     xml_file_template = None
     # xml_file_name = None
 
+    _xmlfile = None
+
+    @property
+    def xmlfile(self):
+        if self._xmlfile is None:
+            self._xmlfile = MediaFile(False, *self.get_xml_file_parts())
+        return self._xmlfile
+
     def get_xml_file_parts(self):
         yield 'xml'
         yield self.get_printable_target_stem() + ".xml"
 
+    def get_xml_file(self):
+        return self.xmlfile
+
     def make_xml_file(self, ar):
         renderer = settings.SITE.plugins.jinja.renderer
         tpl = renderer.jinja_env.get_template(self.xml_file_template)
@@ -45,9 +55,9 @@ class XMLMaker(dd.Model):
         # parts = [
         #     dd.plugins.accounting.xml_media_dir,
         #     self.xml_file_name.format(self=self)]
-        xmlfile = MediaFile(False, *self.get_xml_file_parts())
+        xmlfile = self.xmlfile
         # xmlfile = Path(settings.MEDIA_ROOT, *parts)
-        ar.logger.info("Make %s from %s ...", xmlfile.path, self)
+        ar.logger.debug("Make %s from %s ...", xmlfile.path, self)
         xmlfile.path.parent.mkdir(exist_ok=True, parents=True)
         xmlfile.path.write_text(xml)
         # xmlfile.write_text(etree.tostring(xml))
@@ -71,3 +81,9 @@ class XMLMaker(dd.Model):
         # return mark_safe(f"""<a href="{url}">{url}</a>""")
         # return (xmlfile, url)
         return xmlfile
+
+    @dd.displayfield(_("XML file"))
+    def xml_file(self, ar):
+        mf = self.xmlfile
+        href = settings.SITE.media_root / mf.url
+        return mark_safe(f"<a href=\"{href}\" target=\"blank\">{mf.path.name}</a>")

lino/modlib/linod/mixins.py

@@ -45,15 +45,15 @@ class RunNow(dd.Action):
 
     def run_from_ui(self, ar, **kwargs):
         # print("20231102 RunNow", ar.selected_rows)
+        now = dd.now()
         for obj in ar.selected_rows:
             assert issubclass(obj.__class__, Runnable)
             if True:  # dd.plugins.linod.use_channels:
                 obj.last_start_time = None
                 obj.last_end_time = None
-                obj.requested_at = timezone.now()
-                obj.message = "{} requested to run this task at {}.".format(
-                    ar.get_user(), dd.ftl(timezone.now())
-                )
+                obj.requested_at = now
+                tpl = _("{0} requested to run this task at {1}.")
+                obj.message = tpl.format(ar.get_user(), dd.fdtl(now))
                 # obj.disabled = False
                 obj.full_clean()
                 obj.save()
@@ -155,10 +155,12 @@ class Runnable(Sequenced, RecurrenceSet):
         await ar.adebug("Start %s with logging level %s", self, self.log_level)
         # ar.info("Start %s with logging level %s", astr(self), self.log_level)
         # forget about any previous run:
-        self.last_start_time = timezone.now()
+        now = await sync_to_async(dd.now)()
+        self.last_start_time = now
         self.requested_at = None
         self.last_end_time = None
-        self.message = ""
+        self.message = f"Started at {self.last_start_time} " \
+            f"with logging level {self.log_level}"
         # print("20231102 full_clean")
         await sync_to_async(self.full_clean)()
         # self.full_clean()
@@ -184,19 +186,21 @@ class Runnable(Sequenced, RecurrenceSet):
                 self.disabled = True
                 await ar.awarning("Disabled %s after exception %s", self, e)
                 # ar.warning("Disabled %s after exception %s", astr(self), e)
-        self.last_end_time = timezone.now()
+        now = await sync_to_async(dd.now)()
+        self.last_end_time = now
         self.message = "<pre>" + self.message + "</pre>"
         await sync_to_async(self.full_clean)()
         # self.full_clean()
         await self.asave()
+        await sync_to_async(dd.post_ui_save.send)(sender=self.__class__, instance=self)
 
     @dd.displayfield("Status")
     def status(self, ar=None):
         if self.is_running():
-            return _("Running since {}").format(dd.ftf(self.last_start_time))
+            return _("Running since {}").format(dd.fdtf(self.last_start_time))
         if self.requested_at is not None:
             return _("Requested to run asap (since {})").format(
-                dd.ftf(self.requested_at))
+                dd.fdtf(self.requested_at))
         if self.disabled:
             return _("Disabled")
         if self.last_start_time is None or self.last_end_time is None:
@@ -206,7 +210,7 @@ class Runnable(Sequenced, RecurrenceSet):
         next_time = self.get_next_suggested_date(self.last_end_time)
         if next_time is None:
             return _("Not scheduled")
-        return _("Scheduled to run at {}").format(dd.ftf(next_time))
+        return _("Scheduled to run at {}").format(dd.fdtf(next_time))
 
 
 async def start_task_runner(ar=None, max_count=None):
@@ -219,7 +223,7 @@ async def start_task_runner(ar=None, max_count=None):
     while True:
         await ar.adebug("Start next task runner loop.")
 
-        now = timezone.now()
+        now = await sync_to_async(dd.now)()
         next_time = now + \
             timedelta(seconds=dd.plugins.linod.background_sleep_time)
 
@@ -282,7 +286,8 @@ async def start_task_runner(ar=None, max_count=None):
         if max_count is not None and count >= max_count:
             await ar.ainfo("Stop after %s loops.", max_count)
             return next_time
-        if (to_sleep := (next_time - timezone.now()).total_seconds()) <= 0:
+        now = await sync_to_async(dd.now)()
+        if (to_sleep := (next_time - now).total_seconds()) <= 0:
             continue
         await ar.adebug("Let task runner sleep for %s seconds.", to_sleep)
         await asyncio.sleep(to_sleep)

lino/modlib/linod/models.py

@@ -55,7 +55,7 @@ class SystemTaskChecker(Checker):
     verbose_name = _("Check for missing system tasks")
     model = None
 
-    def get_checkdata_problems(self, obj, fix=False):
+    def get_checkdata_problems(self, ar, obj, fix=False):
         for proc in Procedures.get_list_items():
             if proc.class_name == "linod.SystemTask":
                 if SystemTask.objects.filter(procedure=proc).count() == 0:

lino/modlib/memo/mixins.py

@@ -3,7 +3,6 @@
 # License: GNU Affero General Public License v3 (see file COPYING for details)
 
 from lxml.html import fragments_fromstring
-from lino.utils.html import E, tostring, mark_safe
 import lxml
 
 try:
@@ -19,6 +18,7 @@ from django.utils.html import format_html
 from lino.core.gfks import gfk2lookup
 from lino.core.model import Model
 from lino.core.fields import fields_list, RichTextField, PreviewTextField
+from lino.utils.html import E, tostring, mark_safe
 from lino.utils.restify import restify
 from lino.utils.soup import truncate_comment
 from lino.utils.mldbc.fields import BabelTextField
@@ -33,6 +33,7 @@ def django_truncate_comment(html_str):
         settings.SITE.plugins.memo.short_preview_length, html=True
     )
 
+
 MARKDOWNCFG = dict(
     extensions=["toc"], extension_configs=dict(toc=dict(toc_depth=3, permalink=True))
 )
@@ -293,7 +294,7 @@ class PreviewableChecker(Checker):
             obj.save()
         # self.synchronize_mentions(mentions)
 
-    def get_checkdata_problems(self, obj, fix=False):
+    def get_checkdata_problems(self, ar, obj, fix=False):
         for x in self._get_checkdata_problems(settings.SITE.DEFAULT_LANGUAGE, obj, fix):
             yield x
         if isinstance(obj, BabelPreviewable):