Commit Graph
107918 Commits
Author SHA1 Message Date
Olivier Dony 656accfde4 [FIX] payment_*: remove obsolete license entry
The license entry in the manifest files for these modules were
leftovers from previous versions.

Once integrated in Odoo Community they share the same license as all
other modules, as mentioned in the LICENSE notice at the top of the
files.

opw-743686
2017-05-16 14:58:12 +02:00
Olivier Dony 7a443aabf9 [MERGE] Forward-port saas-11 up to 6490e652c3 2017-05-16 14:56:02 +02:00
Olivier Dony 6490e652c3 [MERGE] Forward-port 9.0 up to 1070b83e14 2017-05-16 14:48:43 +02:00
Wolfgang Taferner 58c928a732 [FIX] mrp: fix unlinking of move lots for production during create_lots
Closes #17007
2017-05-16 13:54:43 +02:00
Olivier Dony 1070b83e14 [FIX] l10n_cn*, payment_sips: clarify obsolete licensing info
The license info for these modules was a leftover from previous versions.

Once integrated in Odoo Community they share the same license as all
other modules, as mentioned in the LICENSE notice at the top of the
files.

opw-743686
2017-05-16 13:42:10 +02:00
Nicolas Martinelli 0fab06204c [ADD] hr_expense_check: add amount in words on check
- Create an expense, approve.
- Register the payment by check
- Print the check

The amount in words is not written.

The core of the issue is that the wizard
`hr.expense.register.payment.wizard` has been defined as a brand new
model instead of inheriting from `account.register.payments`. Hence the
Enterprise module "hr_expense_sepa" and now "hr_expense_check".

opw-742305
2017-05-16 13:27:40 +02:00
Nicolas Martinelli 8fdf046df2 [REF] account_check_printing: avoid duplicated code
Small refactoring necessary for the following commit, to avoid
duplicated code.

opw-742305
2017-05-16 13:27:40 +02:00
Goffin Simon 8c22c8b625 [FIX] sale: _get_invoiced with same sequence for PO and SO
When PO and SO used the same sequence, a vendor bill with a source
document equal to the sequence of a SO could be displayed by this
SO(in the smart button "invcoies").

opw:693991
2017-05-16 13:23:57 +02:00
Nicolas Martinelli 6c38a80b6f [FIX] purchase: modify expected date
- Create a PO with a line scheduled 2017-07-01
- Validate the PO => the expected date of the related stock move is
  2017-07-01.
- Modify the PO line date to 2017-07-15 => the expected date of the
  related stock move is not updated.

Since we allow the modification of the scheduled date, we should make
sure that the related information is changed accordingly.

opw-741815
2017-05-16 12:06:47 +02:00
qsm-odoo 1545995b39 [FIX] web_tour: tips were overlapping the chat windows
Before this commit, the tour tips were overlapping the chat windows.
This was due to the fact they always had a z-index of 1051 wherever
they were. If a tip was placed in a location where none of its
ancestors created a new stacking context, the closed tip was then
positioned above everything else, including the chat windows.

A simple solution would have been the one proposed by this commit:
https://github.com/odoo/odoo/commit/856e78b35a47a8732e5bd4da1cce247df70e5414
However, as it explained, this would not be entirely correct. In the
website, elements are using z-indexes and would thus hide closed
tooltips when it is not wanted if those have no z-index at all.

The solution this commit is making is to force the creation of a
stacking context for tooltips' parent. The z-index will still exist
on all tips (closed and not closed) to force them above every other
elements, but now only elements of the same parent. This is indeed
more correct (but maybe still not entirely): website tooltip are
often placed in the body (as it is the element which scrolls) and
are thus still placed above elements like the navbar as their
z-index is left high. However, when placed in the appswitcher, they
will now be placed above appswitcher elements only (and thus not
above outside elements which are themself above the appswitcher
(e.g. the chat windows)).

opw-743330
2017-05-16 11:17:13 +02:00
Olivier Dony ab8c81a96f [MERGE] Forward-port saas-11 up to 0570033ae7 2017-05-16 10:30:23 +02:00
Olivier Dony 0570033ae7 [MERGE] Forward-port 9.0 up to 2c4512c263 2017-05-16 10:06:36 +02:00
Olivier Dony 2c4512c263 [MERGE] Forward-port saas-6 up to 7dc2ed94c3 2017-05-16 09:22:27 +02:00
qdp-odoo a95a667563 [REF] account: validate of FY lock date into a dedicated method.
in res.company write(), create a sub method _validate_fiscalyear_lock()
so that it is easier to inherit to modify or extend its
functionality.

