As noted in #708434-2: Attributes Pricing in 6? price formatting needs to be updated to use uc_price().

CommentFileSizeAuthor
#3 724900_uc_price.patch6.2 KBfenstrat
#2 724900_uc_price.patch6.06 KBfenstrat
#1 724900_uc_price.patch6.05 KBfenstrat
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

fenstrat’s picture

fenstrat
he/him
Primary language English
Location Australia
CreditAttribution: fenstrat commented
Status: Active » Needs review
FileSize
724900_uc_price.patch6.05 KB

Attached patch implements uc_price().

It also standardises the use of "Default price" when referring to the normal Ubercart sell_price field (rather than "Standard price" as it was called sometimes.

fenstrat’s picture

fenstrat
he/him
Primary language English
Location Australia
CreditAttribution: fenstrat commented
FileSize
724900_uc_price.patch6.06 KB

Updated patch - only change is to add array_filter() when checking if the uc_price_per_role_enabled variable is empty.

fenstrat’s picture

fenstrat
he/him
Primary language English
Location Australia
CreditAttribution: fenstrat commented
FileSize
724900_uc_price.patch6.2 KB

Hmm, always test patches no matter how trivial the change!

Attached is the correct fix for checking for the empty uc_price_per_role_enabled variable.

longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave commented
Status: Needs review » Fixed

Committed with a minor change to hook_uninstall, thanks.

fenstrat’s picture

fenstrat
he/him
Primary language English
Location Australia
CreditAttribution: fenstrat commented

Thanks longwave.

Status: Fixed » Closed (fixed)

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