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

Commit

Permalink
fix(ngView): ensure ngView is terminal and uses its own manual transc…
Browse files Browse the repository at this point in the history
…lusion system
  • Loading branch information
matsko authored and mhevery committed Aug 9, 2013
1 parent 1b5bee4 commit 87405e2
Showing 1 changed file with 42 additions and 38 deletions.
80 changes: 42 additions & 38 deletions src/ngRoute/directive/ngView.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
'use strict';

ngRouteModule.directive('ngView', ngViewFactory);

/**
* @ngdoc directive
* @name ngRoute.directive:ngView
Expand Down Expand Up @@ -169,17 +167,22 @@ ngRouteModule.directive('ngView', ngViewFactory);
* @description
* Emitted every time the ngView content is reloaded.
*/
ngViewFactory.$inject = ['$route', '$anchorScroll', '$compile', '$controller', '$animate'];
function ngViewFactory( $route, $anchorScroll, $compile, $controller, $animate) {
var NG_VIEW_PRIORITY = 500;
var ngViewDirective = ['$route', '$anchorScroll', '$compile', '$controller', '$animate',
function($route, $anchorScroll, $compile, $controller, $animate) {
return {
restrict: 'ECA',
terminal: true,
transclude: 'element',
compile: function(element, attr, linker) {
return function(scope, $element, attr) {
var currentScope,
currentElement,
onloadExp = attr.onload || '';
priority: NG_VIEW_PRIORITY,
compile: function(element, attr) {
var onloadExp = attr.onload || '';

element.html('');
var anchor = jqLite(document.createComment(' ngView '));
element.replaceWith(anchor);

return function(scope) {
var currentScope, currentElement;

scope.$on('$routeChangeSuccess', update);
update();
Expand All @@ -200,41 +203,42 @@ function ngViewFactory( $route, $anchorScroll, $compile, $controller,
template = locals && locals.$template;

if (template) {
var newScope = scope.$new();
linker(newScope, function(clone) {
cleanupLastView();

clone.html(template);
$animate.enter(clone, null, $element);

var link = $compile(clone.contents()),
current = $route.current;

currentScope = current.scope = newScope;
currentElement = clone;

if (current.controller) {
locals.$scope = currentScope;
var controller = $controller(current.controller, locals);
if (current.controllerAs) {
currentScope[current.controllerAs] = controller;
}
clone.data('$ngControllerController', controller);
clone.contents().data('$ngControllerController', controller);
cleanupLastView();

currentScope = scope.$new();
currentElement = element.clone();
currentElement.html(template);
$animate.enter(currentElement, null, anchor);

var link = $compile(currentElement, false, NG_VIEW_PRIORITY - 1),
current = $route.current;

if (current.controller) {
locals.$scope = currentScope;
var controller = $controller(current.controller, locals);
if (current.controllerAs) {
currentScope[current.controllerAs] = controller;
}
currentElement.data('$ngControllerController', controller);
currentElement.children().data('$ngControllerController', controller);
}

current.scope = currentScope;

link(currentScope);
currentScope.$emit('$viewContentLoaded');
currentScope.$eval(onloadExp);
link(currentScope);

// $anchorScroll might listen on event...
$anchorScroll();
});
currentScope.$emit('$viewContentLoaded');
currentScope.$eval(onloadExp);

// $anchorScroll might listen on event...
$anchorScroll();
} else {
cleanupLastView();
}
}
}
}
};
}
}];

ngRouteModule.directive('ngView', ngViewDirective);

0 comments on commit 87405e2

Please sign in to comment.