if the EURO symbol is on the invoice it is not shown correct on the 'print'

I had to add a basic html header and set utf8 for it to show up correct!

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bwynants’s picture

function uc_order_view


  if ($view_mode == 'print') {
    $output ='<html><head>';
    $output .='<meta http-equiv="Content-Type" content="text/html; charset=utf-8">';
    $output .='</head><body>';
    $output .= '<div align="right" style="margin-bottom: 1em; margin-right: 1em;"><input type="button" value="' . t('Print invoice') . '" onclick="window.print();" /> '
              . '<input type="button" value="' . t('Close window') . '" onclick="window.close();" /></div>';
    $output .= theme('uc_order', array('order' => $order, 'op' => 'print', 'template' => variable_get('uc_cust_order_invoice_template', 'customer')));
    $output .='</body></html>';
    print $output;
    exit();
  }
bwynants’s picture

Status: Active » Needs review
FileSize
1.07 KB

patch added

bwynants’s picture

FileSize
1.39 KB

correct patch

TR’s picture

Can you run the resulting HTML through http://validator.w3.org/ to make sure it validates?

bwynants’s picture

FileSize
2.98 KB

one error remaining
Line 11, Column 90: required attribute "ALT" not specified

but that's caused by the theme('uc_order' call...

longwave’s picture

Improved patch attached that adds a new .tpl.php file for easier theming of this page.

longwave’s picture

Status: Needs review » Fixed

Committed #6.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.