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

lino/core/model.py

@@ -180,7 +180,8 @@ class Model(models.Model, fields.TableRow):
                 and b is not models.Model
                 and not b._meta.abstract
             ):
-                msg = b._lino_ddh.disable_delete_on_object(self, [self.__class__])
+                msg = b._lino_ddh.disable_delete_on_object(
+                    self, [self.__class__])
                 if msg is not None:
                     return msg
         return self.__class__._lino_ddh.disable_delete_on_object(self)
@@ -243,7 +244,8 @@ class Model(models.Model, fields.TableRow):
     @classmethod
     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 = frozenset(
+                fields.fields_list(cls, cls.active_fields))
         cls.active_fields = cls.active_fields | fields.fields_list(cls, names)
 
     @classmethod
@@ -274,9 +276,11 @@ class Model(models.Model, fields.TableRow):
     @classmethod
     def on_analyze(cls, site):
         if isinstance(cls.workflow_owner_field, str):
-            cls.workflow_owner_field = cls.get_data_elem(cls.workflow_owner_field)
+            cls.workflow_owner_field = cls.get_data_elem(
+                cls.workflow_owner_field)
         if isinstance(cls.workflow_state_field, str):
-            cls.workflow_state_field = cls.get_data_elem(cls.workflow_state_field)
+            cls.workflow_state_field = cls.get_data_elem(
+                cls.workflow_state_field)
         # for vf in cls._meta.private_fields:
         #     if vf.name == 'detail_link':
         #         if vf.verbose_name is None:
@@ -300,7 +304,8 @@ class Model(models.Model, fields.TableRow):
                     bleached_fields.append(f)
         cls._bleached_fields = tuple(bleached_fields)
         if hasattr(cls, "bleached_fields"):
-            raise ChangedAPI("Replace bleached_fields by bleached=True on each field")
+            raise ChangedAPI(
+                "Replace bleached_fields by bleached=True on each field")
 
     @classmethod
     def create_from_choice(cls, text, ar=None, save=True):
@@ -344,7 +349,8 @@ class Model(models.Model, fields.TableRow):
         if isinstance(lookup_field, str):
             lookup_field = model._meta.get_field(lookup_field)
         if isinstance(lookup_field, BabelCharField):
-            flt = settings.SITE.lookup_filter(lookup_field.name, value, **known_values)
+            flt = settings.SITE.lookup_filter(
+                lookup_field.name, value, **known_values)
         else:
             if isinstance(lookup_field, models.CharField):
                 fkw[lookup_field.name + "__iexact"] = value
@@ -369,7 +375,8 @@ class Model(models.Model, fields.TableRow):
         try:
             obj.full_clean()
         except ValidationError as e:
-            raise ValidationError("Failed to auto_create %s : %s" % (obj2str(obj), e))
+            raise ValidationError(
+                "Failed to auto_create %s : %s" % (obj2str(obj), e))
         obj.save()
         signals.auto_create.send(obj, known_values=known_values)
         return obj
@@ -410,15 +417,15 @@ class Model(models.Model, fields.TableRow):
         pass
 
     def before_ui_save(self, ar, cw):
-        for f, old, new in self.fields_to_bleach():
+        for f, old, new in self.fields_to_bleach(save=True, ar=ar):
             setattr(self, f.name, new)
 
-    def fields_to_bleach(self):
+    def fields_to_bleach(self, **kwargs):
         for f in self._bleached_fields:
             old = getattr(self, f.name)
             if old is None:
                 continue
-            new = sanitize(old)
+            new = sanitize(old, **kwargs)
             # try:
             #     new = bleach.clean(
             #         new,
