When multiple requests are completed in the same HTTP request, they won't try...

When multiple requests are completed in the same HTTP request, they won't try to start back up the chain again.

application/modules/requests/models/ApprovalChainModel.php

@@ -204,11 +204,14 @@ class Requests_ApprovalChainModel extends Unl_Model
         }
     
         // If any advanced requests are completed, drop them from our list
+        $incompleteRequests = new Unl_Model_Collection('Requests_RequestModel');
         foreach ($requests as $requestId => $request) {
             if ($request->isComplete()) {
-                unset($requests[$requestId]);
+                continue;
             }
+            $incompleteRequests[$requestId] = $request;
         }
+        $requests = $incompleteRequests;
 
         // If we don't have any requests left to process (after dropping completed requests), stop.
         if (count($requests) == 0) {
@@ -238,11 +241,14 @@ class Requests_ApprovalChainModel extends Unl_Model
         self::advance($requestsToAdvance);
 
         // If any advanced requests were completed, drop them from our list
+        $incompleteRequests = new Unl_Model_Collection('Requests_RequestModel');
         foreach ($requests as $requestId => $request) {
             if ($request->isComplete()) {
-                unset($requests[$requestId]);
+                continue;
             }
+            $incompleteRequests[$requestId] = $request;
         }
+        $requests = $incompleteRequests;
 
         // If we don't have any requests left to process (after dropping completed requests), stop.
         if (count($requests) == 0) {