From 8d31bf8c071cbb4d4dbfefd21494e9e70f395956 Mon Sep 17 00:00:00 2001
From: "Amir E. Aharoni" <amir.aharoni@mail.huji.ac.il>
Date: Mon, 8 Oct 2012 00:13:34 +0200
Subject: [PATCH] Updated util functions names after refactoring

jquery.uls.js is just updated from GitHub.

Change-Id: I506ab810d7136ba5d96dfdd469394a843110f8de
---
 lib/jquery.uls/jquery.uls.js            | 31 ++++++++++++++-----------
 resources/js/ext.uls.displaysettings.js |  6 ++---
 resources/js/ext.uls.init.js            |  8 +++++--
 tests/webfonts.html                     |  4 ++--
 4 files changed, 29 insertions(+), 20 deletions(-)

diff --git a/lib/jquery.uls/jquery.uls.js b/lib/jquery.uls/jquery.uls.js
index d7df3145..dba46938 100644
--- a/lib/jquery.uls/jquery.uls.js
+++ b/lib/jquery.uls/jquery.uls.js
@@ -728,37 +728,42 @@
 		},
 
 		listen: function () {
-			var that = this;
+			var lcd = this;
+
 			if ( this.options.clickhandler ) {
 				this.$element.on( 'click', 'div.row li', function() {
-					that.options.clickhandler.call( this, $( this ).data( 'code' ) );
+					lcd.options.clickhandler.call( this, $( this ).data( 'code' ) );
 				} );
 			}
 
 			// The region section need to be in sync with the map filter.
-			that.$element.scroll( function () {
-				var scrollTop = $( this ).position().top;
-				var scrollBottom = $( this ).height();
-				if ( this.offsetHeight + this.scrollTop >= this.scrollHeight/2 ) {
-					that.$element.trigger( 'scrollend' );
+			lcd.$element.scroll( function () {
+				var $ulsLanguageList = $( this ),
+					scrollTop = $ulsLanguageList.position().top,
+					scrollBottom = $ulsLanguageList.height();
+
+				if ( this.offsetHeight + this.scrollTop >= this.scrollHeight / 2 ) {
+					lcd.$element.trigger( 'scrollend' );
 				}
+
 				// The region section need to be in sync with the map filter.
 				var inviewRegion = 'WW';
-				that.$element.find( 'div.uls-lcd-region-section' ).each( function () {
-					var top = $( this ).position().top;
-					var height = $( this ).height();
+				lcd.$element.find( 'div.uls-lcd-region-section' ).each( function () {
+					var $lcdRegionSection = $( this ),
+						top = $lcdRegionSection.position().top,
+						height = $lcdRegionSection.height();
+
 					if ( top < scrollTop && height > scrollBottom ) {
-						inviewRegion = $( this ).attr( 'id' );
+						inviewRegion = $lcdRegionSection.attr( 'id' );
 						return true;
 					}
 				} );
+
 				var inview = $.uls.data.regiongroups[inviewRegion];
 				$( '.regionselector' ).removeClass( 'active' );
 				$( '#uls-region-' + inview ).addClass( 'active' );
 			} );
-
 		}
-
 	};
 
 	$.fn.lcd = function( option ) {
diff --git a/resources/js/ext.uls.displaysettings.js b/resources/js/ext.uls.displaysettings.js
index d4be4c66..cc8ac175 100644
--- a/resources/js/ext.uls.displaysettings.js
+++ b/resources/js/ext.uls.displaysettings.js
@@ -129,7 +129,7 @@
 				var language = languages[i];
 				var $button = $( '<button>' )
 					.addClass( 'button uls-language-button' )
-					.text( $.uls.data.autonym( language ) );
+					.text( $.uls.data.getAutonym( language ) );
 				if ( language === this.uiLanguage ) {
 					$button.addClass( 'down' );
 				}
@@ -244,7 +244,7 @@
 			$systemFont.attr( 'selected', savedFont === 'system' || !savedFont );
 			var $fontLabel = this.$template.find( 'label#ui-font-selector-label' );
 			$fontLabel.html( '<strong>'
-				+ $.i18n( 'ext-uls-webfonts-select-for', $.uls.data.autonym( this.uiLanguage ) )
+				+ $.i18n( 'ext-uls-webfonts-select-for', $.uls.data.getAutonym( this.uiLanguage ) )
 				+ '</strong>'
 				+ '<div>'
 				+ $.i18n( 'ext-uls-webfonts-select-for-ui-info' )
@@ -279,7 +279,7 @@
 			var $fontLabel = this.$template.find( '#content-font-selector-label' );
 			$fontLabel.html( '<strong>'
 				+ $.i18n( 'ext-uls-webfonts-select-for',
-					$.uls.data.autonym( this.contentLanguage ) )
+					$.uls.data.getAutonym( this.contentLanguage ) )
 				+ '</strong>'
 				+ '<div>'
 				+ $.i18n( 'ext-uls-webfonts-select-for-content-info' )
diff --git a/resources/js/ext.uls.init.js b/resources/js/ext.uls.init.js
index f32ab1e6..13f11416 100644
--- a/resources/js/ext.uls.init.js
+++ b/resources/js/ext.uls.init.js
@@ -71,18 +71,22 @@
 			]
 				.concat( mw.uls.getPreviousLanguages() )
 				.concat( mw.uls.getAcceptLanguageList() );
+
 		if ( window.GEO ) {
-			list = list.concat( $.uls.data.languagesInTerritory( window.GEO.country_code ) );
+			list = list.concat( $.uls.data.getLanguagesInTerritory( window.GEO.country_code ) );
 		}
+
 		$.each( list, function ( i, v ) {
 			if ( $.inArray( v, unique ) === -1 ) {
 				unique.push( v );
 			}
 		} );
+
 		// Filter out unknown and unsupported languages
 		unique = $.grep( unique, function ( langCode, index ) {
 			return $.fn.uls.defaults.languages[langCode];
 		} );
+
 		return unique;
 	};
 
@@ -170,7 +174,7 @@
 			fade: true,
 			trigger: 'manual',
 			title: function () {
-				var prevLangName = $.uls.data.autonym( previousLang );
+				var prevLangName = $.uls.data.getAutonym( previousLang );
 				var linkClass = 'uls-prevlang-link';
 				var prevLangLink = "<a href='#' lang = '" +
 					previousLang + "' class = '" + linkClass + "' >" +
diff --git a/tests/webfonts.html b/tests/webfonts.html
index 5392ad7e..5b12aaf1 100644
--- a/tests/webfonts.html
+++ b/tests/webfonts.html
@@ -37,7 +37,7 @@
 		$.each( languages, function ( lang, language ) {
 			$langselector.append( $( "<option></option>" )
 				.attr( "value", language )
-				.text( language + " - " + $.uls.data.autonym( language ) ) );
+				.text( language + " - " + $.uls.data.getAutonym( language ) ) );
 		} );
 		$fontSelector.on( 'change', function () {
 			var font = $fontSelector.find( 'option:selected' ).val();
@@ -46,7 +46,7 @@
 		$langselector.on( 'change', function () {
 			var language = $langselector.find( 'option:selected' ).val();
 			listFonts( $webfonts.list( language ) );
-			$( '#webfonts-preview-area' ).text( $.uls.data.autonym( language ) );
+			$( '#webfonts-preview-area' ).text( $.uls.data.getAutonym( language ) );
 		} );
 	} )
 </script>