Permalink
Browse files

FIX: insert javascript requirements before the first inline script.

  • Loading branch information...
1 parent 23ed533 commit 9f6eeb4d0ea7c0ba0b88af92d6725a00778e17ad @simonwelsh simonwelsh committed with wilr Jul 1, 2012
Showing with 3 additions and 2 deletions.
  1. +3 −2 view/Requirements.php
View
@@ -695,9 +695,10 @@ function includeInHTML($templateFile, $content) {
// We put script tags into the body, for performance.
// If your template already has script tags in the body, then we put our script
// tags just before those. Otherwise, we put it at the bottom.
- $p1 = strripos($content, '<script');
$p2 = stripos($content, '<body');
- if($p1 !== false && $p1 > $p2) {
+ $p1 = stripos($content, '<script', $p2);
+
+ if($p1 !== false) {
$content = substr($content,0,$p1) . $jsRequirements . substr($content,$p1);
} else {
$content = preg_replace("/(<\/body[^>]*>)/i", $jsRequirements . "\\1", $content);

0 comments on commit 9f6eeb4

Please sign in to comment.