The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.when the domain access is enabled and each site has it's own site logo, the invoice takes the logo from the wrong site.
1. create a site 1 e.g. site1.mysite
2. create a second site 2 e.g. site2.mysite
3. choose different default themes for each of the sites from admin -> site building -> domains -> (theme link for the domain)
4. set different logos for each of the site
5. now place an order in site 2 and view the order
6. click on invoice tab. the logo of site 1 is shown instead of site 2's logo
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | ubercart-1069590.patch | 1.07 KB | sukr_s |
Comments
Comment #1
sukr_s commentedI'm providing the patch that resolves this issue. would be great if someone could review and integrate the patch.
Comment #2
longwaveThe wrong logo will still be displayed if you place an order on site 2, then go to site 1's admin pages and view the order. http://drupal.org/project/uc_domain fixes this issue and also other issues with order tokens so the invoice is always correct for the domain where the order was placed.
Comment #3
longwaveHm, but $theme_key could still be a better way of finding the current domain than using variable_get(). I will look at this further when I get time, but uc_domain is still the better solution for using Domain Access with Ubercart.
Comment #4
tr commentedLet's get the bot to test this.
Comment #5
tr commentedComment #7
longwaveCommitted a simplification for the site-logo token in http://drupalcode.org/project/ubercart.git/commitdiff/b150b22 which fixes this issue. The original code copied theme_get_setting(), but instead we can just call theme_get_setting().