django-fast-treenode 3.0.0__py3-none-any.whl → 3.0.2__py3-none-any.whl

{django_fast_treenode-3.0.0.dist-info → django_fast_treenode-3.0.2.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: django-fast-treenode
-Version: 3.0.0
+Version: 3.0.2
 Summary: Treenode Framework for supporting tree (hierarchical) data structure in Django projects
 Home-page: https://django-fast-treenode.readthedocs.io/
 Author: Timur Kady
@@ -27,6 +27,7 @@ License: MIT License
         OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
         SOFTWARE.
         
+        
 Project-URL: Homepage, https://github.com/TimurKady/django-fast-treenode
 Project-URL: Documentation, https://django-fast-treenode.readthedocs.io/
 Project-URL: Source, https://github.com/TimurKady/django-fast-treenode

{django_fast_treenode-3.0.0.dist-info → django_fast_treenode-3.0.2.dist-info}/RECORD

@@ -1,14 +1,14 @@
-django_fast_treenode-3.0.0.dist-info/licenses/LICENSE,sha256=T0evsb7y-63fg18ovdNSx3wwWWRwyluQvN9J4zFSvfE,1093
+django_fast_treenode-3.0.2.dist-info/licenses/LICENSE,sha256=SSYqS84FCnAW7tAxmjBKU8qAa8Jv4VGPuSSGeHwWtJE,1095
 treenode/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-treenode/apps.py,sha256=Q4dhmaDYGuq-Z3bzxPyFmgI0izpZXJNEnyGt5IFKJ4c,1833
+treenode/apps.py,sha256=QlwjNDM9rkUoWB8Vm8-OkS6lNx0-aTByuGZlu9wrQMs,1832
 treenode/cache.py,sha256=2jUiiecfFxwB7QFukpU4u0FnDzGH6hNRfo6KAYvs6vM,8447
 treenode/forms.py,sha256=V-upmbYSW1BbuXdSBGExHxw_j5TTUheaHvreK9tSGTE,3155
 treenode/settings.py,sha256=oSkcKXNVd28HXrlWZIH2VYinCMq-UdCDlX4KD0Qc_Xk,631
 treenode/signals.py,sha256=ERrlKjGqhYaPYVKKRk1JBBlPFOmJKpJ6bXsJavcTlo0,518
 treenode/tests.py,sha256=2uDafv3Ns6f7Vy1ekUtgYxCZEi1KRyesZDTAFhYcX-E,63
 treenode/urls.py,sha256=krHvVigc_dxC0z5hEd2rgeH6th8jW7qJY3Qbia-419Y,240
-treenode/version.py,sha256=ilteqE3d-JmmZYyCo0giDxr9qKM3IjC-pyTSxUSEodA,222
-treenode/widgets.py,sha256=VItPvN9XgaSRI_MZjKEmtaHDJcn2bDIQIppwKjXmYQM,4017
+treenode/version.py,sha256=TN9nZNpy8llm9fqIJAWYPQd8TagW5fxnW0I69gWD-oY,220
+treenode/widgets.py,sha256=61ed16bVqb1_R97jekDrbKS5pDVK4nzXSDwL3CDBYEk,4075
 treenode/admin/__init__.py,sha256=XNEYHdF5lKb0vpdlVxdR2fxj5oUgzyx1YyCwsv0gxHw,100
 treenode/admin/admin.py,sha256=xhf6tT5Ydn6i_upUDA1A6TN_muNndCko8gzM_fNIex4,7707
 treenode/admin/changelist.py,sha256=KUYS9MaR8Ck_1xmMqupobxWKarrJEqmHuEG32CL01Bo,1662
@@ -17,7 +17,7 @@ treenode/admin/importer.py,sha256=hK3D-1DZcoowGblRluGzng3n5Bf__hMsbNaIGXRpRdg,62
 treenode/admin/mixin.py,sha256=yXcSpBEfoMYT7tuAbHhGbuqlVQkCR5RizW2bNWJ0QNM,10639
 treenode/managers/__init__.py,sha256=c7F9Ku9489Hv6lTpUY2nbyBlWFCXBWAkNBm4xTKcjL8,186
 treenode/managers/managers.py,sha256=8OaFxtajyR1d7-UHyiUbifMBEF9cjfHTIEYPkYUWmt0,7166
-treenode/managers/queries.py,sha256=rO-zTaIj9EQt1jrQF0lbAQeGgdvHs29gpKR6mfUX2d4,9237
+treenode/managers/queries.py,sha256=Kepax8SDn7G5tOlPRWBCp5Oyp49O5iMITCMBoNCm_Ak,10655
 treenode/managers/tasks.py,sha256=gqZfqeOrdPtTJZ1q83aCFh6htfxP0XMTOkd1KWL9PVU,5227
 treenode/models/__init__.py,sha256=iR4ksCKoayvkIWWgGk6OUGHZC3D0mzAtgdBcS2vQPBw,188
 treenode/models/decorators.py,sha256=N2dcnWqSCiEXDcYCf0zVijrbGUC8kYlqOLi_GKFmECU,1457
@@ -26,7 +26,7 @@ treenode/models/models.py,sha256=4S4SFFSxuIlY1DjuGW2T_bJaOos9chp1cuJ5k4_4gPg,122
 treenode/models/mixins/__init__.py,sha256=aALVKMGAWbgMAeKWS6s-NF3L5FmRX96mQxtpthOX-Ec,805
 treenode/models/mixins/ancestors.py,sha256=9g-0nPHoiF_SX2kN4uDLdbWyw-TDCz1YqxLJngwTZOQ,1971
 treenode/models/mixins/children.py,sha256=H9iMqgucSmwLX-3O3QUj1a2PUQTmmWZ4GPPjRZ9a5E4,2399
-treenode/models/mixins/descendants.py,sha256=2iL3ofbr4KjCeBId7gotDBBLUU8Z2FvLMHtPjPYUUUc,1972
+treenode/models/mixins/descendants.py,sha256=RKowr29JUUO3E_UDvbLbMLhbn5IUkY2eh3AXfU8XSE8,2165
 treenode/models/mixins/family.py,sha256=MB5kWRVvxU_xmSgCekveTP5Vhj4wJki8bU7hzn9RNLE,1673
 treenode/models/mixins/logical.py,sha256=gh5wv5XZDs5GWarU6g9zKXWNwji7SE3zSVNIpywDWjw,2190
 treenode/models/mixins/node.py,sha256=8kqYdFTyZK5WsSxguuX3jBo1nZehDmwIBjm38S6QJw4,8576
@@ -72,19 +72,19 @@ treenode/templates/admin/treenode_rows.html,sha256=S1XtZXWMUdfgjDoKG7OJwZg81a_5I
 treenode/templates/widgets/tree_widget.html,sha256=GKcCU-B2FkkJ2BSOuXOw9e_PdYTtADcvyITEXqOlZ9Y,723
 treenode/utils/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 treenode/utils/db/__init__.py,sha256=RwicAcJSI1nhIPWLdT7j9TFsgOc9834VDn9lVn54GlY,247
-treenode/utils/db/compiler.py,sha256=jvrPE26dDDaPzcMqpQPx3JWtjKrgSyI5F8aa4-2RRzc,3854
+treenode/utils/db/compiler.py,sha256=PgD9ybS5H8OUHw1gkFBQHhnrf5HiCx8QXUMRhydwh7o,3824
 treenode/utils/db/db_vendor.py,sha256=4SyEHl51jVCDB3is4omHCf2bTB_QV3RvemUQYxJP5m0,930
 treenode/utils/db/service.py,sha256=PF85Yhz2xUWFFCzpLYotmiNTZXXEH61rhswslSxEUds,2640
-treenode/utils/db/sqlcompat.py,sha256=tXFgaVy0IavdJ6FAMW7kKW0JyEAm-z_qka8XGWzzM6Q,1649
+treenode/utils/db/sqlcompat.py,sha256=K71ggkKIvpdTtHQ6Y4qcbo6cj2eYiEfy6DlVBr8Po1E,4460
 treenode/utils/db/sqlquery.py,sha256=KXcfKbbaBF-D134H_2DiPQtjedR79SJNXPJc0msZYEc,1938
 treenode/views/__init__.py,sha256=ppxbBx51TUaKstJFpAd_DTmbKjbZGmVMLNYSpgUKnd0,111
 treenode/views/autoapi.py,sha256=o75e8IFsogbhZN_rbx3BKVnoruD96nWelnC5UzOqUDw,3628
 treenode/views/autocomplete.py,sha256=Z7cBnC4Ihdyxm8zlbnG6CkZdVkM3TOTWRpw5mdhaIVA,1469
 treenode/views/children.py,sha256=bygXaEBExxG3zIPL34_PYHLFFIqlQU2naqPIlyQ6e-s,1152
 treenode/views/common.py,sha256=mrmr40R91XVbMWcz5GZT-OjpnQ87F7XQZxu1W6rqpqI,617
-treenode/views/crud.py,sha256=fWuSKCj_m0tqqDWTCscX0r2Kh34-cMunm5pFdEQuu_A,6267
+treenode/views/crud.py,sha256=RI5rdyD4hZTszjZFThByxi_lkAeJlqbDCXFkD8iyzKE,7424
 treenode/views/search.py,sha256=c_GyooT3jyoNa96bBxfoWruRN1wIw-ZGYvwGKkGojTs,1501
-django_fast_treenode-3.0.0.dist-info/METADATA,sha256=ZI4c88wl_4Iyo6Znl8lCL_dmy3AivWETuq6zS4eQcrQ,10245
-django_fast_treenode-3.0.0.dist-info/WHEEL,sha256=ck4Vq1_RXyvS4Jt6SI0Vz6fyVs4GWg7AINwpsaGEgPE,91
-django_fast_treenode-3.0.0.dist-info/top_level.txt,sha256=fmgxHbXyx1O2MPi_9kjx8aL9L-8TmV0gre4Go8XgqFk,9
-django_fast_treenode-3.0.0.dist-info/RECORD,,
+django_fast_treenode-3.0.2.dist-info/METADATA,sha256=Q5sQAXL8xAh-5T9r36G5IsxzvFiQJhK0UxG8xm-xC4o,10255
+django_fast_treenode-3.0.2.dist-info/WHEEL,sha256=wXxTzcEDnjrTwFYjLPcsW_7_XihufBwmpiBeiXNBGEA,91
+django_fast_treenode-3.0.2.dist-info/top_level.txt,sha256=fmgxHbXyx1O2MPi_9kjx8aL9L-8TmV0gre4Go8XgqFk,9
+django_fast_treenode-3.0.2.dist-info/RECORD,,

{django_fast_treenode-3.0.0.dist-info → django_fast_treenode-3.0.2.dist-info}/WHEEL

@@ -1,5 +1,5 @@
 Wheel-Version: 1.0
-Generator: setuptools (80.0.0)
+Generator: setuptools (80.1.0)
 Root-Is-Purelib: true
 Tag: py3-none-any
 

{django_fast_treenode-3.0.0.dist-info → django_fast_treenode-3.0.2.dist-info}/licenses/LICENSE

@@ -19,3 +19,4 @@ AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
 LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
 OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 SOFTWARE.
+

treenode/apps.py

@@ -18,7 +18,7 @@ class TreeNodeConfig(AppConfig):
     """Config Class."""
 
     default_auto_field = "django.db.models.BigAutoField"
-    name = "supertree"
+    name = "treenode"
 
     def ready(self):
         """Ready method."""

treenode/managers/queries.py

@@ -84,6 +84,48 @@ class TreeQuery:
             sql = self.order_by(sql1, "priority")
             return sql, params1
 
+    def get_descendants(self, include_self, depth):
+        """
+        Build SQL for the 'descendants'.
+
+        Relationship using startswith-like logic.
+        Avoids locale-dependent string comparison issues by relying on
+        _path LIKE 'xxx.%'.
+        """
+        like_pattern = self.node._path + '.%'  # emulate startswith
+
+        base_sql = f"""
+            SELECT id, _depth, priority
+            FROM {self.db_table}
+            WHERE _path LIKE %s
+        """
+        params = [like_pattern]
+
+        if depth is not None:
+            depth_val = getattr(self.node, "_depth", None)
+            if depth_val is None:
+                depth_val = type(self.node).objects.values_list(
+                    "_depth", flat=True).get(pk=self.node.pk)
+            base_sql += " AND _depth <= %s"
+            params.append(depth_val + depth)
+
+        if include_self:
+            sql_self = f"""
+                SELECT id, _depth, priority
+                FROM {self.db_table}
+                WHERE id = %s
+            """
+            union_sql, union_params = self.wrap_union_all([
+                (base_sql, params),
+                (sql_self, [self.node.pk])
+            ])
+        else:
+            union_sql, union_params = base_sql, params
+
+        union_sql = self.order_by(union_sql, "_depth, priority")
+        return union_sql, union_params
+
+    '''
     def get_descendants(self, include_self, depth):
         """
         Build SQL for the 'descendants' relationship.
@@ -113,6 +155,7 @@ class TreeQuery:
 
         union_sql = self.order_by(union_sql, "_depth, priority")
         return union_sql, union_params
+    '''
 
     def get_ancestors(self, include_self):
         """

treenode/models/mixins/descendants.py

@@ -32,17 +32,23 @@ class TreeNodeDescendantsMixin(models.Model):
     def get_descendants_queryset(self, include_self=False, depth=None):
         """Get the descendants queryset."""
         path = self.get_order()  # calls refresh and gets the current _path
-        from_path = path + '.'
-        to_path = path + '/'
-        options = {'_path__gte': from_path, '_path__lt': to_path}
-        if depth:
-            options["_depth__lt"] = depth
-        queryset = self._meta.model.objects.filter(**options)
-
-        if include_self:
-            return self._meta.model.objects.filter(pk=self.pk) | queryset
-        else:
-            return queryset
+
+        # from_path = path + '.'
+        # to_path = path + '/'
+
+        # options = {'_path__gte': from_path, '_path__lt': to_path}
+        # if depth:
+        #    options["_depth__lt"] = depth
+        # queryset = self._meta.model.objects.filter(**options)
+        # if include_self:
+        #     return self._meta.model.objects.filter(pk=self.pk) | queryset
+        # else:
+        #    return queryset
+
+        suffix = "" if include_self else '.'
+        path += suffix
+        queryset = self._meta.model.objects.filter(_path__startswith=path)
+        return queryset
 
     @cached_method
     def get_descendants_pks(self, include_self=False, depth=None):

treenode/utils/db/compiler.py

@@ -5,7 +5,7 @@ Tree update task compiler class.
 Compiles tasks to low-level SQL to update the materialized path (_path), depth
 (_depth), and node order (priority) when they are shifted or moved.
 
-Version: 3.0.0
+Version: 3.1.0
 Author: Timur Kady
 Email: timurkady@yandex.com
 """
@@ -34,9 +34,11 @@ class TreePathCompiler:
         _depth) are recalculated.
         """
         db_table = model._meta.db_table
+        # Will eliminate the risk if the user names the model order or user.
+        qname = connection.ops.quote_name(db_table)
 
         sorting_field = model.sorting_field
-        sorting_fields = ["priority", "id"] if sorting_field == "priority" else [sorting_field]  # noqa: D501
+        sorting_fields = ["priority", "id"] if sorting_field == "priority" else [sorting_field]  # noqa: D5017
         sort_expr = ", ".join([
             f"c.{field}" if "." not in field else field
             for field in sorting_fields
@@ -44,7 +46,7 @@ class TreePathCompiler:
 
         cte_header = "(id, parent_id, new_priority, new_path, new_depth)"
 
-        row_number_expr = "ROW_NUMBER() OVER (ORDER BY {sort_expr}) - 1"
+        row_number_expr = f"ROW_NUMBER() OVER (ORDER BY {sort_expr}) - 1"
         hex_expr = SQLCompat.to_hex(row_number_expr)
         lpad_expr = SQLCompat.lpad(hex_expr, SEGMENT_LENGTH, "'0'")
 
@@ -57,7 +59,7 @@ class TreePathCompiler:
                     {row_number_expr} AS new_priority,
                     {new_path_expr} AS new_path,
                     0 AS new_depth
-                FROM {db_table} AS c
+                FROM {qname} AS c
                 WHERE c.parent_id IS NULL
             """
             params = []
