Issue #1196588: use ctype_digit to check if a given string is a number.
[project/print.git] / print_mail / print_mail.module
index 8d8ff57..bf1fca5 100644 (file)
@@ -1,16 +1,19 @@
 <?php
-// $Id$
 
 /**
  * @file
  * Displays Printer-friendly versions of Drupal pages.
+ *
+ * @ingroup print
  */
 
 define('PRINTMAIL_PATH', 'printmail');
 
-define('PRINT_MAIL_FORMAT', 'mail');
+// Defined in print.module
+//define('PRINT_MAIL_FORMAT', 'mail');
 
 define('PRINT_MAIL_LINK_POS_DEFAULT', 'link');
+define('PRINT_MAIL_LINK_TEASER_DEFAULT', 0);
 define('PRINT_MAIL_SHOW_LINK_DEFAULT', 1);
 define('PRINT_MAIL_NODE_LINK_VISIBILITY_DEFAULT', 0);
 define('PRINT_MAIL_NODE_LINK_PAGES_DEFAULT', '');
@@ -22,138 +25,173 @@ define('PRINT_MAIL_BOOK_LINK_DEFAULT', 1);
 define('PRINT_MAIL_HOURLY_THRESHOLD', 3);
 define('PRINT_MAIL_TEASER_DEFAULT_DEFAULT', 1);
 define('PRINT_MAIL_TEASER_CHOICE_DEFAULT', 1);
