Fix template caching.

Author: leonard-thieu

Readme.md

@@ -86,7 +86,7 @@ tooltip-template="" | String() | '' | Set your tooltip template (HTML or just Te
  |  |  | **to know**: don't use it together with `tooltip-template-url` attribute, use only one of them
 tooltip-template-url="" | String() | '' | Set your external tooltip template PATH
  |  |  | **to know**: don't use it together with `tooltip-template` attribute, use only one of them
-tooltip-template-url-cache="" | String(Boolean) | false | This attribute stores and retrieves the template from the cache
+tooltip-template-url-cache="" | String(Boolean) | true | This attribute stores and retrieves the template from the cache
 tooltip-controller="" | String() | '' | Set a controller to your external tooltip template
 tooltip-smart="" | String(Boolean) | false | Set the tooltip to automatically search the best position on the screen
 tooltip-show-trigger="" | String('event1 event2') | 'mouseover' | Show the tooltip on specific event/events
@@ -106,10 +106,10 @@ Sometimes you may need to set all of your tooltips options in one place, you can
 ```javascript
 .config(['tooltipsConfProvider', function configConf(tooltipsConfProvider) {
   tooltipsConfProvider.configure({
-    'smart':true,
-    'size':'large',
+    'smart': true,
+    'size': 'large',
     'speed': 'slow',
-    'tooltipTemplateUrlCache': true
+    'tooltipTemplateUrlCache': false
     //etc...
   });
 }])

demo/views/template-url-cache.html

@@ -0,0 +1 @@
+<div>I'm a tooltip loaded using templateUrlCache!</div>

dist/angular-tooltips.js

@@ -6,7 +6,7 @@
  * http://720kb.github.io/angular-tooltips
  * 
  * MIT license
- * Thu May 18 2017
+ * Fri May 19 2017
  */
 /*global angular,window*/
 (function withAngular(angular, window) {
@@ -225,7 +225,7 @@
       'closeButton': false,
       'size': '',
       'speed': 'steady',
-      'tooltipTemplateUrlCache': false,
+      'tooltipTemplateUrlCache': true,
       'show': null
     };
 
@@ -571,65 +571,79 @@
               registerOnScrollFrom(parentElement);
             }
           }
+          , showTemplate = function showTemplate(template) {
+          
+            tooltipElement.removeClass('_force-hidden'); //see lines below, this forces to hide tooltip when is empty
+            tipTipElement.empty();
+            tipTipElement.append(closeButtonElement);
+            tipTipElement.append(template);
+            $timeout(function doLater() {
+              
+              onTooltipShow();
+            });
+          }
+          , hideTemplate = function hideTemplate() {
+          
+            //hide tooltip because is empty
+            tipTipElement.empty();
+            tooltipElement.addClass('_force-hidden'); //force to be hidden if empty
+          }
+          , getTemplate = function getTemplate(tooltipTemplateUrl) {
+          
+            var template = $templateCache.get(tooltipTemplateUrl);
+
+            if (typeof template === 'undefined') {
+              
+              // How should failing to load the template be handled?
+              template = $http.get(tooltipTemplateUrl).then(function onGetTemplateSuccess(response) {
+                
+                return response.data;
+              });
+              $templateCache.put(tooltipTemplateUrl, template);
+            }
+            
+            return template;
+          }
           , onTooltipTemplateChange = function onTooltipTemplateChange(newValue) {
+          
             if (newValue) {
-              tooltipElement.removeClass('_force-hidden'); //see lines below, this forces to hide tooltip when is empty
-              tipTipElement.empty();
-              tipTipElement.append(closeButtonElement);
-              tipTipElement.append(newValue);
-              $timeout(function doLaterShow() {
-
-                onTooltipShow();
-              });
+              
+              showTemplate(newValue);
             } else {
-              //hide tooltip because is empty
-              tipTipElement.empty();
-              tooltipElement.addClass('_force-hidden'); //force to be hidden if empty
+              
+              hideTemplate();
             }
           }
           , onTooltipTemplateUrlChange = function onTooltipTemplateUrlChange(newValue) {
+          
             if (newValue && !$attrs.tooltipTemplateUrlCache) {
-
-              $http.get(newValue).then(function onResponse(response) {
-
-                if (response &&
-                  response.data) {
-
-                  tooltipElement.removeClass('_force-hidden'); //see lines below, this forces to hide tooltip when is empty
-                  tipTipElement.empty();
-                  tipTipElement.append(closeButtonElement);
-                  tipTipElement.append($compile(response.data)(scope));
-                  $timeout(function doLater() {
-
-                    onTooltipShow();
-                  });
-                }
+              
+              getTemplate(newValue).then(function onGetTemplateSuccess(template) {
+                
+                showTemplate($compile(template)(scope));
+              }).catch(function onGetTemplateFailure(reason) {
+                
+                $log.error(reason);
               });
             } else {
-              //hide tooltip because is empty
-              tipTipElement.empty();
-              tooltipElement.addClass('_force-hidden'); //force to be hidden if empty
+              
+              hideTemplate();
             }
           }
           , onTooltipTemplateUrlCacheChange = function onTooltipTemplateUrlCacheChange(newValue) {
+          
             if (newValue && $attrs.tooltipTemplateUrl) {
+              
+              getTemplate($attrs.tooltipTemplateUrl).then(function onGetTemplateSuccess(template) {
+                
+                showTemplate($compile(template)(scope));
+              }).catch(function onGetTemplateFailure(reason) {
 
-              var template = $templateCache.get($attrs.tooltipTemplateUrl);
-
-              if (typeof template !== 'undefined') {
-
-                tooltipElement.removeClass('_force-hidden'); //see lines below, this forces to hide tooltip when is empty
-                tipTipElement.empty();
-                tipTipElement.append(closeButtonElement);
-                tipTipElement.append($compile(template)(scope));
-                $timeout(function doLater() {
-                  onTooltipShow();
-                });
-              }
+                $log.error(reason);
+              });
             } else {
-              //hide tooltip because is empty
-              tipTipElement.empty();
-              tooltipElement.addClass('_force-hidden'); //force to be hidden if empty
+              
+              hideTemplate();
             }
           }
           , onTooltipSideChange = function onTooltipSideChange(newValue) {