Fixed a bug in the destroyAllMessages function.

The destroyAllMessages was not setting the loop counter correctly and would error if there was only 1 message in the array.
harishreddy-m · Jan 15, 2015 · b7946ce · b7946ce
1 parent 1763c0e
commit b7946ce
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/src/growlMessageService.js b/src/growlMessageService.js
@@ -56,9 +56,8 @@ angular.module("angular-growl").service("growlMessages", ['$sce', '$timeout', fu
   };
 
   this.destroyAllMessages = function (referenceId) {
-    var messages = this.getAllMessages(referenceId),
-      i = messages.length;
-    for (i - 1; i >= 0; i--) {
+    var messages = this.getAllMessages(referenceId);
+    for (var i = messages.length - 1; i >= 0; i--) {
       messages[i].destroy();
     }
     if (this.directives[referenceId]) {