Commit Graph

93956 Commits

Author SHA1 Message Date
rossasa 84dffd2dee [CLA] Rossa S.A. 2015-08-11 09:31:46 +02:00
Juan Jose Scarafia 97fdf7bc6d [IMP] sale: call product_id_change with company_id from wizard
Add company when calling product_id_change from _prepare_advance_invoice_vals on
sale.advance.payment.inv wizard.

This way we have same on change call as the one called from account_invoice
(e.g. on uos_id_change and onchange_account_id).

Closes #7939
2015-08-11 08:57:07 +02:00
Nicolas Lempereur 5844fa4a2a [FIX] survey: partner access to survey input
Allow the partner associated to a survey user input to read the data on
this output.

A survey can be created by a different user than the one
filling it, for example an appraisal interview request can be created by
another user than its interviewer. In this case if the interviewer is
not Survey / Manager an error would happen.

closes #7978
opw-644791
2015-08-10 16:57:27 +02:00
Goffin Simon a1328c9054 [FIX] website: escaped hash and url
To be valid for the redirection, the url and the hash
must be percent encoded.

opw:646837
2015-08-10 16:18:16 +02:00
Paulius Sladkevičius 3074835079 [CLA] HBEE
Closes #7825
2015-08-10 16:02:51 +02:00
Olivier LAURENT 755b09c8dc [FIX] hr_holidays: multiple fixes for search filters
- Wrong 'year' filter that implemented in fact an 'active types' filter
- Add a real 'year' filter
- 'my leaves' uses directly the 'user_id' related field
- 'My Department Leaves' filter that mixed ids from different models

Closes #7685
2015-08-10 12:33:25 +02:00
Nicolas Martinelli f2108e9edd [FIX] stock_account: merge the Reference/Description
When invoices are created from pickings, and that the user chooses to group by
partner, we make sure to keep the Reference/Description of all the source
documents. This is exactly what is done for the Source Document field.

opw-646903
2015-08-10 11:54:28 +02:00
Nicolas Martinelli fb3dd3f0f4 [FIX] stock: do not overwrite date_done when a backorder is created
When the user specifies a Date of Transfer ('date_done') and only transfers the
order partially, we must keep the value instead of overwriting with today's
date.

opw-646908
2015-08-10 11:28:44 +02:00
Richard deMeester 59fef0db21 [FIX] account: Add sequence to tax views
For account.tax.code and account.tax.code.template.

Sequences on account.tax.code are used for reporting.
However, it wasn't possible to edit them through the client.
Neither through the form view or the list view with a handle.

Fixes #1844
Closes #2656
2015-08-10 10:55:32 +02:00
Richard deMeester 9e6bfc9852 [CLA] Willow-IT corporate 2015-08-10 10:55:32 +02:00
Nicolas Lempereur 97dbe0db5f [FIX] web_kanban, web: kanban title direction
In internet explorer, writing direction can lead to wrong text direction
when combined with rotate.

A fix was already in place for IE up to version 9 but the issue also
happens in next versions which are more difficult to differentiate
(especially the current spartan/edge version).

So this fix use the same css property value for all browser and it doesn't
seem to have an impact (other than correcting the issue in IE).

closes #7955
opw-646430
2015-08-10 10:54:23 +02:00
Nicolas Martinelli c16885e6e2 [FIX] hr_timesheet_invoice: invoice line in partner lang
During refactoring at 55f9cbf, lost translation of invoice lines.
The invoice lines should be in the partner language instead of in the user lang.
The analytic lines are browsed explicitely in the user language.

Fixes #7774
Closes #7796
opw-646166
2015-08-10 08:25:29 +02:00
Odoo Translation Bot 4fcc756ef9 [I18N] Update translation terms from Transifex 2015-08-09 02:00:12 +02:00
Denis Ledoux 6ebbbdbae9 [FIX] stock: no uom rounding for DO availability & reservation
Instead of rounding the availability and reversation
of a delivery order line to the current UOM rounding,
we round to the generic Product of measure precision.

