Commit Graph
104099 Commits
Author SHA1 Message Date
Simon Lejeune c524e6a254 [FIX] mail, report: class doesn't exist 2016-08-04 13:56:18 +02:00
Raphael Collet f259e74f8a [FIX] models: avoid recursive recompute() in method unlink()
This fixes an issue when a one2many computed field is recomputed: all computed
lines are created twice.

Consider a simple case, where the value of the field is a single line.  The
execution goes as follows:

 * invoke `recompute()`
    * write value `[(5,), (0, 0, vals)]` to the field
       * command `(5,)`: invoke `unlink()`
          * delete existing lines
          * invoke `recompute()`
             * write value `[(5,), (0, 0, vals)]` to the field
                * command `(5,)`: invoke `unlink()`
                   * no more existing lines, return
                * command `(0, 0, vals)`: invoke `create()`
                   * create a new line
             * mark the field as recomputed
       * command `(0, 0, vals)`: invoke `create()`
          * create a new line
    * mark the field as recomputed

There is a recursive call to `recompute()` inside `unlink()` that recomputes
the field while it is being written.  The fix simply prevents the recursive
call to `recompute()`, just like in methods `create()` and `write()`.
2016-08-04 11:55:00 +02:00
Olivier Dony 2870bdac1b [FIX] base: fix typo in conversion to new API
Introduced by 19561727a7

Fixes #13010
2016-08-04 10:26:07 +02:00
Christophe Simonis 53015e11a9 [MERGE] forward port branch saas-11 up to eb83ab2 2016-08-03 16:16:30 +02:00
Christophe Simonis eb83ab236f [MERGE] forward port branch saas-10 up to c956075 2016-08-03 15:29:57 +02:00
qsm-odoo e7e7b9beb0 [FIX] calendar: use correct bootstrap for invitations
Commit cfb8e8700a introduced a new template for meeting invitation page.
This template used bootstrap row's without any container which added
an ugly useless scrollbar at the bottom.

This commit uses valid bootstrap by simply removing the bootstrap
column layout (as there is no column but only centered content) and
using a simple container. Also, wraps the table in a .table-responsive
div so that the template is mobile-friendly.
Also fixes a typo in the template name.
2016-08-03 15:18:20 +02:00
Christophe Simonis c956075f5f [MERGE] forward port branch 9.0 up to cadb481 2016-08-03 15:07:24 +02:00
Simon Lejeune cadb481e20 [FIX] web: oe-view-manager-content display issue on ie11
This tries to be the smallest patch possible in order to avoid triggering
another display issues, so we only patch in the ms-hight-contrast section
which only applies on ie 10/11.

* The view-manager weren't displayed correctly; they didn't fit 100% of
  the screen width. This is fixed by explicitely seting the top, right,
  left, bottom attributes.

* The modals content were not displayed. This was due to 2 issues:
  * The rule that we patch in the first point set the position of
    `oe-view-manager` as absolute with an importang flag. Because of
    this important, the rule that reset the position of the
    `oe-view-manager` as static in modals.less was not honored. So
    we also patch this rule with an important flag inside the same
    file (so that all the fixes related to ie10/11 are in the same
    place).
  * Now that the modals are displayed, they do not fit 100% of the
    screen. It's because the `max-height: 100%` of `.modal-content`
    set in modals.less does not work in ie10/11, due to some bug
    with the flex displays. We set the height to 450px so that the
    modals are kind of correctly displayed on a 1366x768 screen
    (their content are still scrollable).

related to rev 1ccd87a037
closes #12929
2016-08-03 14:39:19 +02:00
Martin Trigaux 1dbd085743 [FIX] base: add missing part to 04b081ab
The 'amount 42 False' bug could also be reproduced with monetary widget.
2016-08-03 14:26:06 +02:00
Martin Trigaux 04b081ab37 [FIX] tools: do not display False as the currency
The field symbol is not required on a res.currency (it should and will be
changed in master). In case it is not set, do not display the currency in the
rapport.

