lino 25.5.2__py3-none-any.whl → 25.5.3__py3-none-any.whl

lino/core/layouts.py

@@ -14,8 +14,11 @@ from django.db.models.fields.related import ForeignObject
 # from django.contrib.contenttypes.fields import GenericRelation
 
 from lino.core import constants
-from lino.core.fields import fields_list, VirtualField, wildcard_data_elems
+from lino.core import atomizer
+from lino.core import fields
+from lino.core import store
 from lino.core.plugin import Plugin
+from lino.core.utils import Panel
 from lino.modlib.users.utils import get_user_profile
 
 
@@ -36,46 +39,16 @@ def DEBUG_LAYOUTS(lo):
     return False
 
 
-class DummyPanel(object):
+class DummyPanel:
     """
-    A layout panel which does not exist in the current configuration
+    A layout panel that does not exist in the current configuration
     but might exist as a real panel in some other configuration.
     """
 
     pass
 
 
-class Panel(object):
-    """
-    To be used when a panel cannot be expressed using a simple
-    template string because it requires one or more options. These
-    `options` parameters can be:
-
-    - label
-    - required_roles
-    - window_size
-    - label_align
-
-    Unlike a :class:`BaseLayout` it cannot have any child panels
-    and cannot become a tabbed panel.
-    """
-
-    def __init__(self, desc, label=None, **options):
-        # assert not 'required' in options
-        self.desc = desc
-        if label is not None:
-            options.update(label=label)
-        self.options = options
-
-    def replace(self, *args, **kw):
-        """
-        Calls the standard :meth:`string.replace`
-        method on this Panel's template.
-        """
-        self.desc = self.desc.replace(*args, **kw)
-
-
-class LayoutHandle(object):
+class LayoutHandle:
     """
     A layout handle analyzes a layout (an instance of :class:`BaseLayout` or a some
     subclass thereof) and holds the resulting metadata, especially the layout
@@ -109,8 +82,8 @@ class LayoutHandle(object):
         #     main = layout.main_m or layout.main
         # else:
         #     main = layout.main
-        main = layout.main
-        self.define_panel("main", main)
+
+        self.define_panel("main", layout.main)
 
         self.main = self._names.get("main")
         if self.main is None:
@@ -120,8 +93,6 @@ class LayoutHandle(object):
 
         self.width = self.main.width
         self.height = self.main.height
-
-        self.layout.setup_handle(self)
         for k, v in self.layout._labels.items():
             if k not in self._names:
                 raise Exception(
@@ -182,8 +153,8 @@ class LayoutHandle(object):
                     raise Exception("Invalid remote wildcard %s" % rw)
                 rwmodel = fld.remote_field.model
                 rwnames = []
-                for de in wildcard_data_elems(rwmodel):
-                    if isinstance(de, (VirtualField, ForeignObject)):
+                for de in fields.wildcard_data_elems(rwmodel):
+                    if isinstance(de, (fields.VirtualField, ForeignObject)):
                         continue
                     if self.use_as_wildcard(de):
                         k = rwprefix + de.name
@@ -200,7 +171,7 @@ class LayoutHandle(object):
                 if de.name not in explicit_specs:
                     if self.use_as_wildcard(de):
                         wildcard_names.append(de.name)
-                        if len(explicit_specs) or isinstance(de, VirtualField):
+                        if len(explicit_specs) or isinstance(de, fields.VirtualField):
                             self.hidden_elements.add(de.name)
             wildcard_str = self.layout.join_str.join(wildcard_names)
             desc = desc.replace("*", wildcard_str)
@@ -387,7 +358,7 @@ class LayoutHandle(object):
         )
 
 
-class BaseLayout(object):
+class BaseLayout:
     """
     Base class for all Layouts (:class:`FormLayout`, :class:`ColumnsLayout`
     and  :class:`ParamsLayout`).
@@ -482,7 +453,8 @@ class BaseLayout(object):
         self._datasource = ds
         if ds is not None:
             if isinstance(self.hidden_elements, str):