Was PR #16615
2017-05-15 23:37:48 +02:00
Joren Van Onder 7836a83b3b [FIX] point_of_sale: currency and decimal precision on ESC/POS receipt
Decimals were either hardcoded or not set at all.

opw-741752
2017-05-15 11:30:29 -07:00
Joren Van Onder 4d2328cd9e [FIX] point_of_sale: don't reserve pickings with tracked products
Introduced by 8f39ad2be6 which was
intended for 9.0. It does not work well in 10.0 because here users
have the possibility to specify their own tracking numbers directly in
the POS interface. This causes issues when eg. the user specifies the
same lot number as is already reserved by the system.

opw-742657
2017-05-15 09:39:45 -07:00
qsm-odoo bee3253b44 [FIX] mass_mailing: increase email clients compatibility
Creating a mail template which is complex and compatible with all mail
clients is a very difficult task as each mail client supports a
different subset of basic CSS rules.

This commit fixes 3 problems which were occuring in outlook and/or
thunderbird but not in gmail:

- Main layout horizontal overflow
    -> change 24px horizontal padding of main layout to 3% and set the
       width to 94% (so that the sum is 100%)

- Fixed 600px content was full width instead
    -> Change the .o_mail_wrapper to be a <table> element instead of a
       <div> element. The table is full-width and contains one row of 3
       cells: an empty left one, a fixed 600px-width center one and an
       empty right one. This allows to have the same layout as before but
       without using the unsupported max-width css rule.

- Texts were not using the right color and font-size
    -> <p> elements were not properly inheriting the color and font-size
       properties, so force the style on <p> elements.

opw-743156
2017-05-15 15:26:05 +02:00
Odoo Translation Bot 40eca1e3e7 [I18N] Update translation terms from Transifex 2017-05-15 12:23:37 +02:00
Cédric Pigeon 477b8176c1 [REF] stock: allow domain selection hook
Add hook to allow override of domain in orderpoint scheduler.

Closes #16909
opw-743020
2017-05-15 09:57:18 +02:00
Odoo Translation Bot edb8221cbb [I18N] Update translation terms from Transifex 2017-05-14 04:31:11 +02:00
Odoo Translation Bot 12a049e551 [I18N] Update translation terms from Transifex 2017-05-14 00:28:46 +02:00
Joren Van Onder 3d3391da86 [FIX] pos_mercury: take into account rescue sessions
If there are open rescue sessions there will be multiple opened
pos.session records with the same user.

A better solution would be to make the POS client send back the
pos.session id it belongs to and use that instead, but this would be
unsuitable for a stable release. It will instead be done in master.

opw-743318
2017-05-12 19:14:01 -07:00
Nicolas Lempereur 89e644000a [FIX] web_editor: don't resize iframe in content
In 4e0e376b0 an iframe for the readonly preview was added when
displaying html fields.

In e5112ee00 this iframe was only used if style-inline was set in the
widget options.

This brought an incompatibility if the not iframed preview contained an
iframe (eg. a youtube video which is then resize to a 30px height) which
is solved by this commit.

opw-725161
closes #16955
2017-05-12 16:21:14 +02:00
qsm-odoo f5d337680a [FIX] website: fix missed forward-port
A forward-port ported this commit:
https://github.com/odoo/odoo/commit/c12f2cd5ccc72a680ab741b4ce16d3913dd3854e#diff-473476fda8a8b38a9ee4480eb2bf2e2eR150
But ignored that a more recent one did remove the related code:
https://github.com/odoo/odoo/commit/0eafeab1#diff-473476fda8a8b38a9ee4480eb2bf2e2eL173

The result was misindented breaking code.
2017-05-12 14:12:30 +02:00
Josse Colpaert c91c3726a7 [FIX] mrp: for the cost calculation and traceability, we need to include all moves
When calculating the cost price or the traceability between the consumed
and produced quants in a production order, we need to take into account
the extra moves that are added in the action_done (move_validate)

opw-715541
2017-05-12 12:24:12 +02:00
qsm-odoo f83f9d702a [FIX] website: slideshow image gallery was not responsive
The slideshow image gallery code was quiet old and ugly. This commit
refactores it so that it works better, especially its responsiveness.

Before this commit, the number of thumbnail images was limited to 10.
If there were more images than 10, the user was able to navigate
between groups of 10 images thanks to arrow buttons. The problem is
obvious: on small devices, 10 images is too much. Now this "10" limit
is not fixed, it is determined according to the width of the browser
at the time the user loads the snippet.