This is to have the real availability, even for a unit
of measure that normally cannot be split.

For instance, for a product for which the default
uom is 'Unit(s)', but for a specific delivery order
you deliver 1 'Dozens' of this product,
if you have 6 Units available, it will now display
that 0.500 'Dozens' is available and reserved, which
is what is actually happening.

Before, it displayed that 1 dozens was available, even
if only 6 units were available, on the 12 needed.

opw-643312
2015-08-07 15:25:07 +02:00
Jeremy Kersten 5a53d7d8b9 [FIX] website_sale: force refresh cart without cache when product removed
Firefox keep in cache the value of input.
So, when we delete a product line and call location.reload(), firefox
remember the value and state of this input and don't refresh with the new
value.

Adding True as param (forceReload) fixes the problem since it doesn't
use old value. Another way will be to add autocomplete="off".

ForcedReload (Optional): Is a Boolean flag, which, when it is true, causes the
page to always be reloaded from the server. If it is false or not specified,
the browser may reload the page from its cache.

This closes #7888 #3342 #7491
2015-08-07 15:11:54 +02:00
Olivier Dony 1388c0c108 [FIX] auth_openid: typo in rev. 54e0690 2015-08-07 14:15:41 +02:00
Olivier Dony fb55b2fa5e [FIX] res.users: more consistent handling of user private fields 2015-08-07 13:16:45 +02:00
Goffin Simon 6e2026dcda [FIX] account_analytic_analysis: timesheet_ca_invoiced
The function "_timesheet_ca_invoiced_calc" must not take lines from invoice in state "draft" or "cancel"
to compute timesheet_ca_invoiced. Inspired from function "_ca_invoiced_calc".

opw646602
2015-08-07 09:41:34 +02:00
Alexandre Fayolle 0b74fbed74 [FIX] ir.ui.view: pass context as a kwarg
otherwise when a method such as fields_get is overridden
using the new API, self.env.context is an empty directory,
and the translations cannot be performed correctly

Closes #7911
2015-08-07 02:52:19 +02:00
Christophe Simonis 6e59ea5db9 [FIX] tests: explain why we ignore phatomjs return code 2015-08-06 21:27:03 +02:00
Olivier Dony 54e06907c0 [FIX] base, auth_openid: wrong implementation of API by auth_openid
Authentication modules are supposed to override res_users.check_credentials()
in order to plug in their own mechanism, without actually modifying the
behavior of res_users.check(), res_users.authenticate() or
res_users._login().

auth_openid was incorrectly overriding check() instead of
check_credentials(), and unnecessarily accessing private
attributes of res_users. Fixing the implementation of auth_openid
to follow the API means we can completely make those attributes
private.
2015-08-06 19:19:17 +02:00
Christophe Simonis d8d9c7277e [FIX] phantomjs test: `phantom` variable is undefined inside page. 2015-08-06 17:20:37 +02:00
Josse Colpaert fa1842cb1d [FIX] stock: add procurement group for extra moves in picking
When pack operations generate extra moves, they should
take the same procurement group as those of the picking.

That way, when invoicing, they will be put on the same
invoice when there is a different invoice address on the
sale order.
2015-08-06 14:55:44 +02:00
Goffin Simon 5831aca505 [FIX] account_followup: contact with type = "invoice"
In "Payment Follow-up" when clicking on "Send Overdue Email"
if contact with type = "invoice"  and an email exists in the childs then
the email will be send to this contact else the normal behavior
is kept. When the overvue email is sent to a contact who is not the commercial
partner, a message is written in the chatter of the commercial partner to inform him
that the email has been sent to the "invoice" contact.

