The isset() checks for options text color and font_size do not work as the data comes in as empty strings rather than not being set at all. This makes the fields not visible unless the user explicitly defines these.

CommentFileSizeAuthor
#1 views_pdf_template.php_.diff1.08 KBwaylandbill
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

waylandbill’s picture

Here is one possible fix to the issue.

Simon Georges’s picture

Version: 7.x-1.0 » 7.x-1.x-dev
Status: Active » Needs work

Hi,

Could you eventually post your patch in a proper format? See http://drupal.org/node/707484 to guide you.

ram4nd’s picture

Instead of the other guy I am going to thank you. Thanks, the solution worked for me.

killua99’s picture

Priority: Normal » Critical
Status: Needs work » Reviewed & tested by the community

Exist a patch to handle this issues and others. Please see #2032189: Multiple issue fixing in a single patch and review others issues patch.

This is RTBC, and ready to apply.

Simon Georges’s picture

Priority: Critical » Normal
Status: Reviewed & tested by the community » Fixed

A modified version of this has been committed. Thanks for the report.

Status: Fixed » Closed (fixed)

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