Fix multi instances (#67)

Thanks
brianteeman · Jan 9, 2018 · d81bea8 · d81bea8
1 parent 9e69aec
commit d81bea8
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 9 deletions.
diff --git a/media/com_wrapper/js/iframe-height.js b/media/com_wrapper/js/iframe-height.js
@@ -1,17 +1,14 @@
-function iFrameHeight()
+function iFrameHeight(iframe)
 {
-    var height = 0;
-    var iframe = document.querySelectorAll('*[id^="blockrandom"]');
     var doc    = 'contentDocument' in iframe ? iframe.contentDocument : iframe.contentWindow.document;
+    var height = parseInt(doc.body.scrollHeight);
 
     if (!document.all)
     {
-        height = doc.body.scrollHeight;
         iframe.style.height = parseInt(height) + 60 + 'px';
     }
-    else if (document.all)
+    else if (document.all && iframe.id)
     {
-        height = doc.body.scrollHeight;
-        document.all.blockrandom.style.height = parseInt(height) + 20 + 'px';
+        document.all[iframe.id].style.height = parseInt(height) + 20 + 'px';
     }
 }
diff --git a/media/com_wrapper/js/iframe-height.min.js b/media/com_wrapper/js/iframe-height.min.js
diff --git a/modules/mod_wrapper/helper.php b/modules/mod_wrapper/helper.php
@@ -54,7 +54,7 @@ public static function getParams(&$params)
 		// Auto height control
 		if ($params->def('height_auto'))
 		{
-			$load = 'onload="iFrameHeight()"';
+			$load = 'onload="iFrameHeight(this)"';
 		}
 		else
 		{