Closes #7870
opw:646149, 646575
2015-08-06 13:04:49 +02:00
Christophe Simonis d3e9bde738 [MERGE] forward port of branch saas-3 up to 2ab698b 2015-08-06 11:02:24 +02:00
Christophe Simonis 2ab698b4b1 [MERGE] forward port of branch 7.0 up to d2d81ca 2015-08-06 11:01:47 +02:00
Denis Ledoux 5cd1235b06 [FIX] purchase: Purchase invoices stat button group on partner form
The group applied on the purchase invoices stat button
was "Accountant", while "Invoicing & payments" is enough.

That way, the customer and purchase invoices stat buttons
have the same group applied, which makes sense.

opw-646748
2015-08-06 10:05:01 +02:00
Josse Colpaert d789ac7793 [FIX] stock: No rereserve visible if pack operations have been generated already fixes #7725 2015-08-06 09:48:18 +02:00
Olivier Dony d2d81ca9d8 [FIX] crm_partner_assign: search_geo_partner() should not pick deactivated partners 2015-08-06 09:20:24 +02:00
Goffin Simon 4f6eebf698 [FIX] account: tax sign
Division by 0 introduced by c525c6712a
2015-08-05 19:03:15 +02:00
Olivier Dony 5a41a3011a [FIX] ir_translation: undo asymmetric change of set_ids/get_ids
Partial revert of commit 39d17c2580
of PR #4617, that broke translations by introducing a different
set of criterions for filtering translations in `_set_ids()` and
`_get_ids()`, adding the `module` column in `_set_ids`.

This meant `_get_ids` would see translations that `_set_ids`
would never update: the translation interface did not appear to
be working anymore as soon as a translation entry existed with
no `module` value.
This is likely for manual entries and for entries bootstrapped
by `translate_fields()`, as they do not belong to any module.

The situation described in PR #4617 probably requires an extra
fix in the translation import system instead.
2015-08-05 16:29:16 +02:00
Goffin Simon c525c6712a [FIX] account: tax sign
When changing manually the the amount of tax in supplier invoice,
the tax sign of the tax must be kept.

opw:645691
2015-08-05 16:10:37 +02:00
Goffin Simon 4f91dc371c [FIX] website_sale: transaction error
When an error occurs during the transaction, the customer must receive
a message to inform him.

opw:645705
2015-08-05 15:19:45 +02:00
Raphael Collet 9699f510e4 [FIX] res_users: fix wrong index when parsing m2m commands 2015-08-05 14:35:42 +02:00
Denis Ledoux 91417516ae [FIX] web_calendar: use of selection fields as color filters
Using a selection field as color filters in a calendar view
had as effect to display the second letter of the state name
(The state name string, index 1)
as filter, instead of the full state name.

Besides, the colors of the filters did not match.

e.g., Sales > Sales > Sales Orders > Calendar View

opw-646486
2015-08-05 14:25:45 +02:00
Nicolas Martinelli f26b94fd7d [FIX] sale: filter taxes to keep the ones applying to the partner company
The problem originally arises in the frontend (eCommerce). In this case, it is
necessary to switch the user id to the SUPERUSER_ID in order to be allowed to
create a SO. This is done in the method sale_get_order in the module
website_sale. The consequence is that in a multicompany configuration, all taxes
of the product will be retrieved and applied in the frontend.

This fix filters the taxes retrieved to keep only the ones which apply to the
company of the partner, when the user id is SUPERUSER_ID.

opw-645258
opw-645915
2015-08-05 10:39:56 +02:00
Goffin Simon 663be8118f [FIX] account: tax sign in supplier invoice
When changing manually the the amount of tax in supplier invoice,
the tax sign of the tax must be kept.

opw:645691
2015-08-05 10:30:11 +02:00
Nicolas Martinelli 6f355623f0 [IMP] account_cancel: better handling of bank statement cancellation
Before this addition, cancelling a bank statement line was allowed even if the
bank statement was confirmed. However, when the bank statement is confirmed, it
is not possible to reconcile it afterwards. The consequence is that is was not
possible to reconcile this bank statement line without cancelling the whole
bank statement, and therefore forcing the user to do the complete
reconciliation process again.

