[Bug 231] timedout jobs should have their notice field specifying the cause of failure

Applied your fix Patrice, fields now reads "Kill time expired". Thanks!

diff --git a/faxd/faxQueueApp.c++ b/faxd/faxQueueApp.c++
index 1dbbbf3307558d70c372987926257ad07965fd35..c1bf2086fee63493ab43c0c02e35ffe2f1653e41 100644 (file)
--- a/faxd/faxQueueApp.c++
+++ b/faxd/faxQueueApp.c++
@@ -1876,8 +1876,10 @@ faxQueueApp::timeoutJob(Job& job)
        job.remove();                           // remove from sleep queue
        job.state = FaxRequest::state_failed;
        FaxRequest* req = readRequest(job);
-       if (req)
+       if (req) {
+           req->notice = "Kill time expired";
            deleteRequest(job, req, Job::timedout, true);
+       }
        setDead(job);
     } else
        job.killtime = 0;                       // mark job to be removed
@@ -1899,6 +1901,7 @@ faxQueueApp::timeoutJob(Job& job, FaxRequest& req)
     job.state = FaxRequest::state_failed;
     traceQueue(job, "KILL TIME EXPIRED");
     Trigger::post(Trigger::JOB_TIMEDOUT, job);
+    req.notice = "Kill time expired";
     deleteRequest(job, req, Job::timedout, true);
     setDead(job);
 }