Simply minor bug:

if in settings the print class is left empty, the function print_insert_link() creates a link containing en empty class="", which does not validate.

Patch attached.

CommentFileSizeAuthor
#2 print_fix_empty_class0.patch397 bytessuit4
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jcnventura’s picture

jcnventura CreditAttribution: jcnventura commented
Status: Needs review » Postponed (maintainer needs more info)

Patch is not attacched :)

João

suit4’s picture

suit4 CreditAttribution: suit4 commented
Status: Postponed (maintainer needs more info) » Needs review
FileSize
print_fix_empty_class0.patch397 bytes

wait --- there seems to be a problem with attaching files .....

jcnventura’s picture

jcnventura CreditAttribution: jcnventura commented
Status: Needs review » Fixed

Thanks!

It seems to be harmless enough that I am committing it already.

João

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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