+define('PRINT_MAIL_SEND_OPTION_DEFAULT', 'sendpage');
+define('PRINT_MAIL_JOB_QUEUE_DEFAULT', 0);
+
+/**
+ * Implements hook_permission().
+ */
+function print_mail_permission() {
+  return array(
+    'access send to friend' => array(
+      'title' => t('Access the Send To Friend functionality'),
+      'description' => t('Provides the ability to send pages to a friend and the links to them in the original pages.'),
+    ),
+  );
+}
 
 /**
- * Implementation of hook_theme().
+ * Implements hook_theme().
  */
 function print_mail_theme() {
   return array(
     'print_mail_format_link' => array(
-      'arguments' => array(),
+      'variables' => array(),
     ),
     'print_mail_form' => array(
-      'arguments' => array('form'),
+      'variables' => array('form' => NULL),
+      'file' => 'print_mail.inc',
     ),
   );
 }
 
 /**
- * Implementation of hook_menu().
+ * Implements hook_menu().
  */
 function print_mail_menu() {
   $items = array();
 
   $items[PRINTMAIL_PATH] = array(
-    'title' => 'Send page by e-mail',
+    'title' => variable_get('print_mail_text_title', 'Send page by e-mail'),
     'page callback' => 'drupal_get_form',
     'page arguments' => array('print_mail_form'),
-    'access arguments' => array('access print'),
+    'access callback' => '_print_mail_access',
+    'access arguments' => array('access send to friend'),
     'type' => MENU_CALLBACK,
+    'file' => 'print_mail.inc',
   );
-  $items['admin/settings/print/email'] = array(
+  $items[PRINTMAIL_PATH . '/' . PRINTMAIL_PATH] = array(
+    'access callback' => FALSE,
+  );
+  $items['admin/config/user-interface/print/email'] = array(
     'title' => 'e-mail',
     'page callback' => 'drupal_get_form',
     'page arguments' => array('print_mail_settings'),
     'access arguments'  => array('administer print'),
     'weight' => 2,
     'type' => MENU_LOCAL_TASK,
+    'file' => 'print_mail.admin.inc',
+  );
+  $items['admin/config/user-interface/print/email/options'] = array(
+    'title' => 'Options',
+    'weight' => 1,
+    'type' => MENU_DEFAULT_LOCAL_TASK,
+  );
+  $items['admin/config/user-interface/print/email/strings'] = array(
+    'title' => 'Text strings',
+    'page callback' => 'drupal_get_form',
+    'page arguments' => array('print_mail_strings_settings'),
+    'access arguments'  => array('administer print'),
+    'weight' => 2,
+    'type' => MENU_LOCAL_TASK,
+    'file' => 'print_mail.admin.inc',
   );
 
   return $items;
 }
 
 /**
- * Implementation of hook_block().
+ * Implements hook_block_info().
  */
-function print_mail_block($op = 'list', $delta = 0, $edit = array()) {
-  switch ($op) {
-    case 'list':
-      $block[0]['info'] = t('Most emailed');
+function print_mail_block_info() {
+      $block['print_mail-top']['info'] = t('Most emailed');
+      $block['print_mail-top']['cache'] = DRUPAL_CACHE_GLOBAL;
       return $block;
-      break;
-    case 'configure':
-      return '';
-    case 'save':
-      return;
-    case 'view':
+}
+
+/**
+ * Implements hook_block_view().
+ */
+function print_mail_block_view($delta = 0) {
       switch ($delta) {
-      case 0:
+      case 'print_mail-top':
         $block['subject'] = t('Most emailed');
         $result = db_query_range("SELECT path FROM {print_mail_page_counter} ORDER BY sentcount DESC", 0, 3)
                     ->fetchAll();
         if (count($result)) {
           $block['content'] = '<div class="item-list"><ul>';
           foreach ($result as $obj) {
-            $block['content'] .= '<li>'. l(_print_get_title($obj->path), $obj->path) .'</li>';
+            $block['content'] .= '<li>' . l(_print_get_title($obj->path), $obj->path) . '</li>';
           }
           $block['content'] .= '</ul></div>';
         }
         break;
       }
       return $block;
-      break;
-    }
 }
 
 /**
- * Implementation of hook_link().
+ * Implements hook_node_view().
  */
-function print_mail_link($type, $node = NULL, $teaser = FALSE) {
+function print_mail_node_view($node, $build_mode) {
   $print_mail_link_pos = variable_get('print_mail_link_pos', array(PRINT_MAIL_LINK_POS_DEFAULT => PRINT_MAIL_LINK_POS_DEFAULT));
   $print_mail_link_use_alias = variable_get('print_mail_link_use_alias', PRINT_MAIL_LINK_USE_ALIAS_DEFAULT);
-  $allowed_type = print_mail_link_allowed(array('type' => $type, 'node' => $node, 'teaser' => $teaser));
+  $allowed_type = print_mail_link_allowed(array('type' => 'node', 'node' => $node, 'teaser' => ($build_mode == 'teaser')));
   if (($allowed_type) && !empty($print_mail_link_pos['link'])) {
-    drupal_add_css(drupal_get_path('module', 'print') .'/css/printlinks.css');
+    drupal_add_css(drupal_get_path('module', 'print') . '/css/printlinks.css');
     $links = array();
     $format = theme('print_mail_format_link');
 
     // Show book link
     if ($allowed_type === PRINT_ALLOW_BOOK_LINK) {
-      $links['book_mail'] = array('href' => PRINTMAIL_PATH .'/book/export/html/'. $node->nid,
+      $links['book_mail'] = array('href' => PRINTMAIL_PATH . '/book/export/html/' . $node->nid,
                                  'title' => $format['text'],
                                  'attributes' => $format['attributes'],
                                  'html' => $format['html'],
       );
-
-      return $links;
     }
     elseif ($allowed_type === PRINT_ALLOW_NORMAL_LINK) {
       $query_arr = $_GET;
-      if ($type == 'comment') {
+      if ('node' == 'comment') {
         $query_arr['comment'] = $node->cid;
       }
       $query = print_query_string_encode($query_arr, array('q'));
       if (empty($query)) $query = NULL;
 
       if ($print_mail_link_use_alias) {
-        $path = drupal_get_path_alias('node/'. $node->nid);
+        $path = drupal_get_path_alias('node/' . $node->nid);
       }
       else {
         $path = $node->nid;
       }
 
-      $links['print_mail'] = array('href' => PRINTMAIL_PATH .'/'. $path,
+      $links['print_mail'] = array('href' => PRINTMAIL_PATH . '/' . $path,
                                   'title' => $format['text'],
                                   'attributes' => $format['attributes'],
                                   'html' => $format['html'],
                                   'query' => $query,
       );
-
-      return $links;
     }
+
+    $node->content['links']['print_mail'] = array(
+      '#theme' => 'links',
+      '#links' => $links,
+      '#attributes' => array('class' => array('links', 'inline')),
+    );
   }
-  else {
-    return;
+
+  // Insert content corner links
+  if (!empty($print_mail_link_pos['corner']) && ($build_mode == 'full')) {
+    $node->content['print_links']['#markup'] .= print_mail_insert_link(NULL, $node);
   }
 }
 
 /**
- * Implementation of hook_help().
+ * Implements hook_help().
  */
 function print_mail_help($path, $arg) {
   $print_mail_link_pos = variable_get('print_mail_link_pos', array(PRINT_MAIL_LINK_POS_DEFAULT => PRINT_MAIL_LINK_POS_DEFAULT));
-  if ((preg_match('!^node/!i', $path) == 0) &&
-      !(empty($print_mail_link_pos['link']) && empty($print_mail_link_pos['corner']))) {
+  if (($path !== 'node/%') && !(empty($print_mail_link_pos['link']) && empty($print_mail_link_pos['corner']))) {
     static $output = FALSE;
 
     if ($output === FALSE) {
@@ -168,61 +206,62 @@ function print_mail_help($path, $arg) {
 }
 
 /**
- * Implementation of hook_nodeapi_view().
+ * Implements hook_node_load().
  */
-function print_mail_nodeapi_view(&$node, $teaser, $page) {
-      // Insert content corner links
-      if ($teaser === FALSE) {
-        _print_mail_set_node_fields($node);
-      }
-      $print_mail_link_pos = variable_get('print_mail_link_pos', array(PRINT_MAIL_LINK_POS_DEFAULT => PRINT_MAIL_LINK_POS_DEFAULT));
-      if (($teaser === FALSE) && !empty($print_mail_link_pos['corner']) &&
-          (preg_match('!^print!i', $_GET['q']) == 0)) {
-        $link = print_mail_insert_link(NULL, $node);
-        if ($link) {
-          $node->content['print_mail_link'] = array(
-            '#markup' => "<span class='print-link'>$link</span>",
-            '#weight' => -102,
-          );
-        }
-      }
+function print_mail_node_load($nodes, $types) {
+  foreach ($nodes as $node) {
+    _print_mail_set_node_fields($node);
+  }
 }
 
 /**
- * Implementation of hook_nodeapi_prepare().
+ * Implements hook_node_insert().
  */
-function print_mail_nodeapi_prepare(&$node, $teaser, $page) {
-  _print_mail_set_node_fields($node);
+function print_mail_node_insert($node) {
+  if (user_access('administer print') || user_access('node-specific print configuration')) {
+    if ($node->print_mail_display === NULL) $node->print_mail_display = variable_get('print_mail_display_' . $node->type, PRINT_TYPE_SHOW_LINK_DEFAULT);
+    if ($node->print_mail_display_comment === NULL) $node->print_mail_display_comment = variable_get('print_mail_display_comment_' . $node->type, PRINT_TYPE_COMMENT_LINK_DEFAULT);
+    if ($node->print_mail_display_urllist === NULL) $node->print_mail_display_urllist = variable_get('print_mail_display_urllist_' . $node->type, PRINT_TYPE_URLLIST_DEFAULT);
+
+    _print_mail_node_conf_modify($node->nid, $node->print_mail_display, $node->print_mail_display_comment, $node->print_mail_display_urllist);
+  }
 }
 
 /**
- * Implementation of hook_nodeapi_update().
+ * Implements hook_node_update().
  */
-function print_mail_nodeapi_update(&$node, $teaser, $page) {
-  _print_mail_node_conf_modify($node->nid, $node->print_mail_display, $node->print_mail_display_comment, $node->print_mail_display_urllist);
+function print_mail_node_update($node) {
+  if (user_access('administer print') || user_access('node-specific print configuration')) {
+    if ($node->print_mail_display === NULL) $node->print_mail_display = variable_get('print_mail_display_' . $node->type, PRINT_TYPE_SHOW_LINK_DEFAULT);
+    if ($node->print_mail_display_comment === NULL) $node->print_mail_display_comment = variable_get('print_mail_display_comment_' . $node->type, PRINT_TYPE_COMMENT_LINK_DEFAULT);
+    if ($node->print_mail_display_urllist === NULL) $node->print_mail_display_urllist = variable_get('print_mail_display_urllist_' . $node->type, PRINT_TYPE_URLLIST_DEFAULT);
+
+    _print_mail_node_conf_modify($node->nid, $node->print_mail_display, $node->print_mail_display_comment, $node->print_mail_display_urllist);
+  }
 }
 
 /**
- * Implementation of hook_nodeapi_delete().
+ * Implements hook_node_delete().
  */
-function print_mail_nodeapi_delete(&$node, $teaser, $page) {
+function print_mail_node_delete($node) {
   db_delete('print_mail_node_conf')
     ->condition('nid', $node->nid)
     ->execute();
   db_delete('print_mail_page_counter')
-    ->condition('path', $node->path)
+    ->condition('path', 'node/' . $node->nid)
     ->execute();
 }
 
 /**
- * Implementation of hook_form_alter().
+ * Implements hook_form_alter().
  */
-function print_mail_form_alter(&$form, $form_state, $form_id) {
+function print_mail_form_alter(&$form, &$form_state, $form_id) {
   // Add the node-type settings option to activate the mail version link
-  if (($form_id == 'node_type_form') || !empty($form['#node_edit_form'])) {
+  if ((user_access('administer print') || user_access('node-specific print configuration')) &&
+      (($form_id == 'node_type_form') || !empty($form['#node_edit_form']))) {
     $form['print']['mail_label'] = array(
       '#type' => 'markup',
-      '#markup' => '<p><strong>'. t('Send by e-mail') .'</strong></p>',
+      '#markup' => '<p><strong>' . t('Send by e-mail') . '</strong></p>',
     );
 
     $form['print']['print_mail_display'] = array(
@@ -239,16 +278,62 @@ function print_mail_form_alter(&$form, $form_state, $form_id) {
     );
 
     if ($form_id == 'node_type_form') {
-      $form['print']['print_mail_display']['#default_value'] = variable_get('print_mail_display_'. $form['#node_type']->type, PRINT_TYPE_SHOW_LINK_DEFAULT);
-      $form['print']['print_mail_display_comment']['#default_value'] = variable_get('print_mail_display_comment_'. $form['#node_type']->type, PRINT_TYPE_COMMENT_LINK_DEFAULT);
-      $form['print']['print_mail_display_urllist']['#default_value'] = variable_get('print_mail_display_urllist_'. $form['#node_type']->type, PRINT_TYPE_URLLIST_DEFAULT);
+      $form['print']['print_mail_display']['#default_value'] = variable_get('print_mail_display_' . $form['#node_type']->type, PRINT_TYPE_SHOW_LINK_DEFAULT);
+      $form['print']['print_mail_display_comment']['#default_value'] = variable_get('print_mail_display_comment_' . $form['#node_type']->type, PRINT_TYPE_COMMENT_LINK_DEFAULT);
+      $form['print']['print_mail_display_urllist']['#default_value'] = variable_get('print_mail_display_urllist_' . $form['#node_type']->type, PRINT_TYPE_URLLIST_DEFAULT);
     }
     else {
-      $form['print']['print_mail_display']['#default_value'] = $form['#node']->print_mail_display;
-      $form['print']['print_mail_display_comment']['#default_value'] = $form['#node']->print_mail_display_comment;
-      $form['print']['print_mail_display_urllist']['#default_value'] = $form['#node']->print_mail_display_urllist;
+      $node = $form['#node'];
+      $form['print']['print_mail_display']['#default_value'] = isset($node->print_mail_display) ? $node->print_mail_display : variable_get('print_mail_display_' . $node->type, PRINT_TYPE_SHOW_LINK_DEFAULT);
+      $form['print']['print_mail_display_comment']['#default_value'] = isset($node->print_mail_display_comment) ? $node->print_mail_display_comment : variable_get('print_mail_display_comment_' . $node->type, PRINT_TYPE_COMMENT_LINK_DEFAULT);
+      $form['print']['print_mail_display_urllist']['#default_value'] = isset($node->print_mail_display_urllist) ? $node->print_mail_display_urllist : variable_get('print_mail_display_urllist_' . $node->type, PRINT_TYPE_URLLIST_DEFAULT);
+    }
+  }
+}
+
+/**
+ * Implements hook_job_queue_functions().
+ */
+function print_mail_job_queue_functions() {
+  $functions['print_mail'] = array(
+   'title' => t('Send to friend'),
+  );
+  return $functions;
+}
+
+/**
+ * Access callback to check a combination of user_acess() and page access
+ *
+ * @param $permission
+ *   permission required to view the page
+ * @return
+ *   TRUE if the user has permission to view the page, FALSE otherwise
+ */
+function _print_mail_access($permission) {
+  $page_access = TRUE;
+  $parts = explode('/', $_GET['q']);
+  if ($parts[0] == PRINTMAIL_PATH) {
+    if (count($parts) > 1) {
+      unset($parts[0]);
+      $path = implode('/', $parts);
+      if (ctype_digit($parts[1])) {
+        if (drupal_lookup_path('source', $path)) {
+          // This is a numeric alias
+          $path = drupal_get_normal_path($path);
+        }
+        else {
+          // normal nid
+          $path = 'node/' . $path;
+        }
+      }
+      // If the destination page is not accessible, don't show the form
+      if (!($router_item = menu_get_item($path)) || (!$router_item['access'])) {
+       $page_access = FALSE;
+      }
     }
   }
+
+  return (user_access($permission) && $page_access);
 }
 
 /**
@@ -260,14 +345,14 @@ function print_mail_form_alter(&$form, $form_state, $form_id) {
 function _print_mail_set_node_fields(&$node) {
   if (isset($node->nid)) {
     $res = db_query("SELECT link, comments, url_list FROM {print_mail_node_conf} WHERE nid = :nid", array(':nid' => $node->nid))
-             ->fetch();
+              ->fetch();
   }
   else {
     $res = FALSE;
   }
-  $node->print_mail_display = $res ? intval($res->link) : PRINT_TYPE_SHOW_LINK_DEFAULT;
-  $node->print_mail_display_comment = $res ? intval($res->comments) : PRINT_TYPE_COMMENT_LINK_DEFAULT;
-  $node->print_mail_display_urllist = $res ? intval($res->url_list) : PRINT_TYPE_URLLIST_DEFAULT;
+  $node->print_mail_display = $res ? intval($res->link) : variable_get('print_mail_display_' . $node->type, PRINT_TYPE_SHOW_LINK_DEFAULT);
+  $node->print_mail_display_comment = $res ? intval($res->comments) : variable_get('print_mail_display_comment_' . $node->type, PRINT_TYPE_COMMENT_LINK_DEFAULT);
+  $node->print_mail_display_urllist = $res ? intval($res->url_list) : variable_get('print_mail_display_urllist_' . $node->type, PRINT_TYPE_URLLIST_DEFAULT);
 }
 
 /**
@@ -284,13 +369,6 @@ function _print_mail_set_node_fields(&$node) {
  *   value of the url_list field (0 or 1)
  */
 function _print_mail_node_conf_modify($nid, $link, $comments, $url_list) {
-  if (($link == PRINT_TYPE_SHOW_LINK_DEFAULT) && ($comments == PRINT_TYPE_COMMENT_LINK_DEFAULT) &&
-      ($url_list == PRINT_TYPE_URLLIST_DEFAULT)) {
-    db_delete('print_mail_node_conf')
-      ->condition('nid', $nid)
-      ->execute();
-  }
-  else {
     db_merge('print_mail_node_conf')
       ->key(array('nid' => $nid))
       ->fields(array(
@@ -299,7 +377,6 @@ function _print_mail_node_conf_modify($nid, $link, $comments, $url_list) {
         'url_list' => $url_list,
       ))
       ->execute();
-  }
 }
 
 /**
@@ -312,9 +389,9 @@ function _print_mail_node_conf_modify($nid, $link, $comments, $url_list) {
 function theme_print_mail_format_link() {
   $print_mail_link_class  = variable_get('print_mail_link_class', PRINT_MAIL_LINK_CLASS_DEFAULT);
   $print_mail_show_link = variable_get('print_mail_show_link', PRINT_MAIL_SHOW_LINK_DEFAULT);
-  $print_mail_link_text = variable_get('print_mail_link_text', t('Send to friend'));
+  $print_mail_link_text = filter_xss(variable_get('print_mail_link_text', t('Send to friend')));
 
-  $img = drupal_get_path('module', 'print') .'/icons/mail_icon.gif';
+  $img = drupal_get_path('module', 'print') . '/icons/mail_icon.gif';
   $title = t('Send this page by e-mail.');
   $class = strip_tags($print_mail_link_class);
   $new_window = FALSE;
@@ -344,7 +421,7 @@ function theme_print_mail_format_link() {
 function print_mail_insert_link($path = NULL, $node = NULL) {
   if ($node !== NULL) {
     $nid = $node->nid;
-    $path = 'node/'. $nid;
+    $path = 'node/' . $nid;
     $allowed_type = print_mail_link_allowed(array('node' => $node));
   }
   else {
@@ -361,7 +438,7 @@ function print_mail_insert_link($path = NULL, $node = NULL) {
   if ($allowed_type) {
     if ($nid !== NULL) {
       if ($allowed_type === PRINT_ALLOW_BOOK_LINK) {
-        $path = 'book/export/html/'. $nid;
+        $path = 'book/export/html/' . $nid;
       }
       else {
         if (variable_get('print_mail_link_use_alias', PRINT_MAIL_LINK_USE_ALIAS_DEFAULT)) {
@@ -371,7 +448,7 @@ function print_mail_insert_link($path = NULL, $node = NULL) {
           $path = $nid;
         }
       }
-      $path = PRINTMAIL_PATH .'/'. $path;
+      $path = PRINTMAIL_PATH . '/' . $path;
       $query = print_query_string_encode($_GET, array('q'));
       if (empty($query)) {
         $query = NULL;
@@ -380,9 +457,9 @@ function print_mail_insert_link($path = NULL, $node = NULL) {
     else {
       $query = NULL;
     }
-    drupal_add_css(drupal_get_path('module', 'print') .'/css/printlinks.css');
+    drupal_add_css(drupal_get_path('module', 'print') . '/css/printlinks.css');
     $format = theme('print_mail_format_link');
-    return '<span class="print_mail">'. l($format['text'], $path, array('attributes' => $format['attributes'], 'query' => $query, 'absolute' => TRUE, 'html' => $format['html'])) .'</span>';
+    return '<span class="print_mail">' . l($format['text'], $path, array('attributes' => $format['attributes'], 'query' => $query, 'absolute' => TRUE, 'html' => $format['html'])) . '</span>';
   }
   else {
     return FALSE;
@@ -401,13 +478,14 @@ function print_mail_insert_link($path = NULL, $node = NULL) {
  *   PRINT_ALLOW_BOOK_LINK if a link is allowed in a book node
  */
 function print_mail_link_allowed($args) {
-  if (!empty($args['teaser']) || !user_access('access print')) {
-    // If showing only the teaser or the user is not allowed or link is disabled
+  if ((!empty($args['teaser']) && !variable_get('print_mail_link_teaser', PRINT_MAIL_LINK_TEASER_DEFAULT))
+      || !user_access('access send to friend')) {
+    // If the teaser link is disabled or the user is not allowed
     return FALSE;
   }
   if (!empty($args['path'])) {
     $nid = preg_replace('!^node/!', '', drupal_get_normal_path($args['path']));
-    if (is_numeric($nid)) {
+    if (ctype_digit($nid)) {
       $args['node'] = node_load($nid);
     }
   }
@@ -415,38 +493,34 @@ function print_mail_link_allowed($args) {
     static $node_type = FALSE;
 
     $node = $args['node'];
-    if ($node_type === FALSE) {
-      if (isset($node->type)) {
-        $node_type = $node->type;
-      }
-      else {
-        $node_type = '';
-      }
+    if (isset($node->type)) {
+      $node_type = $node->type;
     }
     // Node
     $print_mail_node_link_visibility = variable_get('print_mail_node_link_visibility', PRINT_MAIL_NODE_LINK_VISIBILITY_DEFAULT);
     $print_mail_node_link_pages = variable_get('print_mail_node_link_pages', PRINT_MAIL_NODE_LINK_PAGES_DEFAULT);
 
-    if (!empty($node->printing) ||
-        !_print_page_match($print_mail_node_link_visibility, $print_mail_node_link_pages)) {
-      // Page not in visibility list or we are working!
+    if (!_print_page_match($print_mail_node_link_visibility, "node/" . $node->nid, $print_mail_node_link_pages)) {
+      // Page not in visibility list
       return FALSE;
     }
     elseif (isset($args['type']) && ($args['type'] == 'comment') && isset($node_type)) {
       // Link is for a comment, return the configured setting
-      $res = db_query("SELECT comments FROM {print_mail_node_conf} WHERE nid = :nid", array(':nid' => $node->nid))
-               ->fetch();
-      $print_display_comment = $res ? intval($res->comments) : PRINT_TYPE_COMMENT_LINK_DEFAULT;
-      if (($print_display_comment) ||
-          variable_get('print_mail_display_comment_'. $node_type, PRINT_TYPE_COMMENT_LINK_DEFAULT)) {
+      // Cache this statically to avoid duplicate queries for every comment.
+      static $res = array();
+      if (!isset($res[$node->nid])) {
+        $res[$node->nid] = db_query("SELECT comments FROM {print_mail_node_conf} WHERE nid = :nid", array(':nid' => $node->nid))
+                                    ->fetch();
+      }
+      $print_display_comment = $res ? intval($res[$node->nid]->comments) : variable_get('print_mail_display_comment_' . $node_type, PRINT_TYPE_COMMENT_LINK_DEFAULT);
+      if ($print_display_comment) {
         return PRINT_ALLOW_NORMAL_LINK;
       }
     }
     else {
       // Node link
-      if ((!$node->print_mail_display) || (isset($node_type) &&
-          !variable_get('print_mail_display_'. $node_type, PRINT_TYPE_SHOW_LINK_DEFAULT))) {
-        // Link for this node type is disabled
+      if (!$node->print_mail_display) {
+        // Link for this node is disabled
         return FALSE;
       }
       elseif (isset($node->book)) {
@@ -472,7 +546,7 @@ function print_mail_link_allowed($args) {
     $print_mail_sys_link_visibility = variable_get('print_mail_sys_link_visibility', PRINT_MAIL_SYS_LINK_VISIBILITY_DEFAULT);
     $print_mail_sys_link_pages = variable_get('print_mail_sys_link_pages', PRINT_MAIL_SYS_LINK_PAGES_DEFAULT);
 
-    return _print_page_match($print_mail_sys_link_visibility, $print_mail_sys_link_pages);
+    return _print_page_match($print_mail_sys_link_visibility, $_GET['q'], $print_mail_sys_link_pages);
   }
   return FALSE;
 }