Price per role is being managed per node so far.

It means that price must be set every node. If "uc_attribute" is used on the node/product. Prices per role on those options must be set on "node/%node/edit/option_prices" page for every node.

This behaviour is not user-friendly, mainly if the same price per role should be used on all products.

Implementing defaults for option prices per role makes the workflow much nicer and standard-like.

Default prices for options are being set on page: "admin/store/products/classes/%uc_product_class/options", so I suggest to administer/manage the options default prices per role on the same page.

CommentFileSizeAuthor
#4 uc_price_per_role_default_option_prices-2696543-4.patch802 bytesdabbor
#2 uc_price_per_role_default_option_prices-2696543-2.patch3.47 KBdabbor
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dabbor created an issue. See original summary.

dabbor’s picture

dabbor CreditAttribution: dabbor at Morpht commented
Related issues: +#2345421: set default value for roles
FileSize
uc_price_per_role_default_option_prices-2696543-2.patch3.47 KB

Providing patch with implementation what was mentioned in issue's description.

It works for me nice.

dabbor’s picture

dabbor CreditAttribution: dabbor at Morpht commented
Assigned: dabbor » Unassigned
Status: Active » Needs review
dabbor’s picture

dabbor CreditAttribution: dabbor at Morpht commented
FileSize
uc_price_per_role_default_option_prices-2696543-4.patch802 bytes

Adding new minimalistic patch that adds drupal_alter into function uc_price_per_role_load_option_prices($nid):

/**
   * Alter option prices.
   *
   * Default option prices can be provided or node's option prices can be
   * overridden.
   */
  $context = array(
    'nid' => $nid,
  );
  drupal_alter('uc_price_per_role_option_prices', $prices, $context);

This way "options default prices per role" can be managed in custom module and just a minimum change to the "Ubercart Price Per Role" contrib is required.