Issue #1188232: remove deprecated call to AliasNbPages() in TCPDF.
[project/print.git] / print_pdf / print_pdf.pages.inc
index a3ed60d..29443e3 100644 (file)
@@ -1,5 +1,4 @@
 <?php
-// $Id$
 
 /**
  * @file
@@ -7,29 +6,59 @@
  *
  * This file is included by the print_pdf module and includes the
  * functions that interface with the PDF generation packages.
+ *
+ * @ingroup print
  */
 
-require_once(DRUPAL_ROOT .'/'. drupal_get_path('module', 'print') .'/print.pages.inc');
+module_load_include('inc', 'print', 'print.pages');
 
 /**
  * Generate a PDF version of the printer-friendly page
  *
  * @see print_controller()
- * @see _print_get_template()
  * @see _print_pdf_dompdf()
  * @see _print_pdf_tcpdf()
  */
 function print_pdf_controller() {
   global $base_url;
 
+  // Disable caching for generated PDFs, as Drupal doesn't ouput the proper headers from the cache
+  $GLOBALS['conf']['cache'] = FALSE;
+
   $args = func_get_args();
-  // Remove the printpdf/ prefix
   $path = implode('/', $args);
   $cid = isset($_GET['comment']) ? (int)$_GET['comment'] : NULL;
 
-  $print_pdf_pdf_tool = variable_get('print_pdf_pdf_tool', PRINT_PDF_PDF_TOOL_DEFAULT);
+  $pdf_filename = variable_get('print_pdf_filename', PRINT_PDF_FILENAME_DEFAULT);
+  if (!empty($pdf_filename)) {
+    $pdf_filename = token_replace($pdf_filename, array('node' => $node)) . '.pdf';
+  }
+  else {
+    $pdf_filename = str_replace('/', '_', $path) . '.pdf';
+  }
+  if (function_exists('transliteration_clean_filename')) {
+    $pdf_filename = transliteration_clean_filename($pdf_filename, language_default('language'));
+  }
 
-  $print = print_controller($path, $cid);
+  $pdf = print_pdf_generate_path($path, $cid, $pdf_filename);
+  if ($pdf == NULL) {
+    drupal_goto($print['url']);
+    exit;
+  }
+
+  $nodepath = (isset($node->path)) ? drupal_get_normal_path($node->path) : 'node/' . $path;
+  db_merge('print_pdf_page_counter')
+    ->key(array('path' => $nodepath))
+    ->fields(array(
+        'totalcount' => 1,
+        'timestamp' => REQUEST_TIME,
+    ))
+    ->expression('totalcount', 'totalcount + :inc', array(':inc' => 1))
+    ->execute();
+}
+
+function print_pdf_generate_path($path, $cid = NULL, $pdf_filename = NULL) {
+  $print = print_controller($path, $cid, PRINT_PDF_FORMAT);
   if ($print === FALSE) {
     return;
   }
@@ -39,31 +68,78 @@ function print_pdf_controller() {
   $print['content'] = preg_replace_callback($pattern, '_print_rewrite_urls', $print['content']);
   $print['logo'] = preg_replace_callback($pattern, '_print_rewrite_urls', $print['logo']);
   $print['footer_message'] = preg_replace_callback($pattern, '_print_rewrite_urls', $print['footer_message']);
-  // And converted from private to public paths
-  $file_downloads = variable_get('file_downloads', FILE_DOWNLOADS_PUBLIC);
-  if ($file_downloads == FILE_DOWNLOADS_PRIVATE) {
-    $pattern = "!(<img\s[^>]*?src\s*?=\s*?['\"]?${base_url}/)system/files(/[^>]*?>)!is";
-    $replacement = '$1'. file_directory_path() .'$2';
-    $print['content'] = preg_replace($pattern, $replacement, $print['content']);
-    $print['logo'] = preg_replace($pattern, $replacement, $print['logo']);
-    $print['footer_message'] = preg_replace($pattern, $replacement, $print['footer_message']);
-  }
+
+  // Send to printer option causes problems with PDF
+  $print['sendtoprinter'] = '';
 
   $node = $print['node'];
-  ob_start();
-  include_once(DRUPAL_ROOT .'/'. _print_get_template('pdf', $print['type']));
-  $html = ob_get_contents();
-  ob_end_clean();
+  $html = theme('print', array('print' => $print, 'type' => PRINT_PDF_FORMAT, 'node' => $node));
+
+  // Convert the a href elements, to make sure no relative links remain
+  $pattern = '!<(a\s[^>]*?)>!is';
+  $html = preg_replace_callback($pattern, '_print_rewrite_urls', $html);
+  // And make anchor links relative again, to permit in-PDF navigation
+  $html = preg_replace("!${base_url}/" . PRINTPDF_PATH . "/.*?%2523!", '#', $html);
+
+  return print_pdf_generate_html($print, $html, $pdf_filename);
+}
+
+function print_pdf_generate_html($print, $html, $filename = NULL) {
+  $print_pdf_pdf_tool = variable_get('print_pdf_pdf_tool', PRINT_PDF_PDF_TOOL_DEFAULT);
 
   if (basename($print_pdf_pdf_tool) == 'dompdf_config.inc.php') {
-    _print_pdf_dompdf($print, $html, $path .'.pdf');
+    return _print_pdf_dompdf($print, $html, $filename);
   }
   elseif (basename($print_pdf_pdf_tool) == 'tcpdf.php') {
-    _print_pdf_tcpdf($print, $html, $path .'.pdf');
+    return _print_pdf_tcpdf($print, $html, $filename);
   }
-  else {
+  elseif (substr(basename($print_pdf_pdf_tool, '.exe'), 0, 11) == 'wkhtmltopdf') {
+    return _print_pdf_wkhtmltopdf($print, $html, $filename);
+  }
+  elseif ($filename) {
     return drupal_not_found();
   }
+  return NULL;
+}
+
+/**
+ * Convert image paths to the file:// protocol
+ *
+ * In some Drupal setups, the use of the 'private' filesystem or Apache's
+ * configuration prevent access to the images of the page. This function
+ * tries to circumnvent those problems by accessing files in the local
+ * filesystem.
+ *
+ * @param $html
+ *   contents of the post-processed template already with the node data
+ * @see print_pdf_controller()
+ */
+function _print_pdf_file_access_images($html) {
+  global $base_url, $language;
+
+  // TODO: This needs to be rewritten from scratch for Drupal 7
+/*
+  // And converted from private to public paths
+  switch (variable_get('language_negotiation', LANGUAGE_NEGOTIATION_NONE)) {
+    case LANGUAGE_NEGOTIATION_PATH_DEFAULT:
+    case LANGUAGE_NEGOTIATION_PATH:
+      $lang = $language->language;
+      break;
+    default:
+      $lang = '';
+      break;
+  }
+  $file_downloads = variable_get('file_default_scheme', 'public');
+  if ($file_downloads == 'private') {
+    $pattern = "!(<img\s[^>]*?src\s*?=\s*?['\"]?)${base_url}/(?:(?:index.php)?\?q=)?(?:${lang}/)?system/files/([^>]*?>)!is";
+    $replacement = '$1file://' . realpath(file_directory_path()) . '/$2';
+    $html = preg_replace($pattern, $replacement, $html);
+  }
+  $pattern = "!(<img\s[^>]*?src\s*?=\s*?['\"]?)${base_url}/(?:(?:index.php)?\?q=)?(?:${lang}/)?([^>]*?>)!is";
+  $replacement = '$1file://' . dirname($_SERVER['SCRIPT_FILENAME']) . '/$2';
+  $html = preg_replace($pattern, $replacement, $html);
+*/
+  return $html;
 }
 
 /**
@@ -77,21 +153,33 @@ function print_pdf_controller() {
  *   name of the PDF file to be generated
  * @see print_pdf_controller()
  */
-function _print_pdf_dompdf($print, $html, $filename) {
+function _print_pdf_dompdf($print, $html, $filename = NULL) {
   $print_pdf_pdf_tool = variable_get('print_pdf_pdf_tool', PRINT_PDF_PDF_TOOL_DEFAULT);
   $print_pdf_paper_size = variable_get('print_pdf_paper_size', PRINT_PDF_PAPER_SIZE_DEFAULT);
   $print_pdf_page_orientation = variable_get('print_pdf_page_orientation', PRINT_PDF_PAGE_ORIENTATION_DEFAULT);
   $print_pdf_content_disposition = variable_get('print_pdf_content_disposition', PRINT_PDF_CONTENT_DISPOSITION_DEFAULT);
-  require_once(DRUPAL_ROOT .'/'. $print_pdf_pdf_tool);
+
+  if (variable_get('print_pdf_autoconfig', PRINT_PDF_AUTOCONFIG_DEFAULT)) {
+    define("DOMPDF_ENABLE_PHP", FALSE);
+    define("DOMPDF_ENABLE_REMOTE", TRUE);
+    define("DOMPDF_TEMP_DIR", file_directory_temp());
+    define("DOMPDF_UNICODE_ENABLED", variable_get('print_pdf_dompdf_unicode', PRINT_PDF_DOMPDF_UNICODE_DEFAULT));
+  }
+
+  require_once(DRUPAL_ROOT . '/' . $print_pdf_pdf_tool);
+  spl_autoload_register('DOMPDF_autoload');
+
+  // Try to use local file access for image files
+  $html = _print_pdf_file_access_images($html);
 
   // dompdf seems to have problems with something in system.css so let's not use it
   $html = preg_replace('!<link.*?modules/system/system.css.*?/>!', '', $html);
 
   $url_array  = parse_url($print['url']);
 
-  $protocol = $url_array['scheme'] .'://';
+  $protocol = $url_array['scheme'] . '://';
   $host = $url_array['host'];
-  $path = dirname($url_array['path']) .'/';
+  $path = dirname($url_array['path']) . '/';
 
   $dompdf = new DOMPDF();
   $dompdf->set_base_path($path);
@@ -99,12 +187,44 @@ function _print_pdf_dompdf($print, $html, $filename) {
   $dompdf->set_paper(drupal_strtolower($print_pdf_paper_size), $print_pdf_page_orientation);
   $dompdf->set_protocol($protocol);
 
-  $html = theme('print_pdf_dompdf_footer', $html);
+// dompdf can't handle footers cleanly, so disable the following
+//  $html = theme('print_pdf_dompdf_footer', array('html' => $html));
+
+  // If dompdf Unicode support is disabled, try to convert to ISO-8859-1 and then to HTML entities
+  if (!variable_get('print_pdf_dompdf_unicode', PRINT_PDF_DOMPDF_UNICODE_DEFAULT)) {
+  // Convert the euro sign to an HTML entity
+  $html = str_replace('€', '&#0128;', $html);
+
+  // Convert from UTF-8 to ISO 8859-1 and then to HTML entities
+  if (function_exists('utf8_decode')) {
+    $html = utf8_decode($html);
+  }
+// iconv fails silently when it encounters something that it doesn't know, so don't use it
+//  else if (function_exists('iconv')) {
+//    $html = iconv('UTF-8', 'ISO-8859-1', $html);
+//  }
+  elseif (function_exists('mb_convert_encoding')) {
+    $html = mb_convert_encoding($html, 'ISO-8859-1', 'UTF-8');
+  }
+  elseif (function_exists('recode_string')) {
+    $html = recode_string('UTF-8..ISO_8859-1', $html);
+  }
+  $html = htmlspecialchars_decode(htmlentities($html, ENT_NOQUOTES, 'ISO-8859-1'), ENT_NOQUOTES);
+  }
+
+  //must get rid of tbody (dompdf goes into recursion)
+  $html = preg_replace('!<tbody[^>]*?>|</tbody>!i', '', $html);
 
   $dompdf->load_html($html);
 
   $dompdf->render();
-  $dompdf->stream($filename, array('Attachment' => ($print_pdf_content_disposition == 2)));
+  if ($filename) {
+    $dompdf->stream($filename, array('Attachment' => ($print_pdf_content_disposition == 2)));
+    return TRUE;
+  }
+  else {
+    return $dompdf->output();
+  }
 }
 
 /**
@@ -118,8 +238,9 @@ function _print_pdf_dompdf($print, $html, $filename) {
  *   name of the PDF file to be generated
  * @see print_pdf_controller()
  */
-function _print_pdf_tcpdf($print, $html, $filename) {
-  global $base_url;
+function _print_pdf_tcpdf($print, $html, $filename = NULL) {
+  global $base_url, $language;
+
   $print_pdf_pdf_tool = variable_get('print_pdf_pdf_tool', PRINT_PDF_PDF_TOOL_DEFAULT);
   $print_pdf_paper_size = variable_get('print_pdf_paper_size', PRINT_PDF_PAPER_SIZE_DEFAULT);
   $print_pdf_page_orientation = variable_get('print_pdf_page_orientation', PRINT_PDF_PAGE_ORIENTATION_DEFAULT);
@@ -127,28 +248,35 @@ function _print_pdf_tcpdf($print, $html, $filename) {
 
   $pdf_tool_path = realpath(dirname($print_pdf_pdf_tool));
 
-  define('K_TCPDF_EXTERNAL_CONFIG', TRUE);
-  define('K_PATH_MAIN', dirname($_SERVER['SCRIPT_FILENAME']));
-  define('K_PATH_URL', $base_url);
-  define('K_PATH_FONTS', $pdf_tool_path .'/fonts/');
-  define('K_PATH_CACHE', $pdf_tool_path .'/cache/');
-  define('K_PATH_IMAGES', '');
-  define('K_BLANK_IMAGE', $pdf_tool_path .'/images/_blank.png');
-  define('K_CELL_HEIGHT_RATIO', 1.25);
-  define('K_SMALL_RATIO', 2/3);
-
-  require_once(DRUPAL_ROOT .'/'. $print_pdf_pdf_tool);
-  if (strpos(PDF_PRODUCER, 'PHP4') === FALSE) {
-    require_once(DRUPAL_ROOT .'/'. drupal_get_path('module', 'print_pdf') .'/print_pdf.class.inc');
-  }
-  else {
-    // Use only the PHP5 version of TCPDF
+  if (variable_get('print_pdf_autoconfig', PRINT_PDF_AUTOCONFIG_DEFAULT)) {
+    define('K_TCPDF_EXTERNAL_CONFIG', TRUE);
+    define('K_PATH_MAIN', dirname($_SERVER['SCRIPT_FILENAME']));
+    define('K_PATH_URL', $base_url);
+    define('K_PATH_FONTS', $pdf_tool_path . '/fonts/');
+    define('K_PATH_CACHE', $pdf_tool_path . '/cache/');
+    define('K_PATH_IMAGES', '');
+    define('K_BLANK_IMAGE', $pdf_tool_path . '/images/_blank.png');
+    define('K_CELL_HEIGHT_RATIO', 1.25);
+    define('K_SMALL_RATIO', 2/3);
   }
 
+  // Try to use local file access for image files
+  $html = _print_pdf_file_access_images($html);
+
+  // Decode HTML entities in image filenames
+  $pattern = "!<img\s[^>]*?src\s*?=\s*?['\"]?{$base_url}[^>]*?>!is";
+  $html = preg_replace_callback($pattern, create_function('$matches', 'return html_entity_decode($matches[0], ENT_QUOTES);'), $html);
+  // Remove queries from the image URL
+  $pattern = "!(<img\s[^>]*?src\s*?=\s*?['\"]?{$base_url}[^>]*?)(?:%3F|\?)[\w=&]+([^>]*?>)!is";
+  $html = preg_replace($pattern, '$1$2', $html);
+
+  require_once(DRUPAL_ROOT . '/' . $print_pdf_pdf_tool);
+  module_load_include('inc', 'print_pdf', 'print_pdf.class');
+
   $font = Array(
-    variable_get('print_pdf_font_family', PRINT_PDF_FONT_FAMILY_DEFAULT),
+    check_plain(variable_get('print_pdf_font_family', PRINT_PDF_FONT_FAMILY_DEFAULT)),
     '',
-    variable_get('print_pdf_font_size', PRINT_PDF_FONT_SIZE_DEFAULT),
+    check_plain(variable_get('print_pdf_font_size', PRINT_PDF_FONT_SIZE_DEFAULT)),
   );
   $orientation = drupal_strtoupper($print_pdf_page_orientation[0]);
 
@@ -158,29 +286,125 @@ function _print_pdf_tcpdf($print, $html, $filename) {
   // set document information
   $pdf->SetAuthor(strip_tags($print['submitted']));
   $pdf->SetCreator(variable_get('site_name', 'Drupal'));
-  $pdf->SetTitle($print['title']);
+  $pdf->SetTitle(html_entity_decode($print['title'], ENT_QUOTES, 'UTF-8'));
   $keys = implode(' ', explode("\n", trim(strip_tags($print['taxonomy']))));
   $pdf->SetKeywords($keys);
   $pdf->setPDFVersion('1.6');
+  $pdf->setFontSubsetting(variable_get('print_pdf_font_subsetting',FALSE));
 
-  $pdf = theme('print_pdf_tcpdf_header', $pdf, $html, $font);
-  $pdf = theme('print_pdf_tcpdf_footer', $pdf, $html, $font);
-  $pdf = theme('print_pdf_tcpdf_page', $pdf);
+  if ($language->direction == LANGUAGE_RTL) {
+    $pdf->setRTL(TRUE);
+  }
 
-  //initialize document
-  $pdf->AliasNbPages();
+  $pdf = theme('print_pdf_tcpdf_header', array('pdf' => $pdf, 'html' => $html, 'font' => $font));
+  $pdf = theme('print_pdf_tcpdf_footer', array('pdf' => $pdf, 'html' => $html, 'font' => $font));
+  $pdf = theme('print_pdf_tcpdf_page', array('pdf' => $pdf));
 
   // add a page
   $pdf->AddPage();
 
-  $pdf = theme('print_pdf_tcpdf_content', $pdf, $html, $font);
+  $pdf = theme('print_pdf_tcpdf_content', array('pdf' => $pdf, 'html' => $html, 'font' => $font));
 
   // reset pointer to the last page
   $pdf->lastPage();
 
-  //Close and output PDF document
-  $output_dest = ($print_pdf_content_disposition == 2) ? 'D' : 'I';
-  $pdf->Output($filename, $output_dest);
+  // try to recover from any warning/error
+  ob_clean();
+
+  if ($filename) {
+    //Close and output PDF document
+    $output_dest = ($print_pdf_content_disposition == 2) ? 'D' : 'I';
+    $pdf->Output($filename, $output_dest);
+    return TRUE;
+  }
+  else {
+    return $pdf = $pdf->Output('', 'S');
+  }
+}
+
+/**
+ * Generate the PDF file using wkhtmltopdf
+ *
+ * @param $print
+ *   array containing the configured data
+ * @param $html
+ *   contents of the post-processed template already with the node data
+ * @param $filename
+ *   name of the PDF file to be generated
+ * @see print_pdf_controller()
+ */
+function _print_pdf_wkhtmltopdf($print, $html, $filename = NULL) {
+  $print_pdf_pdf_tool = variable_get('print_pdf_pdf_tool', PRINT_PDF_PDF_TOOL_DEFAULT);
+  $print_pdf_paper_size = variable_get('print_pdf_paper_size', PRINT_PDF_PAPER_SIZE_DEFAULT);
+  $print_pdf_page_orientation = variable_get('print_pdf_page_orientation', PRINT_PDF_PAGE_ORIENTATION_DEFAULT);
+  $print_pdf_content_disposition = variable_get('print_pdf_content_disposition', PRINT_PDF_CONTENT_DISPOSITION_DEFAULT);
+  $print_pdf_wkhtmltopdf_options = variable_get('print_pdf_wkhtmltopdf_options', PRINT_PDF_WKHTMLTOPDF_OPTIONS);
+
+  $dpi = 96;
+
+  if (!empty($print_pdf_wkhtmltopdf_options)) {
+    $print_pdf_wkhtmltopdf_options = token_replace($print_pdf_wkhtmltopdf_options, array('node' => $print['node']));
+  }
+
+  $version = _print_pdf_wkhtmltopdf_version();
+
+  // 0.10.0 beta2 identifies itself as 0.9.9
+  if (version_compare($version, '0.9.9', '>=')) {
+    $print_pdf_wkhtmltopdf_options = '--disable-local-file-access ' . $print_pdf_wkhtmltopdf_options;
+  }
+  elseif (version_compare($version, '0.9.6', '>=')) {
+    $print_pdf_wkhtmltopdf_options = '--disallow-local-file-access ' . $print_pdf_wkhtmltopdf_options;
+  }
+  else {
+    drupal_goto($print['url']);
+    exit;
+  }
+
+  $descriptor = array(0 => array('pipe', 'r'), 1 => array('pipe', 'w'), 2 => array('pipe', 'w'));
+  $cmd = realpath($print_pdf_pdf_tool) . " --page-size $print_pdf_paper_size --orientation $print_pdf_page_orientation --dpi $dpi $print_pdf_wkhtmltopdf_options - -";
+
+  $process = proc_open($cmd, $descriptor, $pipes, NULL, NULL);
+
+  if (is_resource($process)) {
+    fwrite($pipes[0], $html);
+    fclose($pipes[0]);
+
+    $pdf = stream_get_contents($pipes[1]);
+    fclose($pipes[1]);
+
+    stream_set_blocking($pipes[2], 0);
+    $error = stream_get_contents($pipes[2]);
+    if (!empty($error)) {
+      watchdog('print_pdf', 'wkhtmltopdf: ' . $error);
+    }
+    fclose($pipes[2]);
+
+    $retval = proc_terminate($process);
+  }
+
+  if (!empty($pdf)) {
+    if ($filename) {
+      if (headers_sent()) {
+        die("Unable to stream pdf: headers already sent");
+      }
+      header("Cache-Control: private");
+      header("Content-Type: application/pdf");
+
+      $attachment =  ($print_pdf_content_disposition == 2) ?  "attachment" :  "inline";
+
+      header("Content-Disposition: $attachment; filename=\"$filename\"");
+
+      echo $pdf;
+      flush();
+      return TRUE;
+    }
+    else {
+      return $pdf;
+    }
+  }
+  else {
+    return NULL;
+  }
 }
 
 /**
@@ -190,9 +414,9 @@ function _print_pdf_tcpdf($print, $html, $filename) {
  *   contents of the body of the HTML from the original node
  * @see theme_print_pdf_tcpdf_footer()
  */
-function theme_print_pdf_dompdf_footer(&$html) {
-  preg_match('!<div class="print-footer">(.*?)</div>!si', $html, $tpl_footer);
-  $html = str_replace($tpl_footer[0], '', $html);
+function theme_print_pdf_dompdf_footer($vars) {
+  preg_match('!<div class="print-footer">(.*?)</div>!si', $vars['html'], $tpl_footer);
+  $html = str_replace($tpl_footer[0], '', $vars['html']);
 
   $text = '<script type="text/php">
     if (isset($pdf)) {
@@ -211,7 +435,7 @@ function theme_print_pdf_dompdf_footer(&$html) {
       $pdf->line(15, $y, $w - 15, $y, $color, 1);
 
       $y += $text_height / 2;
-      $pdf->page_text(15, $y, \''. addslashes(strip_tags($tpl_footer[1])) .'\', $font, $size, $color);
+      $pdf->page_text(15, $y, \'' . addslashes(strip_tags($tpl_footer[1])) . '\', $font, $size, $color);
 
       $pdf->close_object();
       $pdf->add_object($footer, "all");
@@ -237,10 +461,11 @@ function theme_print_pdf_dompdf_footer(&$html) {
  *   array with the font definition (font name, styles and size)
  * @see theme_print_pdf_tcpdf_header()
  */
-function theme_print_pdf_tcpdf_header(&$pdf, &$html, $font) {
-  preg_match('!<div class="print-logo">(.*?)</div>!si', $html, $tpl_logo);
-  preg_match('!<h1 class="print-title">(.*?)</h1>!si', $html, $tpl_title);
-  preg_match('!<div class="print-site_name">(.*?)</div>!si', $html, $tpl_site_name);
+function theme_print_pdf_tcpdf_header($vars) {
+  $pdf = $vars['pdf'];
+  preg_match('!<div class="print-logo">(.*?)</div>!si', $vars['html'], $tpl_logo);
+  preg_match('!<h1 class="print-title">(.*?)</h1>!si', $vars['html'], $tpl_title);
+  preg_match('!<div class="print-site_name">(.*?)</div>!si', $vars['html'], $tpl_site_name);
 
   $ratio = 0;
   $logo = '';
@@ -248,15 +473,15 @@ function theme_print_pdf_tcpdf_header(&$pdf, &$html, $font) {
   if ($logo_ret) {
     $logo = trim($matches[1], '\'"');
     $size = getimagesize($logo);
-    $ratio = $size[0] / $size[1];
+    $ratio = $size ? ($size[0] / $size[1]) : 0;
   }
 
   // set header font
-  $pdf->setHeaderFont($font);
+  $pdf->setHeaderFont($vars['font']);
   // set header margin
   $pdf->SetHeaderMargin(5);
   // set header data
-  $pdf->SetHeaderData($logo, 10 * $ratio, $tpl_title[1], strip_tags($tpl_site_name[1]));
+  $pdf->SetHeaderData($logo, 10 * $ratio, html_entity_decode($tpl_title[1], ENT_QUOTES, 'UTF-8'), html_entity_decode(strip_tags($tpl_site_name[1]), ENT_QUOTES, 'UTF-8'));
 
   return $pdf;
 }
@@ -268,13 +493,14 @@ function theme_print_pdf_tcpdf_header(&$pdf, &$html, $font) {
  *   current TCPDF object
  * @see theme_print_pdf_tcpdf_page()
  */
-function theme_print_pdf_tcpdf_page(&$pdf) {
+function theme_print_pdf_tcpdf_page($vars) {
+  $pdf = $vars['pdf'];
   // set margins
   $pdf->SetMargins(15, 20, 15);
   // set auto page breaks
   $pdf->SetAutoPageBreak(TRUE, 15);
   // set image scale factor
-  $pdf->setImageScale(4);
+  $pdf->setImageScale(1);
   // set image compression quality
   $pdf->setJPEGQuality(100);
 
@@ -292,20 +518,29 @@ function theme_print_pdf_tcpdf_page(&$pdf) {
  *   array with the font definition (font name, styles and size)
  * @see theme_print_pdf_tcpdf_content()
  */
-function theme_print_pdf_tcpdf_content(&$pdf, &$html, $font) {
+function theme_print_pdf_tcpdf_content($vars) {
+  $pdf = $vars['pdf'];
   // set content font
-  $pdf->setFont($font[0], $font[1], $font[2]);
+  $pdf->setFont($vars['font'][0], $vars['font'][1], $vars['font'][2]);
 
-  preg_match('!<body.*?>(.*)</body>!sim', $html, $matches);
+  preg_match('!<body.*?>(.*)</body>!sim', $vars['html'], $matches);
   $pattern = '!(?:<div class="print-(?:logo|site_name|breadcrumb|footer)">.*?</div>|<hr class="print-hr" />)!si';
   $matches[1] = preg_replace($pattern, '', $matches[1]);
 
+  // Make CCK fields look better
+  $matches[1] = preg_replace('!(<div class="field.*?>)\s*!sm', '$1', $matches[1]);
+  $matches[1] = preg_replace('!(<div class="field.*?>.*?</div>)\s*!sm', '$1', $matches[1]);
+  $matches[1] = preg_replace('!<div( class="field-label.*?>.*?)</div>!sm', '<strong$1</strong>', $matches[1]);
 
   // Since TCPDF's writeHTML is so bad with <p>, do everything possible to make it look nice
-  $matches[1] = preg_replace('!(<p\s*/>|</p>)!i', '<br />', $matches[1]);
+  $matches[1] = preg_replace('!<(?:p(|\s+.*?)/?|/p)>!i', '<br$1 />', $matches[1]);
   $matches[1] = str_replace(array('<div', 'div>'), array('<span', 'span><br />'), $matches[1]);
+  do {
+    $prev = $matches[1];
+    $matches[1] = preg_replace('!(</span>)<br />(\s*?</span><br />)!s', '$1$2', $matches[1]);
+  } while ($prev != $matches[1]);
 
-  $pdf->writeHTML($matches[1]);
+  @$pdf->writeHTML($matches[1]);
 
   return $pdf;
 }
@@ -321,13 +556,14 @@ function theme_print_pdf_tcpdf_content(&$pdf, &$html, $font) {
  *   array with the font definition (font name, styles and size)
  * @see theme_print_pdf_tcpdf_footer()
  */
-function theme_print_pdf_tcpdf_footer(&$pdf, &$html, $font) {
-  preg_match('!<div class="print-footer">(.*?)</div>!si', $html, $tpl_footer);
+function theme_print_pdf_tcpdf_footer($vars) {
+  $pdf = $vars['pdf'];
+  preg_match('!<div class="print-footer">(.*?)</div>!si', $vars['html'], $tpl_footer);
   $footer = trim(preg_replace('!</?div[^>]*?>!i', '', $tpl_footer[1]));
 
   // set footer font
-  $font[2] *= 0.8;
-  $pdf->setFooterFont($font);
+  $vars['font'][2] *= 0.8;
+  $pdf->setFooterFont($vars['font']);
   // set footer margin
   $pdf->SetFooterMargin(10);
   // set footer data
@@ -343,9 +579,10 @@ function theme_print_pdf_tcpdf_footer(&$pdf, &$html, $font) {
  *   current TCPDF object
  * @see theme_print_pdf_tcpdf_footer2()
  */
-function theme_print_pdf_tcpdf_footer2(&$pdf) {
+function theme_print_pdf_tcpdf_footer2($vars) {
+  $pdf = $vars['pdf'];
   //Position at 1.5 cm from bottom
-  $pdf->writeHTMLCell(0, 15, 15, 0, $pdf->footer, 0, 0, 0, TRUE, '');
+  $pdf->writeHTMLCell(0, 15, 15, $pdf->getPageHeight()-15, $pdf->footer);
 
   $ormargins = $pdf->getOriginalMargins();
   $pagenumtxt = t('Page !n of !total', array('!n' => $pdf->PageNo(), '!total' => $pdf->getAliasNbPages()));