@@ -527,6 +534,7 @@ class Model(models.Model, fields.TableRow):
         """
         watcher = ChangeWatcher(row)
         # assert hasattr(row, state_field.attname)
+        row.before_ui_save(ar, watcher)  # added 20250312 for #5976
         old = getattr(row, state_field.attname)
         target_state.choicelist.before_state_change(row, ar, old, target_state)
         row.before_state_change(ar, old, target_state)
@@ -628,7 +636,8 @@ class Model(models.Model, fields.TableRow):
         # print("20210325", navinfo)
         # raise Exception("20")
 
-        qs = sliced_data_iterator(ar.data_iterator, navinfo["offset"], ar.limit)
+        qs = sliced_data_iterator(
+            ar.data_iterator, navinfo["offset"], ar.limit)
         for obj in qs:
             if obj.pk == self.pk:
                 items.append(E.b(str(obj)))
@@ -672,21 +681,22 @@ class Model(models.Model, fields.TableRow):
 
         df = actor.get_disabled_fields(obj, ar)
         # print(20170909, df)
-        for ba in actor.get_actions():
-            assert ba.actor == actor  # 20170102
-            if ba.action.show_in_workflow:
-                # if actor.model.__name__ == 'Vote':
-                #     if ba.action.__class__.__name__ == 'MarkVoteAssigned':
-                #         print(20170115, actor, ar.get_user())
-                if ba.action.action_name not in df:
-                    if actor.get_row_permission(obj, ar, state, ba):
-                        if show and isinstance(ba.action, ChangeStateAction):
-                            show_state()
-                            sep = " \u2192 "  # "→"
-                            show = False
-                        l.append(sep)
-                        l.append(ar.action_button(ba, obj))
-                        sep = " "
+        if 'workflow_buttons' not in df:
+            for ba in actor.get_actions():
+                assert ba.actor == actor  # 20170102
+                if ba.action.show_in_workflow:
+                    # if actor.model.__name__ == 'Vote':
+                    #     if ba.action.__class__.__name__ == 'MarkVoteAssigned':
+                    #         print(20170115, actor, ar.get_user())
+                    if ba.action.action_name not in df:
+                        if actor.get_row_permission(obj, ar, state, ba):
+                            if show and isinstance(ba.action, ChangeStateAction):
+                                show_state()
+                                sep = " \u2192 "  # "→"
+                                show = False
+                            l.append(sep)
+                            l.append(ar.action_button(ba, obj))
+                            sep = " "
         if state and show:
             show_state()
         return E.span(*l)
@@ -710,7 +720,6 @@ class Model(models.Model, fields.TableRow):
     #     return self.preview(ar)
     #     #~ username = kw.pop('username',None)
 
-
     @classmethod
     def get_chooser_for_field(cls, fieldname):
         d = getattr(cls, "_choosers_dict", {})
@@ -718,7 +727,7 @@ class Model(models.Model, fields.TableRow):
         #     print("20200425 Model.get_chooser_for_field", cls, fieldname, d)
         return d.get(fieldname, None)
 
-    def filename_root(self):
+    def get_printable_target_stem(self):
         return self._meta.app_label + "." + self.__class__.__name__ + "-" + str(self.pk)
 
     @classmethod
@@ -743,7 +752,8 @@ class Model(models.Model, fields.TableRow):
             return set([fld.choicelist.get_by_name(x) for x in states.split()])
         elif isinstance(states, set):
             return states
-        raise Exception("Cannot resolve stateset specifier {!r}".format(states))
+        raise Exception(
+            "Cannot resolve stateset specifier {!r}".format(states))
 
     @classmethod
     def add_picker(model, fldname):
@@ -879,7 +889,8 @@ class Model(models.Model, fields.TableRow):
         collect(self)
         s = "\n".join(
             [
-                '    "%s" -> "%s"' % (p._meta.verbose_name, c._meta.verbose_name)
+                '    "%s" -> "%s"' % (p._meta.verbose_name,
+                                      c._meta.verbose_name)
                 for p, c in pairs
             ]
         )
@@ -970,6 +981,7 @@ LINO_MODEL_ATTRIBS = (
     "error2str",
     "print_subclasses_graph",
     "disable_create",
+    "override_column_headers",
     "grid_post",
     "submit_insert",
     "delete_veto_message",
@@ -1021,7 +1033,8 @@ def pre_delete_handler(sender, instance=None, **kw):
                 for obj in qs:
                     setattr(obj, gfk.name, None)
             elif qs.count():
-                raise Warning(instance.delete_veto_message(qs.model, qs.count()))
+                raise Warning(instance.delete_veto_message(
+                    qs.model, qs.count()))
     for qs in must_cascade:
         if qs.count():
             logger.info(

lino/core/plugin.py

@@ -1,10 +1,9 @@
 # -*- coding: UTF-8 -*-
-# Copyright 2008-2024 Rumma & Ko Ltd
+# Copyright 2008-2025 Rumma & Ko Ltd
 # License: GNU Affero General Public License v3 (see file COPYING for details)
 
-import os
 import inspect
-from os.path import exists, join, dirname, isdir, abspath
+from os.path import join, dirname, isdir, abspath
 from collections.abc import Iterable
 from urllib.parse import urlencode
 from lino.core.exceptions import ChangedAPI
@@ -56,7 +55,8 @@ class Plugin:
 
     def hide(self):
         if self.site._startup_done:
-            raise Exception("Tried to deactivate plugin {} after startup".format(self))
+            raise Exception(
+                "Tried to deactivate plugin {} after startup".format(self))
         self.hidden = True
 
     def configure(self, **kw):

lino/core/requests.py

@@ -1,5 +1,5 @@
 # -*- coding: UTF-8 -*-
-# Copyright 2009-2024 Rumma & Ko Ltd
+# Copyright 2009-2025 Rumma & Ko Ltd
 # License: GNU Affero General Public License v3 (see file COPYING for details)
 """
 See introduction in :doc:`/dev/ar` and API reference in
