Commit Graph

92084 Commits

Author SHA1 Message Date
Denis Ledoux 37c805f26f [FIX] purchase: replace company_id widget selection by option no_create
For selection fields, name_get calls to resolve the display name are client-side initiated, and will display "Unknown" if the user does not have read access to the selected company. And the reason for using a selection widget in the first place was to prevent inadvertent creation of companies. This is now doable via the no_create option, so we can remove the selection widget.
2014-12-05 14:41:30 +01:00
Denis Ledoux 38124e7e31 [FIX] sale_stock: product_id_change_with_wh on sale order popup too
product_id_change_with_wh onchange was not anymore called when using the popup form in sale order line.

The behavior was different concerning the product onchange when being in the editable list or in the popup form
For instance, for make to stock products not having enough stock, the warning was displayed when being in editable list, but not in the popup form

This is a regression of rev. 86f785ae1b

opw-619624
2014-12-05 14:00:37 +01:00
sve-openerp ff3cb41d5c [FIX] purchase: remove redundancy in report
The warehouse name and address name is often the same (name of the company).
Remove the name of the address as two warehouse may use the same address.
Fixes #4062
2014-12-05 13:51:28 +01:00
Samus CTO 55fa50891d [FIX] Use _unknown model for missing models in relation fields
During a migration of database, it is possible that some custom field
("x_", state is 'manual') are relational to model from a module that is
not provided.

Note: this used to work in Odoo 7.0 but crashed in 8.0.

Closes #3877
2014-12-04 20:58:46 +01:00
Denis Ledoux 4e9613609d [FIX] sale: rounded sales team invoiced gauge
The price_total field of the account invoice report is not rounded (it cannot be easily rounded, as this has to be done in the sql view)
In a multi currencies environment, this is possible that the price_total value has a lot of digits
We therefore round it manually, for the gauge of the sales team kanban view
2014-12-04 14:48:54 +01:00
Sandy Carter aa10972d13 Raise error on read of a browse object with bad id
Check if id is valid by searching record columns when a key error is raised
If the record has the column, the key error is actually an error on a
missing or inaccessible id.

Signed-off-by: Sandy Carter <sandy.carter@savoirfairelinux.com>

Closes #3658
2014-12-04 13:24:26 +01:00
Denis Ledoux 9c7fb721f0 [FIX] translate: allow common english 2 chars terms
Before, all isolated (between xml/html tags) two chars words coming from views were not translated (by choice).
But, for some words, allowing them is useful. For instance, the word 'or' located between two buttons.

opw-616716
2014-12-04 13:13:07 +01:00
Denis Ledoux 6629729f2d [FIX] account_analytic_analysis: tree fields access rights
Add invoicing related fields on anlytic account tree view for the invoicing group only
Otherwise, when a user not having the invoicing access rights displays the analytic account list, he gets an access right error.

opw-619485
2014-12-04 11:37:22 +01:00
Arthur Maniet 35431de125 [FIX] res.currency: correctly compute the number of decimal places for currencies formatting in 'get_format_currencies_js_function'. 2014-12-04 10:51:54 +01:00
Denis Ledoux 04f3fffdb4 Revert "[ADD] purchase: average price computation test"
This reverts commit 5f9280e854.

These tests have been introduced in 7.0/saas-3, but can no longer be applied in 8.0, as they uses models that do not exist anymore in the new wms.

if such tests do not exist yet in Odoo 8.0, then these tests needs re-work, they cannot be applied like that

Conflicts:
	addons/purchase/tests/test_average_price.py
2014-12-04 10:20:55 +01:00
Christophe Simonis 5ca7fa18fc [MERGE] forward port of branch 8.0 up to 31a01ea 2014-12-03 19:24:57 +01:00
Christophe Simonis 7c49c6d79d [FIX] purchase: do not use deprecated api 2014-12-03 18:40:36 +01:00
Alexis de Lattre 31a01ea772 Fixes #3851 Create invoice from picking copies some parameters from the order to the invoice 2014-12-03 17:53:25 +01:00
Christophe Simonis 2ed212dcbf [MERGE] forward port of branch 7.0 up to d6daf5f 2014-12-03 17:51:06 +01:00
Holger Brunn 8eae018b27 [FIX] mrp: keep traceability of procurement.group
Pass group_id information when a manufacturing order is created from
a procurement to keep a better traceability and know the reason of stock.move
creation.
Fixes #4019
2014-12-03 17:30:44 +01:00
Cedric Snauwaert d6daf5fa2f [FIX] product: add store=dict to field name_template
Recompute the name_template field if the template changes.

