Added toastr to logger

Author: nareshbhatia

bower.json

@@ -19,8 +19,10 @@
   },
   "dependencies": {
     "angular": "~1.4.0",
+    "angular-animate": "~1.4.0",
     "angular-bootstrap": "~0.13.0",
     "angular-sanitize": "~1.4.0",
+    "angular-toastr": "~1.4.1",
     "angular-ui-router": "~0.2.15",
     "bootstrap-sass": "~3.3.4",
     "lodash": "~3.9.3"

karma.conf.js

@@ -15,9 +15,12 @@ module.exports = function (config) {
             // list of files / patterns to load in the browser
             files: [
                 'bower_components/angular/angular.js',
+                'bower_components/angular-animate/angular-animate.js',
                 'bower_components/angular-bootstrap/ui-bootstrap.js',
                 'bower_components/angular-bootstrap/ui-bootstrap-tpls.js',
                 'bower_components/angular-sanitize/angular-sanitize.js',
+                'bower_components/angular-toastr/dist/angular-toastr.js',
+                'bower_components/angular-toastr/dist/angular-toastr.tpls.js',
                 'bower_components/angular-ui-router/release/angular-ui-router.js',
                 'bower_components/angular-mocks/angular-mocks.js',
                 'bower_components/bardjs/dist/bard.js',

src/components/dashboard/dashboard.directive.js

@@ -33,7 +33,7 @@
         activate();
 
         function activate() {
-            logger.info('Activated Dashboard View');
+            logger.log('Activated Dashboard View');
         }
     }
 

src/core/core.module.js

@@ -2,13 +2,13 @@
     'use strict';
 
     angular.module('app.core', [
-        // Angular modules
-        'ngSanitize',
+        // Angular modules (ngAnimate 1.4.x is not compatible with ui.bootstrap 0.13.0)
+        /* 'ngAnimate', */ 'ngSanitize',
 
         // Our reusable framework
         'fw.exception', 'fw.logger',
 
         // 3rd Party modules
-        'ui.bootstrap', 'ui.router'
+        'toastr', 'ui.bootstrap', 'ui.router'
     ]);
 })();

src/framework/exception/exception-handler.provider.js

@@ -40,17 +40,19 @@
         $provide.decorator('$exceptionHandler', extendExceptionHandler);
     }
 
-    extendExceptionHandler.$inject = ['$delegate', 'exceptionHandler', 'logger'];
+    extendExceptionHandler.$inject = ['$delegate', 'exceptionHandler', '$injector'];
 
     /**
      * Extend the $exceptionHandler service to log via the logger service.
      * @param  {Object} $delegate
      * @param  {Object} exceptionHandler
-     * @param  {Object} logger
+     * @param  {Object} $injector
      * @return {Function} the decorated $exceptionHandler service
      */
-    function extendExceptionHandler($delegate, exceptionHandler, logger) {
+    function extendExceptionHandler($delegate, exceptionHandler, $injector) {
         return function(exception, cause) {
+            // Need to load logger at runtime to avoid circular dependency with toastr
+            var logger = $injector.get('logger');
             var appErrorPrefix = exceptionHandler.config.appErrorPrefix || '';
             var errorData = {exception: exception, cause: cause};
             exception.message = appErrorPrefix + exception.message;

src/framework/logger/logger.js

@@ -5,10 +5,10 @@
         .module('fw.logger')
         .factory('logger', logger);
 
-    logger.$inject = ['$log'];
+    logger.$inject = ['$log', 'toastr'];
 
     /* @ngInject */
-    function logger($log) {
+    function logger($log, toastr) {
         var service = {
             log     : log,
             info    : info,
@@ -25,14 +25,17 @@
         }
 
         function info(message) {
+            toastr.info(message, 'Information');
             $log.info('info: ' + message);
         }
 
         function warn(message) {
+            toastr.warning(message, 'Warning');
             $log.warn('warn: ' + message);
         }
 
         function error(message) {
+            toastr.error(message, 'Error');
             $log.error('error: ' + message);
         }
 

src/index.html

@@ -19,6 +19,7 @@
     <link href='http://fonts.googleapis.com/css?family=Open+Sans:400italic,700italic,700,400' rel='stylesheet' type='text/css'>
 
     <!-- build:css styles/lib.css -->
+    <link href="/bower_components/angular-toastr/dist/angular-toastr.css" rel="stylesheet" type="text/css">
     <!-- endbuild -->
 
     <!-- build:css styles/app.css-->
@@ -34,9 +35,12 @@
 
     <!-- build:js js/lib.js -->
     <script src="/bower_components/angular/angular.js"></script>
+    <script src="/bower_components/angular-animate/angular-animate.js"></script>
     <script src="/bower_components/angular-bootstrap/ui-bootstrap.js"></script>
     <script src="/bower_components/angular-bootstrap/ui-bootstrap-tpls.js"></script>
     <script src="/bower_components/angular-sanitize/angular-sanitize.js"></script>
+    <script src="/bower_components/angular-toastr/dist/angular-toastr.js"></script>
+    <script src="/bower_components/angular-toastr/dist/angular-toastr.tpls.js"></script>
     <script src="/bower_components/angular-ui-router/release/angular-ui-router.js"></script>
     <script src="/bower_components/lodash/lodash.js"></script>
     <!-- endbuild -->