fix(directive): newlines before/after translation ids should be ignored

Fixes #909

src/directive/translate.js

@@ -87,6 +87,19 @@ angular.module('pascalprecht.translate')
  */
 .directive('translate', ['$translate', '$q', '$interpolate', '$compile', '$parse', '$rootScope', function ($translate, $q, $interpolate, $compile, $parse, $rootScope) {
 
+  /**
+   * @name trim
+   * @private
+   *
+   * @description
+   * trim polyfill
+   *
+   * @returns {string} The string stripped of whitespace from both ends
+   */
+  var trim = function() {
+    return this.replace(/^\s+|\s+$/g, '');
+  };
+
   return {
     restrict: 'AE',
     scope: true,
@@ -124,7 +137,7 @@ angular.module('pascalprecht.translate')
 
           if (angular.equals(translationId , '') || !angular.isDefined(translationId)) {
             // Resolve translation id by inner html if required
-            var interpolateMatches = iElement.text().match(interpolateRegExp);
+            var interpolateMatches = trim.apply(iElement.text()).match(interpolateRegExp);
             // Interpolate translation id if required
             if (angular.isArray(interpolateMatches)) {
               scope.preText = interpolateMatches[1];

test/unit/directive/translate.spec.js

@@ -93,6 +93,12 @@ describe('pascalprecht.translate', function () {
         expect(element.text()).toBe('TEXT');
       });
 
+      it('should return translation id if translation doesn\'t exist (innerHTML with newlines)', function () {
+        element = $compile("<div translate>\nTEXT\n</div>")($rootScope);
+        $rootScope.$digest();
+        expect(element.text()).toBe('TEXT');
+      });
+
       it('should return translation if translation id exist', function () {
         element = $compile('<div translate>TRANSLATION_ID</div>')($rootScope);
         $rootScope.$digest();
@@ -103,6 +109,16 @@ describe('pascalprecht.translate', function () {
         expect(element.text()).toBe('');
       });
 
+      it('should return translation if translation id exist (innerHTML with newlines)', function () {
+        element = $compile("<div translate>\nTRANSLATION_ID\n</div>")($rootScope);
+        $rootScope.$digest();
+        expect(element.text()).toBe('foo');
+
+        element = $compile("<div translate>\nBLANK_VALUE\n</div>")($rootScope);
+        $rootScope.$digest();
+        expect(element.text()).toBe('');
+      });
+
       it('should return translation id if translation doesn\'t exist and is passed as interpolation', function () {
         $rootScope.translationId = 'TEXT';
         element = $compile('<div translate>{{translationId}}</div>')($rootScope);