Commit Graph

10 Commits

Author SHA1 Message Date
Denis Ledoux 78a29b3738 [FIX] point_of_sale: no need to specify numeric digits
In the report pos order, average_price was set as a numeric(16,2), therefore, if the amount was too big, it led to a psql crash:
A field with precision 16, scale 2 must round to an absolute value less than 10^14.
2014-10-21 10:13:44 +02:00
Denis Ledoux 18d365ce05 "[REVERT][FIX] point_of_sale: field date of report.pos.order should be a datetime"
This reverts commit 97d097a2af.

As explained in the commit comments (on Github), this patch leads to an infinite loop in 7.0, the filter of the pos orders report using the '=' operator in its domain, which is not available for datetime fields, but is for date fields.

This should not be forward ported to newer release (saas-3)
2014-08-29 20:40:31 +02:00
Denis Ledoux 99ce17cfca [FIX] point_of_sale: report, product_id of line is product_product
product_id column of pos_order_line is a product_product
the left join of l.product_id was done on product_template, instead of product_product

It worked as long as the ids product_product were the same as product_template. Meaning that, if you used variants, this report view was screwed.
2014-08-21 11:20:09 +02:00
Denis Ledoux 97d097a2af [FIX] point_of_sale: field date of report.pos.order should be a datetime
The date_order field of the pos.order is a datetime, not a date
As, in report.pos.order, the date field is related to date_order of the pos.order

We usually do not commit fixes altering the model fields structure in 7.0, but this one is retro-compatible, as the database structure won't change
2014-08-14 18:01:37 +02:00
Vo Minh Thu e7f4891d46 [IMP] use the openerp namespace.
bzr revid: vmt@openerp.com-20121206145632-0h1coh5aaem65wsy
2012-12-06 15:56:32 +01:00
Amit Patel d48856e81a [FIX]:point_of_sale: In the same way the %age of discount was wrong in the reporting > point of sale > Sale details, there's a problem of conversion from %age to amount in the other report (reporting > point of sale > ORDER ANALYSIS), where the Total discount is the sum of %ages converted in €. So two times 50% discount = 100€ even though the item cost 1 bazillion, which is not correct.
bzr revid: apa@tinyerp.com-20121010052920-7d2zl85war0bn8jr
2012-10-10 10:59:20 +05:30
Saurang Suthar (OpenERP) 2ad3b32160 [IMP]:improved title Minimum Salesman by Salesperson everywhere
bzr revid: ssu@tinyerp.com-20120504121326-nqpx2eyqc1qrp04w
2012-05-04 17:43:26 +05:30
Saurang Suthar (OpenERP) a2a05cfc19 [IMP]:improved titles State by Status everywhere
bzr revid: ssu@tinyerp.com-20120504115748-2vd1dns7bxptg384
2012-05-04 17:27:48 +05:30
Fabien Pinckaers d4febfe788 [IMP+FIX] Complete cleaning of point_of_sale module
- refactorisation of the code: 2000 lines of code removed
   - fixed all the bugs
      - support for tax included/excluded prices
      - fixed rounding troubles
      - don't print the ticket if partial payment
   - cleaning features:
      - workflow reviewed: New -> Done -> Posted / Invoiced
      - generation of journal entries/invoice is now correct
      - only one picking per PoS order
      - wizard to apply global discount (on each lines)
   - useability:
      - reviewed the main point_of_sale screen
      - statements are open by default
   - removed non-useful features
      - advances (you use a special product for this)
      - contracts numbers
      - refund with exchange (you just create a new sale with negatives qty)
      - removed validation steps
      - scan product wizard
   - clean and complete YAML tests (work in progress)

bzr revid: fp@tinyerp.com-20110925140930-s6zibvn51k3914in
2011-09-25 16:09:30 +02:00
ARA (OpenERP) e04b0e3e8a [ADD] ponit_of_sale: add a missining report files
bzr revid: ara@tinyerp.com-20101028060715-8ft3ft16kllhk8pb
2010-10-28 11:37:15 +05:30