@@ -70,16 +72,18 @@ class TreePathCompiler:
                     {row_number_expr} AS new_priority,
                     {path_expr} AS new_path,
                     p._depth + 1 AS new_depth
-                FROM {db_table} c
-                JOIN {db_table} p ON c.parent_id = p.id
+                FROM {qname} c
+                JOIN {qname} p ON c.parent_id = p.id
                 WHERE p.id = %s
             """
             params = [parent_id]
 
-        recursive_row_number_expr = "ROW_NUMBER() OVER (PARTITION BY c.parent_id ORDER BY {sort_expr}) - 1"   # noqa: D501
+        recursive_row_number_expr = f"ROW_NUMBER() OVER (PARTITION BY c.parent_id ORDER BY {sort_expr}) - 1"
         recursive_hex_expr = SQLCompat.to_hex(recursive_row_number_expr)
-        recursive_lpad_expr = SQLCompat.lpad(recursive_hex_expr, SEGMENT_LENGTH, "'0'")   # noqa: D501
-        recursive_path_expr = SQLCompat.concat("t.new_path", "'.'", recursive_lpad_expr)   # noqa: D501
+        recursive_lpad_expr = SQLCompat.lpad(
+            recursive_hex_expr, SEGMENT_LENGTH, "'0'")
+        recursive_path_expr = SQLCompat.concat(
+            "t.new_path", "'.'", recursive_lpad_expr)
 
         recursive_sql = f"""
             SELECT