Fixes #11346
2016-08-03 14:23:11 +02:00
Christophe Simonis 1cca5dea13 [MERGE] forward port branch saas-6 up to 20db912 2016-08-03 14:14:00 +02:00
Christophe Simonis 20db912f77 [MERGE] forward port branch 8.0 up to 1213de3 2016-08-03 14:09:30 +02:00
Christophe Simonis e9dc273060 [FIX] account: use correct attribute name
Wrongly forward-ported from saas~6.
opw-683311
2016-08-03 14:02:35 +02:00
Goffin Simon 1213de3ff5 [FIX] account: get_reconciliation_proposition
The bank statement reconciliation widget should look for the name
of the account.move if the ref is not found.

Backport of 11f8e8d

opw:683741
2016-08-03 13:38:44 +02:00
Jeremy Kersten 6ffc8b6261 [FIX] mail: controller avatar
FIX 1
-----
If the partner_id is not allowed, we return the placeholder.

Before this commit, the status as undefined and raise traceback.
UnboundLocalError: local variable 'status' referenced before assignment

FIX 2
-----
Fix the defaultheader to avoid to have:
   [[('Content-Type', 'image/png')], ('Content-Length', 43)]
And so, during parsing by werkzeug:
   ValueError: need more than 1 value to unpack
2016-08-03 13:02:31 +02:00
Christophe Simonis e3ab323e29 [FIX] base: add method allowing to call ir.server.action from crons 2016-08-03 12:39:20 +02:00
Richard Mathot d6d4ee0183 [REV] mrp: ce419242d0.
Too unstable fix (requires module upgrade).

Odoo 9 will keep its limit of 16 chars for mrp.bom's
`code` field
2016-08-03 11:08:50 +02:00
Richard deMeester e401a0bbbb [FIX] l10n_au, l10n_nz: account definitions
Change default definitions of funds clearing accounts to be assets
instead of liquidity.  Without this fix, reconciling to these
accounts during bank reconciliation is not generating the correct
journal, but is treating the process as a matching step (which it
is not).

For existing data, the accounts will need to be manually changed
from liquidity to current assets.

Closes #12996
2016-08-03 10:57:54 +02:00
qsm-odoo b907c9654f [FIX] web: restore missing groups since web refactoring
Commit 977db823c7 refactored the web module so that it gathers
common features of community and enterprise. The problem is that the
web.webclient_bootstrap template content lost its groups attributes.

