Fix Javascript ReferenceError by using target=_blank instead of popup() in a link
authorNicolas Iooss <nicolas.iooss_git@polytechnique.org>
Sun, 3 Nov 2013 17:45:51 +0000 (18:45 +0100)
committerNicolas Iooss <nicolas.iooss_git@polytechnique.org>
Sun, 3 Nov 2013 17:48:34 +0000 (18:48 +0100)
Fix following reported Javascript error when clicking on the education link
present in search results: "Uncaught ReferenceError: popup is not defined"

Signed-off-by: Nicolas Iooss <nicolas.iooss_git@polytechnique.org>
plugins/function.display_education.php

index 4690f74..143c474 100644 (file)
@@ -32,7 +32,7 @@ function display_education($name, $url, $degree, $gradYear, $field, $program, $f
         }
     }
     if ($url != '') {
-        $txt .= '<a href="' . $url . '" onclick="return popup(this)">' . $name . '</a>';
+        $txt .= '<a href="' . $url . '" target="_blank">' . $name . '</a>';
     } else {
         $txt .= $name;
     }