Fairly Minor Bug Report:

Even if your current theme does not have a site logo enabled (toggle switch turned off), Ubercart is still trying to place an image on the order invoice and in the customer order e-mail. The result is a broken image link.

Instead, Ubercart should probably check to see if the site has a current site logo enabled before trying to link to the image.

Thanks!

CommentFileSizeAuthor
#3 394428.empty_logo.2.x.patch1004 bytescha0s

Comments

rszrama’s picture

rszrama commented
Assigned: Unassigned » rszrama

Excellent. Thanks for the feedback on that, Ben. Will bookmark it and get on it when I'm back in KY. Good luck on your interview!

rszrama’s picture

rszrama commented
Issue tags: +dc2009 code sprint
cha0s’s picture

cha0s commented
Status: Active » Needs review
StatusFileSize
new 394428.empty_logo.2.x.patch1004 bytes
Island Usurper’s picture

Island Usurper commented
Status: Needs review » Fixed

Looks good to me. Committed.

rszrama’s picture

rszrama commented
Assigned: rszrama » Unassigned

Status: Fixed » Closed (fixed)
Issue tags: -dc2009 code sprint

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

Issue tags: +dc2009 code sprint

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.