Use last successful task for classifier last trained

src-ui/messages.xlf

@@ -4519,10 +4519,6 @@
           <context context-type="sourcefile">src/app/components/common/system-status-dialog/system-status-dialog.component.html</context>
           <context context-type="linenumber">119</context>
         </context-group>
-        <context-group purpose="location">
-          <context context-type="sourcefile">src/app/components/common/system-status-dialog/system-status-dialog.component.html</context>
-          <context context-type="linenumber">135</context>
-        </context-group>
       </trans-unit>
       <trans-unit id="46628344485199198" datatype="html">
         <source>Classifier</source>
@@ -4531,6 +4527,13 @@
           <context context-type="linenumber">121</context>
         </context-group>
       </trans-unit>
+      <trans-unit id="6096684179126491743" datatype="html">
+        <source>Last Trained</source>
+        <context-group purpose="location">
+          <context context-type="sourcefile">src/app/components/common/system-status-dialog/system-status-dialog.component.html</context>
+          <context context-type="linenumber">135</context>
+        </context-group>
+      </trans-unit>
       <trans-unit id="6732151329960766506" datatype="html">
         <source>Copy Raw Error</source>
         <context-group purpose="location">

src-ui/src/app/components/admin/settings/settings.component.spec.ts

@@ -413,7 +413,7 @@ describe('SettingsComponent', () => {
         index_last_modified: new Date().toISOString(),
         index_error: null,
         classifier_status: SystemStatusItemStatus.OK,
-        classifier_last_modified: new Date().toISOString(),
+        classifier_last_trained: new Date().toISOString(),
         classifier_error: null,
       },
     }

src-ui/src/app/components/common/system-status-dialog/system-status-dialog.component.html

@@ -122,7 +122,7 @@
               <dd>
                 {{status.tasks.classifier_status}}
                 @if (status.tasks.classifier_status === 'OK') {
-                  @if (isStale(status.tasks.classifier_last_modified)) {
+                  @if (isStale(status.tasks.classifier_last_trained)) {
                     <i-bs name="exclamation-triangle-fill" class="text-warning ms-1" [ngbPopover]="classifierStatus" triggers="mouseenter:mouseleave"></i-bs>
                   } @else {
                     <i-bs name="check-circle-fill" class="text-primary ms-1" [ngbPopover]="classifierStatus" triggers="mouseenter:mouseleave"></i-bs>
@@ -132,7 +132,7 @@
                 }
               </dd>
               <ng-template #classifierStatus>
-                <h6><ng-container i18n>Last Updated</ng-container>:</h6> <span class="font-monospace small">{{status.tasks.classifier_last_modified | customDate:'medium'}}</span>
+                <h6><ng-container i18n>Last Trained</ng-container>:</h6> <span class="font-monospace small">{{status.tasks.classifier_last_trained | customDate:'medium'}}</span>
               </ng-template>
             </dl>
           </div>

src-ui/src/app/components/common/system-status-dialog/system-status-dialog.component.spec.ts

@@ -45,7 +45,7 @@ const status: SystemStatus = {
     index_last_modified: new Date().toISOString(),
     index_error: null,
     classifier_status: SystemStatusItemStatus.OK,
-    classifier_last_modified: new Date().toISOString(),
+    classifier_last_trained: new Date().toISOString(),
     classifier_error: null,
   },
 }

src-ui/src/app/data/system-status.ts

@@ -35,7 +35,7 @@ export interface SystemStatus {
     index_last_modified: string // ISO date string
     index_error: string
     classifier_status: SystemStatusItemStatus
-    classifier_last_modified: string // ISO date string
+    classifier_last_trained: string // ISO date string
     classifier_error: string
   }
 }

src/documents/views.py

@@ -14,6 +14,7 @@ from unicodedata import normalize
 from urllib.parse import quote
 
 import pathvalidate
+from django.apps import apps
 from django.conf import settings
 from django.contrib.auth.models import User
 from django.db import connections
@@ -1621,12 +1622,21 @@ class SystemStatusView(GenericAPIView, PassUserMixin):
         try:
             load_classifier()
             classifier_status = "OK"
-            classifier_last_modified = make_aware(
+            task_result_model = apps.get_model("django_celery_results", "taskresult")
-                datetime.fromtimestamp(os.path.getmtime(settings.MODEL_FILE)),
+            result = (
+                task_result_model.objects.filter(
+                    task_name="documents.tasks.train_classifier",
+                    status="SUCCESS",
+                )
+                .order_by(
+                    "-date_done",
+                )
+                .first()
             )
+            classifier_last_trained = result.date_done if result else None
         except Exception as e:
             classifier_status = "ERROR"
-            classifier_last_modified = None
+            classifier_last_trained = None
             classifier_error = "Error loading classifier, check logs for more detail."
             logger.exception(f"System status error loading classifier: {e}")
 
@@ -1660,7 +1670,7 @@ class SystemStatusView(GenericAPIView, PassUserMixin):
                     "index_last_modified": index_last_modified,
                     "index_error": index_error,
                     "classifier_status": classifier_status,
-                    "classifier_last_modified": classifier_last_modified,
+                    "classifier_last_trained": classifier_last_trained,
                     "classifier_error": classifier_error,
                 },
             },