@@ -10,13 +10,12 @@ See introduction in :doc:`/dev/ar` and API reference in
 
 import sys
 import json
-import html
 import logging
 from io import StringIO
 from lino import logger
 from contextlib import contextmanager
 from types import GeneratorType
-from copy import copy, deepcopy
+from copy import deepcopy
 from xml.sax.saxutils import escape
 from etgen import html as xghtml
 
@@ -34,7 +33,7 @@ from django.db import models
 from django.db.models.query import QuerySet
 from asgiref.sync import sync_to_async
 
-from lino.utils.html import E, tostring, iselement
+from lino.utils.html import E, tostring
 from lino.utils import AttrDict
 from lino.utils import capture_output
 from lino.utils import MissingRow
@@ -49,11 +48,9 @@ from lino.core.diff import ChangeWatcher
 from lino.core.utils import getrqdata
 from lino.core.utils import obj2unicode
 from lino.core.utils import obj2str
-from lino.core.utils import UnresolvedModel
 from lino.core.utils import format_request
 from lino.core.utils import PhantomRow
 from lino.core.store import get_atomizer
-from lino.core.exceptions import ChangedAPI
 
 # try:
 #     from django.contrib.contenttypes.models import ContentType
@@ -82,6 +79,7 @@ Subject: {subject}
 def noop(ar):
     return ar.success(gettext("Aborted"))
 
+
 def mi2bp(master_instance, bp):
     if master_instance is not None:
         if isinstance(master_instance, (models.Model, TableRow)):
@@ -114,6 +112,7 @@ class PrintLogger(logging.Logger):
         self.addHandler(h)
         # print("20231012", logging.getLevelName(self.level), self.__class__)
 
+
 class StringLogger(logging.Logger):
     # Instantiated by BaseRequest.capture_logger()
 
@@ -271,7 +270,7 @@ class ValidActionResponses(object):
 
 inheritable_attrs = frozenset(
     ["user", "subst_user", "renderer", "requesting_panel", "master_instance",
-    "logger", "show_urls"] )
+     "logger", "show_urls"])
 
 
 def bool2text(x):
@@ -280,7 +279,6 @@ def bool2text(x):
     return _("No")
 
 
-
 class SearchQuerySet:
     pass
 
@@ -440,7 +438,7 @@ class BaseRequest:
                 self.known_values.setdefault(k, v)
 
             self.obvious_fields |= self.actor.obvious_fields
-                # self.obvious_fields.update(obvious_fields.split())
+            # self.obvious_fields.update(obvious_fields.split())
             # if parent.obvious_fields is not None:
             if parent is not None and parent.actor is self.actor and parent.obvious_fields:
                 self.obvious_fields |= parent.obvious_fields
@@ -453,7 +451,6 @@ class BaseRequest:
                         self.actor, locals()
                     )
                 )
-            # if isinstance(self.master_instance, UnresolvedModel):
             if isinstance(self.master_instance, MissingRow):
                 raise exceptions.BadRequest(self.master_instance)
                 # raise Exception(
