Pagecontainer: create public method to retrieve the activePage

jquery-archive · gseguin · Sep 24, 2013 · Sep 24, 2013 · Sep 24, 2013 · Sep 24, 2013
commit 08f04755eae875d3f8994e2a38de776c5381df3d
diff --git a/js/widgets/pagecontainer.js b/js/widgets/pagecontainer.js
@@ -151,7 +151,7 @@ define( [
 		},
 
 		// TODO active page should be managed by the container (ie, it should be a property)
-		_getActiveContent: function() {
+		getActivePage: function() {
 			return this.activePage;
 		},
 
@@ -223,7 +223,7 @@ define( [
 		},
 
 		_handleDialog: function( changePageOptions, data ) {
-			var to, active, activeContent = this._getActiveContent();
+			var to, active, activeContent = this.getActivePage();
 
 			// If current active page is not a dialog skip the dialog and continue
 			// in the same direction

diff --git a/tests/unit/content/content_core.js b/tests/unit/content/content_core.js
@@ -91,7 +91,7 @@
 	test( "continues backward when the active content isn't a dialog", function() {
 		expect( 2 );
 
-		proto._getActiveContent = function() {
+		proto.getActivePage = function() {
 			return $( "<div>" );
 		};
 
@@ -105,7 +105,7 @@
 	test( "continues forward when the active content isn't a dialog", function() {
 		expect( 2 );
 
-		proto._getActiveContent = function() {
+		proto.getActivePage = function() {
 			return $( "<div>" );
 		};
 
@@ -119,7 +119,7 @@
 	test( "extends changePageOptions when current content is a dialog", function() {
 		var result, opts = {};
 
-		proto._getActiveContent = function() {
+		proto.getActivePage = function() {
 			return $( "<div>", {"class": "ui-dialog"} );
 		};