Some animation and style was also wrong in some cases.
This commit was made as a 10.0 fix, the snippet should keep improving
in advanced versions.
2017-05-12 10:08:27 +02:00
Richard Mathot d994f51c60 [FIX] website_sale: hide horizontal rule if there is no availability warning 2017-05-11 18:18:25 +02:00
qsm-odoo 7fc788b694 [FIX] website_sale: "Confirm order" button had fixed width
Putting fixed width on button is obviously always a bad idea as
traduction may change the inner text size. Besides, removing the
fixed width does not change the design anyway.
2017-05-11 17:05:38 +02:00
Aaron Bohy d068c4cac1 [FIX] sale: group_sale_layout shouldn't imply group_user
Because when it does, it can create a cycle in the graph of
implied groups. This happens when the option 'Sort products
in sections with subtotals and page-breaks' is checked in the
Sales settings. In this case, being in group_user implies to be
in group_sale_layout, and being in group_sale_layout implies to
be in group_user.

This is a problem for groups display in the 'Users' form view: the
employee groups are no more linear, and thus no more displayed as
a selection field, but there is an 'Employee' section where each
of the employee group can be checked. Activating the option in the
Sales setting should have no impact of the groups hierarchy of HR.

Moreover, there have been several imilar issues on production
databases, probably due to this cycle, but that we haven't been
able to fully understand/reproduce yet: with this option in the
'Sales' settings activated, after a mysterious operation, the
'Users' form view was broken, because the selection field
'sel_groups_'for HR was still referenced in the view, but didn't
exist anymore (as HR groups weren't displayed as a selection field
anymore).
2017-05-11 15:14:07 +02:00
Pierre Masereel 36aebfb0f7 [FIX] base: ir_cron active_test in context
When the list view of 'ir_cron' is opened, the key 'active_test' is set
to False in the context to see all cron, even the archived ones.

The problem by passing 'active_test' in context is that it's propagated
to all actions launched from this view, for example whan you run
manually a cron, this key is passed to the cron action, which may leads
to call some actions on archived records which is not intended.

To fix this issue, we removed the key 'active_test' from context and
instead create a filter that show all crons set by default.
2017-05-11 11:25:11 +02:00
Aaron Bohy 067815f97d [FIX] purchase: sort order_line by sequence
Commit 9d4efc8 added the sequence field on model
'purchase.order.line' and added this field in the
tree view (with 'handle' widget), to allow to
re-order the purchase order lines in the purchase
order form view.

However, for this to actually work, the default order
on the model should be on sequence field, otherwise
the rows will still be ordered by id even if the user
changes their sequence value by re-ordering them.
2017-05-11 10:15:10 +02:00
Olivier Dony 7dc2ed94c3 [MERGE] Forward-port 8.0 up to e0014387 2017-05-11 09:05:32 +02:00
Olivier Dony e0014387c5 [FIX] report: always close pdf files after merge
If an exception during the merges (such as a file descriptor overrun), we
would otherwise depend on the next garbage collection to close the
files. But the next GC may never come.

For example if we ran out of OS file descriptors during merge, all future
requests will crash for the same reason, and the process will never recover
because the GC will never run.

Much easier to explicitly close the files all the time.
2017-05-11 09:01:12 +02:00
Olivier Dony 3978713996 [FIX] mass-mailing: unsubscribe even when archived
A record (Partner/Lead) that was archived can still be unsubscribed.
2017-05-11 09:00:44 +02:00
Aaron Bohy 4e8e546ed7 [FIX] web: DebugManager: open view: search and qweb views
can't be opened that way.

For now, only views that can be opened with a view manager are
supported, i.e. views like form, list, kanban, graph...

So this fix filters out the views of type qweb and search from the
list.
2017-05-11 08:52:24 +02:00
Juan Jose Scarafia 53dfb12394 [FIX] sale: deposit product with multiple company taxes
When creating a down payment for a SO, the applied taxes must
be the taxes of the company order. In this way, it's the same behavior
as a product with multiple taxes is added on a SO, only the order company
taxes are applied.

opw:741958
2017-05-10 16:38:48 +02:00
Goffin Simon 508e5ed9dd [FIX] stock: PO line with unit_price = 0 and cost_method = 'real' or 'average'
Let's consider the stockable product P with an internal category set
with cost_method = 'real' and 'average'. When creating a PO with P
and a unit_price=0, after delivering the product, the cost transfered to
the stock valuation account  was equal to the product cost multplied by
the product quantity instead of the unit_price set on the PO multiplied
by the quantity.

PS: The quant is created with the stock.move in _quant_create_from_move and the unit_price
set on the stock.move is taken with function get_price_unit.

