First, thanks so much for this module. My previous attempt to do webform/ubercart event registration in D6 was cobbled together with spit, chewing gum, and hours of sweat. This is so much better.

I would think that the most common use case of attributes in this module would be to set levels of ticket pricing, meaning that the attribute price is the price, rather than being an add-on price as Ubercart intends. For this reason, I would think many people would like to remove the + sign next to the price in the attribute display so that instead of:

Gold Sponsor, +2,500

you'd see:

Gold Sponsor, $2,500

My php is good enough that I was able to to this by hacking the module, changing line 201 in uc_attribute.inc from $display_price = ($option->price != 0 ? ', '. ($option->price > 0 ? '+' : '') . uc_price($price_info, $context) : ''); to $display_price = ($option->price != 0 ? ', '. uc_price($price_info, $context) : '');, but I don't know how to do this the Drupal way without hacking your module.

So I was thinking that others might find it useful if either there were an option to display the plus sign or not, or (maybe a better idea, and easier to implement) to only display the plus sign if both the attribute price > 0 AND the base price of the event = 0. That way if you had a required attribute that by itself determined the price of the event, you wouldn't see that pesky plus sign. Thoughts?

Comments

thoughtcat’s picture

I know this issue is very old and this isn't entirely relevant but in D7 you can remove the + sign from within the module by going to /admin/store/settings/products and checking "Display total price" in the attribute settings.

sano’s picture

thank you @thoughtcat