Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Article quantity do not add up to #units-to-order #175

Closed
antil0pa opened this issue Feb 2, 2016 · 4 comments
Closed

[BUG] Article quantity do not add up to #units-to-order #175

antil0pa opened this issue Feb 2, 2016 · 4 comments

Comments

@antil0pa
Copy link

antil0pa commented Feb 2, 2016

*Follow up to http://foodsoft.51229.x6.nabble.com/problem-updating-foodsoft-adam-td1189.html

After updating to Foodsoft v3.3.9+adam+ß there is a problem of adding articles to the total order. The
'Everyone' column and the tollerance bar zero out after refreshing the page.

I tested it on again with the foodcoopamsterdam demo on 3 different browser and 2 different machines.

In the screenshot of the interface , it is visible that though some orders should be full, none are marked to be ordered.

foodsoft

@antil0pa
Copy link
Author

antil0pa commented Feb 3, 2016

I found the source of the confusion. The shopping cart function has been added, therefore individual orders are not added to the Everyone column until paid for. This seems logical, But when you add or remove articles it still affects the units-to-order count in the Everyone column. This is only temporary - refreshing the page sets it back to zero.
This causes conflicts between the units-to-order column and the Amount column, when units-to-order can even show a negative quantity when removing article-units.

@wvengen
Copy link
Member

wvengen commented Apr 25, 2016

I got bitten by this today as well. It's even more confusing after the amount coloring.
This is the offending line (plus tolerance equivalent). Replacing it by

data_delta[:value_others] = order_article.quantity - \
  group_order_article.group_order_article_quantities.confirmed.sum(:quantity)

when the payorder plugin is enabled would do wonders. Haven't been able to do that in a somewhat clean way, though (deface messes up stuff).

@wvengen
Copy link
Member

wvengen commented Apr 25, 2016

I added a workaround for this. Better now?

@wvengen
Copy link
Member

wvengen commented Jan 29, 2021

The focus of this fork has become getting relevant functionality upstream. If you need this feature, please open an issue with upstream foodsoft.

@wvengen wvengen closed this as completed Jan 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants