uc_taxes_ca_action() provides a tax action for each tax. It currently sites the action title to "Charge a tax", which becomes very confusing when you have more than one tax. Attached is a patch to change the title to t('Apply tax: @name', array('@name' => $rate->name)), allowing each tax action to have a custom name, which matches the name of the tax.

CommentFileSizeAuthor
#1 530902_tax_action_name.patch550 bytesIsland Usurper
uc_taxes_ca_action_description.patch555 bytesmikejoconnor

Comments

Island Usurper’s picture

Island Usurper commented
Priority: Normal » Minor
StatusFileSize
new 530902_tax_action_name.patch550 bytes

The action names are run through check_plain() when they are displayed on the CA form, so we should use ! instead of @ here. The default predicates also change the action names to something similar, so we should probably keep them consistent.

Island Usurper’s picture

Island Usurper commented
Status: Needs review » Fixed

Looks OK to me on the actions form. Committed.

Status: Fixed » Closed (fixed)

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