Commit Graph
106846 Commits
Author SHA1 Message Date
Haresh Shyara c02b2ca7d1 [IMP] base_import: Improve import preview for related fields
normal mode: Importing only regular/basic fields from source file
advanced mode: Importing any related fields from source file(i.e. if detected pattern in header like '<relational_fieldname>/<fname>')

- Rename following checkbox "Show all fields for completion (advanced)" in "Show fields of relation fields (advanced)"
- If the import file contains a column with a field of a relation field, show the column label in the preview, even if "Show fields of relation fields (advanced)" is not checked > to make it less confusing when importing advanced files (e.g. products with attributes)

This is a backport of 1ba4fbe640, which was performed due to several
tickets mentioning confusion with the new import system.
2017-01-03 12:07:47 +01:00
Pin Zhang 4c798a2690 [FIX] base: use defined variable
The directive is `mname` not `name`.

CLA signature for PinZhang
Closes #14895
2017-01-03 11:17:43 +01:00
Raphael Collet 46b51cb4b2 [FIX] expression: multi-level expressions with magic columns 2017-01-03 10:17:13 +01:00
Nicolas Lempereur a988653157 [FIX] web: error if chart with only zero values
We could get an js error if the chart contained zero values.

opw-703245
2017-01-02 17:31:06 +01:00
Nicolas Lempereur 983237a53c [FIX] account: fix css congratulation message
The css clear was using jquery ui class for enterprise which now may
not be available on the given part. This commit replace it with class
defined in account_reconciliation.

With this the style could have issues such as not displaying the
congratulation message after reconciliation.

fixes #14743
2017-01-02 16:27:27 +01:00
Christophe Simonis c08a590bb1 [FIX] purchase: correct fifo test
Oversight of previous forward-port.
2017-01-02 15:07:14 +01:00
Christophe Simonis 81f59cee2c [MERGE] forward port branch saas-11 up to b844d93889 2017-01-02 14:38:59 +01:00
Christophe Simonis b844d93889 [MERGE] forward port branch 9.0 up to 3805eb620a 2017-01-02 12:49:55 +01:00
Christophe Simonis 3805eb620a [MERGE] forward port branch saas-6 up to 7be1a1c55b 2017-01-02 11:42:52 +01:00
Christophe Simonis 7be1a1c55b [MERGE] forward port branch 8.0 up to 2b8fcfc08b 2017-01-02 11:22:45 +01:00
Damien Bouvy d3538dbe47 [FIX] payment_ogone: migration error
Only happens if Ogone answers with "wait", which explains why it
took so long to happen.
2017-01-02 11:11:49 +01:00
Goffin Simon 7fc1a3ca7b [FIX] mrp: Computing the qty of the BOM Structure report
Each qty of a line in the BOM Structure report must be computed in
the uom of its BOM line.

Example:

With 2 products: A, B

BOM A: 100 kg of A is produced by 5t of B

So 1kg of A needs how many t of B ?:

In the code:
qty_per_bom = 100 kg expressed in the uom of B => qty_per_bom = 0.1t
qty = 1 kg expressed in the uom of B => qty = 0.001t
l.product_qty = 5t
qty of B = (l.product_qty *qty)/ qty_per_bom = 0.05t

opw:697502
2017-01-02 10:37:08 +01:00
Odoo Translation Bot e8b0c1db69 [I18N] Update translation terms from Transifex 2017-01-01 04:42:12 +01:00
Odoo Translation Bot 2b8fcfc08b [I18N] Update translation terms from Transifex 2017-01-01 02:58:08 +01:00
Odoo Translation Bot 072906795c [I18N] Update translation terms from Transifex 2017-01-01 00:30:05 +01:00
Thibault Delavallée 84a650e33a [FIX] resource: fix computation of limited attendances
Due to an error introduced when migrating the module at new API
at commit a4e57ea16c limited attendances
having date_from and / or date_to were not correctly taken into
account.

Tests have been added to avoid regression.
2016-12-30 16:38:02 +01:00
Nicolas Lempereur 5869cd6bd0 [FIX] website_customer: error if customer without countries
For customer without countries, we get False instead of tuple:
 (country_id, 'country name')

Introduced by a35cce2008.

opw-703234
2016-12-30 15:27:48 +01:00
Nicolas Martinelli a71cdd02ec [FIX] payment_paypal: documentation link
The Paypal URL doesn't exist anymore. Redirect to Odoo documentation
instead.

opw-697606
2016-12-30 15:01:43 +01:00
Nicolas Lempereur eb2d598935 [FIX] auth_oauth: dup fail if odoo oauth provider removed
It is unwise to delete it (it can simply be disabled) but it can be done
(and other part of the code take it into account).

opw-703196
2016-12-30 11:29:48 +01:00
Nicolas Martinelli e1deead1cb [FIX] account: reintroduce tax lines in invoice
Commit d861cdc1c4 introduced the tax group feature. The tax line
details were removed from the printed invoice in favor of the tax group.