backport of 41628fc + f25c5d8 to 7.0 plus some optimisations in triggers
Fixes #3277
2014-12-03 16:39:14 +01:00
Adrien Peiffer def9222bd9 [FIX] Fix wrong unit price on invoice genereted by stock picking when uos is used
[IMP] Rounding not necessary
2014-12-03 16:23:56 +01:00
Mart Raudsepp 7a979b2d3e [FIX] Fix floating point handling in purchase _prepare_order_line_move
Without this better floating point handling, an extra stock move might
be created for zero quantity for some order lines upon PO confirmation,
because qty is equal to something closer to e.g 1.14e-13, but this is
larger than 0, so it creates a stock.move, which gets rounded too late
to 0.0

Closes #3346
2014-12-03 16:10:58 +01:00
Mart Raudsepp 05de60cbf9 [FIX] Correct floating number handling for _procure_orderpoint_confirm
Fixes comparison with min_quantity orderpoint in scheduler - basic floating
point math issue in procurement scheduler when comparing current quantity
with orderpoint minimum quantity. In certain cases floating point comparison
could result in e.g 400.0 < 400.0 == True due to typical floating point
comparison issues, as Odoo doesn't use Decimal types where the issue
doesn't exist.

Fixes early exiting out of the loop cycle, in case qty is already near zero.
Fixes the new procurement creation check, to not do that if it's close
enough to zero already, to be considered a floating point math error, not
really non-zero.

These combined (or at least the last one) avoid each supply_method == buy
pending in draft PO's getting a zero quantity extra procurement order each
time the scheduler runs. Otherwise there could be hundreds of zero quantity
procurement orders pending, which makes the confirming of the PO take hours,
due to creating hundreds of stock moves for each order line.

Use float_compare helper to solve all these with floating point type for now,
instead of the more evasion possibility of converting to Decimal module.
Two potential bad comparisons remain, add FIXME notes for now until further
analysis.

Also: Float rounding on reste when comparing and on the procurement qty
2014-12-03 16:10:58 +01:00
Alexis de Lattre a300b6e3e7 [FIX] delivery: display the label of the field number_of_packages
Fixes #4011
2014-12-03 15:34:51 +01:00
Martin Trigaux a765876ffb [IMP] web: do not fully hide pager in grouped view
When the list view is grouped, the page count should be hidden as irrelevant.
However if it's fully hidden, the limit can no longer be changed.
Instead of hidding the pager, this commit hides the arrows and replaces
the content by the current limit to allow to be changed.
2014-12-03 15:02:36 +01:00
Christophe Simonis 2e092ace29 [FIX] must manage() Environment before loading test file 2014-12-03 14:48:54 +01:00
Xavier Morel 2275ab5d95 [FIX] doc: remaining instances of ruby-style interpolation in templates
fixes #3157
2014-12-03 14:21:00 +01:00
Raphael Collet 894a898e9e [FIX] models: exists() should not consider record with id 0 as existing 2014-12-03 13:59:27 +01:00
Yajo 4bcb31f706 [FIX] packaging: redhat dependencies
Install psycogreen from easy_install (not in EPEL), required for openerp-gevent
2014-12-03 11:40:59 +01:00
Grover Menacho c7b6a96239 [FIX] action_confirm was not asking for purchase pricelist, otherwise it was asking for sale pricelist 2014-12-03 11:28:59 +01:00
Benoit Guillot bc2a52fb28 [IMP] mrp: add prepare method for manufacturing orders to allow override
Fixes #3973
2014-12-03 10:48:23 +01:00
Josse Colpaert b6f3795e33 [IMP] Manufacture should be by default in all warehouses 2014-12-03 10:13:46 +01:00
Raphael Collet d82aa69ccc [IMP] models: add check for common conversion error in field definitions 2014-12-03 09:55:17 +01:00
Julien Legros 5f9280e854 [ADD] purchase: average price computation test 2014-12-02 19:33:23 +01:00
Julien Legros bfadcdd5d6 [FIX] stock: compute average price on action_done
The average price computation is now deduplicated and moved to
a separate function called in stock_move action_done.
This makes sure it is always called when a stock.move is
processed, even without going through the partial picking wizard.

Fixes #2991
Closes #3949
OPW-615491
2014-12-02 19:32:49 +01:00
Josse Colpaert 536d00d2ba [FIX] Avoid endless loop when doing a return picking 2014-12-02 18:54:25 +01:00
Mack ffda023295 [FIX] http.py: no traceback for SessionExpiredException
Rebase and close #3618
2014-12-02 18:34:28 +01:00
Christophe Simonis bc24d1b010 [FIX] product: force empty view_id for action `product_variant_action` 2014-12-02 17:45:31 +01:00
Christophe Simonis 640e77eaa3 [FIX] base: deleting a view must not delete linked actions 2014-12-02 17:45:31 +01:00
Mario Arias Badila bd2633753d [ADD] pyeval: date.replace method
Needed to create filters like "previous month"

It was just defined for "datetime", but is also needed for "date"

closes #2915
2014-12-02 16:59:24 +01:00
Abhishek Jaiswal b374c51474 [ADD] document @default_order in list views
closes #2623
2014-12-02 16:59:23 +01:00
Denis Ledoux 2592f149a0 [FIX] mail: model can be None or uninstalled
This is related to rev. 83b28737e9
2014-12-02 16:50:45 +01:00
Josse Colpaert 8c8056d95f [IMP] Query should pass through orm and as such the refreshes can be removed in the code, which will go faster as it is done automatically by the new api 2014-12-02 16:38:36 +01:00
Nicolas Lempereur 38f0508bba [FIX] doc: typos and code samples from howtos
* The dashboard from the backend howto doesn't specify "name" field
  which causes non-null constraint violation.
* Some useful examples uses idea.idea" model which isn't available in
  8.0 so I replaced them with "note.note".

closes #3993
2014-12-02 16:22:55 +01:00
Denis Ledoux 83b28737e9 [FIX] mail: no_auto_thread default True if the model has no mail.thread 2014-12-02 16:06:51 +01:00
Josse Colpaert c255fe3384 [IMP] Add a method half-up for uom conversion in case we want to convert from default UoM to something else + remaining_qty round + default factor on UoM 2014-12-02 15:51:58 +01:00
Josse Colpaert 49c8fc21ba [FIX] When creating a procurement from a move, respect the right UoM conversions 2014-12-02 15:51:58 +01:00
Josse Colpaert bb45636ba7 [IMP] When preparing pack operations, use the smallest UoM of the moves if they are smaller than the default UoM. Otherwise use default UoM.
The same is done when extra moves are generated.  It is going to check if the UoM of the operation is smaller if it has one.

Throw an error when a key can not be found in action_done because there were links on a move
that was not supposed to be done (e.g. 0.5 Dozen when Dozen is rounded at 1)

[IMP] Throw an error when a key can not be found because of UoMs/picking + extra float_compare

[IMP] Integrate remarks qdp
2014-12-02 15:51:58 +01:00
Josse Colpaert 910376a157 [IMP] Rounding of the UoM should have an unlimited precision 2014-12-02 15:51:58 +01:00
Josse Colpaert 77f8732680 [IMP] Rounding should be done on move immediately to default UoM and quants should be rounded to the default UoM + remaining_qty
[FIX] Remaining qty should each time be in the default UoM of the product

Even with different UoM we want a consistent matching between moves and pack operations.
When calculating the remaining qty on move/pack operation we always start by converting the
qty on the move/operation to the default UoM and afterwards we subtract the links between them
which will also be in the default UoM of the product.
In order to create backorders / extra moves these quantities are used.
2014-12-02 15:51:58 +01:00
Josse Colpaert 244c8ee625 [FIX] Propate propagated_from_id when reconciling negative quants to the quant that loses it
When a negative quant is created but the positive quant counterpart is reconciling
a negative quant that of course also has a positive counterpart, the latter should eventually
let its field propagated_from_id tell that it originated from the very first negative quant as the
second negative quant will have disappeared through reconciliation.
2014-12-02 15:51:57 +01:00
Josse Colpaert cf16632b44 [IMP] No digits_compute on product_qty of move + rounding on qty_available fields + add float_compare 2014-12-02 15:51:57 +01:00
Martin Trigaux 73db5b3eef [FIX] account_voucher: period selection in on_change
The find method returns a list of ids while the on_chage expects an int
2014-12-02 15:12:23 +01:00
Raphael Collet 591e329bb0 [FIX] fields: inherited fields get their attribute 'state' from their base field 2014-12-02 14:58:17 +01:00