I would like to give the user a little more freedom in the adjusted prices part of the attribute system.
The part in question is in the product node where the options of the attribute are shown like ( option 1 + $ 3.44). The $3.44 part is now the total price or price increase for the default qty (or 1 if there is no default quantity). I would like to add an extra check box in /admin/store/settings/attributes giving the end user the chance to specify if he would like the current situation (default_qty) or prefers '1'. My wholesale business uses quite big default quantities and customers are mostly interested in the price (total or increase for 1 product).
I could code it very simple in a few lines, keeping the functionality working as it is now (with 1 extra choice) and would be happy to make a patch, just checking if anyone thinks this is a good idea and will it make any chance to get into core?
Thanks
Mark
(ps. Ubercart site is down?)
Comment | File | Size | Author |
---|---|---|---|
#8 | 590918-uc_attribute-qty-1.patch | 1.65 KB | longwave |
#7 | attribute-quantity-is-1-590918-2694604.patch | 489 bytes | splash112 |
#1 | single_product_attribute.patch | 1.85 KB | splash112 |
Comments
Comment #1
splash112 CreditAttribution: splash112 commentedSmall patch file attached for the idea above.
It's absolutely non intrusive and any site will work as before, only with some extra functionality.
Mark
Comment #2
splash112 CreditAttribution: splash112 commentedAlso see:
http://drupal.org/node/595354
Comment #3
TR CreditAttribution: TR commentedDoes *anyone* need to see the total price or price increase for the default qty? Shouldn't this *always* be for qty=1? I personally don't use default qty. Is the display price set to be the sell_price multiplied by the default qty? It seems it would have to, to be consistent with what's done for the attribute price adjustments. That sounds like it would confuse the customer - I think prices should always be for 1 piece. If necessary, the display price could be themed to say "each" after the price if default qty > 1.
Comment #4
splash112 CreditAttribution: splash112 commentedFair point. If somebody would like to have the price per default quantity, a simple hook_price_alter would probably do the trick..
Comment #5
toitimhcm CreditAttribution: toitimhcm commentedit worked, thanks
--------------
Comment #6
longwaveThinking we should revisit this, and agreeing with TR that there is no real need to multiply by qty, so perhaps we should just remove that part. I guess most users have a default qty of 1 anyway so this doesn't affect them.
Comment #7
splash112 CreditAttribution: splash112 commentedHi Longwave,
Thanks for checking back on this. Attached is a patch conform TR's comment.
Thanks
Mark
Comment #8
longwaveAlternative patch that simplifies the code further.
Comment #9
longwaveCommitted, needs porting to 7.x.
Comment #10
longwaveCommitted.