@@ -88,27 +92,22 @@ class TreePathCompiler:
                 {recursive_row_number_expr} AS new_priority,
                 {recursive_path_expr} AS new_path,
                 t.new_depth + 1 AS new_depth
-            FROM {db_table} c
+            FROM {qname} c
             JOIN tree_cte t ON c.parent_id = t.id
         """
 
-        final_sql = f"""
-            WITH RECURSIVE tree_cte {cte_header} AS (
-                {base_sql}
-                UNION ALL
-                {recursive_sql}
-            )
-            UPDATE {db_table} AS orig
-            SET
-                priority = t.new_priority,
-                _path = t.new_path,
-                _depth = t.new_depth
-            FROM tree_cte t
-            WHERE orig.id = t.id;
-        """
+        final_sql = SQLCompat.update_from(
+            db_table=db_table,
+            cte_header=cte_header,
+            base_sql=base_sql,
+            recursive_sql=recursive_sql,
+            update_fields=["priority", "_path", "_depth"]
+        )
 
         with connection.cursor() as cursor:
-            cursor.execute(final_sql.format(sort_expr=sort_expr), params)
+            # Make params read-only
+            params = tuple(params)
+            cursor.execute(final_sql, params)
 
 
 # The End

treenode/utils/db/sqlcompat.py

@@ -17,11 +17,12 @@ Instead of LPAD(...)
 old: LPAD(...)
 new: SQLCompat.lpad(...)
 
-Version: 3.0.0
+Version: 3.1.0
 Author: Timur Kady
 Email: timurkady@yandex.com
 """
 
