Commit Graph
118790 Commits
Author SHA1 Message Date
William Henrotin daec62ab68 [FIX] stock: inventory detail view.
In the inventory form view, the stock move table into the inventory
adjustment tab shows useless
columns. This commit add inventory adjustment related columns only.

Task : 1857705
2018-08-13 14:23:19 +02:00
Simon Lejeune d7b291a98a [REF] stock: expose a method to get the theoretical quantity
- use the `gather` method instead of calling search with a custom domain
- expose the method so that the barcode client action is able to call it
  when a line is added without being backed by an inventory line (yet).
2018-08-13 14:23:19 +02:00
Simon Lejeune d3d576cf8e [IMP] web: add infos key to on_close from special action 2018-08-13 14:23:19 +02:00
Alexandre Kühn 65cc630050 [IMP] web: provide infos for close action
This is a backport of 3210f10827

With this commit, when we perform a close action, we can provide infos to
the `on_close` handler, so that it can handle different cases that caused
the close action.
2018-08-13 14:23:19 +02:00
Simon Lejeune 2502b34a72 [REF] stock: move: rework kanban view
Note: stock moves are created from multiple objects. In case of inventory
adjustments, some field are useless like the initial quantity or the
reserved quantity. This commit hides those useless fields in the stock_move
kanban view in case of move related to inventory.
2018-08-13 14:23:19 +02:00
Simon Lejeune 9f2bbcbd61 [REF] stock: move line: add tracking fields
These fields aren't use in the main form views because we prepare the
context before opening the move lines wizard, but we need them in the
barcode client action because it is using a form view on this model
without clean possibility to prepare the context.
2018-08-13 14:23:19 +02:00
Odoo Translation Bot 566d1586bc [I18N] Update translation terms from Transifex 2018-08-12 07:14:34 +02:00
Christophe Simonis edd586002e [FIX] sale: uses float_compare in test 2018-08-10 12:14:35 +02:00
Christophe Simonis 7717f082c0 [MERGE] forward port branch saas-11.3 up to af35aea6b0 2018-08-09 19:33:32 +02:00
Christophe Simonis af35aea6b0 [FIX] sale_stock: adapt new test to renamed xmlid 2018-08-09 18:56:26 +02:00
Christophe Simonis a98131d5bc [FIX] test_mail: bump query count
An extra query is made because of new group added to fields in 57ea939eda.
2018-08-09 18:52:46 +02:00
Christophe Simonis 9cb97a8c2d [MERGE] forward port branch saas-11.2 up to 499af66727 2018-08-09 15:06:31 +02:00
Christophe Simonis 499af66727 [MERGE] forward port branch 11.0 up to a10ea93dbe 2018-08-09 14:50:55 +02:00
Goffin Simon a10ea93dbe [FIX] account: rounding problem with Payment Matching in multi-currency
Steps to reproduce:

- activate the multi-currency option in the Accounting App
- activate the EUR currency (USD being the base currency)
- create a EUR Bank Journal with an EUR account
- on 31/07 ==> rate = 1.1
- on 08/08 ==> rate = 1.2
- create a vendor bill for Asustek on 31/07 for 39794,40 EUR
- create a vendor bill for Asustek on 31/07 for 61668,00 EUR
- create a vendor bill for Asustek on 31/07 for 33487,70 EUR
- create a supplier payment for Asustek for 134950,1 EUR
- use the payment matching smart button
- match the 3 invoices you entered;

Bug:
Odoo says there is an open balance to write-off and prevents you from reconciling

Expected Behavior: the payment total amount in EUR matches the 3 vendor bills total
amount in EUR so I would expect Odoo to consider I have a perfect match between the
payment and the vendor bills and the difference in USD (base currency) should be
automatically reflected through an exchange difference entry.

if you follow the same operation with 3 invoices of 115EUR each and a payment of 345EUR,
it will work as expected.

opw:1872978
2018-08-09 13:43:38 +02:00
len-odoo 908ddbfeb7 [FIX] account_asset: do not show deactivated assets in the report view
When canceling invoices, Odoo deactivates related assets.
However these assets are not filtered in the pivot view, screwing the report.

opw 1864252
2018-08-09 13:17:16 +02:00
William Henrotin d3df57bf90 [FIX] mrp: remove mrp.message
The commit cfbfaf38be283e68c5a923ff2aafd1293135c4f1 remove the views and 
menus related to mrp.message. This commit complete it by removing the 
model, all the related fields and the access rules
2018-08-09 13:12:18 +02:00
Nicolas Martinelli 1f42cf9f9c [FIX] sale, sale_timesheet: re-introduce default invoice policy
Related to f89d4201c7