opw:740865
2017-05-10 15:14:57 +02:00
Goffin Simon a1aed26d52 [FIX] account: Payment cancelled when cancelling statement line
Steps to reproduce:

1. Create a payment of 999 with bank
2. Create a statement line of 1000 on bank journal
3. Go to reconcile interface, choose the 999 line and make an adjustment of 1
4. The payment has been reconciled with the statement line, and one new journal
entry has been created for adjustment of "1"
5. Cancel statement line (you need to enable this functionality on bank journal)

Bug:The payment journal entry of 999 is deleted and payment remains confirmed

After the fix: The payment journal entry of 999 is UNRECONCILED and the payment
entry still exists.

opw:741619
2017-05-10 15:10:02 +02:00
zeroheure 7a9d57bd1f [FIX] doc: various typos and missing words
Closes #16903
2017-05-10 13:54:21 +02:00
zeroheure 94f37fa9c8 [CLA] signature for zeroheure (Xavier Brochard)
Done at #16903
2017-05-10 13:54:21 +02:00
Aaron Bohy e4e2735e87 [FIX] web_kanban: traceback when quickly drag&dropping
... records on models having a sequence field (e.g. project.task).

When a record is dragged and dropped, the following RPCs are done:
 - write (the new group id on the moved record)
 - read (the moved record)
 - resequence (the column where the record has been moved to)

To perform the resequence RPC, we need to retrieve the list of ids
of the records being in the column. We do that by iterating over
the cards in the column and reading the id in their data attribute.

However, a traceback might occur if the resequence was triggered
exactly when the user was holding a record over the column being
resequenced, because jquery-ui creates a placeholder for the record
being dragged and dropped, and this placeholder doesn't have the
required data attribute.
2017-05-10 11:10:38 +02:00
Nicolas Martinelli 4e205b7549 [FIX] l10n_fr_hr_payroll: identification number
The field `ssnid` is used on the payslip. However, it is not available
in any view, therefore it is impossible to fill it.

We replace by the `identification_id`, which is the field commonly used
for identification.

opw-742204
Closes #16798
2017-05-10 09:39:35 +02:00
Martin Trigaux ecf3555dee [FIX] pad: bad reference to old menu
The configuration has moved.

Fixes #16828
2017-05-09 17:20:58 +02:00
Nicolas Martinelli 3bdea9eb54 [FIX] account, account_tax_cash_basis: incorrect entry
- Activate multi-currency: EUR is main currency, USD is secondary
  currency
- USD currency rate is:
  1.2 on 2017-01-01
  1.5 on 2017-02-01
- Create a journal in USD (e.g. Cash)
- Create a tax with Cash Basis activated
- Create an invoice of 100 USD on 2017-01-15
- Register a payment of 100 USD on this invoice on 2017-02-15
- 4 journal entries are created:
  the payment itself
  the exchange rate difference
  the cash basis linked to the payment
  the cash basis linked to the exchange rate difference

The cash basis linked to the exchange rate difference should not be
created, it is not necessary. The amounts are already included in the
cash basis linked to the payment.

opw-728767
2017-05-09 15:13:29 +02:00
Aaron Bohy 7e98e32959 [FIX] web: traceback when quickly clicking on back button
Before this rev., there might be a crash when the user quickly
clicked several times on the browser's back button. This crash
occurred in the code saving/restoring the scroll position of the
current action.

When clicking fast on the back button, several actions are
executed but don't wait for each other (those actions are triggered
by the hashchange event), so when it is fast, and mainly when the
appswitcher is involved, it may happen that we try to get the
scroll position of an action that isn't loaded yet.

opw~741814
2017-05-09 14:54:15 +02:00
Goffin Simon e25563dca6 [FIX] stock: Impossible to fill the partner field in mobile
The field partner_id was hidden in mobile.

opw:728272
2017-05-09 13:29:24 +02:00
Josef Kaser a37bcaa197 [CLA] signature for JosefKaser
Closes #16887
2017-05-09 13:19:26 +02:00
Nicolas Martinelli c401219f97 [FIX] sale_service: new line in task description
Backport of ee6e2f5771

opw-741240
2017-05-09 12:04:52 +02:00
Ayoub Zahid 5da86716be [IMP] pos_restaurant : export the TableWidget and the FloorScreenWidget
These widgets were not exported, which means that there was no good way
to extend them.
2017-05-09 11:53:16 +02:00
Raphael Collet faba6daf7c [FIX] models: attribute pool on a model's class's may be falsy but not None
Modify the test to match `None` instead of any falsy value.  The latter
triggers a bug when installing a just-deinstalled module (point_of_sale).
2017-05-09 11:31:52 +02:00