-                self.hidden_elements = set(fields_list(ds, self.hidden_elements))
+                self.hidden_elements = set(
+                    atomizer.fields_list(ds, self.hidden_elements))
             self.hidden_elements |= ds.hidden_elements
             # self.editable = not ds.hide_editing(user_type)
             # ~ if str(ds).endswith('Partners'):
@@ -515,9 +487,6 @@ class BaseLayout(object):
         for name in args:
             self.main = self.main.replace(name, "")
 
-    def setup_handle(self, lh):
-        pass
-
     def setup_element(self, lh, e):
         pass
 
@@ -626,7 +595,7 @@ add_tabpanel() on %s horizontal 'main' panel %r."""
         # ~ if kw:
         # ~ print 20120525, self, self.detail_layout._element_options
 
-    def get_layout_handle(self, front_end=None):
+    def get_layout_handle(self):  # , front_end=None):
         """
         Return the LayoutHandle for this layout.
         Create a LayoutHandle instance if this is the first call.
@@ -638,11 +607,11 @@ add_tabpanel() on %s horizontal 'main' panel %r."""
         changed my mind.
 
         """
-        if front_end is None:
-            front_end = settings.SITE.kernel.editing_front_end
+        # if front_end is None:
+        front_end = settings.SITE.kernel.editing_front_end
         hname = front_end.ui_handle_attr_name
         if hname is None:
-            raise Exception("{0} has no `ui_handle_attr_name`!".format(front_end))
+            raise Exception(f"{front_end} has no `ui_handle_attr_name`!")
 
         # we do not want any inherited handle
         h = self.__dict__.get(hname, None)
@@ -800,7 +769,7 @@ class ParamsLayout(BaseLayout):
 
     join_str = " "
     url_param_name = constants.URL_PARAM_PARAM_VALUES
-    params_store = None
+    _params_store = None
 
     def get_data_elem(self, name):
         de = self._datasource.get_param_elem(name)
@@ -813,12 +782,16 @@ class ParamsLayout(BaseLayout):
         #     print("20200425 ParamsLayout.get_data_elem()", name, de)
         return de
 
-    def setup_handle(self, lh):
-        # if str(self._datasource) == 'courses.Pupils':
-        #     print("20160329 setup_handle")
-        from lino.core.store import ParameterStore
-
-        self.params_store = ParameterStore(lh, self.url_param_name)
+    @property
+    def params_store(self):
+        if self._params_store is None:
+            k = settings.SITE.kernel.editing_front_end.ui_handle_attr_name
+            lh = getattr(self, k, None)
+            if lh is None:
+                # raise Exception(f"20250523 {self} has no lh")
+                return None
+            self._params_store = store.ParameterStore(lh, self.url_param_name)
+        return self._params_store
 
 
 class ActionParamsLayout(ParamsLayout):

lino/core/model.py

@@ -23,11 +23,13 @@ from lino.utils.html import E, forcetext, tostring, join_elems
 from lino.utils.soup import sanitize
 
 from lino.core import fields
+from lino.core import atomizer
 from lino.core import signals
 from lino.core import actions
 from lino.core import inject
 
-from .fields import make_remote_field, RichTextField, displayfield
+from lino.core.atomizer import make_remote_field
+from .fields import RichTextField, displayfield
 from .utils import error2str
 from .utils import obj2str
 from .diff import ChangeWatcher
@@ -246,8 +248,8 @@ class Model(models.Model, fields.TableRow):
     def add_active_field(cls, names):
         if isinstance(cls.active_fields, str):
             cls.active_fields = frozenset(
-                fields.fields_list(cls, cls.active_fields))
-        cls.active_fields = cls.active_fields | fields.fields_list(cls, names)
+                atomizer.fields_list(cls, cls.active_fields))
+        cls.active_fields = cls.active_fields | atomizer.fields_list(cls, names)
 
     @classmethod
     def hide_elements(self, *names):
@@ -491,7 +493,7 @@ class Model(models.Model, fields.TableRow):
         self.delete()
 
     def get_row_permission(self, ar, state, ba):
