Replaced 'catch' with 'then(null, err)' to support IE8. Fixes #21

CHANGELOG.md

@@ -1,3 +1,8 @@
+## v0.6.6
+
+* Removed calls to $q `catch` and replaced with `then(null, f)` so that the code works in 
+  IE8 (ECMAScript 3).
+
 ## v0.6.5
 
 * Reverted changes below as they led to a bug with injected `$element` in modal controller.

bower.json

@@ -1,6 +1,6 @@
 {
   "name": "angular-modal-service",
-  "version": "0.6.5",
+  "version": "0.6.6",
   "homepage": "https://github.com/dwmkerr/angular-modal-service",
   "authors": [
     "Dave Kerr (github.com/dwmkerr)"

dst/angular-modal-service.js

@@ -40,8 +40,7 @@
                 // save template into the cache and return the template
                 $templateCache.put(templateUrl, result.data);
                 deferred.resolve(result.data);
-              })
-              .catch(function(error) {
+              }, function(error) {
                 deferred.reject(error);
               });
           }
@@ -119,15 +118,14 @@
             //  Parse the modal HTML into a DOM element (in template form).
             var modalElementTemplate = angular.element(template);
 
-            //  Create the controller, explicitly specifying the scope to use.
-            var modalController = $controller(controllerName, inputs);
-
             //  Compile then link the template element, building the actual element.
             //  Set the $element on the inputs so that it can be injected if required.
             var linkFn = $compile(modalElementTemplate);
             var modalElement = linkFn(modalScope);
             inputs.$element = modalElement;
 
+            //  Create the controller, explicitly specifying the scope to use.
+            var modalController = $controller(controllerName, inputs);
 
             //  Finally, append the modal to the dom.
             if (options.appendElement) {
@@ -150,7 +148,7 @@
             deferred.resolve(modal);
 
           })
-          .catch(function(error) {
+          .then(null, function(error) { // 'catch' doesn't work in IE8.
             deferred.reject(error);
           });
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "angular-modal-service",
-  "version": "0.6.5",
+  "version": "0.6.6",
   "description": "AngularJS Service for showing Modals and Popups",
   "main": "./dst/angular-modal-service.js",
   "scripts": {

src/angular-modal-service.js

@@ -40,8 +40,7 @@
                 // save template into the cache and return the template
                 $templateCache.put(templateUrl, result.data);
                 deferred.resolve(result.data);
-              })
-              .catch(function(error) {
+              }, function(error) {
                 deferred.reject(error);
               });
           }
@@ -149,7 +148,7 @@
             deferred.resolve(modal);
 
           })
-          .catch(function(error) {
+          .then(null, function(error) { // 'catch' doesn't work in IE8.
             deferred.reject(error);
           });