Commit Graph
103429 Commits
Author SHA1 Message Date
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
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
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
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
Christophe Simonis 65a67cedb4 [MERGE] forward port branch saas-10 up to 09e9434 2016-07-30 13:02:20 +02:00
Christophe Simonis 09e943429e [FIX] website_sale: unable to edit product price
The less refactoring in 5b65befcd8
wrongly removed the css directive that hide some elements in edit mode.

As a side effect, the product price was appearing twice when editing a
product. Commit 44ca4317a9 tempt to fix
it but removed the edition part.

opw:683343
2016-07-30 12:43:21 +02:00
Josse Colpaert ec841b7fa1 [FIX] inventory: when creating partial inventory, it should also go to partial state 2016-07-29 16:04:48 +02:00
Josse Colpaert f295bdccad [FIX] stock: fix traceback of unlink of picking 2016-07-29 16:04:33 +02:00
Simon Lejeune 3aea814e3c [FIX] web: inputs of an editable list inside a modal are broken
This is triggered by the interaction of the `-webkit-overflow-scrolling` [1]
rule that is set on the top of the modal (div.modal) and the fact that inputs
elements are displayed over the table with an absolute positioning (that's
how the editable lists work).

Indeed, the `-webkit-overflow-scrolling` rule creates a new stacking context [2]
and is inherited by all the children of the modal (and so on), resulting in this
situation:

                                                       .modal
                                                 /             \
                                         ... (new S.T.)    ... (new S.T.)
                                           /                      \
                                  ... (new S.T.)              ... (new S.T.)
                                 /
                       .o_list_editable (new S.T.)
                      /                            \
           .o_form_view (new S.T.)         .table-responsive (new S.T.)
      /            |           \                           \
input            input           input                      table (new S.T.)

This explain why the input are not visible: actually, they are displayed
behind the table. In this case the  document order is used to know which
div should be in front of another div[3], that's what rev odoo/enterprise@304b790004 [4]
tried to change. However, this was not enough for iOS 9.3 / iphone 6
because, for some unknown reason (but probably due to the fixed position
of the modal that seems kind of broken on safari mobile), the inputs where
still displayed under. Note that the fix was ok for ios 9.3 / ipad air 2.

The fact that `-webkit-overflow-scrolling` creates a new stacking context
at every node seems odd, unfortunately removing the rule result in a non
scrolling div at all.

The fix is to explicitely tell the input to appear over the .table-responsive.
This is done by sharing the same stacking context under the common parent
between the form and the list view: .o_list_editable. For that, we prevent
the creation of a new stacking context under .o_form_view by reseting the
`-webkit-overflow-scrolling` to "auto" and we set a dummy z-index to the
inputs element. As they are the only one to have a z-index in this dom
range, they will always appear on top of the table.

With this solution, the `-webkit-overflow-scrolling` is still inherited
correctly and the div is still scrollbale.

This fix unveils another issue with the editable list: if the screen is
too small resulting in an horizontal scrollbar, the table in itself is
scrollable but the inputs aren't.

opw 683169

cherry-pick of odoo/enterprise@35f285e50a

[1] this rule allow a smooth scrolling of the content of a div on iOS
https://developer.mozilla.org/en-US/docs/Web/CSS/-webkit-overflow-scrolling
[2] https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_Positioning/Understanding_z_index/The_stacking_context
[3] https://www.w3.org/TR/CSS2/zindex.html
[4] This commit has been reverted with odoo/enterpise@287cff02c7
and never forward-ported to saas-10.
2016-07-29 14:44:26 +02:00
Christophe Simonis 685fa525e7 [FIX] product_visible_discount: use define variable
Once again, a failed forward-port...
2016-07-29 14:01:53 +02:00
Christophe Simonis 529f88dbd3 [MERGE] forward port branch saas-10 up to 4ca04fd 2016-07-29 13:01:29 +02:00
Christophe Simonis 4ca04fdd41 [MERGE] forward port branch 9.0 up to 6d43f28 2016-07-29 12:55:36 +02:00
Christophe Simonis 6d43f280c9 [FIX] stock: do not quote static domain 2016-07-29 12:47:08 +02:00
Christophe Simonis 4c7d9cb6a6 [MERGE] forward port branch 9.0 up to e0c0a5f 2016-07-29 12:04:24 +02:00
Christophe Simonis e0c0a5fb1b [MERGE] forward port branch saas-6 up to 50e1486 2016-07-29 11:56:15 +02:00
Christophe Simonis 50e148648d [MERGE] forward port branch 8.0 up to 9fe930e 2016-07-29 11:45:47 +02:00
Erwin van der Ploeg 9b64a2f869 [IMP] l10n_nl: correct module description
Remove outdated information
Closes #12274
2016-07-29 11:22:39 +02:00
aetna-softwares 9fe930e6ba [FIX] survey: prevent saving issues of answers to multiple_choice questions, when id starts with id of other questions
Fixes #11873 #12215
2016-07-29 11:13:08 +02:00
mva ec5410581e [FIX] doc: change memory hard and soft
The hardware limit should be higher than the soft, otherwise it is killed before
reaching the limit.