@@ -465,6 +462,7 @@ class BaseRequest:
         user=None,
         subst_user=None,
         current_project=None,
+        display_mode=None,
         master=None,
         master_instance=None,
         master_key=None,
@@ -494,6 +492,7 @@ class BaseRequest:
         else:
             self.user = user
         self.current_project = current_project
+        self.display_mode = display_mode
         if renderer is None:
             renderer = settings.SITE.kernel.text_renderer
             # renderer = settings.SITE.kernel.default_renderer
@@ -509,9 +508,11 @@ class BaseRequest:
                 master = self.actor.master
             if master_type and ContentType is not None:
                 try:
-                    master = ContentType.objects.get(pk=master_type).model_class()
+                    master = ContentType.objects.get(
+                        pk=master_type).model_class()
                 except ContentType.DoesNotExist:
-                    raise exceptions.BadRequest("Invalid master_type {}".format(master_type)) from None
+                    raise exceptions.BadRequest(
+                        "Invalid master_type {}".format(master_type)) from None
             self.master = master
 
             if self.master is not None and self.master_instance is None:
@@ -520,7 +521,8 @@ class BaseRequest:
                     master_instance = self.get_master_instance(
                         self.master, master_key, master_type
                     )
-                self.master_instance = self.actor.cast_master_instance(master_instance)
+                self.master_instance = self.actor.cast_master_instance(
+                    master_instance)
 
         self.row_meta = dict(meta=True)
 
@@ -642,7 +644,10 @@ class BaseRequest:
             kw.update(user=request.user)
             kw.update(subst_user=request.subst_user)
         kw.update(requesting_panel=request.requesting_panel)
-        kw.update(current_project=rqdata.get(constants.URL_PARAM_PROJECT, None))
+        kw.update(current_project=rqdata.get(
+            constants.URL_PARAM_PROJECT, None))
+        kw.update(display_mode=rqdata.get(
+            constants.URL_PARAM_DISPLAY_MODE, None))
 
         # If the incoming request specifies an active tab, then the
         # response must forward this information. Otherwise Lino would
@@ -713,7 +718,8 @@ class BaseRequest:
                 offset = rqdata.get(constants.URL_PARAM_START, None)
                 if offset:
                     kw.update(offset=int(offset))
-                limit = rqdata.get(constants.URL_PARAM_LIMIT, self.actor.preview_limit)
+                limit = rqdata.get(constants.URL_PARAM_LIMIT,
+                                   self.actor.preview_limit)
                 if limit:
                     kw.update(limit=int(limit))
             except ValueError:
@@ -1007,7 +1013,8 @@ class BaseRequest:
 
     debug = lambda self, *args, **kwargs: self.logger.debug(*args, **kwargs)
     info = lambda self, *args, **kwargs: self.logger.info(*args, **kwargs)
-    warning = lambda self, *args, **kwargs: self.logger.warning(*args, **kwargs)
+    warning = lambda self, * \
+        args, **kwargs: self.logger.warning(*args, **kwargs)
 
     async def adebug(self, *args, **kwargs):
         return await self.alogger.debug(*args, **kwargs)
@@ -1043,7 +1050,8 @@ class BaseRequest:
             )
             return
 
-        self.logger.info("Send email '%s' from %s to %s", subject, sender, recipients)
+        self.logger.info("Send email '%s' from %s to %s",
+                         subject, sender, recipients)
 
         recipients = [a for a in recipients if "@example.com" not in a]
         if not len(recipients):
@@ -1535,18 +1543,17 @@ class BaseRequest:
         if cls is not None:
             user_type = self.get_user().user_type
             for ba in cls.get_toolbar_actions(self.bound_action.action, user_type):
-                if not ba.action.select_rows:
-                    if ba.action.show_in_plain:
-                        ir = ba.request_from(self)
-                        # assert ir.user is self.user
-                        if ir.get_permission():
-                            # try:
-                            #     btn = ir.ar2button(**btnattrs)
-                            # except AttributeError:
-                            #     raise Exception("20200513 {}".format(ir))
-                            btn = ir.ar2button(**btnattrs)
-                            # assert iselement(btn)
-                            buttons.append(btn)
+                if ba.action.show_in_plain and not ba.action.select_rows:
+                    ir = ba.request_from(self)
+                    # assert ir.user is self.user
+                    if ir.get_permission():
+                        # try:
+                        #     btn = ir.ar2button(**btnattrs)
+                        # except AttributeError:
+                        #     raise Exception("20200513 {}".format(ir))
+                        btn = ir.ar2button(**btnattrs)
+                        # assert iselement(btn)
+                        buttons.append(btn)
         # print("20181106", cls, self.bound_action, buttons)
         return buttons
         # if len(buttons) == 0:
