Commit Graph
107659 Commits
Author SHA1 Message Date
Goffin Simon cfb06c83e0 [FIX] base: preview_rml_report
Typo error

opw:728334
2017-04-12 09:19:25 +02:00
Raphael Collet 3481f0faff [FIX] fields: recompute recursive stored fields 2017-04-11 15:34:34 +02:00
Wolfgang Taferner 46e10baeac [FIX] point of sale: readonly attributes incomplete
We do not want to allow the change of a related config via a pos.order
and in case the pos.order is done or invoiced we should not be able to
change the user_id anymore and the fiscal position should never ever be
changed after the order is not in draft anymore.
User editable introduced at dca370d7.

Closes #16372
2017-04-11 15:09:14 +02:00
Yannick Tivisse bbc361d389 [FIX] sale: Use the shipping partner in tax computation 2017-04-11 14:38:01 +02:00
Fabien Meghazi 68a9a4ce54 [FIX] portal: fix regression of API migration in the portal wizard
Before the migration, send_mail() used to be called with an untainted
context but after the API migration it inherited the altered context
with active_test=False which caused subsequent calls to
ir.mail_server#connect() to fetch inactive records.
2017-04-11 12:28:34 +02:00
Adrien Peiffer (ACSONE) 3a88f7826b [REF] hr_expense: inheritance on move line preparation values
Closes #16351
opw-728271
2017-04-11 10:46:57 +02:00
Nicolas Martinelli c9c110d088 [FIX] payment_authorize: no partner email
When no email is defiend on the partner, a traceback is raised when
paying with Authorize.net.

The `text` attribute requires a string, not a boolean (`False` in this
case).

opw-728301
2017-04-11 10:43:09 +02:00
Odoo Translation Bot 69b45530dc [I18N] Update translation terms from Transifex 2017-04-11 10:37:54 +02:00
Nicolas Martinelli 1d29649e57 [FIX] account_tax_cash_basis: zero line
- Create a new account of type "Expenses". Allow the reconciliation and
  assign this account on the internal category of a product on both
  "Expense account", "Stock Input" and "Stock Output".
- Create a fiscal position to get the following tax behavior: apply two
  taxes of the same amount but with opposite sign (+ 21% on the one
  side; - 21% on the other side). Activate the tax cash basis feature on
  these taxes. It should give, on the same FP:
  Tax 21% -> Tax 21%
  Tax 21% -> Tax -21%
- Create a journal for the tax cash basis, set it in the company
  settings.
- Create a PO (with the previously created FP) for several products
  belonging to the same internal category and receive the products. Each
  line generates an entry recorded in the stock journal. The expense
  account is used in those entries.
- Create the supplier invoice for the PO and validate which also
  generates a journal entry with as many line on the expense account as
  there are some products on the invoice.
- In the journal items, reconcile the expenses accounts entries
  generated by the stock input and the supplier invoice.

It triggers a lot of lines on this expense account for 0 each.

opw-726120
2017-04-11 09:00:38 +02:00
Alexis de Lattre 91470a37fe [FIX] account: traceback on mass reconciliation of payments and invoices
Fix Bug #14866. Was PR #16149. Courtesy of Alexis de Lattre (Akretion)
2017-04-11 08:51:06 +02:00
Christophe Simonis 2c897d4199 [FIX] base: allow users to delete own filters, even if not creator 2017-04-10 15:17:49 +02:00
Pierre Masereel 57baccb14a [FIX] maintenance: scheduled maintenance request
When we set a period for preventive maintenance on an equipment which
has no maintenance team set, the cron cannot create the maintenance
requests.

To fix this issue, the maintenance team is now required when a period is
set on an equipment.
2017-04-10 14:54:53 +02:00
Nicolas Martinelli 0b7b5650f1 [FIX] purchase: select supplier
Complement of previous commit, allows to retrieve procurement in the
method as well.
2017-04-10 13:13:22 +02:00
Nicolas Martinelli a07447e85a [FIX] purchase: select supplier
When a PO is created through the resolution of a procurement, the first
supplier of the list is automatically chosen. This is hardcoded and
cannot be extended.