With this addition, it is not possible to cancel a bank statement line if the
corresponding bank statement is confirmed. However, we offer the user the
possibility to reset the bank statement to draft through a new button. This new
button will only change the state of the bank statement, and will not modify
the state of the associated bank statement lines. The consequence is that the
user will be able to first set the bank statement to draft, then cancel the
bank statement lines he wants, and finally launch the reconciliation process
only on the lines which were cancelled.

opw-645903
2015-08-05 08:18:21 +02:00
Denis Ledoux 4d9cef5412 [FIX] google_calendar: Google doesn't allow wrong attendee emails
Odoo allows bad formatted emails for partners (and attendees).
Google doesn't.
Therefore, upon sync, if the email is wrong, we do
not send it to Google, to avoid Google being mad at us.

opw-646369
2015-08-04 18:54:58 +02:00
Denis Ledoux dd500b34b7 [FIX] google_calendar: log google bad requests reason
For debugging easyness.
For instance, when trying to sync an event with an attendee
with a wrong email, google failed, it's wasn't easy
to know why from the logs

It is now a bit easier.

opw-646369
2015-08-04 18:54:58 +02:00
Laetitia Gangloff a56319c390 [FIX] l10n_be_coda: Add date_invoice in demo data
To be sure than the invoice is integrated into the create fiscal year in demo data
Closes #7723
2015-08-04 17:18:38 +02:00
Ronak Baxi f8efc8a6c3 [FIX] payment: typo in track_visibility 2015-08-04 17:03:51 +02:00
Olivier Dony c96174da5f [MERGE] Forward-port of latest bugfixes from saas-3 up to 733cb3e76f 2015-08-04 15:22:49 +02:00
Denis Ledoux be98d30e79 [FIX] website_mail: advanced email features button snippet colors
Due to the fact the background color was hardcoded, it
wasn't possible to edit the colors of the button link
with the website editor wysiwyg

opw-646655
2015-08-04 15:18:18 +02:00
Olivier Dony 733cb3e76f [MERGE] Forward-port from 7.0 up to 8b3d69a0d7 2015-08-04 15:17:51 +02:00
Daniel Reis 8b3d69a0d7 [FIX] orm: performance of regex to check search `order` spec
This regex is used for a quick sanity check of
the order_spec in `search(order=<order_spec>)`.
Because it was build on the repetition of a
group ending with a series of optional patterns,
it could cause expensive backtracking when the
order spec did not actually match the regex
(the regex engine was trying all possible ways
to split the groups)

Forcing the repeating group to either end
with a comma or the end of the string prevents
prohibitive backtracking, while being even
more restrictive with regard to the syntax of
the order spec.

Closes #7755
2015-08-04 11:45:01 +02:00
Olivier Dony df002518ee [FIX] models: access error due to prefetch of indirectly referenced records
Complements commit af9393d505
in light of commit 62b0d99cfe,
to really have the correct effect.

When the prefetching failed due to the presence of
extra records in the cache (for which the access is denied),
the `read` operation was indeed retried. However the
result was not stored in the cache because the cache
already held a FailedValue (automatically added when the
prefetch failed).
2015-08-04 00:00:31 +02:00
Kiril Vangelovski d1d4688521 [FIX] analytic_user_function: hr.timesheet user onchange
Make additional argument to on_change_user_id in analytic_user_function
optional so that it wont break other modules.
2015-08-03 23:30:43 +02:00
Pedro M. Baeza 4fdd99588c [FIX] board: Put keyword argument on fields_view_get super call 2015-08-03 19:04:13 +02:00
Denis Ledoux 28f89db7b7 [FIX] stock: call to `create` of ir.sequence using args=
By default, the argument to pass the record values is `vals`,
not `values`.
Besides, it's not mandatory to provide the argument name
when it's not optional.

opw-646374
2015-08-03 18:04:57 +02:00