bryce/jquery.uls
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #120 from siebrand/7

Browse Source 
Various fixes suggested by static code analysis
This commit is contained in:
Kartik Mistry
2013-12-03 05:11:12 -08:00
parent 52cadffd06 520e1b1181
commit 1d98d61dc1
4 changed files with 9 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
data/ulsdata2json.php
View File

@@ -76,6 +76,7 @@ $js = <<<JAVASCRIPT
( function ( $ ) { ( function ( $ ) {
'use strict'; 'use strict';
$.uls = $.uls || {}; $.uls = $.uls || {};
//noinspection JSHint
$.uls.data = $json; $.uls.data = $json;
} ( jQuery ) ); } ( jQuery ) );

2
examples/index-autonymfont.html
View File

@@ -1,4 +1,4 @@
s<!DOCTYPE html> <!DOCTYPE html>
<html lang="en" dir="ltr"> <html lang="en" dir="ltr">
<head> <head>
<meta charset="utf-8"/> <meta charset="utf-8"/>

1
src/jquery.uls.data.js
View File

File diff suppressed because one or more lines are too long

10
test/jquery.uls.test.js
View File

@@ -20,11 +20,13 @@
module( 'jquery.uls' ); module( 'jquery.uls' );
var orphanScripts, badRedirects, doubleRedirects, languagesWithoutAutonym;
/* /*
* Runs over all script codes mentioned in langdb and checks whether * Runs over all script codes mentioned in langdb and checks whether
* they belong to the 'Other' group. * they belong to the 'Other' group.
*/ */
var orphanScripts = function () { orphanScripts = function () {
var language, script, var language, script,
result = []; result = [];
@@ -42,7 +44,7 @@
/* /*
* Runs over all languages and checks that all redirects have a valid target. * Runs over all languages and checks that all redirects have a valid target.
*/ */
var badRedirects = function () { badRedirects = function () {
var language, target, var language, target,
result = []; result = [];
@@ -61,7 +63,7 @@
* Runs over all languages and checks that all redirects point to a language. * Runs over all languages and checks that all redirects point to a language.
* There's no reason to have double redirects. * There's no reason to have double redirects.
*/ */
var doubleRedirects = function () { doubleRedirects = function () {
var language, target, var language, target,
result = []; result = [];
@@ -80,7 +82,7 @@
* Runs over all script codes mentioned in langdb and checks whether * Runs over all script codes mentioned in langdb and checks whether
* they have something that looks like an autonym. * they have something that looks like an autonym.
*/ */
var languagesWithoutAutonym = function () { languagesWithoutAutonym = function () {
var language, var language,
result = []; result = [];