We add a hook so it's possible to customize the selection of the
supplier, for example by selecting the one who offers the best price.

opw-726309
2017-04-10 11:12:48 +02:00
Florent de Labarre aefd8813b9 [FIX] report: check error codes for big pdf files
Closes #16060
2017-04-10 09:43:42 +02:00
qsm-odoo 4aab300070 [FIX] web_editor: allow to save unchanged icons with media dialog
There was a bug with the icon edition when following these steps:
- Open the Media dialog to edit an existing icon
- Hit the "save" button in the dialog without choosing the icon
- The edited icon disappears

This happenned for icons whose original values were aliases of the
related icon. Indeed, following these steps to edit an "fa-facebook-f"
was leaving the icon unchanged as expected but doing it for a
"fa-facebook" icon made the icon disappeared. This is because the
last one is an alias of the first one and that the media dialog was
ignoring aliasies when saving the original value.

opw-725813
2017-04-07 17:46:07 +02:00
Pierre Masereel 8d65763f93 [FIX] stock: MTO move status
When we validate a SO with an MTO product, then an RFQ and a delivery
picking are created. The status of the delivery picking is in state
'confirmed', but should be 'waiting' as the delivery waits for a
specific delivery input.

The problem is caused by the unreserve method, which is called in the
'procurement_jit' module. This method only checks that a move has
ancestors to determine if the move should be in waiting or in confirmed,
which is not the case for our delivery move because the RFQ has not been
validated yet so there is not ancestors yet.

To fix this issue, we check the 'procure_method' of the move, and if it
is in 'make_to_order' we put it in 'waiting' instead of 'confirmed'.

We also fixed the function recalculate_move_state that could cause the
same problem.
2017-04-07 16:35:57 +02:00
Yannick Tivisse a3778ceada [FIX] account,sale: Select correclty the delivery partner
To deduct the fiscal position, we should use the shipping partner.

The field `partner_shipping_id` is defined in the sale module.

Thus, we should choose this partner if the module sale is installed and if this field is set, or fallback to the original partner instead
2017-04-07 16:17:49 +02:00
jaredkipe cf81b665e5 [FIX] hr_timesheet_attendance: convert field type
Bad migration of new API, was a float field.
Probably introduced at 510d260

Closes #16290
2017-04-07 15:12:22 +02:00
Yannick Tivisse 1452a9903c [FIX] account: Use the invoice shipping address to select the fiscal position
Tribute to Ray Carnes

PURPOSE
=======

Since the fields shipping_partner_id has been added on the invoice model, the onchange triggered by the partner_id hasn't been modified.

SPECIFICATION
=============

The shipping_partner_id will be picked by default to select the fiscal position. If there's no shipping partner, it will check if a 'shipping' partner is a child on the partner_id.
2017-04-07 14:33:02 +02:00
Miku Laitinen b3c949b0c5 [FIX] doc: migration from community to enterprise
In the beginning Odoo Enterprise addons contained a second web module. At some
point that was replaced with web_enterprise, but the documentation wasn't
updated.

Describe the proper changes.

Closes #16302
2017-04-07 14:00:25 +02:00
Goffin Simon 428d9e8177 [FIX] odoo: Copy a record in other language than US
When copying a record in an other language L than US, first a translation
was created for L by the create called in the copy function and afterwards,
a second translation was created by the function copy_translations

So there were two duplicated translations for L instead of one.

opw:706865
2017-04-06 17:13:15 +02:00
Raphael Collet 5219dcbae3 [FIX] base: malformed base.pot 2017-04-06 16:30:29 +02:00
Raphael Collet 6621ade382 [FIX] ir_model: prevent renaming/removing fields that are used in views 2017-04-06 15:06:06 +02:00
Raphael Collet 5a1672fab8 [FIX] ir_model: field names must be unique per model 2017-04-06 15:05:52 +02:00
Raphael Collet 6b48098c81 [FIX] ir_model: field names must be valid pg identifiers 2017-04-06 15:05:52 +02:00
Nicolas Martinelli e79d94438e [FIX] website_sale{_options}: quantity over 999
The fields containing the quantity of product doesn't display numbers
over 999 correctly: only 3 digits display in the field.