+from django.db import connection
 from .db_vendor import is_mysql, is_mariadb, is_sqlite, is_mssql
 from ...settings import TREENODE_PAD_CHAR
 
@@ -42,19 +43,103 @@ class SQLCompat:
 
     @staticmethod
     def to_hex(value):
-        """Convert integer to hexadecimal string."""
+        """Convert integer to uppercase hexadecimal string."""
         if is_sqlite():
-            return f"printf('%x', {value})"
+            return f"UPPER(printf('%x', {value}))"
+        elif is_mysql() or is_mariadb():
+            return f"UPPER(CONV({value}, 10, 16))"
         else:
-            return f"TO_HEX({value})"
+            return f"UPPER(TO_HEX({value}))"
 
     @staticmethod
     def lpad(value, length, char=TREENODE_PAD_CHAR):
         """Pad string to the specified length."""
         if is_sqlite():
-            return (f"substr(replace(hex(zeroblob({length})), '00', {char}), "
-                    f"1, {length} - length({value})) || {value}")
+            return f"printf('%0{length}s', {value})"
         else:
             return f"LPAD({value}, {length}, {char})"
 
+    @staticmethod
+    def update_from(db_table, cte_header, base_sql, recursive_sql, update_fields):
+        """
+        Generate final SQL for updating via recursive CTE.
+
+        PostgreSQL uses UPDATE ... FROM.
+        Other engines use vendor-specific strategies.
+        """
+        qt = connection.ops.quote_name(db_table)
+        def qf(f): return connection.ops.quote_name(f)
+
+        cte_alias = {
+            "priority": "new_priority",
+            "_path": "new_path",
+            "_depth": "new_depth",
+        }
+
+        if connection.vendor == "postgresql":
+            set_clause = ", ".join(
+                f"{qf(f)} = t.{cte_alias.get(f, f)}" for f in update_fields
+            )
+            return f"""
+                WITH RECURSIVE tree_cte {cte_header} AS (
+                    {base_sql}
+                    UNION ALL
+                    {recursive_sql}
+                )
+                UPDATE {qt} AS orig
+                SET {set_clause}
+                FROM tree_cte t
+                WHERE orig.id = t.id;
+            """
+
+        elif connection.vendor in {"microsoft", "mssql"}:
+            set_clause = ", ".join(
+                f"{qt}.{f} = t.{f}" for f in update_fields
+            )
+            return f"""
+                WITH tree_cte {cte_header} AS (
+                    {base_sql}
+                    UNION ALL
+                    {recursive_sql}
+                )
+                UPDATE orig
+                SET {set_clause}
+                FROM {qt} AS orig
+                JOIN tree_cte t ON orig.id = t.id;
+            """
+
+        elif connection.vendor == "oracle":
+            set_clause = ", ".join(
+                f"orig.{f} = t.{f}" for f in update_fields
+            )
+            return f"""
+                WITH tree_cte {cte_header} AS (
+                    {base_sql}
+                    UNION ALL
+                    {recursive_sql}
+                )
+                MERGE INTO {qt} orig
+                USING tree_cte t
+                ON (orig.id = t.id)
+                WHEN MATCHED THEN UPDATE SET
+                    {set_clause};
+            """
+
+        else:
+            set_clause = ", ".join(
+                f"{qf(f)} = (SELECT t.{f} FROM tree_cte t WHERE t.id = {qt}.id)"
+                for f in update_fields
+            )
+            where_clause = f"id IN (SELECT id FROM tree_cte)"
+            return f"""
+                WITH RECURSIVE tree_cte {cte_header} AS (
+                    {base_sql}
+                    UNION ALL
+                    {recursive_sql}
+                )
+                UPDATE {qt}
+                SET {set_clause}
+                WHERE {where_clause};
+            """
+
 # The End