Since the invoice policy is becoming more and more simple, it is
always nice to add as much logic as possible to make things even more
simple.

For the record, 5 fields are taken into account when it comes to
invoicing policy:
service_policy
service_type
service_tracking
invoice_policy
expense_policy

They are partially linked one with the other, and not all make sense for
a given product type...
2018-08-09 13:04:45 +02:00
len-odoo f4ec8b95b7 [FIX] mail: only get one mail channel partner per user
When more than one mail.channel.partner have been linked to a given user, he may
be unable to open some discussion channel because of a traceback.
The issue is that we access a channel_partner field while the recordset is not a
singleton.
We limit the search to 1 to ensure that this recordset is a singleton.
2018-08-09 11:24:27 +02:00
Christophe Simonis cdef8740ec [MERGE] forward port branch 11.0 up to 252bc396b4 2018-08-09 11:10:38 +02:00
Simon Lejeune 252bc396b4 [FIX] stock: button_validate: forced move lines
When a move line is not linked to a move, it'll be linked to the first
move of the same product at validation time. We slightly improve this
logic by linking it to the first non-completed move. To be completely
correct, we should split the move line if necessary and link it to all
needed moves, but at this time we don't  want to make the logic too
complex.

Use case: create a sale order with three sale order lines of the same
products, use the detailed operations view and add three move lines of
the same products, validate the picking. Before the patch, all move
lines were linked to the first move and thus only the first sale order
line had delivered quantities.

Fixes #25488
opw-1861765
2018-08-09 08:38:17 +02:00
Jigar A d92f17cebe [FIX] signup: allow non-admin to invite portal users 2018-08-08 23:44:40 +02:00
Christophe Simonis 325399ab44 [FIX] {sale_,}stock: revert 0ab71f1247
Esentially forward-porting a backport is not a good idea...
2018-08-08 19:38:19 +02:00
Christophe Simonis 4c2bdafe69 [MERGE] forward port branch saas-15 up to e7ccf38c48 2018-08-08 18:42:27 +02:00
Christophe Simonis e7ccf38c48 [MERGE] forward port branch saas-14 up to 42f87c73fd 2018-08-08 17:58:05 +02:00
Christophe Simonis 42f87c73fd [MERGE] forward port branch 10.0 up to 6ded4d9ce8 2018-08-08 17:26:04 +02:00
Pedro M. Baeza b122526666 [FIX] pad_project: avoid rewriting project on task saving
Due to the side effect on related fields that are not readonly, the value is saved
in project although not changed. Putting it readonly avoids the problem.

Fixes #26156
Closes #26171
2018-08-08 15:42:13 +02:00
Nicolas Martinelli d32a5eddc8 [FIX] point_of_sale: wrong behavior Price Control
This is not expected since it leads to an inconsistent result: the user
and the access rights do not match. Better add a PIN to the manager.

This reverts commit bb8bb3cbfe.
2018-08-08 14:30:07 +02:00
Nicolas Martinelli 6ded4d9ce8 [FIX] payment_stripe: disable button
- Create an order from the eCommerce
- Choose to pay with Stripe
- Enter the credit card info, validate

There is a short window of time (before the reload of the page) where
the user can click on 'Pay Now' again.

This is because the HTML is completely replaced, including the button.
We restore it when necessary.

opw-1866332
2018-08-08 14:18:17 +02:00
Martin Trigaux c119ea40bb [FIX] point_of_sale: show the invoice button for paid orders
Introduced at e434690fe3
The button should be displayed when the order is paid to generate the invoice
Once the invoice is generated, the state is invoiced and a statbutton lead to
the generated invoice.

Remove the readonly attrs as it has no effect on a button
2018-08-08 14:13:46 +02:00
Lucas Perais (lpe) f2b2d87236 [FIX] account: default name for analytic line from aml
Create an account_move, with at least one move line having an analytic account on it
AND no label

Save and Post

Before this commit, the post failed because an analytic.line should have a label (required)

After this commit, we fill in a default name for the analytic line, composed of the data of the line,
if available, or '/'