@@ -1662,7 +1669,8 @@ class BaseRequest:
         else:
             # print("20190703", self.actor, self.actor.default_action)
             sar = self.spawn_request(actor=self.actor)
-            list_title = tostring(sar.href_to_request(sar, list_title, icon_name=None))
+            list_title = tostring(sar.href_to_request(
+                sar, list_title, icon_name=None))
             # return list_title + " » " + self.get_detail_title(elem)
             return format_html("{} » {}", list_title,  self.get_detail_title(elem))
 
@@ -1858,16 +1866,17 @@ class ActionRequest(BaseRequest):
                     else:
                         raise Exception(
                             "20160329 params_layout {0} has no params_store "
-                            "in {1!r}".format(self.actor.params_layout, self.actor)
+                            "in {1!r}".format(
+                                self.actor.params_layout, self.actor)
                         )
             else:
                 for k in param_values.keys():
                     if k not in pv:
-                        raise Exception(
-                            "Invalid key '%s' in param_values of %s "
-                            "request (possible keys are %s)"
-                            % (k, self.actor, list(pv.keys()))
-                        )
+                        msg = ("Invalid key '%s' in param_values of %s "
+                               "request (possible keys are %s)"
+                               % (k, self.actor, list(pv.keys())))
+                        # print(msg)  # 20250309
+                        raise Exception(msg)
                 pv.update(param_values)
             # print("20160329 ok", pv)
             self.param_values = AttrDict(**pv)
@@ -1883,7 +1892,8 @@ class ActionRequest(BaseRequest):
                 #     self.selected_rows, action)
                 # raise Exception(msg)
             if request is not None:
-                apv.update(action.params_layout.params_store.parse_params(request))
+                apv.update(
+                    action.params_layout.params_store.parse_params(request))
             self.action_param_values = AttrDict(**apv)
             # action.check_params(action_param_values)
             self.set_action_param_values(**action_param_values)
@@ -2001,19 +2011,20 @@ class ActionRequest(BaseRequest):
         except Exception as e:
             if not settings.SITE.catch_layout_exceptions:
                 raise
-            # Report this exception. But since such errors may occur
-            # rather often and since exception loggers usually send an
-            # email to the local system admin, make sure to log each
-            # exception only once.
-            self.no_data_text = f"{e} (set catch_layout_exceptions to see details)"
+            # Report this exception. But since such errors may occur rather
+            # often and since exception loggers usually send an email to the
+            # local system admin, make sure to log each exception only once.
+            e = str(e)
+            self.no_data_text = f"{e} (set catch_layout_exceptions to see \
+                details)"
             self._data_iterator = []
-            w = WARNINGS_LOGGED.get(str(e))
+            w = WARNINGS_LOGGED.get(e)
             if w is None:
-                WARNINGS_LOGGED[str(e)] = True
+                WARNINGS_LOGGED[e] = True
                 # raise
                 # logger.exception(e)
                 logger.warning(f"Error while executing {repr(self)}: {e}\n"
-                    "(Subsequent warnings will be silenced.)")
+                               "(Subsequent warnings will be silenced.)")
 
         if self._data_iterator is None:
             raise Exception(f"No data iterator for {self}")
