In the commerce_reports_commerce_reports_dashboard function of commerce_reports.module, t() function should be used for all the section titles, including Year / Month / Week, Quantity / revenue and Billing / shipping.

BTW, thanks for this excellent module that seems to work quite well !

CommentFileSizeAuthor
#2 2216879-use-t-function-for-section-title-3.patch2.48 KBmglaman
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

eme’s picture

eme CreditAttribution: eme commented

Same in dashboard-block.tpl.php : "from" should be passed in t() function.

mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman commented
Status: Active » Needs review
FileSize
2216879-use-t-function-for-section-title-3.patch2.48 KB

Here is patch that properly wraps mentioned strings in t().

mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman commented
Status: Needs review » Fixed

Committed.

  • Commit df40278 on 7.x-4.x by mglaman: 
    Issue #2216879 by mglaman | eme: Use t() function for section title.

Status: Fixed » Closed (fixed)

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

  • Commit df40278 on 7.x-4.x, 2023381-commerce-coupon by mglaman: 
    Issue #2216879 by mglaman | eme: Use t() function for section title.