MDL-46116 mod_imscp: Fix some jshint issues.
authorDavid Balch <david.balch@conted.ox.ac.uk>
Wed, 19 Nov 2014 15:17:29 +0000 (15:17 +0000)
committerDavid Balch <david.balch@conted.ox.ac.uk>
Tue, 2 Dec 2014 10:33:11 +0000 (10:33 +0000)
mod/imscp/module.js

index 5b072f7..f3ce6ea 100644 (file)
@@ -35,7 +35,7 @@ M.mod_imscp.init = function(Y) {
 
         var imscp_activate_item_by_index = function(index) {
             imscp_activate_item(Y.YUI2.widget.TreeView.getNode('imscp_tree', index));
-        }
+        };
 
         var imscp_activate_item = function(node) {
             if (!node) {
@@ -45,20 +45,21 @@ M.mod_imscp.init = function(Y) {
             imscp_current_node.highlight();
 
             var content = new Y.YUI2.util.Element('imscp_content');
+            var obj;
             if (node.href) {
                 try {
                     // First try IE way - it can not set name attribute later
                     // and also it has some restrictions on DOM access from object tag.
-                    var obj = document.createElement('<iframe id="imscp_object" src="' + node.href + '">');
+                    obj = document.createElement('<iframe id="imscp_object" src="' + node.href + '">');
                 } catch (e) {
-                    var obj = document.createElement('object');
+                    obj = document.createElement('object');
                     obj.setAttribute('id', 'imscp_object');
                     obj.setAttribute('type', 'text/html');
                     obj.setAttribute('data', node.href);
                 }
             } else {
                 // No href, so create links to children.
-                var obj = document.createElement('div');
+                obj = document.createElement('div');
                 obj.setAttribute('id', 'imscp_child_list');
 
                 var title = document.createElement('h2');
@@ -153,7 +154,7 @@ M.mod_imscp.init = function(Y) {
         };
 
         var imscp_resize_frame = function() {
-            var obj = Y.YUI2.util.Dom.get('imscp_object');
+            obj = Y.YUI2.util.Dom.get('imscp_object');
             if (obj) {
                 var content = imscp_layout_widget.getUnitByPosition('center').get('wrap');
                 // Basically trap IE6 and 7.
@@ -180,7 +181,7 @@ M.mod_imscp.init = function(Y) {
             } else if (node.children) {
                 return imscp_firstlinked(node.children[0]);
             } else {
-                return null
+                return null;
             }
         };
 
@@ -254,11 +255,11 @@ M.mod_imscp.init = function(Y) {
 
         // Ugly resizing hack that works around problems with resizing of iframes and objects.
         left._resize.on('startResize', function() {
-            var obj = Y.YUI2.util.Dom.get('imscp_object');
+            obj = Y.YUI2.util.Dom.get('imscp_object');
             obj.style.display = 'none';
         });
         left._resize.on('endResize', function() {
-            var obj = Y.YUI2.util.Dom.get('imscp_object');
+            obj = Y.YUI2.util.Dom.get('imscp_object');
             obj.style.display = 'block';
             imscp_resize_frame();
         });