Merge pull request #404 from danbarretodev/BT8849

Fix get link method to replace & by & - refs BT#8849

main/newscorm/learnpath.class.php

@@ -3517,6 +3517,8 @@ class learnpath
                 default :
                     break;
             }
+            // Replace & by & because & will break URL with params
+            $file = !empty($file) ? str_replace('&', '&', $file) : '';
         }
         if ($this->debug > 2) {
             error_log('New LP - In learnpath::get_link() - returning "' . $file . '" from get_link', 0);

main/newscorm/lp_view.php

@@ -439,7 +439,6 @@ if ($is_allowed_to_edit) {
     <!-- right zone -->
     <div id="learning_path_right_zone" style="margin-left:<?php echo $margin_left;?>;height:100%">
     <?php
-    $src = !empty($src) ? str_replace('&amp;', '&', $src) : '';
         // hub 26-05-2010 Fullscreen or not fullscreen
         $height = '100%';
         if ($_SESSION['oLP']->mode == 'fullscreen') {