Following up on my long standing threat to remove internal_link hook. Templates...

[squirrelmail.git] / functions / page_header.php
diff --git a/functions/page_header.php b/functions/page_header.php

index 9561f0f4a1e3603179126b85b4a318b316873ba2..8fed43b9dc722fa41f3c0c8e4aaaf8eb8c7a4bb1 100644 (file)
--- a/functions/page_header.php
+++ b/functions/page_header.php
@@ -96,9 +96,18 @@ function makeInternalLink($path, $text, $target='') {
      if ($target != '') {
          $target = " target=\"$target\"";
      }
-    $hooktext = do_hook_function('internal_link',$text);
-    if ($hooktext != '')
-        $text = $hooktext;
+
+    // This is an inefficient hook and is only used by 
+    // one plugin that still needs to patch this code,
+    // plus if we are templat-izing SM, visual hooks 
+    // are not needed.  However, I am leaving the code 
+    // here just in case we find a good (non-visual?) 
+    // use for the internal_link hook.
+    //
+    //$hooktext = do_hook_function('internal_link',$text);
+    //if ($hooktext != '')
+    //    $text = $hooktext;
+
      return '<a href="'.$base_uri.$path.'"'.$target.'>'.$text.'</a>';
  }
  
@@ -431,4 +440,4 @@ function compose_Header($color, $mailbox) {
      echo "<body text=\"$color[8]\" bgcolor=\"$color[4]\" link=\"$color[7]\" vlink=\"$color[7]\" alink=\"$color[7]\" $onload>\n\n";
  }
  
-?>
-\ No newline at end of file
+?>