@@ -2023,7 +2034,8 @@ class ActionRequest(BaseRequest):
             self._data_iterator = tuple(self._data_iterator)
         if isinstance(self._data_iterator, (SearchQuery, SearchQuerySet)):
             self._sliced_data_iterator = tuple(
-                self.actor.get_rows_from_search_query(self._data_iterator, self)
+                self.actor.get_rows_from_search_query(
+                    self._data_iterator, self)
             )
         else:
             self._sliced_data_iterator = sliced_data_iterator(
@@ -2147,7 +2159,8 @@ class ActionRequest(BaseRequest):
 
         if self.request is not None:
             for e in elems:
-                self.ah.store.form2obj(self, self.request.POST or self.rqdata, e, True)
+                self.ah.store.form2obj(
+                    self, self.request.POST or self.rqdata, e, True)
         for e in elems:
             e.full_clean()
         return elems
@@ -2158,7 +2171,8 @@ class ActionRequest(BaseRequest):
             self.actor.handle_uploaded_files(elem, self.request)
 
         if self.request is not None:
-            self.ah.store.form2obj(self, self.request.POST or self.rqdata, elem, True)
+            self.ah.store.form2obj(
+                self, self.request.POST or self.rqdata, elem, True)
         elem.full_clean()
         return elem
 
@@ -2171,7 +2185,8 @@ class ActionRequest(BaseRequest):
         if self._status is not None and not kw:
             return self._status
         if self.actor.parameters:
-            pv = self.actor.params_layout.params_store.pv2dict(self, self.param_values)
+            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}")
             # print(f"20250121c {self} {self.param_values.keys()}\n{self.actor.params_layout}\n{pv.keys()}")
             # print(f"20250121c {self}|{self.actor.params_layout.params_store}\n{}")
@@ -2181,6 +2196,8 @@ class ActionRequest(BaseRequest):
         bp = kw.setdefault("base_params", {})
         if self.current_project is not None:
             bp[constants.URL_PARAM_PROJECT] = self.current_project
+        if self.display_mode is not None:
+            bp[constants.URL_PARAM_DISPLAY_MODE] = self.display_mode
         if self.subst_user is not None:
             # raise Exception("20230331")
             bp[constants.URL_PARAM_SUBST_USER] = self.subst_user.id
@@ -2284,7 +2301,6 @@ class ActionRequest(BaseRequest):
         # ~ s = self.get_title()
         # ~ return s.encode('us-ascii','replace')
 
-
     def to_rst(self, *args, **kw):
         """Returns a string representing this table request in
         reStructuredText markup.
@@ -2373,7 +2389,8 @@ class ActionRequest(BaseRequest):
             # if cellwidths and self.renderer.is_interactive:
             if cellwidths:
                 totwidth = sum([int(w) for w in cellwidths])
-                widths = [str(int(int(w) * 100 / totwidth)) + "%" for w in cellwidths]
+                widths = [str(int(int(w) * 100 / totwidth))
+                          + "%" for w in cellwidths]
                 for i, td in enumerate(headers):
                     # td.set('width', six.text_type(cellwidths[i]))
                     td.set("width", widths[i])
@@ -2381,7 +2398,8 @@ class ActionRequest(BaseRequest):
         # ~ print 20120623, ar.actor
         recno = 0
         for obj in data_iterator:
-            cells = ar.row2html(recno, columns, obj, sums, **self.renderer.cellattrs)
+            cells = ar.row2html(recno, columns, obj, sums,
+                                **self.renderer.cellattrs)
             if cells is not None:
                 recno += 1
                 tble.body.append(xghtml.E.tr(*cells))
@@ -2420,7 +2438,8 @@ class ActionRequest(BaseRequest):
                 columns = None
             else:
                 data = getrqdata(ar.request)
-                columns = [str(x) for x in data.getlist(constants.URL_PARAM_COLUMNS)]
+                columns = [str(x) for x in data.getlist(
+                    constants.URL_PARAM_COLUMNS)]
             if columns:
                 all_widths = data.getlist(constants.URL_PARAM_WIDTHS)
                 hiddens = [
@@ -2464,10 +2483,12 @@ class ActionRequest(BaseRequest):
                     # except AttributeError as ex:
                     #     raise AttributeError("20160529 %s : %s" % (e, ex))
                 #
-                fields = [e for e in fields if not e.value.get("hidden", False)]
+                fields = [e for e in fields if not e.value.get(
+                    "hidden", False)]
                 fields = [e for e in fields if not e.hidden]
 
-                widths = ["%d" % (e.width or e.preferred_width) for e in fields]
+                widths = ["%d" % (e.width or e.preferred_width)
+                          for e in fields]
                 columns = [e.name for e in fields]
 
             headers = [column_header(col) for col in fields]