I have an item that has 1 attribute that is a radio button and another attribute that is a text field. It seems that the prices first come out properly, but then switch to what each attribute would increase the base.

While I know that if the product has more then one attribute that it has to work in this manner as each attribute option may change the price of the product, it should be able to detect if the other attributes will not affect the price, like a text attribute that has no options.

should be as easy as changing the select statement looking for the number of attributes to exclude the text field type attribute.

granville

Comments

cyu’s picture

cyu commented

Oddly enough I addressed this a couple months ago here: http://drupal.org/node/275367 but then never came back to uc_aac and applied the fix. I'll take care of that and possibly make a release if I can find any other clean-up that needs done.

cyu’s picture

cyu commented

In the meantime, you could make the following replacement if you want to test this out. Support for uc_attribute_priced_attributes() is only available in Ubercart 1.1 or later.

function _uc_aac_markup($form_id, $nid) {
  $result = db_query('SELECT sell_price FROM {uc_products} WHERE nid=%d AND vid=(SELECT vid FROM {node} WHERE nid=%d)', $nid, $nid);
  $priced_attributes = count(uc_attribute_priced_attributes($nid));
  $price = 0;
  if ($product = db_fetch_object($result)) {
    $price = $product->sell_price;
  }
  $update_attributes = 0;
  if (variable_get('uc_aac_attribute_reprice', '1') == 1 && (variable_get('uc_attribute_option_price_format', 'adjustment') == 'adjustment' || $priced_attributes > 1) && (variable_get('uc_attribute_option_price_format', 'adjustment') != 'none')) {
    $update_attributes = 1;
  }
  return theme('uc_aac_js', $nid, $price, $update_attributes);
}
cyu’s picture

cyu commented
Status: Active » Fixed

This is fixed in the latest dev release by commit: http://drupal.org/cvs?commit=146425

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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