OPW 1863418
closes #26013
2018-08-08 10:54:18 +02:00
Gert Pellin bb8bb3cbfe [FIX] point_of_sale: wrong behavior Price Control (#26232)
Before the fix price control only worked like should on initialisation of a pos section. On switching cashier during a setting manager would lose price change rights and normal users would gain them.
2018-08-08 09:59:20 +02:00
Wolfgang Taferner a4a9196c8d [FIX] account: regression - display partner_name as placeholder 2018-08-08 09:50:31 +02:00
Damien Bouvy c61cddf814 [FIX] hr_gamification: csv space EoL
Can cause some issues with importing the files
2018-08-08 09:40:16 +02:00
Nicolas Lempereur 8ff138a0da [FIX] barcodes: ok PageUp/Down/Home/End in firefox
Same reason as 8d948d92.

In firefox if barcode is installed, for example on a form view the keys
to navigate the possible scrolling PageUp/PageDown/Home/End don't work
at all.

This is caused by the interface barcode catching and differences of the
key event handling between firefox and other browser.

So in this change, we add these keys so they are not catched by barcode.

note: aso backport 5bc29744c so the code is the same between 10 and 11.

related to opw-706352
closes #26219
2018-08-08 09:11:06 +02:00
len-odoo 0ab71f1247 [FIX] stock, sale_stock: copy the to_refund field when splitting a move
The to_refund attribute is set to copy=False.
However when splitting a move to create a backorder, a new move is created as
a copy with the remaining work to do.
This attribute is needed so that the backorder refund is consistently applied.

Essentially a backport of commit:
https://github.com/odoo/odoo/commit/c5ffca151160f9d99bec036330def108e4e91e0b

opw 1865324
2018-08-08 08:49:11 +02:00
Goffin Simon ceb5c03c93 [FIX] hr_holidays: Allocation Request not always in the Summary View
Steps to reproduce the bug:

- connect as the demo user

- go to the Leaves app

- check the Leaves Summary menu: by default, it is empty

- go to Allocation Request

- create one Allocation Request for 1.00 day using the Compensatory Days type

- repeat the operation several time

Bug:

Only the first Allocation Request appears in the Leaves Summary

opw:1872365
2018-08-08 08:42:08 +02:00
Damien Bouvy 19f477299b [FIX] hr_gamification: csv space EoL
Can cause some issues with importing the files
2018-08-08 07:59:45 +02:00
Christophe Simonis 476e1b5646 [MERGE] forward port branch saas-14 up to 2c942e5ff8 2018-08-07 19:14:24 +02:00
Christophe Simonis 2c942e5ff8 [MERGE] forward port branch 10.0 up to bec639c094 2018-08-07 19:12:16 +02:00
Christophe Simonis bec639c094 [MERGE] forward port branch 9.0 up to 5a64fffe9a 2018-08-07 19:10:25 +02:00
Christophe Simonis efe2dcd7ae [MERGE] forward port branch 11.0 up to ced9156a97 2018-08-07 19:08:07 +02:00
Christophe Simonis 5a64fffe9a [FIX] base: explicitly allow superuser to write on attachment's assets 2018-08-07 19:06:02 +02:00
Christophe Simonis ced9156a97 [MERGE] forward port branch saas-15 up to 65a80cf3fd 2018-08-07 18:11:15 +02:00
Swapnesh Shah 174bd3a7ac [FIX] website: remove unnecessary ensure_one
The write works on multiple records

Closes #26033
2018-08-07 17:47:57 +02:00
Nicolas Martinelli 6470ee74b8 [FIX] point_of_sale: return with rounding < 0.01
- Set the currency to a rounding of 0.001
- Create a POS order:
  Unit Price: 0.12
  Quantity: 0.1
- Pay and validate
- From the backend, return the products on the create order
- Click on 'Payment'

The proposed amount is 0.01, not 0.012. It is not possible to complete
the payment process since the field is rounded at DB level to 2 digits
of precision.

We make consistent the `amount` field with the various amounts on the
`pos.order`. By using a `digits=0` and forcing the rounding, we avoid
inconsistent rounding between the order and the payment.

opw-1864344
2018-08-07 17:33:57 +02:00
Christophe Simonis 65a80cf3fd [MERGE] forward port branch saas-14 up to 0c5cbbe0bb 2018-08-07 17:01:16 +02:00
Christophe Simonis 0c5cbbe0bb [MERGE] forward port branch 10.0 up to 692a32e780 2018-08-07 16:54:18 +02:00
Christophe Simonis 692a32e780 [MERGE] forward port branch 9.0 up to 1b76b6168f 2018-08-07 16:52:50 +02:00
Martin Trigaux 3d3a9f3169 [FIX] event_sale: make action_confirm work with multi
The other overrides are working with multiple records but not event_sale.
The ensure_one() is not needed if you make a proper loop

Replaces and closes #26078
2018-08-07 14:58:40 +02:00
Christophe Simonis 33e7479865 [MERGE] forward port branch saas-11.2 up to d7e2c767ce 2018-08-07 13:59:40 +02:00