Needs work
Project:
Drupal Commerce Extra Price Formatters
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
29 Nov 2012 at 14:22 UTC
Updated:
4 Feb 2016 at 08:41 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
ikos commentedHi,
Sorry it's take me a while to look at this.
I have looked at what your module does and can see some issues in the queue for this module that are asking for things you have done here.
The attached patch does the following:
I have not merged the discount on one line part as Bojan commented that it should be part of the discount module.
Can you have a look and see if the patch works for you?
many thanks
Richard
Comment #2
MarcElbichon commentedI've tested your patch with success but i have to do some minor changes :
Here's a new patch correcting these malfunctions.
Comment #3
matsjacobsson commentedI would love to have this functionallity included in the module too. Whats the status for this?
Cheers! /Mats
Comment #4
summit commentedHi, me too.
WOuld love to see this committed.
Will test this patch and report back.
Does anybody have latest dev including this patch as zip? Than I can test easier.
Thanks for reply in advance,
greetings,
Martijn
Comment #5
summit commentedHi,
I build the module inclusive patch myself and it is working great. Great to be able to select the different components to be included in the price myself.
See attached .txt file. Remove .txt and copy over latest dev.
Thanks for committed this in advance!
Greetings, Martijn
Comment #6
discipolo commentedpatch from #2 didnt apply for me. this is the same patch again.
Comment #7
discipolo commentedafter applying this patch where should i look for new configuration options allowing to hide or reorder price components?
EDIT: found it by looking at the code: admin/commerce/config/checkout/form/pane/cart_contents
Comment #8
discipolo commentedthe interface still shows up but it doesnt reorder components anymore for me, not sure what changed yet
Comment #9
joelpittetLet's put this to needs work.
Comment #10
heliogabalI applied path #6 just now and it works for me.
Comment #11
psf_ commentedAfter apply the path #6 we need flush the cache.