Avoid some of the strict JS warnings in Firefox.
authorRotem Liss <rotem@users.mediawiki.org>
Tue, 10 Jun 2008 18:55:32 +0000 (18:55 +0000)
committerRotem Liss <rotem@users.mediawiki.org>
Tue, 10 Jun 2008 18:55:32 +0000 (18:55 +0000)
skins/common/mwsuggest.js
skins/common/wikibits.js

index d7d03f9..5247006 100644 (file)
@@ -96,14 +96,16 @@ function os_encodeQuery(value){
   if(escape) {
     return escape(value);
   }
+  return null;
 }
 function os_decodeValue(value){
   if (decodeURIComponent) {
     return decodeURIComponent(value);
-  } 
+  }
   if(unescape){
        return unescape(value);
   }
+  return null;
 }
 
 /** Brower-dependent functions to find window inner size, and scroll status */
@@ -452,7 +454,7 @@ function os_updateSearchQuery(r,newText){
 
 /** Find event target */
 function os_getTarget(e){
-       if (!e) var e = window.event;
+       if (!e) e = window.event;
        if (e.target) return e.target;
        else if (e.srcElement) return e.srcElement;
        else return null;
@@ -525,7 +527,7 @@ function os_eventKeypress(e){
 
 /** Catch the key code (Firefox bug)  */
 function os_eventKeydown(e){
-       if (!e) var e = window.event;
+       if (!e) e = window.event;
        var targ = os_getTarget(e);
        var r = os_map[targ.id];
        if(r == null)
index 2982607..802b595 100644 (file)
@@ -58,7 +58,7 @@ function importScript(page) {
 var loadedScripts = {}; // included-scripts tracker
 function importScriptURI(url) {
        if (loadedScripts[url]) {
-               return;
+               return null;
        }
        loadedScripts[url] = true;
        var s = document.createElement('script');