Thank you for a great module! We are planning on using it soon in production for a youth hostel even though it is still in alpha, it seems robust enough for us.

One big problem though. Currently when creating an order and adding products, the taxes are calculated in the order total but are not shown on the page. They are not printed on the receipt either.

Any clue or example would be greatly appreciated, I could probably provide a patch if need be. Thx!

CommentFileSizeAuthor
#5 taxes-not-showing-2097597-5.patch1.14 KBleducdubleuet

Comments

devv69’s picture

devv69 commented

I have the same exact problem.

This is actually the only major bug who stop me for using it in production

thanks a lot :)

devv69’s picture

devv69 commented

hi

any update on this bug ?

jessewwilson’s picture

jessewwilson commented
Issue summary: View changes

I'm having this problem as well.

Items show in POS with their prices. Total is shown with Tax included however no line items for the taxes are displayed.

Receipt print out shows the same way. Total includes tax but there is no line item for taxes.

leducdubleuet’s picture

leducdubleuet commented
Version: 7.x-1.0-alpha3 » 7.x-1.0-alpha4

The current issue has yet to be acknowledged by the maintainer, it is still present as of version Alpha4.

leducdubleuet’s picture

leducdubleuet commented
Version: 7.x-1.0-alpha4 » 7.x-1.x-dev
Status: Active » Needs work
StatusFileSize
new taxes-not-showing-2097597-5.patch1.14 KB

Here is my patch to get the taxes showing. I mark it as "Needs work" since it may not be the way to go at all to get the taxes but I am certainly going in the right direction... I am also using t($component['name']) as the line title in POS and it may not be desirable for everybody. I put it inside a t() and I translate what it returns in french and since I am not using english at all, it has no side effect for me. Your mileage may vary...

Hope it helps someone and that some version of this get into the module someday!

jason ruyle’s picture

jason ruyle commented

Patch works for me. Display of text is little strange, but at least it works now.

rbayliss’s picture

rbayliss commented
Assigned: Unassigned » rbayliss

Thanks for your patience on this everyone. I consider this a blocker to a stable release, and we're going to be working on it. Ultimately, we want to be able to display all of the total components, just like when you're viewing an order through the regular Commerce Order UI. Unfortunately, we also need to display the amount paid and the balance due in the same table, which means that we can't just use the regular commerce_price_formatted_components formatter, and we need to deal with more than one field at once. Anyway, this is all to say I appreciate the patches above and if they work for you temporarily, that's awesome. We're going to try to address this in a way that will works well with taxes, discounts, and any other price components I haven't even heard about yet.

leducdubleuet’s picture

leducdubleuet commented

@rbayliss Thanks for the update, let me know if I can help.

By the way, we love Commerce POS, thank you for a great solution!

smccabe’s picture

smccabe commented
Issue tags: +Commerce Sprint
caldenjacobs’s picture

caldenjacobs commented

This seems to be working fine in 2.x

smccabe’s picture

smccabe commented
Status: Needs work » Fixed

Not an issue in 2.x, closing

Status: Fixed » Closed (fixed)

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