accrete 0.0.38__tar.gz → 0.0.40__tar.gz

{accrete-0.0.38 → accrete-0.0.40}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: accrete
-Version: 0.0.38
+Version: 0.0.40
 Summary: Django Shared Schema Multi Tenant
 Author-email: Benedikt Jilek <benedikt.jilek@pm.me>
 License: Copyright (c) 2023 Benedikt Jilek

{accrete-0.0.38 → accrete-0.0.40}/accrete/annotation.py

@@ -1,4 +1,4 @@
-from django.db.models import Field, QuerySet
+from django.db.models import Field
 from django.db.models.expressions import Func
 from django.db.models.aggregates import Aggregate
 
@@ -34,13 +34,10 @@ class AnnotationModelMixin:
 
 class AnnotationManagerMixin:
 
-    @staticmethod
-    def add_annotations(queryset: QuerySet):
-        model = queryset.model
-        if not hasattr(model, 'get_annotations'):
-            return queryset
-        return queryset.annotate(**{
+    def get_annotations(self):
+        if not hasattr(self.model, 'get_annotations'):
+            return {}
+        return {
             annotation['name']: annotation['annotation'].function
-            for annotation in model.get_annotations()
-        })
-
+            for annotation in self.model.get_annotations()
+        }

{accrete-0.0.38 → accrete-0.0.40}/accrete/contrib/ui/__init__.py