We add a specific class `quantity` for input fields, with a padding of
zero. The class `js_quantity` is not used since it is linked to jQuery
events.

opw-727640
opw-725671
2017-04-06 11:05:30 +02:00
Nicolas Martinelli d4684ba7cd [FIX] sale: 100 % discount
- Set discount policy: Show public price & discount to the customer
- Set a product with 100% discount

When added to a SO, the product discount is 0 %.

opw-727519
2017-04-06 08:37:59 +02:00
Nicolas Martinelli 48aefe61f0 [FIX] mrp: warning for lot
Complement of commit 85132f68d2

opw-727679
2017-04-06 08:35:43 +02:00
Jeremy Kersten bd17dfe019 [FIX] payment_paypal: pdt parsing - accept error as response
In case of error, paypal return some lines where error message and not a
string with 'key=value'.

This commit fix traceback:
ValueError: dictionary update sequence element #0 has length 1; 2 is required
2017-04-05 17:21:07 +02:00
Nicolas Lempereur dff2cf3ca1 [FIX] base: field monetary lose context of t-lang
In monetary field refactoring (3407fb70e4) there was a small issue that
made context lost so the language use to render the field could be
different from the one to render the view.

For example: if the user has english, and print a report using the lang
of a partner in language spanish, the report would use spanish to format
number for all fields but monetary (which would use english).

opw-708197
2017-04-05 14:49:03 +02:00
Martin Trigaux 0ac2ca9b14 [FIX] point_of_sale: translate close button
Evaluated when loading the widgets
2017-04-05 14:26:12 +02:00
Richard Mathot a16da9f9c6 [IMP] mrp: price on product should be changed when receiving from production in average price, not only from supplier 2017-04-05 14:05:15 +02:00
Wolfgang Taferner bf781cf740 [FIX] mrp: BoM count detection in case it is a single variant
In case a product is a single variant we should not leave it with a BoM
of zero as soon there is a BoM for its template. This would for example
hide the `compute_price` object button which relies on the `bom_count`to
be visible.

BoM is not found for a single variant product and compute price not
possible from the `product.product` view although it is implemented
actually there.

Closes #16237
2017-04-05 12:00:39 +02:00
Nicolas Martinelli 85132f68d2 [FIX] mrp: warning for lot
The constraint should only check for move lots with a lot number already
filled in. Indeed, in case we use work orders, we can potentially create
move lots without S/N, and fill in the S/N later.

opw-727679
2017-04-05 09:47:54 +02:00
Martin Trigaux c6b4154fa6 [FIX] point_of_sale: do not create picking for service only orders
opw-727322
2017-04-05 09:37:21 +02:00
Goffin Simon 130e71f2e4 [FIX] sale: _get_display_price with pricelist item based on other pricelist
Create a product P with a price = 1799
Make a pricelist P1 in USD with an item I1 for a product P with a
fixed price to 100.
Set P1 with discount_policy = "with_discount"(include in price)
Make another pricelist P2 in USD with an item I2 based on P1.
I2 is a formula = 50% of the price
Set P2 with discount_policy = "without_discount"
Create a SO with P2 for the product P

Bug: The subtotal of P is 899.5 instead of 50

The problem came from _get_display_price wich in this case recomputed
the price according to the discount(in this case equal to 50% and this is
the good discount to display because P1 include the discount) and the lst_price
of the product(=1799) So the displayed price was equal to 899.5 instead of
50.

All the other cases worked because

1.  P1 without_discount and P2 without_discount:

Price returned by _get_display_price was equal to 1799 with a discount of 97,2%
So the subtotal was equal to 50

2. P1 without_discount and P2 with_discount:

Price returned by _get_display_price was equal to 100 with a discount of 50%
So the subtotal was equal to 50

3. P1 with_discount and P2 with_discount:

Price returned by _get_display_price was equal to 50 with a discount of 0%
So the subtotal was equal to 50

And the last case(P1 with_discount and P2 without_discount) didn't work because:

