Add "updated_at" field to Orders model

[jmerro-c3d]

This PR adds an extra field to the purchase order model to show when it or any of the line items were last updated.

Was writing a PO sync service between InvenTree and another software and wanted a way to check if a purchase order was modified since the last sync job.

[netlify]

✅ Deploy Preview for inventree-web-pui-preview ready!

Name	Link
🔨 Latest commit	10072fd
🔍 Latest deploy log	https://app.netlify.com/projects/inventree-web-pui-preview/deploys/699ee1fc3175cc00081df494
😎 Deploy Preview	https://deploy-preview-11374--inventree-web-pui-preview.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
Lighthouse	1 paths audited Performance: 94 (🟢 up 1 from production) Accessibility: 81 (no change from production) Best Practices: 100 (no change from production) SEO: 78 (no change from production) PWA: - View the detailed breakdown and full score reports

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[SchrodingersGat]

@jmerro-c3d I like this idea, notes below:

• This should be added to the Order generic model, so that it applies to SalesOrder and ReturnOrder also
• The signal hooks should then accommodate those orders too
• API endpoints should be able to order the results by this field
• The "purchase order" table (and other order tables) should display this in a column
• That column(s) should be sortable
• Also add API filters for before and after date ranges (see existing examples of this)
• The purchase order detail page in the UI should display this date

[jmerro-c3d]

@jmerro-c3d I like this idea, notes below:

• This should be added to the Order generic model, so that it applies to SalesOrder and ReturnOrder also
• The signal hooks should then accommodate those orders too
• API endpoints should be able to order the results by this field
• The "purchase order" table (and other order tables) should display this in a column
• That column(s) should be sortable
• Also add API filters for before and after date ranges (see existing examples of this)
• The purchase order detail page in the UI should display this date

@SchrodingersGat Thanks, does this work for you? ... If you don't want to show the updated_at time in the details I can remove that.

Is there something else I need to do to generate the API schema or change tests?

[SchrodingersGat]

Is there something else I need to do to generate the API schema or change tests?

API

Simply bump the API version in ./src/backend/InvenTree/InvenTree/api_version.py - according to the format already there

Tests

Add some unit tests to ensure that:

• The updated_at field is correctly set when the order is saved
• The updated_at field is correctly set when any line item is added / updated / deleted

[codspeed-hq]

Merging this PR will improve performance by 6.81%

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

⚡ 2 improved benchmarks
✅ 42 untouched benchmarks

Performance Changes

	Mode	Benchmark	BASE	HEAD	Efficiency
⚡	WallTime	test_search_performance[purchaseorder]	25.8 ms	24.3 ms	+6.02%
⚡	WallTime	test_api_options_performance[/api/stock/location/]	25.4 ms	23.8 ms	+6.81%

---

_{Comparing jmerro-c3d:po-updated-at-field (10072fd) with master (ac9a1f2)}

[codecov]

Codecov Report

❌ Patch coverage is 99.00990% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 88.01%. Comparing base (edc639d) to head (91f50f7).

Additional details and impacted files

@@            Coverage Diff             @@
##           master   #11374      +/-   ##
==========================================
+ Coverage   87.99%   88.01%   +0.01%     
==========================================
  Files        1295     1296       +1     
  Lines       58806    58901      +95     
  Branches     1939     1939              
==========================================
+ Hits        51745    51839      +94     
- Misses       6579     6580       +1     
  Partials      482      482              

Flag	Coverage Δ
backend	89.20% <99.00%> (+0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components	Coverage Δ
Backend Apps	91.71% <99.00%> (+0.01%)	⬆️
Backend General	93.37% <ø> (ø)
Frontend	70.85% <ø> (ø)

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[SchrodingersGat]

Is this going to update every existing order to have an updated_at time which corresponds to the time of the database update to add this column?

Please check this - when you run invoke migrate - do the existing entries in the database get filled with a constant value, or null?

[jmerro-c3d]

Ah yep, will change them to be null by default. When ordering by "updated" DESC do you want the ones with null to be at the bottom or top of the list? By default they are at the top.

[SchrodingersGat]

Is there a way to control that?

[jmerro-c3d]

This works for me:

diff --git a/src/backend/InvenTree/InvenTree/filters.py b/src/backend/InvenTree/InvenTree/filters.py
index 78c5c5e45..d87395dd7 100644
--- a/src/backend/InvenTree/InvenTree/filters.py
+++ b/src/backend/InvenTree/InvenTree/filters.py
@@ -4,6 +4,7 @@ from datetime import datetime
 
 from django.conf import settings
 from django.core.exceptions import ValidationError
+from django.db.models import F
 from django.utils import timezone
 from django.utils.timezone import make_aware
 
@@ -114,9 +115,10 @@ class InvenTreeOrderingFilter(filters.OrderingFilter):
         ordering = super().get_ordering(request, queryset, view)
 
         aliases = getattr(view, 'ordering_field_aliases', None)
+        nulls_last = getattr(view, 'nulls_last_fields', set())
 
         # Attempt to map ordering fields based on provided aliases
-        if ordering is not None and aliases is not None:
+        if ordering is not None and (aliases is not None or nulls_last):
             """Ordering fields should be mapped to separate fields."""
 
             ordering_initial = ordering
@@ -129,7 +131,7 @@ class InvenTreeOrderingFilter(filters.OrderingFilter):
                     field = field[1:]
 
                 # Are aliases defined for this field?
-                alias = aliases.get(field, field)
+                alias = aliases.get(field, field) if aliases else field
 
                 """
                 Potentially, a single field could be "aliased" to multiple field,
@@ -153,10 +155,14 @@ class InvenTreeOrderingFilter(filters.OrderingFilter):
                     continue
 
                 for a in alias:
-                    if reverse:
-                        a = '-' + a
-
-                    ordering.append(a)
+                    if a in nulls_last:
+                        ordering.append(
+                            F(a).desc(nulls_last=True)
+                            if reverse
+                            else F(a).asc(nulls_last=True)
+                        )
+                    else:
+                        ordering.append(('-' + a) if reverse else a)
 
         return ordering

diff --git a/src/backend/InvenTree/order/api.py b/src/backend/InvenTree/order/api.py
index a7bdee756..a3723a4ef 100644
--- a/src/backend/InvenTree/order/api.py
+++ b/src/backend/InvenTree/order/api.py
@@ -431,6 +431,8 @@ class PurchaseOrderList(
         'updated_at',
     ]
 
+    nulls_last_fields = {'updated_at'}
+
     ordering = '-reference'
 
 
@@ -894,6 +896,8 @@ class SalesOrderList(
         'updated_at',
     ]
 
+    nulls_last_fields = {'updated_at'}
+
     search_fields = [
         'customer__name',
         'reference',
@@ -1562,6 +1566,8 @@ class ReturnOrderList(
         'updated_at',
     ]
 
+    nulls_last_fields = {'updated_at'}
+
     search_fields = [
         'customer__name',
         'reference',

[SchrodingersGat]

@jmerro-c3d interesting, I have been wondering how to control that for a while. It would be useful to submit that as a separate PR, but please keep it out of scope for this one.

[SchrodingersGat]

Looks like you've left some git cruft in here - please clean this up!

[jmerro-c3d]

Oh, weird... Resolved in Github editor and it... didn't resolve?? whatever, fixed now.

[SchrodingersGat]

Sorry, API version has got out of date again

[jmerro-c3d]

Sorry, API version has got out of date again

All good, fixed.

[jmerro-c3d]

aw, playright tests job timed out downloading deb packages