fix(rating): evaluate max attribute on parent scope

Author: bekos

src/rating/docs/demo.html

@@ -1,5 +1,6 @@
 <div ng-controller="RatingDemoCtrl">
-    <rating value="rate" max="10" readonly="isReadonly" on-hover="hoveringOver(value)" on-leave="overStar = null"></rating>
+    <rating value="rate" max="max" readonly="isReadonly" on-hover="hoveringOver(value)" on-leave="overStar = null"></rating>
+    <span class="badge badge-inverse" ng-show="overStar && !isReadonly">{{overStar}} / {{max}}</span>
 
     <hr/>
     <pre>Rate: <b>{{rate}}</b> - Readonly is: <i>{{isReadonly}}</i> - Hovering over: <b>{{overStar || "none"}}</b></pre>

src/rating/docs/demo.js

@@ -1,5 +1,6 @@
 var RatingDemoCtrl = function ($scope) {
   $scope.rate = 7;
+  $scope.max = 10;
   $scope.isReadonly = false;
   $scope.hoveringOver = function(value) {
     $scope.overStar = value;

src/rating/rating.js

@@ -16,7 +16,7 @@ angular.module('ui.bootstrap.rating', [])
     replace: true,
     link: function(scope, element, attrs) {
 
-      var maxRange = angular.isDefined(attrs.max) ? scope.$eval(attrs.max) : ratingConfig.max;
+      var maxRange = angular.isDefined(attrs.max) ? scope.$parent.$eval(attrs.max) : ratingConfig.max;
 
       scope.range = [];
       for (var i = 1; i <= maxRange; i++) {

src/rating/test/rating.spec.js

@@ -77,6 +77,13 @@ describe('rating directive', function () {
     expect(getStars().length).toBe(7);
   });
 
+  it('shows different number of icons when `max` attribute is from scope variable', function() {
+    $rootScope.max = 15;
+    element = $compile('<rating value="rate" max="max"></rating>')($rootScope);
+    $rootScope.$digest();
+    expect(getStars().length).toBe(15);
+  });
+
   it('handles readonly attribute', function() {
     $rootScope.isReadonly = true;
     element = $compile('<rating value="rate" readonly="isReadonly"></rating>')($rootScope);