Browse Source

#3712 - Fixed issue with rating firing onRate on initial load

pull/3763/head
jlukic 9 years ago
parent
commit
44837faf57
1 changed files with 20 additions and 1 deletions
  1. 21
      src/definitions/modules/rating.js

21
src/definitions/modules/rating.js

@ -48,6 +48,7 @@ $.fn.rating = function(parameters) {
$module = $(this), $module = $(this),
$icon = $module.find(selector.icon), $icon = $module.find(selector.icon),
initialLoad,
module module
; ;
@ -66,7 +67,9 @@ $.fn.rating = function(parameters) {
else { else {
module.disable(); module.disable();
} }
module.set.initialLoad();
module.set.rating( module.get.initialRating() ); module.set.rating( module.get.initialRating() );
module.remove.initialLoad();
module.instantiate(); module.instantiate();
}, },
@ -170,6 +173,9 @@ $.fn.rating = function(parameters) {
$module $module
.off(eventNamespace) .off(eventNamespace)
; ;
},
initialLoad: function() {
initialLoad = false;
} }
}, },
@ -189,6 +195,12 @@ $.fn.rating = function(parameters) {
; ;
}, },
is: {
initialLoad: function() {
return initialLoad;
}
},
get: { get: {
initialRating: function() { initialRating: function() {
if($module.data(metadata.rating) !== undefined) { if($module.data(metadata.rating) !== undefined) {
@ -236,7 +248,12 @@ $.fn.rating = function(parameters) {
.addClass(className.active) .addClass(className.active)
; ;
} }
settings.onRate.call(element, rating);
if(!module.is.initialLoad()) {
settings.onRate.call(element, rating);
}
},
initialLoad: function() {
initialLoad = true;
} }
}, },
@ -432,6 +449,8 @@ $.fn.rating.settings = {
maxRating : 4, maxRating : 4,
clearable : 'auto', clearable : 'auto',
fireOnInit : false,
onRate : function(rating){}, onRate : function(rating){},
error : { error : {

Loading…
Cancel
Save