Closes #12957
2016-07-29 10:43:20 +02:00
Martin Trigaux b146335484 [FIX] doc: correct typo in deployment page 2016-07-29 10:11:59 +02:00
Nicolas Martinelli 37ae7a7334 [FIX] account: account filter
Searching on the name of an account can lead to a traceback ("This
domain is syntactically not correct"). This is for example the case in
Accounting -> Configuration -> Taxes -> Custom filter on "Tax account
doesn't contain 'xxx'".

The reason is that the negation of the domain is not correct.

opw-684525
2016-07-28 17:58:00 +02:00
Olivier Dony c372708b01 [FIX] website_google_map: do not allow invalid partner prefix
The only supported prefixes are the following:
 - /partners/ when website_partner and/or website_crm_partner_assign are installed
 - /customers/ when website_customer is installed
2016-07-28 17:27:54 +02:00
Nicolas Martinelli de2d7d9acd [FIX] sale_stock: incorrect value in error message
The error message has an hardcoded precision of 2 digits, which is not
enough of the product precision is higher than 2. Since this error
message is often used, we should avoid its modification to prevent
breaking the translations.

opw-683639
Fixes #12629
2016-07-28 15:10:59 +02:00
Goffin Simon 8bc590a7b9 [FIX] stock: default domain on a many2many field
The default domain on a many2many field must be list and not a string.

opw:684474
2016-07-28 14:46:28 +02:00
Nicolas Martinelli f57f55966d [FIX] sale_stock: UoM in warning
The warning message of the product packaging change does not take into
account the appropriate UoM.

opw-12827
Fixes #12827
2016-07-28 13:36:30 +02:00
Christophe Simonis a2d9b72112 [MERGE] forward port branch 9.0 up to 4af0054 2016-07-28 13:24:50 +02:00
Christophe Simonis 4af005456f [FIX] payment_paypal: adapt test following commit dbf44a9680
Note: the same test exits in 8.0, but is not executed (package not imported)
2016-07-28 13:01:57 +02:00
Christophe Simonis 73cc3611b4 [MERGE] forward port branch 9.0 up to d35f425 2016-07-28 12:31:29 +02:00
Christophe Simonis d35f4250cc [MERGE] forward port branch saas-6 up to 7079a0b 2016-07-28 12:22:03 +02:00
Goffin Simon 6a711060ae [FIX] account: fiscal position in compute_refund
When choosing to refund an invoice with refund method="create refund, reconcile and create a new draft invoice"
, the fiscal position of the refund invoice must be kept in the new draft invoice.

opw:683311
2016-07-28 11:59:11 +02:00
David Monjoie 2cfa7b5626 [FIX] l10n_fr_fec: exclude initial balance at 0
This is the proper fix for 3180aec56a
and is the continuation of f7e4241a91.

A sum without groupby will return NULL so we forced it back to 0,00.
Then, it can happen that you have no move before the start date, so you
don't have any opening "OUV" row.

opw-682365
2016-07-27 19:41:50 +02:00
Christophe Simonis 7079a0be30 [MERGE] forward port branch 8.0 up to 8ceb943 2016-07-27 18:13:51 +02:00
Guewen Baconnier 6cc7c03212 [FIX] tools: use without_demo specified in configuration file
When using the -i command line option without the --without-demo=all command
line option, demo data was installed even if the configuration file has
a line without_demo = all

This change is safe as just above, opt.without_demo is copied into
self.options['without_demo'] so we are sure to consider the 2 options.

Closes #12730
Fixes #9293
2016-07-27 17:39:09 +02:00
Gourav Chawla 8ceb9438d1 [CLA] signature for user gouravchawla
Closes #12738
2016-07-27 17:26:38 +02:00
Simon Lejeune ee228cd837 [FIX] ir_qweb: issue if multiple IRA are created for a bundle
If multiple transactions start knowing that they should generate the
bundle, each one will run lessc and create a ir.attachment record
referencing the path on the filestore. While the file is not duplicated
on the filestore (as it is stored according to its hash), the final
result of `t-call-assets` will source multiple time the same file.

This patch fixes this issue by adapting get_attachment to group its
results by file name and only select the one with the max id (latest
generated).

We also remove the `inc` argument as it is not used.
2016-07-27 16:56:42 +02:00
Goffin Simon 138cf9bbce [FIX] sale_timesheet: _get_timesheet_cost
A user must be set to get the right timesheet_cost

opw:684036
2016-07-27 15:54:51 +02:00
Nicolas Martinelli 2cf80e0075 [IMP] purchase: set scheduled date of lines
Commit 4326d864 removes the inverse function of the `date_planned`. This
is necessary to avoid unexpected changes of the scheduled dates.
However, this removes the ability to set the scheduled dates of all
lines at once.

This commit introduces back the functionality thanks to a new button.
This way, the functionality remains without unexpected behavior.
2016-07-27 15:19:28 +02:00
Cedric Snauwaert 0da7184b67 [FIX] account: fix dependencies for amount_residual computation
when we unreconcile an entry, amount_residual is not recompute due to missing dependencies. Resulting in problem when trying to reconcile the move later on as amount_residual is zero and aml is considered already reconciled
2016-07-27 15:07:29 +02:00
David Monjoie 3180aec56a [FIX] l10n_fr_frc: fix null values in fec
The previous fix didn't take into account the fact that some parts
of the file can truly be empty. Because of this, you end up with
'0,00' in some date fields.
2016-07-27 14:44:24 +02:00
qsm-odoo 931e5567b6 [FIX] base: let the debug value be 'assets'
This commit fixes a bug introduced by the forward port 1fca5e01cb.

In saas-11, the commit f970906c9e introduced the 'assets' value
for the debug mode, to active the assets splitting. To do that, the
commit had to stop the conversion to a boolean of the debug value
(initially introduced by commit 522dec2fcd).

The problem is that the forward port mentionned above did not resolve
correctly the conflict for the commit 3455daf948, reintroducing the
boolean conversion.
2016-07-27 14:26:57 +02:00
Martin Trigaux 5f97e9faad [FIX] account: avoid inconsistent menu translation
The menu `account.menu_finance` is named "Invoicing" in account and renamed
"Accounting" in account_accountant.
The renaming introduce 2 translations that are both in account.pot (as the
translation of an object is based on the module that created it, deduced from
the external id, here account).
When installing account module, two translations for the same item are loaded.

Remove one of the translation (keeping "Accounting" as it is probably what most
people using accounting will have) to avoid uncertainty from a duplicated
translation.

A correct fix would be to avoid renaming the module (will be done in master).

opw-683314
2016-07-27 09:36:30 +02:00
Martin Trigaux cc18c8d0c8 [FIX] doc: correct typo 2016-07-26 20:36:31 +02:00
Martin Trigaux f9a1308e8a [FIX] doc: correct megabyte abreviation
Mo is French.
Courtesy of Erwin van der Ploeg.
2016-07-26 20:34:33 +02:00
Nicolas Martinelli 6742d1fafe [FIX] account: wrong initial balance in GL
When the General Ledger is printed, the initial balance is zero if the
filtering is done by period.

It appears that in this case, the method `_query_get` selects all the
periods before the selected period range thanks to:
`build_ctx_periods(cr, uid, first_period, context['period_from'])`
On the other hand, `_query_get` builds the query as:
`date_start <= %(date_start)s AND id NOT IN %(period_ids)s`

That doesn't make sense since we first choose the periods before the
selected period range, then we exclude them. What the method
`_query_get` is doing seems wrong, but since this method is used in many
reports, it is safer to only fix the GL report directly.

Another solution could be
https://gist.github.com/nim-odoo/453176d9ae820615e69f9a809a3780cc

opw-681601
2016-07-26 18:12:14 +02:00
David Monjoie 92c716229a [FIX] l10n_fr_fec: fix null values in aml balance
The aml.balance in the sql request is null, so the string encoding
fails with an error. This is a quick fix because the customer needs
this report for tomorrow, but a proper fix is required !
2016-07-26 17:57:21 +02:00
Thomas Binsfeld a3fe9eb440 [FIX] account: allow to override abstract widgets
Improve possibility of inheritance.
Return abstract widgets in order to override these.

It closes #12870
2016-07-26 16:17:51 +02:00
Jeremy Kersten 49d789cf99 [FIX] base: widget duration - allow to round to avoid seconds
Allow to round the duration to a minimal unit.

Eg:
   0.33 hour = 19.8 min = "19 minutes 48 seconds"
   if option {'widget'='duration', 'unit'='hour', 'round'='minute'}
   0.33 hour => 20 min = "20 minutes"
2016-07-26 16:17:50 +02:00
Goffin Simon a7b68c288b [FIX] delivery: Many fields still available for editing after delivery is done
carrier_id, carrier_tracking_ref and number_of_packages must be in readonly if
the stock.picking is in done state.

opw:683810
2016-07-26 16:08:03 +02:00
Goffin Simon dbf44a9680 [FIX] payment_paypal: country and state code required for paypal
As explained in:

-https://developer.paypal.com/docs/classic/api/country_codes/
-https://developer.paypal.com/docs/classic/api/state_codes/

The country and state code is required to create a paypal transaction.

But in some cases, the complete name of the state/province is required.
For example, for India, sometimes the complete name of the state is required.
Check the list of those state/province in the documentation.

Hint: form_preprocess_values

opw:682872
2016-07-26 15:59:55 +02:00