Price returned by _get_display_price was equal to 1799 with a discount of 50%
So the subtotal was equal to 899.5

NB: The subtotal is computed with function "_compute_amount" with the following line:

price = line.price_unit * (1 - (line.discount or 0.0) / 100.0)

opw:724924
2017-04-05 09:25:18 +02:00
Lucas Perais (lpe) d24f86337e [FIX] account: account_reconciliation: t-values shouldn't be translatable
+ .pot update
OPW 706759

Closes #16239
2017-04-05 08:51:10 +02:00
Nicolas Martinelli c028e3806b [FIX] account: tax base on invoice report
Commit e1deead1cb introduces back the table of taxes. But it is still
incomplete:
- it should appear all the time
- the tax base should be included as well

Sources:
http://ccff02.minfin.fgov.be/KMWeb/document.do?method=view&id=21786bb3-69a0-4323-8631-aa898b988097#findHighlighted
http://www.ipcf.be/Uploads/Documents/doc_1704.pdf
http://eur-lex.europa.eu/LexUriServ/LexUriServ.do?uri=OJ:L:2002:015:0024:0028:FR:PDF

opw-724519
2017-04-05 08:23:58 +02:00
Sergio Teruel Albert 5b670aa864 [FIX] fleet: access record fields on api.multi
Would not work if executed on several records

Closes #16218
2017-04-05 07:52:34 +02:00
Pedro M. Baeza dc8010c9b1 [CLA] Signature for Tecnativa
Done at #16241
2017-04-05 07:50:53 +02:00
Nicolas Lempereur 3ad1867ea4 [FIX] sales_team: get good team in multi company
When getting the default sales team, we could get erroneously the
team of another company (either because the use is in a team of another
company or the default sales team was in non-bethren company).

This fix when searching a sales team:

- search only teams the user should see (with access rules)
- make default teams "Website Sales" and "Direct Sales" not restricted
  to one company (and it's ancestors).

opw-726461
2017-04-04 17:57:07 +02:00
Josse Colpaert 11f0a5c29f [FIX] sale, purchase: better info on qty update in purchase/sale order line
When the quantities on a purchase or sale line decrease,
the amount of the receipt or delivery order is not adapted.
Also, when the quantities increase, it is good to follow it up.

That is why, when we change some quantity, a message is
posted in the chatter of the purchase/sale to indicate about
the change.  There is also a message when you add a new line.
2017-04-04 17:36:02 +02:00
Goffin Simon 42e1efded5 [FIX] web_kanban: column_add_record
When grouping by a field datetime in kanban view, the name of the
column is an abreviated datetime. Afterwards, when making a quick
create, the abreviated datime was set in the context and
used as default value. So it raised a tracebacks because the value
of the datetime was not in the correct format.

Now it's only possible to make a quick create if the type of the field grouped
is a char, a boolean or a many2one.

opw:727266
2017-04-04 15:19:42 +02:00
Lucas Perais (lpe) fc0d2e2c83 [FIX] account: Dashboard used to open provider bills instead of customer invoice
Correcting patch on PR16042 by Alexis delatre
OPW 726679
Closes #16210
2017-04-04 09:48:54 +02:00
Martin Trigaux ec17d636a9 [FIX] point_of_sale: allow to create more than one rescue session
Fixes #15101 (cf comment 273208287 for details)
Closes #15862

Restore the rescue mechanism from v9, without the field it will be based on
the session name containing '(RESCUE FOR'

Add tests
2017-04-03 15:19:37 +02:00
Goffin Simon ef5fee54b2 [FIX] account_asset: typo error
To follow this commit: 07cbde54d6

opw:724571
2017-04-03 14:42:56 +02:00
David Tran 3d696be967 [CLA] add team members to the tvtmarine-automation CLA
Done at #16200
2017-04-03 10:56:51 +02:00
Jerther 925d8175f5 [CLA] Corporate signature for MetalSartigan
Métal Sartigan accepts the CCLA

Done at #16185
2017-04-03 10:37:29 +02:00
Wolfgang Taferner 51975ccf5a [FIX] doc: wrong header syntax for field 2017-04-03 10:08:37 +02:00