However, it requires that all tax groups are correctly defined in the
localization, otherwise the printed invoice doesn't show the required
tax details.

The correct fix, which is the complete definition of the tax groups,
will be done for the upcoming v11 version. In the meantime, this simply
introduces back the tax details table.
2016-12-30 09:16:08 +01:00
Jose Morales 3c67bf038f [FIX] l10n_mx: add missing account in chart template.
Some cash basis taxes were assigning the account 208.01.01 but this account was not created. Also add tax group
2016-12-29 22:31:16 +01:00
Nicolas Martinelli 1e22870d26 [FIX] product_margin: crash of read_group
Introduced by 9a479e6572
2016-12-29 14:55:31 +01:00
Nicolas Martinelli 3667516a74 [FIX] product_margin: computed values
After commit 9a479e6572, all values in the report are 0.0.

opw-695649
2016-12-29 14:54:06 +01:00
Nicolas Martinelli 8a1b55ae04 [FIX] website_sale: default company on SO
In multi-company:
- Create 2 companies A and B
- Set the superuser on company B
- Set the website on company A

When a SO is generated from the eCommerce, it is defined on company B,
while it should be on company A.

opw-695253
2016-12-29 13:49:28 +01:00
Denis Vermylen (dve) 1879964116 [FIX] hr_holidays: add followers to employee leave
from odoo#14848

Those subscribed to the department of the employee didn't get notified when
the employee encoded a leave.

It was due to the department_id not being given as value in create when the
employee made the leave.

This is because there's a group group_hr_holidays_user on that field in the
view, so only when they created a leave the followers would be added correctly
(as department_id was given as value in create).

This commit also remove the useless subscribe_followers method.
2016-12-29 11:52:47 +01:00
Goffin Simon 30fb3f7c4c [FIX] stock: Making inventory adjustments in multi companies
With Company A (Warehouse Wa) and Company B (Warehouse Wb)

When making inventory adjustments for Warehouse Wb in company A,
the inventory lines displayed must only be the lines of the company A in
the Wb.

In this way, the user avoids confusion with the adjustments he is creating.

opw:697182
2016-12-28 15:51:15 +01:00
Goffin Simon b58b334ace [FIX] stock: Inventory adjustments in multi company
When making an inventory adjustments in multi company, the company field
must be set according to the location_id set. In this case, it avoids creating
by default different inventories for the same warehouse.

opw:697182
2016-12-28 15:45:49 +01:00
Denis Ledoux 7befe67a75 [FIX] account: compute_invoice_totals with amount currency set, but to False
This is related to revision dfcc0c40b1

If the invoice line provided the amount currency, but
set to False, the amount_currency must be computed anyway

opw-702045
2016-12-28 14:57:27 +01:00
Denis Ledoux dfcc0c40b1 [FIX] account, stock_account: anglo saxong cost price currency rounding
When selling in a foreign currency,
A double currency rate change, one in each way, was applied when creating the good cost
move line of the invoice, which leaded to some precision loss when the decimal accuracy was
set with a number of digits greater than the currencies digits rounding

e.g.
Product cost price: 52.56422
CAD to USD rate: 0.7383

The product cost price is always in the company currency
while `_anglo_saxon_sale_move_lines` always returned the price
in the currency of the invoice. A first conversion rate was applied
from the company currency to the invoice currency.

52.56422 * 0.7383 = 38.808163625999995, rounded to 38.81

Then, when validating the invoice, and creating the move line,
the price was then converted again, to the company currency,
as the debit/credit of a move line is always in the company currency:
38.81 / 0.7383 = 52.566707300555336, rounded to 52.57

The cost price of the product in the good cost move line was therefore
set to 52.57, insteaf of 52.56, because of this precision loss.

With this revision, we avoid to do the currency rate conversion
two times, in both ways, to avoid the precision loss.

opw-702045
2016-12-28 14:19:18 +01:00
Nicolas Martinelli 26d1a3884b [FIX] account: crash when unlink taxes
If `get_default` returns an empty value (`None`), the `set` instruction
crashes.

opw-702597
2016-12-28 13:12:12 +01:00
Nicolas Martinelli 66732acabf [FIX] hr_holidays: leaves by department
Backport of commit 80963c33b0

opw-702202
2016-12-28 11:24:13 +01:00
Fabien Pinckaers 347494c06b [FIX] gamification: avoid sending blank emails 2016-12-27 15:28:01 -08:00
Christophe Simonis aa09c52205 [FIX] account: correct query
fix up of commit e98c329039.

opw:702908 opw:702909 opw:702910 opw:702911 opw:702918
2016-12-27 21:02:14 +01:00
Nicolas Lempereur 2639d634bb [FIX] calendar: day of month is expected filled in code
Before this commit there was an error when we chose a monthly recurrence
with "Day of Month" of type "Day of" because the code expect them to be
set. In any case they are required field so if they are not field we
should just ignore it.

opw-702852
2016-12-27 16:50:39 +01:00
Nicolas Lempereur d36dca2d2d [FIX] website_sale: 2 state tags and same shipping change
If website_sale_delivery is installed, when we choose same invoice
address as shipping or another one, a new set of country/state are
displayed.