-        """Returns True or False whether this database object gives permission
+        """Returns True or False whether this database row gives permission
         to the ActionRequest `ar` to run the specified action.
 
         """

lino/core/permissions.py

@@ -12,6 +12,7 @@ from lino import logger
 from django.conf import settings
 
 from lino.core.utils import obj2str
+from lino.modlib.users.utils import get_user_profile
 
 from .roles import check_required_roles
 from .exceptions import ChangedAPI
@@ -242,3 +243,20 @@ def make_permission_handler_(
             return v
 
     return allow
+
+
+def get_view_permission(e):
+    if isinstance(e, Permittable) and not e.get_view_permission(get_user_profile()):
+        return False
+    # e.g. pcsw.ClientDetail has a tab "Other", visible only to system
+    # admins but the "Other" contains a GridElement RolesByPerson
+    # which is not per se reserved for system admins.  js of normal
+    # users should not try to call on_master_changed() on it
+    parent = e.parent
+    while parent is not None:
+        if isinstance(parent, Permittable) and not parent.get_view_permission(
+            get_user_profile()
+        ):
+            return False  # bug 3 (bcss_summary) blog/2012/0927
+        parent = parent.parent
+    return True

lino/core/renderer.py

@@ -300,10 +300,14 @@ class HtmlRenderer(Renderer):
             yield ar.actor.get_table_as_list(ar.master_instance, ar)
             return
 
-        if display_mode == constants.DISPLAY_MODE_STORY:
+        elif display_mode == constants.DISPLAY_MODE_STORY:
             yield ar.actor.get_table_story(ar.master_instance, ar)
             return
 
+        elif display_mode == constants.DISPLAY_MODE_TILES:
+            yield ar.actor.get_table_as_tiles(ar.master_instance, ar)
+            return
+
         yield ar.table2xhtml(**kwargs)
 
         # if show_toolbar:

lino/core/requests.py

@@ -50,7 +50,7 @@ from lino.core.utils import obj2unicode
 from lino.core.utils import obj2str
 from lino.core.utils import format_request
 from lino.core.utils import PhantomRow
-from lino.core.store import get_atomizer
+from lino.core.atomizer import get_atomizer
 
 # try:
 #     from django.contrib.contenttypes.models import ContentType
@@ -62,7 +62,7 @@ if settings.SITE.is_installed("contenttypes"):
 else:
     ContentType = None
 
-from .fields import RemoteField, FakeField, TableRow
+from lino.core.fields import FakeField, TableRow, RemoteField
 
 
 CATCHED_AJAX_EXCEPTIONS = (Warning, exceptions.ValidationError)
@@ -1668,7 +1668,7 @@ class BaseRequest:
     def get_breadcrumbs(self, elem=None):
         list_title = self.get_title()
         # TODO: make it clickable so that we can return from detail to list view
-        if elem is None:
+        if elem is None or self.actor.default_record_id is not None:
             return list_title
         else:
             # print("20190703", self.actor, self.actor.default_action)
@@ -1769,6 +1769,7 @@ class ActionRequest(BaseRequest):
     _data_iterator = None
     _sliced_data_iterator = None
     _insert_sar = None
+    _ah = None
 
     def __init__(
         self,
@@ -1782,13 +1783,17 @@ class ActionRequest(BaseRequest):
         self.rqdata = rqdata
         self.bound_action = action or actor.default_action
         super().__init__(**kw)
-        if not actor.is_abstract():
-            self.ah = actor.get_request_handle(self)
+
+    @property
+    def ah(self):
+        if self._ah is None and not self.actor.is_abstract():
+            self._ah = self.actor.get_request_handle(self)
             # if self.ah.store is None:
             #     raise Exception("20240530 {} has no store!?".format(self.ah))
+        return self._ah
 
     def parse_req(self, request, rqdata, **kw):
-        if not "selected_pks" in kw and not "selected_rows" in kw:
+        if "selected_pks" not in kw and "selected_rows" not in kw:
             selected = rqdata.getlist(constants.URL_PARAM_SELECTED)
             kw.update(selected_pks=selected)
         return super().parse_req(request, rqdata, **kw)
@@ -1897,6 +1902,7 @@ class ActionRequest(BaseRequest):
                 # msg = "20170116 selected_rows is {} for {!r}".format(
                 #     self.selected_rows, action)
                 # raise Exception(msg)
+            # print(f"20250523 {self.bound_action} {request}")
             if request is not None:
                 apv.update(
                     action.params_layout.params_store.parse_params(request))
@@ -2203,7 +2209,7 @@ class ActionRequest(BaseRequest):
         # print("20230331 712 get_status() {}".format(self.subst_user))
         if self._status is not None and not kw:
             return self._status
-        if self.actor.parameters:
+        if self.actor.parameters and self.actor.params_layout.params_store is not None:
             pv = self.actor.params_layout.params_store.pv2dict(
                 self, self.param_values)
             # print(f"20250121c {self}\n{self.actor.params_layout.params_store.__class__}\n{self.actor.params_layout.params_store.param_fields}")

lino/core/signals.py

@@ -24,5 +24,5 @@ pre_ui_save = Signal()  # ['instance', 'ar'])
 post_ui_save = Signal()  # ['instance', 'ar'])
 pre_ui_delete = Signal()  # ['request'])
 pre_ui_build = Signal()
-post_ui_build = Signal()
+# post_ui_build = Signal()
 # database_connected = Signal()

lino/core/site.py

@@ -247,7 +247,7 @@ class Site(object):
 
     legacy_data_path = None
     """