This had the effect to make the client crash when accessing the
template for calendar invitation for example, as the template was
trying to render menus without menu data.
2016-08-03 10:41:12 +02:00
qsm-odoo a1e1e27ec8 [FIX] base: deletion of elements with attribute 'groups'
The test made to perform the deletion was not correctly migrated (see
https://github.com/odoo/odoo/commit/4ddc32313996c8cf6e8e6dffd2253aad8a144ccf#diff-3ad2ad1e9b88257a01a42d04ca1c0605L274): the user to
test must be the request user and not the qwebcontext user.

Thanks to @rco-odoo.
2016-08-03 10:41:12 +02:00
Olivier Dony 28bf5b1507 [I18N] payment_transfer: remove duplicate msgid
Introduced by forward-port 56ef99d050
of 8.0 rev 9a3b711989, while the term
had been previously added in 9.0.
2016-08-03 10:37:11 +02:00
Martin Trigaux 77fad18f8a [IMP] pos_cache: clarify button function
The button does not recompute the cache, it removes it which will be recomputed
when you launch a POS.
Hide the button when there is no cache (e.g. after installation).
2016-08-02 18:15:32 +02:00
Martin Trigaux 683471ae6f [FIX] pos_cache: repair broken xpath
The button `set_deprecated` no longer exists in the parent view.
Add the button next to the pos_cache field, at the end of the form.
2016-08-02 18:15:20 +02:00
Nicolas Martinelli f0d6138712 [FIX] im_odoo_support: link "Odoo Support"
The free livechat support doesn't exist anymore, therefore the link
"Odoo Support" does nothing.

opw-683486
2016-08-02 17:45:20 +02:00
qsm-odoo 30207886b7 [FIX] web: set correct web client height
The main content of the web client was set to grow/shrink so that it
fills the window (thanks to flex). However, to be able to shrink as
much as necessary the min-height needs to be forced to 0.

Without this fix, the web client was overflowing on the bottom of the
page, making impossible to access some menu items.
2016-08-02 16:35:14 +02:00
Nicolas Martinelli 65142a6a45 [FIX] website_sale_delivery: incorrect delivery price
The price of the delivery is incorrectly converted to the currency of
the user in the payment template (next to "Total").

Instead, it should be in the currency of the sales order.

opw-682306
2016-08-02 13:07:32 +02:00
Josse Colpaert 415773505c [FIX] mrp: added recursion constraint on BoM and improved error message on bom explosion 2016-08-02 11:28:51 +02:00
cod-odoo 21215dda37 [IMP] mrp: calendar on workcenter should be editable (there is no other menu to edit it otherwise) 2016-08-02 11:28:51 +02:00
Jeremy Kersten 944db71f21 [FIX] website_event: fix get_formated_date to avoid traceback
Avoid AttributeError: 'WebsiteEventController (extended by WebsiteEventSa' object has no attribute 'env'
Self is the controller and not the event.
2016-08-02 11:18:05 +02:00
Goffin Simon 353dfa22cb [FIX] sale: currency of amount in _get_invoice_price
The currency of amount is the currency of the company as defined in
account.analytic.line in module analytic and in module account.

opw:682378
2016-08-02 09:50:02 +02:00
Goffin Simon 9bb16c5ce1 [FIX] sale: refunds with _get_invoiced
Steps to reproduce:

-Create a first journal with a sequence
-Create a second journal with the same sequence
-Create a first SO, create the invoice for the fist journal and validate it
-Create a second SO, create the invoice for the second journal and validate it (the second invoice has the exact same name as the first one)
-Refund the second invoice (linked to the second SO)
-Go to the first SO => In the invoice button, you see 2 linked invoices (there is normally only one)

When searching the refunds corresponding to the invoices in function
_get_invoiced, the refund must take the journal of the invoice into account.

opw:684144
2016-08-02 09:36:35 +02:00
Goffin Simon ce419242d0 [FIX] mrp: useless size for field code on mrp.bom
The size of 16 characters for the field size on mrp.bom
is useless.

opw:684441
2016-08-02 09:16:25 +02:00
Christophe e106691ecb [CLA] Corporate signature for adiczion
Done at #11662
2016-08-01 17:49:41 +02:00
Christophe 980ff5ef10 [FIX] account: correct typo in css
Closes #11662
2016-08-01 17:49:41 +02:00
Josse Colpaert a7fb3e0fac [FIX] stock: inventory adjustment import should go to the next step too
This commit closes #12970
2016-08-01 17:36:24 +02:00
Ondřej Kuzník fafeb9fb83 [FIX] stock: avoid traceback when owner on picking and transfer more.
If you create a picking, with an owner, for qty x, and confirm it.
Next transfer x+1, it will raise a traceback 'cannot adapt res.partner'

Now, we pass the id, and not the browse record.

This commit closes #12978
2016-08-02 00:47:05 +11:00
JoshuaJan ecf9613ee5 [FIX] calendar: fix get_range_domain to match all events.
In case of event not all day, the timezone has an impact.
So if we search for date >= 2016-01-31 (00:00:00) and that the current timezone
 is +5, we should search for date >= 2016-01-30 19:00:00.

date_to_str will drop the time.

This commit closes #12878
This commit closes #12971
2016-08-01 23:49:09 +11:00
Jeremy Kersten 04d496a326 [FIX] website_forum: t-else qweb don't exist in saas-11
Wrong cherrypick from saas-12.
T-else option has been added in saas-12 only.
2016-08-01 13:57:12 +02:00
Nicolas Martinelli 2ee4afda2f [FIX] base: currency exchange rate date
When creating a new exchange rate, the default date is always the date
of the last server restart.

This is because `fields.Date.today() + ' 00:00:00'` is a fixed string
and the field is instanciated only at startup.

opw-684622
2016-08-01 13:29:16 +02:00
Nicolas Martinelli 35cd054cc8 [FIX] point_of_sale: total revenue in report
In some cases, the "Sales total(Revenue)" of the report is different
from the sum of the payments. To reproduce:
- create a product with a sale price of 1.00
- sell 3 times a quantity of 0.333 of this product
- the "Sales total(Revenue)" will be equal to 1.00, while the total of
  payments is 0.99.

This is because in the report, we first sum all order lines, then we
perform a global rounding.

The solution is to use the total amount calculated for each order, which
is rounded at the appropriate time.

opw-682782
2016-08-01 13:15:30 +02:00
Jeremy Kersten b94a53c61a [FIX] website_forum: when first post is spam, remove more karma.
Now, if the first post of a user is marked as spam, we remove 10x the karma.
(by default 10 * -100= -1000)

That will avoid to the moderators too many click to 'kill' a spammer.

This commit is related to #12803
2016-08-01 12:54:58 +02:00
Jeremy Kersten d1a160aa93 [FIX] website_forum: don't allow a user to submit 2 pending post simutaneously.
Before this commit, a user was able to post withtout limit new post.
They was added to the modarator queue.

Now we only support 1 post by user.

This commit is related to #12803
2016-08-01 12:54:58 +02:00
Mohammed Shekha dcd03d778e [FIX] website: repair customize menu for groups restricted views
The id of the user to browse has been omitted
during the below forward port revision:
95a2f8b121

We can see in the original revision that it was
correctly there:
cbb51610ee

This had as wrong side-effect to not display
views with a group set in the customize menu,
as it checked the groups of a null user browse
recordset.

opw-684326
Closes #12895
2016-08-01 11:23:52 +02:00
Christophe Simonis ac8b882d8f [FIX] website_sale: use new t-options python-formated attribute 2016-08-01 11:16:40 +02:00
Denis Ledoux 79ff7a5233 [FIX] project: remove module_sale_service from project config
The module `sale_service` has been merged in the module
`sale_timesheet` in the revision
234f88a6af

We only remove the field from the view due
to the fact saas-11 is a stable. The field
has already been removed from the model
in newest releases (`saas-12`)

opw-682816
2016-08-01 10:54:46 +02:00
Yajo 4b4bc68815 [FIX] web_editor: broken JS/jQuery helper
A jQuery element is not callable.
2016-08-01 10:02:45 +02:00
Cedric Snauwaert 9b5fed4976 [FIX] account: fix full reconciliation.
when doing a full reconciliation in the same currency, full_reconcile_id was not set on account_partial_reconcile object
resulting in problem when trying to unreconcile entries.

Also add matching number to the exchanged rate entries created in case of full reconciliation.
2016-08-01 10:01:12 +02:00
Jeremy Kersten 7f86272b94 [FIX] hr_timesheet_sheet: set timesheet widget dirty when editing
Without this commit, there was no warning displayed when clicking
on a button that would make the user lose all he entered in the
widget, which is insonsistent with other odoo widgets.

opw-684276
2016-08-01 09:04:58 +02:00
Odoo Translation Bot 337049baa8 [I18N] Update translation terms from Transifex 2016-07-31 03:07:08 +02:00
Odoo Translation Bot b7eb3c6bd0 [I18N] Update translation terms from Transifex 2016-07-31 00:38:56 +02:00