@@ -8,7 +8,7 @@ from .context import (
     list_page,
     detail_page,
     cast_param,
-    prepare_url_params,
+    url_param_dict,
     extract_url_params,
     exclude_params,
     url_param_str,

{accrete-0.0.38 → accrete-0.0.40}/accrete/contrib/ui/context.py

@@ -1,4 +1,5 @@
 import logging
+from urllib.parse import quote_plus
 from dataclasses import dataclass, field
 from typing import TypedDict
 from django.utils.translation import gettext_lazy as _t
@@ -56,6 +57,7 @@ class TableContext(Context):
     endless_scroll: bool
     filter: Filter
     object_param_str: str = field(default='', kw_only=True)
+    field_selection: bool = True
 
 
 @dataclass
@@ -69,6 +71,7 @@ class ListContext(Context):
     endless_scroll: bool = True
     column_height: int = 4
     column_width: int = 12
+    field_selection: bool = False
 
     def __post_init__(self):
         super().__post_init__()
@@ -107,8 +110,8 @@ def cast_param(params: dict, param: str, cast_to: callable, default):
         return default
 
 
-def prepare_url_params(get_params: dict) -> dict:
-    return {key: f'&{key}={value}' for key, value in get_params.items()}
+def url_param_dict(get_params: dict) -> dict:
+    return {key: f'&{key}={quote_plus(value)}' for key, value in get_params.items()}
 
 
 def url_param_str(params: dict, extract: list[str] = None) -> str:
@@ -118,9 +121,9 @@ def url_param_str(params: dict, extract: list[str] = None) -> str:
     them, so that each value is formatted e.g. {'page': '&page=1'}
     """
     if extract:
-        params = prepare_url_params(extract_url_params(params, extract))
+        params = extract_url_params(params, extract)
     param_str = (
-        "".join(str(value) for value in params.values())
+        "".join(str(value) for value in url_param_dict(params).values())
         .replace('&&', '&')
         .replace('?&', '?')
         .strip('?&')
@@ -185,7 +188,7 @@ def get_table_fields(
         field_definition: dict[str | TableField] = None
 ) -> list[TableField]:
 
-    def get_alignment(field_attr):
+    def get_alignment(field_attr) -> TableFieldAlignment:
         number_field_types = (
             models.DecimalField, models.IntegerField, models.FloatField
         )
@@ -193,12 +196,11 @@ def get_table_fields(
                 isinstance(field_attr, number_field_types)
                 or (
                 isinstance(field_attr, Annotation)
-                and field_attr.field in number_field_types
-        )):
+                and field_attr.field in number_field_types)
+        ):
             return TableFieldAlignment.RIGHT
         return TableFieldAlignment.LEFT
 
-
     def get_field_definition(f_name: str) -> TableField:
         if definition := field_definition.get(f_name):
             return definition
@@ -217,12 +219,18 @@ def get_table_fields(
             elif hasattr(attr, 'field'):
                 attr = attr.field
         names.append(attr.verbose_name)
+        is_relation = (
+                attr.related_model
+                and hasattr(attr.related_model, 'get_absolute_url')
+                or False
+        )
         return TableField(
             label=' / '.join([str(name) for name in names[1:]]),
             name=f_name,
             header_info=str(attr.help_text),
             truncate_after=50,
-            alignment=get_alignment(attr)
+            alignment=get_alignment(attr),
+            is_relation=is_relation
         )
 
     if field_definition is None:
@@ -233,6 +241,7 @@ def get_table_fields(
         try:
             table_fields.append(get_field_definition(field_name))
         except AttributeError as e:
-            print(e)
+            _logger.error(e)
             pass
+
     return table_fields

{accrete-0.0.38 → accrete-0.0.40}/accrete/contrib/ui/elements.py

@@ -69,6 +69,7 @@ class TableField:
     suffix: str = ''
     truncate_after: int = 0
     template: str = None
+    is_relation: bool = False
 
 
 @dataclass

{accrete-0.0.38 → accrete-0.0.40}/accrete/contrib/ui/filter.py

@@ -298,14 +298,25 @@ class Filter:
 
     def field_path_selection(self):
         html = ''
-        for field in sorted(self.field_paths, key=lambda f: (len(f.split('__')), f.lower())):
-            label = self.get_field_path_label(field)
-            if not label:
-                continue
+        filter_exclude = getattr(self.model, 'filter_exclude', [])
+        filter_exclude.extend(self.default_exclude)
+        foreign_keys = [x.name for x in filter(
+            lambda x: x.is_relation and x.name not in filter_exclude,
+            self.model._meta.get_fields()
+        )]
+        field_paths = [
+            (self.get_field_path_label(field_path), field_path)
+            for field_path in filter(
+                lambda x: len(x.split('__')) <= 1,
+                self.field_paths + foreign_keys
+            )
+        ]
+        sorted_field_paths = sorted(filter(lambda x: x[0], field_paths), key=lambda x: x[0].lower())
+        for field in sorted_field_paths:
             html += f"""
-                <label class="checkbox my-1" style="width: 100%">
-                    <input type="checkbox" name="{field}">
-                    {label}
+                <label class="checkbox is-unselectable my-1" style="width: 100%">
+                    <input type="checkbox" name="{field[1]}" data-label="{field[0].lower()}">
+                    <span>{field[0]}</span>
                 </label>
             """
         return html

{accrete-0.0.38 → accrete-0.0.40}/accrete/contrib/ui/static/js/filter.js

@@ -1,6 +1,10 @@
+document.body.addEventListener('htmx:afterSwap', function(evt) {
+    checkSelectedFields();
+});
 window.addEventListener('click', showHide);
 buildQueryTagsFromUrl();
 resetInput();
+checkSelectedFields();
 
 function buildQueryTagsFromUrl() {
     const queryTags = document.getElementById('query-tags');
@@ -249,11 +253,30 @@ function toggleFilterModalButton() {
     const button = document.getElementById('modal-filter-button');
     const query = document.getElementById('query-tags');
     if (query.firstElementChild && query.firstElementChild.childElementCount) {
-        button.classList.add('is-primary');
+        button.classList.add('is-success');
     }
     else {
-        button.classList.remove('is-primary');
-    }
+        button.classList.remove('is-success');
+    }
+}
+
+function checkSelectedFields() {
+    const table = document.getElementById('content-table');
+    const header = table.querySelectorAll('th')
+    document.getElementById(
+        'field-path-checkboxes'
+    ).querySelectorAll(
+        'input:checked'
+    ).forEach((el) => {el.checked = ''});
+    header.forEach((th) => {
+        const name = th.getAttribute('data-name');
+        if (name) {
+            document.getElementById(
+                'field-path-checkboxes'
+            ).querySelector(`input[name=${name}]`
+            ).checked = 'checked';
+        }
+    })
 }
 
 //-----------------------------------------------------------------------------
@@ -441,25 +464,57 @@ function changeOperator(event) {
 
 function toggleFieldSelection() {
     const fieldPaths = document.getElementById('field-paths');
+    const input = document.getElementById('field-path-search-input')
+    const checkBoxes = document.getElementById('field-path-checkboxes');
     this.event.target.classList.toggle('is-active');
     fieldPaths.classList.toggle('is-hidden');
+    input.value = ''
+    checkBoxes.querySelectorAll(':scope * input').forEach(
+        (el) => {
+            el.parentElement.classList.remove('is-hidden')
+        });
 }
 
 function filterFieldPaths() {
-    const input = document.getElementById('field-path-search-input');
-    const paths = document.getElementById('field-path-checkboxes').children;
-    console.log(paths)
-    console.log(paths.length)
-    for (let i = 0; paths.length < i; i++) {
-        if (!paths[i].firstElementChild.name.contains(input.value)) {
-            paths[i].classList.add('is-hidden');
-        }
+    const value = document.getElementById('field-path-search-input').value.toString().toLowerCase();
+    const checkBoxes = document.getElementById('field-path-checkboxes');
+    if (!value) {
+        checkBoxes.querySelectorAll(':scope * input').forEach(
+            (el) => {
+                el.parentElement.classList.remove('is-hidden')
+            });
+        return
     }
-    // paths.forEach(el => {
-    //     if (!el.getAttribute('name').contains(input.value)) {
-    //         el.classList.add('is-hidden');
-    //     }
-    // })
+    const pathsToShow = checkBoxes.querySelectorAll(`:scope * input[data-label*=${value}]`);
+    const pathsToHide = checkBoxes.querySelectorAll(`:scope * input:not([data-label*=${value}])`);
+    pathsToShow.forEach((el) => {el.parentElement.classList.remove('is-hidden')});
+    pathsToHide.forEach((el) => {el.parentElement.classList.add('is-hidden')})
+}
+
+function changeFieldSelection() {
+    const fieldPathApply = document.getElementById('field-path-apply');
+    const fieldPathApplyEvent = new Event('click');
+    const url = new URL(window.location)
+    const fieldPaths = document.getElementById('field-path-checkboxes');
+    const selected = fieldPaths.querySelectorAll('input:checked')
+    let fields = []
+    selected.forEach((input) => {
+        fields.push(input.name)
+    })
+    url.searchParams.set('fields', JSON.stringify(fields));
+    fieldPathApply.setAttribute('hx-get', url.toString());
+    htmx.process(fieldPathApply);
+    fieldPathApply.dispatchEvent(fieldPathApplyEvent);
+}
+
+function resetFieldPaths() {
+    const fieldPathApply = document.getElementById('field-path-apply');
+    const fieldPathApplyEvent = new Event('click');
+    const url = new URL(window.location)
+    url.searchParams.delete('fields');
+    fieldPathApply.setAttribute('hx-get', url.toString());
+    htmx.process(fieldPathApply);
+    fieldPathApply.dispatchEvent(fieldPathApplyEvent);
 }
 
 //-----------------------------------------------------------------------------

{accrete-0.0.38 → accrete-0.0.40}/accrete/contrib/ui/templates/ui/list.html

@@ -11,7 +11,7 @@
                     hx-get="{{ url_params }}&page={{ page.next_page_number }}"
                     hx-trigger="intersect once"
                     hx-select=".list-column"
-                    hx-select-oob="#list-pagination-end-index,#list-pagination-next-button"
+                    hx-select-oob="#list-pagination-end-index,#list-pagination-next-button,#list-pagination-prev-button"
                     hx-swap="afterend"
                 {% endif %}
             >
@@ -24,6 +24,16 @@
         {% endfor %}
     </div>
     {% if endless_scroll %}
-        <progress class="htmx-indicator progress is-small is-primary" max="100">15%</progress>
+        <div id="endless-scroller"
+            {% if list_page.has_next %}
+                hx-get="{{ url_params }}&page={{ page.next_page_number }}"
+                hx-trigger="intersect once"
+                hx-select=".list-column"
+                hx-select-oob="#list-pagination-end-index,#list-pagination-next-button,#list-pagination-prev-button,#endless-scroller"
+                hx-swap="afterend"
+            {% endif %}
+        >
+        </div>
+        <progress class="htmx-indicator progress is-small is-success" max="100">15%</progress>
     {% endif %}
 {% endblock %}

{accrete-0.0.38 → accrete-0.0.40}/accrete/contrib/ui/templates/ui/partials/filter.html

@@ -54,27 +54,27 @@
     <div id="query-params-dropdown" class="box mt-0 mx-0 p-1 is-hidden is-fullwidth" tabindex="-1" style="z-index: 900; background: white; word-break: break-word; position: absolute; width: 300px; max-height: 300px; overflow-y: auto" onclick="toggleParams()">
         {{ filter.to_html.params }}
     </div>
-    <div id="field-selection" class="button mt-1" style="z-index: 10; position: relative" onclick="toggleFieldSelection()">
-        {% translate 'Fields' %}
-    </div>
-    <div id="field-paths" class="box mt-1 mx-0 p-1 is-hidden" tabindex="-1" style="z-index: 800; outline: none; background: white; word-break: break-word; position: absolute; width: 300px; max-height: 300px; overflow-y: auto">
-        <div class="field has-addons mr-1">
-            <p class="control is-expanded">
-                <input id="field-path-search-input" class="input is-small" type="text" aria-label="Query Input" oninput="filterFieldPaths()">
-            </p>
-            <p class="control">
-                <button id="field-path-search-clear" class="button is-small has-icon"><i class="icon-clear"></i></button>
-            </p>
-            <p class="control">
-                <button id="field-path-search-reset" class="button is-small has-icon"><i class="icon-delete-filter"></i></button>
-            </p>
-        </div>
-        <div id="field-path-checkboxes">
-            {{ filter.to_html.field_paths }}
+    {% if field_selection %}
+        <div id="field-selection" class="button mt-1" style="z-index: 10; position: relative" onclick="toggleFieldSelection()">
+            {% translate 'Fields' %}
         </div>
-        <div id="field-path-change">
-
+        <div id="field-paths" class="box mt-1 mx-0 p-1 is-hidden" tabindex="-1" style="z-index: 800; outline: none; background: white; word-break: break-word; position: absolute; width: 300px; max-height: 300px; overflow-y: auto">
+            <div class="field has-addons mr-1">
+                <p class="control is-expanded">
+                    <input id="field-path-search-input" class="input is-small" type="text" aria-label="Query Input" oninput="filterFieldPaths()">
+                </p>
+                <p class="control">
+                    <button id="field-path-search-reset" class="button is-small has-icon" onclick="resetFieldPaths()">
+                        <i class="icon-delete-filter"></i>
+                    </button>
+                </p>
+            </div>
+            <div id="field-path-checkboxes" onchange="changeFieldSelection()">
+                {{ filter.to_html.field_paths }}
+            </div>
+            <div id="field-path-apply" hx-get="" hx-trigger="click" hx-replace-url="true"
+                 hx-select-oob="#content,#field-path-apply,#list-pagination">
+            </div>
         </div>
-    </div>
-
+    {% endif %}
 </div>

{accrete-0.0.38 → accrete-0.0.40}/accrete/contrib/ui/templates/ui/partials/pagination_list.html

@@ -1,7 +1,7 @@
 {% load i18n %}
 <div id="list-pagination" class="field has-addons" style="min-width: 100%; max-width: 100%">
     <p class="control">
-        <button class="button"
+        <button id="list-pagination-prev-button" class="button"
                 hx-get="{{ pagination_param_str }}&page={% if list_page.has_previous %}{{ list_page.previous_page_number }}{% else %}{{ list_page.paginator.num_pages }}{% endif %}"
                 hx-replace-url="true"
                 hx-select-oob="#content,#list-pagination,#panel-actions,#header-actions,#breadcrumbs"

{accrete-0.0.38 → accrete-0.0.40}/accrete/contrib/ui/templates/ui/table.html

@@ -4,14 +4,14 @@
 {% load accrete_ui %}
 
 {% block content %}
-    <table class="table is-fullwidth is-hoverable hax-box" hx-indicator=".htmx-indicator">
+    <table id="content-table" class="table is-fullwidth is-hoverable hax-box" hx-indicator=".htmx-indicator">
         <thead style="position: sticky; top: 0; background: white; z-index: 10">
             {% block table_header_row %}
                 <tr>
                     {% block table_header %}
                         <th>{{ object_label }}</th>
                         {% for field in fields %}
-                            <th style="text-align: {% firstof field.header_alignment.value field.alignment.value %}">{{ field.label }}{% if field.header_info %}({{ field.header_info }}){% endif %}</th>
+                            <th data-name="{{ field.name }}" style="text-align: {% firstof field.header_alignment.value field.alignment.value %}">{{ field.label }}{% if field.header_info %}({{ field.header_info }}){% endif %}</th>
                         {% endfor %}
                         {% if unselect_button %}
                             <th></th>
@@ -32,18 +32,20 @@
                                 {% if field.template %}
                                     {% include field.template %}
                                 {% else %}
-                                    {% with val=obj|get_attr:field.name|default_if_none:'---'  %}
+                                    {% with val=obj|get_attr:field.name|default_if_none:'---' is_relation=field.is_relation %}
                                         <td style="text-align: {{ field.alignment.value }}" {% if val|length > field.truncate_after %}title="{{ val }}"{% endif %}>
                                             <span class="responsive-heading has-text-weight-light" style="margin-right: .2rem">{{ field.label }}:</span>
-                                            {{ field.prefix }}
-                                            {% block td_content %}
-                                                {% if field.truncate_after > 0 %}
-                                                    {{ val|truncatechars:field.truncate_after }}
-                                                {% else %}
-                                                    {{ val }}
-                                                {% endif %}
-                                            {% endblock %}
-                                            {{ field.suffix }}
+                                            {% if is_relation %}<a href="{{ obj|related_obj_url:field.name }}" class="is-underlined">{% endif %}
+                                                {{ field.prefix }}
+                                                {% block td_content %}
+                                                    {% if field.truncate_after > 0 %}
+                                                        {{ val|truncatechars:field.truncate_after }}
+                                                    {% else %}
+                                                        {{ val }}
+                                                    {% endif %}
+                                                {% endblock %}
+                                                {{ field.suffix }}
+                                            {% if is_relation %}</a>{% endif %}
                                         </td>
                                     {% endwith %}
                                 {% endif %}
@@ -65,10 +67,11 @@
                  hx-select="tbody > tr"
                  hx-target="#content-table-body"
                  hx-swap="beforeend"
-                 hx-select-oob="#list-pagination-end-index,#list-pagination-next-button,#endless-scroller"
+                 hx-replace-url="true"
+                 hx-select-oob="#list-pagination-end-index,#list-pagination-next-button,#list-pagination-prev-button,#endless-scroller"
              {% endif %}
         >
         </div>
-        <progress class="htmx-indicator progress is-small is-primary" max="100">15%</progress>
+        <progress class="htmx-indicator progress is-small is-success" max="100">15%</progress>
     {% endif %}
 {% endblock %}

{accrete-0.0.38 → accrete-0.0.40}/accrete/contrib/ui/templatetags/accrete_ui.py

@@ -1,4 +1,5 @@
 import logging
+import operator
 from django import template
 from django.db.models import Model
 from django.core.cache import cache
@@ -28,18 +29,25 @@ def combine_templates(template_name, request=None):
 
 @register.filter(name='get_attr')
 def get_attr_from_string(param: object, value: str):
-    attr_name = value.split('__')
-    attr = param
-    for name in attr_name:
-        try:
-            attr = getattr(attr, name)
-        except AttributeError:
-            _logger.exception(f'Object {attr} has no attribute {name}')
-            return ''
+    try:
+        attr = operator.attrgetter(value)(param)
+    except AttributeError:
+        _logger.exception(f'Object {param} has no attribute {value}')
+        return ''
+    if attr is None:
+        return None
     if callable(attr):
-        return attr()
+        return mark_safe(attr())
     else:
-        return attr
+        return mark_safe(attr)
+
+
+@register.filter(name='related_obj_url')
+def related_obj_url(param: object, value: str):
+    attr = getattr(param, value, False)
+    if attr:
+        return attr.get_absolute_url()
+    return '#'
 
 
 @register.filter(name='message_class')

{accrete-0.0.38 → accrete-0.0.40}/accrete/contrib/user/views.py

@@ -1,15 +1,12 @@
-from django.utils import timezone
 from django.contrib.auth.forms import AuthenticationForm
 from django.contrib.auth import views, update_session_auth_hash
 from django.contrib.auth.decorators import login_required
-from django.contrib.sessions.models import Session
 from django.contrib import messages
-from django.db.models import Q
 from django.shortcuts import redirect, render, reverse, resolve_url
 from django.utils.translation import gettext_lazy as _
 from django.conf import settings
 
-from accrete.forms import save_form
+from accrete.utils import save_form
 from accrete.contrib import ui
 from .forms import UserForm, ChangePasswordForm, ChangeEmailForm
 

{accrete-0.0.38 → accrete-0.0.40}/accrete/forms.py

@@ -290,22 +290,3 @@ class TenantModelForm(One2ManyModelForm):
             self.save_o2m()
 
         return self.instance
-
-
-def save_form(form, reraise=False):
-    form.is_saved = False
-    form.save_error = None
-    form.save_error_id = None
-    try:
-        if form.is_valid():
-            with transaction.atomic():
-                form.save()
-            form.is_saved = True
-    except Exception as e:
-        form.save_error = repr(e)
-        error_id = str(uuid4())[:8]
-        _logger.exception(f'{error_id}: {e}')
-        form.save_error_id = error_id
-        if reraise:
-            raise e
-    return form

accrete-0.0.40/accrete/managers.py

@@ -0,0 +1,35 @@
+from django.db import models
+from accrete.tenant import get_tenant
+from accrete.annotation import AnnotationManagerMixin
+
+
+class TenantManager(models.Manager, AnnotationManagerMixin):
+
+    def get_queryset(self):
+        queryset = super().get_queryset()
+        tenant = get_tenant()
+        if tenant:
+            queryset = queryset.filter(tenant=tenant)
+        return queryset.annotate(**self.get_annotations())
+
+    def bulk_create(
+            self,
+            objs,
+            batch_size=None,
+            ignore_conflicts=False,
+            update_conflicts=False,
+            update_fields=None,
+            unique_fields=None,
+    ):
+        tenant = get_tenant()
+        if tenant is None and not all(obj.tenant_id for obj in objs):
+            raise ValueError(
+                'Tenant must be set for all objects when calling bulk_create'
+            )
+        for obj in objs:
+            obj.tenant_id = tenant.pk
+        return super().bulk_create(
+            objs, batch_size=batch_size, ignore_conflicts=ignore_conflicts,
+            update_conflicts=update_conflicts, update_fields=update_fields,
+            unique_fields=unique_fields
+        )

{accrete-0.0.38 → accrete-0.0.40}/accrete/models.py

@@ -3,39 +3,8 @@ from django.conf import settings
 from django.utils.translation import gettext_lazy as _
 from django.contrib.auth.validators import UnicodeUsernameValidator
 from accrete.tenant import get_tenant
-from accrete.annotation import AnnotationModelMixin, AnnotationManagerMixin
-
-
-class TenantManager(models.Manager, AnnotationManagerMixin):
-
-    def get_queryset(self):
-        queryset = super().get_queryset()
-        tenant = get_tenant()
-        if tenant:
-            queryset = queryset.filter(tenant=tenant)
-        return self.add_annotations(queryset)
-
-    def bulk_create(
-            self,
-            objs,
-            batch_size=None,
-            ignore_conflicts=False,
-            update_conflicts=False,
-            update_fields=None,
-            unique_fields=None,
-    ):
-        tenant = get_tenant()
-        if tenant is None and not all(obj.tenant_id for obj in objs):
-            raise ValueError(
-                'Tenant must be set for all objects when calling bulk_create'
-            )
-        for obj in objs:
-            obj.tenant_id = tenant.pk
-        return super().bulk_create(
-            objs, batch_size=batch_size, ignore_conflicts=ignore_conflicts,
-            update_conflicts=update_conflicts, update_fields=update_fields,
-            unique_fields=unique_fields
-        )
+from accrete.annotation import AnnotationModelMixin
+from accrete.managers import TenantManager
 
 
 class TenantModel(models.Model, AnnotationModelMixin):

accrete-0.0.40/accrete/utils/__init__.py

@@ -0,0 +1,4 @@
+from . import dates
+from .forms import save_form
+from .http import filter_from_querystring
+from .models import get_related_model

accrete-0.0.40/accrete/utils/forms.py

@@ -0,0 +1,24 @@
+import logging
+from uuid import uuid4
+from django.db import transaction
+
+_logger = logging.getLogger(__name__)
+
+
+def save_form(form, reraise=False):
+    form.is_saved = False
+    form.save_error = None
+    form.save_error_id = None
+    try:
+        if form.is_valid():
+            with transaction.atomic():
+                form.save()
+            form.is_saved = True
+    except Exception as e:
+        form.save_error = repr(e)
+        error_id = str(uuid4())[:8]
+        _logger.exception(f'{error_id}: {e}')
+        form.save_error_id = error_id
+        if reraise:
+            raise e
+    return form