-    Used by custom fixtures that import data from some legacy
+    Deprecated. Was used by tim2lino and pp2lino to import data from some legacy
     database.
 
     """

lino/core/store.py

@@ -30,11 +30,11 @@ import datetime
 
 from django.conf import settings
 from django.db import models
-from django.db.models.fields import Field
+# from django.db.models.fields import Field
 from django.core import exceptions
 from django.utils.translation import gettext_lazy as _
 from django.utils.encoding import force_str
-from lino import AFTER17
+# from lino import AFTER17
 
 from lino.utils.jsgen import py2js
 from lino.utils.quantities import parse_decimal
@@ -42,11 +42,12 @@ from lino.core.utils import getrqdata
 from lino.core.gfks import GenericForeignKey
 from lino.core import constants
 from lino.core import fields
-from lino.core import actors
+# from lino.core import actors
 from lino.core import actions
 from lino.core import frames
+from lino.core import atomizer
 from lino.core.utils import PhantomRow
-from lino.utils import choosers
+# from lino.utils import choosers
 from lino.utils import curry
 from lino.utils import iif
 from lino.utils.format_date import fds
@@ -291,8 +292,12 @@ class ComboStoreField(StoreField):
         # v = self.full_value_from_object(None,obj)
         if v is None or v == "":
             return (None, None)
+        # print("20250518", obj, obj.__class__, ar.actor, self.field.name)
         if obj is not None:
-            ch = obj.__class__.get_chooser_for_field(self.field.name)
+            # ch = obj.__class__.get_chooser_for_field(self.field.name)
+            # if ch is not None:
+            #     return (v, ch.get_text_for_value(v, obj))
+            ch = ar.actor.get_chooser_for_field(self.field.name)
             if ch is not None:
                 return (v, ch.get_text_for_value(v, obj))
         for i in self.field.choices:
@@ -925,154 +930,6 @@ class OneToOneRelStoreField(StoreField):
             return None
 
 
-def get_atomizer(holder, fld, name):
-    """
-    Return the :term:`atomizer` for this database field.
-
-    An atomizer is an instance of a subclass of :class:`StoreField`.
-
-    """
-    sf = getattr(fld, "_lino_atomizer", None)
-    if sf is None:
-        sf = create_atomizer(holder, fld, name)
-        if sf is None:
-            # print("20210623 {} on {} has no StoreField".format(name, holder))
-            return
-        assert isinstance(sf, StoreField)
-        # if not isinstance(sf, StoreField):
-        #     raise Exception("{} is not a StoreField".format(sf))
-        if isinstance(fld, type):
-            raise Exception("20240913 trying to set class attribute")
-        setattr(fld, "_lino_atomizer", sf)
-    return sf
-
-
-def create_atomizer(holder, fld, name):
-    """
-    The holder is where the (potential) choices come from. It can be
-    a model, an actor or an action. `fld` is a data element.
-    """
-    if name is None:
-        # print("20181023 create_atomizer() no name {}".format(fld))
-        return
-        # raise Exception("20181023 create_atomizer() {}".format(fld))
-    if isinstance(fld, fields.RemoteField):
-        """
-        Hack: we create a StoreField based on the remote field,
-        then modify some of its behaviour.
-        """
-        sf = create_atomizer(holder, fld.field, fld.name)
-
-        # print("20180712 create_atomizer {} from {}".format(sf, fld.field))
-
-        def value_from_object(unused, obj, ar=None):
-            return fld.func(obj, ar)
-
-        def full_value_from_object(unused, obj, ar=None):
-            return fld.func(obj, ar)
-
-        def set_value_in_object(sf, ar, instance, v):
-            # print("20180712 {}.set_value_in_object({}, {})".format(
-            #     sf, instance, v))
-            old_value = sf.value_from_object(instance, ar.request)
-            if old_value != v:
-                return fld.setter(instance, v)
-
-        sf.value_from_object = curry(value_from_object, sf)
-        sf.full_value_from_object = curry(full_value_from_object, sf)
-        sf.set_value_in_object = curry(set_value_in_object, sf)
-        return sf
-
-    meth = getattr(fld, "_return_type_for_method", None)
-    if meth is not None:
-        # uh, this is tricky...
-        return MethodStoreField(fld, name)
-
-    # sf_class = getattr(fld, 'lino_atomizer_class', None)
-    # if sf_class is not None:
-    #     return sf_class(fld, name)
-
-    if isinstance(fld, fields.DummyField):
-        return None
-    if isinstance(fld, fields.RequestField):
-        delegate = create_atomizer(holder, fld.return_type, fld.name)
-        return RequestStoreField(fld, delegate, name)
-    if isinstance(fld, type) and issubclass(fld, actors.Actor):
-        return
-    # if isinstance(fld, type) and issubclass(fld, actors.Actor):
-    #     raise Exception("20230219")
-    #     # 20210618 & 20230218
-    #     if settings.SITE.kernel.default_ui.support_async:
-    #         return SlaveTableStoreField(fld, name)
-    #     return DisplayStoreField(fld, name)
-
-    if isinstance(fld, fields.VirtualField):
-        delegate = create_atomizer(holder, fld.return_type, fld.name)
-        if delegate is None:
-            # e.g. VirtualField with DummyField as return_type
-            return None
-            # raise Exception("No atomizer for %s %s %s" % (
-            #     holder, fld.return_type, fld.name))
-        return VirtStoreField(fld, delegate, name)
-    if isinstance(fld, models.FileField):
-        return FileFieldStoreField(fld, name)
-    if isinstance(fld, models.ManyToManyField):
-        return StoreField(fld, name)
-    if isinstance(fld, fields.PasswordField):
-        return PasswordStoreField(fld, name)
-    if isinstance(fld, models.OneToOneField):
-        return OneToOneStoreField(fld, name)
-    if isinstance(fld, models.OneToOneRel):
-        return OneToOneRelStoreField(fld, name)
-
-    if settings.SITE.is_installed("contenttypes"):
-        from lino.core.gfks import GenericForeignKey, GenericRel
-        from lino.modlib.gfks.fields import GenericForeignKeyIdField
-
-        if isinstance(fld, GenericForeignKey):
-            return GenericForeignKeyField(fld, name)
-        if isinstance(fld, GenericRel):
-            return GenericRelField(fld, name)
-        if isinstance(fld, GenericForeignKeyIdField):
-            return ComboStoreField(fld, name)
-
-    if isinstance(fld, models.ForeignKey):
-        return ForeignKeyStoreField(fld, name)
-    if isinstance(fld, models.TimeField):
-        return TimeStoreField(fld, name)
-    if isinstance(fld, models.DateTimeField):
-        return DateTimeStoreField(fld, name)
-    if isinstance(fld, fields.IncompleteDateField):
-        return IncompleteDateStoreField(fld, name)
-    if isinstance(fld, models.DateField):
-        return DateStoreField(fld, name)
-    if isinstance(fld, models.BooleanField):
-        return BooleanStoreField(fld, name)
-    if isinstance(fld, models.DecimalField):
-        return DecimalStoreField(fld, name)
-    if isinstance(fld, models.AutoField):
-        return AutoStoreField(fld, name)
-        # kw.update(type='int')
-    if isinstance(fld, models.SmallIntegerField):
-        return IntegerStoreField(fld, name)
-    if isinstance(fld, fields.DisplayField):
-        return DisplayStoreField(fld, name)
-    if isinstance(fld, models.IntegerField):
-        return IntegerStoreField(fld, name)
-    if isinstance(fld, fields.PreviewTextField):
-        return PreviewTextStoreField(fld, name)
-    if isinstance(fld, models.ManyToOneRel):
-        # raise Exception("20190625 {} {} {}".format(holder, fld, name))
-        return
-    if (sft := FIELD_TYPES.get(fld.__class__, None)) is not None:
-        return sft(fld, name)
-    kw = {}
-    if choosers.uses_simple_values(holder, fld):
-        return StoreField(fld, name, **kw)
-    else:
-        return ComboStoreField(fld, name, **kw)
-
-
 class BaseStore(object):
     pass
 
@@ -1086,7 +943,7 @@ class ParameterStore(BaseStore):
         # print(f"20250121e {holder}")
         # debug = False
         for pf in params_layout_handle._data_elems:
-            sf = create_atomizer(holder, pf, pf.name)
+            sf = atomizer.create_atomizer(holder, pf, pf.name)
             if sf is not None:
                 # if "__" in pf.name:
                 #     print("20200423 ParameterStore", pf.name, sf)
@@ -1221,7 +1078,7 @@ class Store(BaseStore):
         form = rh.actor.insert_layout
         if form:
             if isinstance(form, str):
-                raise Exception(f"20250306 insert_layout {repr(rh.actor)}")
+                raise Exception(f"20250306 insert_layout {repr(rh.actor)} is a str!")
             dh = form.get_layout_handle()
             self.collect_fields(self.detail_fields, dh)
 
@@ -1305,7 +1162,7 @@ class Store(BaseStore):
                 self.add_field_for(fields, self.pk)
 
     def add_field_for(self, field_list, df):
-        sf = get_atomizer(self.actor, df, df.name)
+        sf = atomizer.get_atomizer(self.actor, df, df.name)
         # if df.name == 'humanlinks_LinksByHuman':
         #     raise Exception("20181023 {} ({}) {}".format(
         #         self, df, sf))

lino/core/tables.py

@@ -16,10 +16,11 @@ from django.utils.translation import gettext_lazy as _
 # from django.core.exceptions import BadRequest
 
 from lino import logger
+from lino.core import constants
 from lino.core import actors
 from lino.core import actions
 from lino.core import fields
-from lino.core import constants
+from lino.core import layouts
 from lino.core.utils import resolve_fields_list
 
 # class InvalidRequest(Exception):
@@ -159,6 +160,8 @@ class AbstractTable(actors.Actor):
     abstract = True
 
     _handle_class = TableHandle
+    _detail_action_class = actions.ShowDetail
+    _params_layout_class = layouts.ParamsLayout
 
     hide_zero_rows = False
     """Set this to `True` if you want to remove rows which contain no
@@ -314,7 +317,7 @@ class AbstractTable(actors.Actor):
 
     table_as_calendar = False
     """Whether the primereact Datatable is used to display a calendar view.
-    
+
     Used only in :term:`React front end`.
     """