treenode/version.py

@@ -4,10 +4,9 @@ TreeNode Version Module
 
 This module defines the current version of the TreeNode package.
 
-Version: 3.0.0
+Version: 3.0.2
 Author: Timur Kady
 Email: timurkady@yandex.com
 """
 
-
-__version__ = '3.0.0'
+__version__ = '3.0.2'

treenode/views/crud.py

@@ -10,14 +10,17 @@ Email: timurkady@yandex.com
 """
 
 import json
-
+import logging
+from django.core.exceptions import ValidationError
 from django.forms.models import model_to_dict
 from django.http import (
     JsonResponse, HttpResponseBadRequest, HttpResponseNotFound,
 )
-
+from django.utils.translation import gettext_lazy as _
 from django.views import View
 
+logger = logging.getLogger("treenode.views.crud")
+
 
 class TreeNodeBaseAPIView(View):
     """Simple API View for TreeNode-based models."""
@@ -128,8 +131,14 @@ class TreeNodeBaseAPIView(View):
             obj.full_clean()
             obj.save()
             return JsonResponse(model_to_dict(obj), status=201)
-        except Exception as e:
-            return HttpResponseBadRequest(str(e))
+        except ValidationError as ve:
+            # give the client clear field errors
+            return JsonResponse({"errors": ve.message_dict}, status=400)
+        except Exception:
+            # log full information for development
+            logger.exception("Failed to create node")
+            return JsonResponse(
+                {"error": _("Failed to create node")}, status=400)
 
     def put(self, request, pk):
         """
@@ -155,10 +164,16 @@ class TreeNodeBaseAPIView(View):
             obj.full_clean()
             obj.save()
             return JsonResponse(model_to_dict(obj))
+        except ValidationError as ve:
+            return JsonResponse({"errors": ve.message_dict}, status=400)
         except self.model.DoesNotExist:
-            return HttpResponseNotFound("Node not found.")
-        except Exception as e:
-            return HttpResponseBadRequest(str(e))
+            # give the client clear field errors
+            return HttpResponseNotFound(_("Node not found (pk={pk})."))
+        except Exception:
+            # log full information for development
+            logger.exception("Error replacing node %s", pk)
+            return JsonResponse(
+                {"error": _("Failed to update node")}, status=400)
 
     def patch(self, request, pk):
         """
@@ -200,10 +215,16 @@ class TreeNodeBaseAPIView(View):
                 "id": obj.pk,
                 "cascade": cascade
             })
+        except ValidationError as ve:
+            # give the client clear field errors|
+            return JsonResponse({"errors": ve.message_dict}, status=400)
         except self.model.DoesNotExist:
-            return HttpResponseNotFound("Node not found.")
-        except Exception as e:
-            return HttpResponseBadRequest(str(e))
+            return HttpResponseNotFound(_("Node not found (pk={pk})."))
+        except Exception:
+            # log full information for development
+            logger.exception("Error deleting node %s", pk)
+            return JsonResponse(
+                {"error": _("Error deleting node")}, status=400)
 
 
 # The End

treenode/widgets.py

@@ -21,6 +21,8 @@ from django.utils.translation import gettext_lazy as _
 class TreeWidget(forms.Widget):
     """Custom widget for hierarchical tree selection."""
 
+    template_name = "django/forms/widgets/select.html"
+
     class Media:
         """Meta class to define required CSS and JS files."""