Sort alert definitions so recoveries come after problems. (HHQ-3142)

hyperic · May 22, 2009 · a6c5ef3 · a6c5ef3
1 parent 6877f5c
commit a6c5ef3
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/hqu/hqapi1/app/AlertdefinitionController.groovy b/hqu/hqapi1/app/AlertdefinitionController.groovy
@@ -199,7 +199,7 @@ public class AlertdefinitionController extends ApiController {
                     out << failureXml
                 } else {
                     out << getSuccessXML()
-                    for (definition in definitions) {
+                    for (definition in definitions.sort {a, b -> a.id <=> b.id}) {
                         out << getAlertDefinitionXML(definition, false)
                     }
                 }
@@ -219,7 +219,8 @@ public class AlertdefinitionController extends ApiController {
         renderXml() {
             out << AlertDefinitionsResponse() {
                 out << getSuccessXML()
-                for (definition in definitions) {
+                // Order by id's so recoveries come after problem alerts
+                for (definition in definitions.sort {a, b -> a.id <=> b.id}) {
                     out << getAlertDefinitionXML(definition, noIds)
                 }
             }