It is done since if we are using the invoicing address as shipping, the
address must be in the allowed shipping countries and states.

This was solved for some instances with 164b8ed01a but failed in other
instances.

With this fix the two state select tags have references to their own
options which allow us to have it alright for the several possibilities.

opw-693127
2016-12-27 15:44:55 +01:00
Nicolas Martinelli 80963c33b0 [FIX] hr_holidays: leaves by department
In the report "Leaves by Department", the start/end dates of the
holidays are computed based on the UTC datetime stored in the database.
This makes the report inconsistent with what is displayed in the
interface.

opw-702202
2016-12-27 15:20:30 +01:00
Nicolas Martinelli bf88647e8d [FIX] account: show maturity date
When creating a journal entry manually, the user needs to be able to
define the maturity date. It was always shown in v8.

opw-702652
2016-12-27 14:28:28 +01:00
Nicolas Lempereur a497300be8 [FIX] website_sale: star rating for first comment
For first comments on a product in the ecommerce we would not
have the star rating displayed till a page refresh, and the star widget
would not be reset after posting.

Also in other situations, we would get a 0 star widget when posting
comment without rating, but after a refresh the widget would dispappear.

With this change the new comment get the same stars as when refreshed.

opw-702608
2016-12-27 13:16:25 +01:00
Goffin Simon fe705b514c [FIX] point_of_sale: Product quantity in Orders Analysis
The product quantity in Orders Analysis must be expressed in the uom
of the product like in the Sales Analysis. In a POS order line, the uom
is always expressed in the uom of the product.

Closes #14599

opw:696927
2016-12-27 11:09:08 +01:00
Nicolas Martinelli cd3988e689 [FIX] base: duplicate ir.model.constraint
There is a contraint on `ir.model.constraint` that contrains the name to
be unique per module.

opw-702516
2016-12-27 10:14:02 +01:00
Nicolas Martinelli 285964b710 [FIX] event_sale: ticket price currency conversion
When adding a ticket to a SO, the currency of the SO is not taken into
account in order to convert the ticket price.

opw-702683
2016-12-26 16:25:45 +01:00
Goffin Simon e98c329039 [FIX] account: where_clause in function _credit_debit_get
The where_clause needed a "AND" to work.
The where_clause didn't contain allias for "account_move_line" so when
injecting a condition with "account_move_line" it crashed.

opw:702538
2016-12-26 13:35:11 +01:00
Nicolas Martinelli 634fb9f390 [FIX] purchase: purchase count
The field `purchase_count` is described as '# Purchases' (which is an
integer), so the user should expect the number of POs associated to the
product, not the quantity ordered as it is the case before this commit.

opw-697815
2016-12-26 13:27:01 +01:00
Nicolas Lempereur 46f84b0284 [FIX] mail: linkify livechat message so frontend has them
In ff56304f58 a solution was tried to have message posted from
backend livechat linkified for visitor in frontend. This was
forward-ported when it should not have been and introduced issues
including:

- `href="{link}"` would become `href="<a href="{link}">{link}</a>"`

- if html entities such as the ones for U+00A0, and entities <, >,
  ', " or ` were next to the link, the entity would be linkified
  too (up to the ;)

This happen because utils.linkify function is to be used with text
content and not html which was the case in this instance.

opw-693457
opw-702410
2016-12-26 12:18:09 +01:00
Nicolas Lempereur 88d7e7f646 Revert "[FIX] mail: livechat escaping"
This reverts commit 1c60a631df.

The fix was done to conciliate 83956d4 and ff56304f5 but ff56304f5 was
forward-ported erroneously and introduced other issues.

opw-702410
2016-12-26 10:26:21 +01:00
Nicolas Martinelli 562f141ffc [FIX] stock: hide "Run Reordering Rules"
This should only be accessible in debug mode, since it is already
possible to run the schedulers (with a simialr effect) manually.
2016-12-26 09:59:45 +01:00
Nicolas Martinelli bdbda19b6d [FIX] stock: prevent concurrent procurement creation
When the user clicks on "Run Reordering Rules", it might create
procurements. However, the cron "Run mrp scheduler" will also create
procurements and might already be running in the background. Therefore,
we need to make sure that both actions don't run concurrently.

Note that the same check is done for the wizard "Run Schedulers".

opw-695607
2016-12-26 09:59:45 +01:00
Goffin Simon 22fb76a002 [FIX] mrp: field resource_type in mrp.workcenter tree view
The field resource_type in mrp.workcenter tree view must be invisible because
this field was not available in the form view before f9f4bba

Adding resource_type in the form view had no business case.

opw:698183
2016-12-26 09:47:56 +01:00
Odoo Translation Bot b21d1ae98e [I18N] Update translation terms from Transifex 2016-12-25 04:42:15 +01:00
Odoo Translation Bot 534eab0c3b [I18N] Update translation terms from Transifex 2016-12-25 02:57:37 +01:00