Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Commit 41cec4d

Browse files
committed
refactor(select): remove code duplication
1 parent 0f3ea45 commit 41cec4d

File tree

1 file changed

+13
-13
lines changed

1 file changed

+13
-13
lines changed

Diff for: src/ng/directive/select.js

+13-13
Original file line numberDiff line numberDiff line change
@@ -414,21 +414,13 @@ var selectDirective = ['$compile', '$parse', function($compile, $parse) {
414414

415415
ctrl.$render = render;
416416

417-
scope.$watchCollection(valuesFn, function () {
418-
if (!renderScheduled) {
419-
scope.$$postDigest(render);
420-
renderScheduled = true;
421-
}
422-
});
423-
if ( multiple ) {
424-
scope.$watchCollection(function() { return ctrl.$modelValue; }, function () {
425-
if (!renderScheduled) {
426-
scope.$$postDigest(render);
427-
renderScheduled = true;
428-
}
429-
});
417+
scope.$watchCollection(valuesFn, scheduleRendering);
418+
419+
if (multiple) {
420+
scope.$watchCollection(function() { return ctrl.$modelValue; }, scheduleRendering);
430421
}
431422

423+
432424
function getSelectedSet() {
433425
var selectedSet = false;
434426
if (multiple) {
@@ -448,6 +440,14 @@ var selectDirective = ['$compile', '$parse', function($compile, $parse) {
448440
}
449441

450442

443+
function scheduleRendering() {
444+
if (!renderScheduled) {
445+
scope.$$postDigest(render);
446+
renderScheduled = true;
447+
}
448+
}
449+
450+
451451
function render() {
452452
renderScheduled = false;
453453

0 commit comments

Comments
 (0)