From: tjhunt <tjhunt>
Date: Tue, 31 Mar 2009 03:24:21 +0000 (+0000)
Subject: MDL-15484 Avoid JavaScript errors after a popup has been blocked.
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=214f5850d12a94e0408e1c85138da439826c36d6;p=moodle.git

MDL-15484 Avoid JavaScript errors after a popup has been blocked.

Also clean up coding style of some nearby functions.
---

diff --git a/lib/javascript-static.js b/lib/javascript-static.js
index 791a00475f..e163821a08 100644
--- a/lib/javascript-static.js
+++ b/lib/javascript-static.js
@@ -10,21 +10,21 @@ function popupchecker(msg) {
 }
 
 function checkall() {
-  var el = document.getElementsByTagName('input');
-  for(var i=0; i<el.length; i++) {
-    if(el[i].type == 'checkbox') {
-      el[i].checked = true;
+    var inputs = document.getElementsByTagName('input');
+    for (var i = 0; i < inputs.length; i++) {
+        if (inputs[i].type == 'checkbox') {
+            inputs[i].checked = true;
+        }
     }
-  }
 }
 
 function checknone() {
-  var el = document.getElementsByTagName('input');
-  for(var i=0; i<el.length; i++) {
-    if(el[i].type == 'checkbox') {
-      el[i].checked = false;
+    var inputs = document.getElementsByTagName('input');
+    for (var i = 0; i < inputs.length; i++) {
+        if (inputs[i].type == 'checkbox') {
+            inputs[i].checked = false;
+        }
     }
-  }
 }
 
 function lockoptions(formid, master, subitems) {
@@ -770,9 +770,12 @@ function getElementsByClassName(oElm, strTagName, oClassNames) {
 function openpopup(url, name, options, fullscreen) {
     var fullurl = url;
     if (!url.match(/https?:\/\//)) {
-        var fullurl = moodle_cfg.wwwroot + url;
+        fullurl = moodle_cfg.wwwroot + url;
     }
     var windowobj = window.open(fullurl,name,options);
+    if (!windowobj) {
+        return true;
+    }
     if (fullscreen) {
         windowobj.moveTo(0,0);
         windowobj.resizeTo(screen.availWidth,screen.availHeight);