Commit Graph
120038 Commits
Author SHA1 Message Date
Florent de Labarre 7d581fa66f [IMP] product: make product name clickable again
Feature present in V8 but removed at 0d68a012d0
Reintroduce the name of the template to be able to click on it

closes odoo/odoo#26821
2018-09-10 08:19:21 +00:00
Martin Trigaux d65969d572 [IMP] base: do not set a value for new translations
When generating new translations (e.g. 'edit translations' button on a view or
on a translatable field), the new generated translations have the source as
translation value.

Set the state to 'to_translate' as it is the default value

This change requires to add a fallback in web_editor when computing translated
dom

When a translation has no value, there is a fallback done in the different
translation method (e.g. xml_translate) but, as the node was manually created
in edit_translation_mapping, the fallback was missing

closes odoo/odoo#24391
2018-09-04 08:22:15 +00:00
Frederik Kramer 5a53e5b8e9 [CLA] signature for initos
Added Florian Kantelberg to our team of contributors

Closes #26638
2018-09-03 16:00:53 +02:00
Adrien Dieudonne 671155c376 [FIX] web: domain: quarter and last quarter
Before this commit, it was not possible to compare data if the time
range selected was quarter or last quarter.

String and integer was concatenated and the returned domain to evaluate
was wrong:
-3 3 - (context_today().month - 1) % 3
instead of:
-3 + 3 - (context_today().month - 1) % 3

Offset period param is always an integer but it's not the case of
boundary params. For 'quarter' and 'last quarter', it's an
expression to evaluate.

Anyway, this part of code will be refactored in a while but it's
important to make the test to avoid this king of issue.
2018-08-31 09:53:00 +02:00
XavierDo 334b57fbf6 [FIX] base: remove error_context message from test log
Some error_context messages were logged during tests
causing some useless noise in log.
Corresponding mute logger are now fixed/added.
2018-08-29 16:51:42 +02:00
Martin Trigaux 228225a1eb [IMP] hr_payroll: log the error during evaluation
If the number fails, it is probably an programming error in the safe_eval
Log the error for easier debug

Closes #26173
2018-08-28 14:48:58 +02:00
Martin Trigaux e398e1674a [IMP] base: log the name of the group that is failing
Closes #26173
2018-08-28 14:47:13 +02:00
Christophe Simonis 9947133361 [MERGE] forward port branch saas-11.5 up to 40430699df 2018-08-27 16:44:17 +02:00
Olivier Colson 40430699df [IMP] base: res.partner.bank : always display "account holder name" field
[IMP] account: partner form view: add a small link under the bank accounts list, allowing to open the tree view of the linked res.partner.bank, so that they can be fully edited

Before that, as the bank_ids field of res.partner is displayed as an editable tree view with only two columns, it was impossible to edit more specific fields of the bank accounts through the partner form view, such as acc_holder_name.
2018-08-27 15:30:19 +02:00
Christophe Simonis 8185970270 [FIX] web: correct test selector
Avoid selecting <td> that contains only a non-breaking space, leading to
wrong string comparison.
2018-08-27 14:35:17 +02:00
Christophe Simonis 1aca7be521 [FIX] point_of_sale: adapt view to renamed field 2018-08-27 14:35:17 +02:00
Thibault Delavallée 2b70abe484 [FIX] hr_holidays: allow to reinitialize module
Demo data prevent from reinitializing hr_holidays module. Indeed confirming
already confirmed leaves or allocations does not work. Demo data are
updated so that we can initialize hr_holidays several times.

This commit is linked to task ID 1876795 and PR #26423.
2018-08-27 14:25:33 +02:00
Thibault Delavallée 94d7c48185 [MOV] mail: move activity code
This recent code should not have been placed at this point.
2018-08-27 14:25:32 +02:00
Christophe Monniez 6dc790d2da [FIX] packaging: find admin uid when testing packages
Since 2f7c03d the admin user could have any uid and the uid 1 is
reserved for the system user. As the uid was hardcoded as 1 when testing
packages, the tests were failing.

With this commit, the uid of the admin user is searched before testing
the packages.

Closes #26603
2018-08-27 13:25:43 +02:00
Xavier Morel 0e307f8280 [FIX] doc building and warnings
doc:

* fix incorrect doc comments (documenting params which don't exist)
* correctly quote non-refs
* fix role label syntax (backticks must be escaped)
* add newline to fix warning

JS doc parser:

* fix handling of already-resolved objects
* fix handling of non-string properties (e.g. foo[0] at module toplevel)
* don't blow up if subject of .include call can't be resolved (just ignore)

Others:

* add translator support for ``problematic`` node (apparently used for
  some refs by recent Sphinx)
2018-08-27 12:55:21 +02:00
Josse Colpaert 8233e524c2 [FIX] iot: remove vim lines for mat 2018-08-27 11:51:40 +02:00
David Beguin 29f3b155c8 [REF] mass_mailing : modify demo data to use user Admin and not System
Fix Issue : With Mass Mailing Campaigns option activated only: if I go to Campaigns menu, I see 'Newsletter', I click on it and I am redirected to Mailings but don't see anything
because the filter 'My Mailings' is activated. In my opinion, there is an inconstancy somewhere. Is it only due to demo data?
Reason : Since System and Admin users refactoring, in one case, we see the data of system, because no filter, ine other case, we see only admin data because filter. As the mailing creator is system and not admin anymore.
2018-08-27 11:37:33 +02:00
Raphael Collet c36fc4a435 [FIX] doc: API of create 2018-08-27 11:18:33 +02:00
Christophe Simonis a7a30791de [MERGE] forward port branch saas-11.4 up to a5187cef10 2018-08-27 11:16:44 +02:00
Christophe Simonis a5187cef10 [MERGE] forward port branch saas-11.3 up to 9eef533e98 2018-08-27 10:37:42 +02:00
Christophe Simonis 9eef533e98 [MERGE] forward port branch 11.0 up to 3c9aa50449 2018-08-27 10:04:06 +02:00
Dipalee Bhalodia 3c9aa50449 [FIX] web: Graph: group by two fields on line chart
Before this commit, when we applied two or more groupbys in line
graph, the groups displayed on the X axis where shifted to the left,
and the first one wasn't displayed at all.

related task: #1848289
Closes: #25037
2018-08-27 08:21:40 +02:00
Moises Lopez - https://www.vauxoo.com/ 8bda4744b5 [FIX] requirements: improve python3.7 compatibility
Continuation from https://github.com/odoo/odoo/commit/27c017ad98b4c9ac5fdcd28c9fd84d3690993118
PyYAML was missed in the list
Closes #26587
2018-08-27 08:19:53 +02:00
Odoo Translation Bot f0ccdcb209 [I18N] Update translation terms from Transifex 2018-08-26 07:07:10 +02:00
Odoo Translation Bot 7526b64e8a [I18N] Update translation terms from Transifex 2018-08-26 04:10:40 +02:00
Odoo Translation Bot 0da9da4c44 [I18N] Update translation terms from Transifex 2018-08-26 02:17:37 +02:00
Jeremy Kersten 70d18276ec [IMP] website_payment: make it website specific
Allow to specity on a payment acquirer on if it is specific to one website or common to all.
2018-08-25 00:03:01 +02:00
Jeremy Kersten 2fb95cbfe5 [FIX] website_theme_install: uninstall current theme
Before this commit we only remove upstream but not itself.
2018-08-25 00:03:01 +02:00
Christophe Simonis 84b22fbab2 [FIX] point_of_sale: adapt view to field rename 2018-08-24 23:21:40 +02:00
Jeremy Kersten d8064099da [FIX] website: move action action_website_add_features
Since we use it in website_form which one is included before res_config_setting,
we need to declare it before.
2018-08-24 22:58:55 +02:00
Christophe Simonis 9f99eb9cb0 [MERGE] forward port branch saas-11.3 up to 3be46428cc 2018-08-24 22:12:20 +02:00
Christophe Simonis 3be46428cc [MERGE] forward port branch 11.0 up to 302ad0632c 2018-08-24 21:12:27 +02:00
Christophe Simonis 302ad0632c [FIX] base: set filename of javascript files of assets
A side effect of this bug was that other bugs like [1] wasn't caught by
the test suite.

[1] 2ed13c583e
2018-08-24 21:07:09 +02:00
Christophe Simonis 2ed13c583e [FIX] web: run all tests, not *only* this one 2018-08-24 21:05:50 +02:00
Romain Derierde-odoojke-odoo
65d8177b55 [REF] website*: revamp res_config_settings for website
Reorder res.config.settings for website
Make website_id in no create in most view
Revamp res.configsetting and view Form simplified

Transform old 'useless' stored field for presentation by computed field.
We can consider that if there are not key, feature is disable.

Co-authored-by: rde-odoo
Co-authored-by: jke-odoo
2018-08-24 20:01:02 +02:00
Christophe Simonis 2438aff60c [FIX] web: remove broken test
This test doesn't pass for unknown reason. We can safely remove it as it
behavior is already tested in test above (create a column in grouped on
m2o).
2018-08-24 19:36:29 +02:00
Christophe Simonis 4a6c3da238 [MERGE] forward port branch 11.0 up to 33981790de 2018-08-24 18:22:27 +02:00
Christophe Simonis 33981790de [MERGE] forward port branch saas-15 up to 4703d649dc 2018-08-24 18:03:00 +02:00
qsm-odoo 0e43512f86 [REF] web: add a scss function to swap two values in a list
Needed for themes migration.
2018-08-24 17:28:13 +02:00
len-odoo 818c18e55d [FIX] web: fix js data during consecutive sequence of resequence moves
Let x_1, ..., x_a, x_{a+1}, ..., x_{a+k}, x_{a+k+1}, ..., x_{n} be a record
sequence ordered by its 'sequence' field.
Suppose the reordering occurs between elements x_{a+k} and x_{a+1}.
If the subsequence x_1, ..., x_a is ordered, then it does not change.
This is also true of the subsequence x_{a+k+1}, ..., x_{n}.
The resquence function makes exactly this assumption, and only sends the server
the subsequence x_{a+1}, ..., x_{a+k} to update its 'sequence' field.
The update of x is done as follows: it is index + offset, where index is the
position of x in the subsequence, and offset is the 'sequence' of the first
element of the sequence (i.e. x_{a+1}.sequence).

It is easy to see that we need another hypothesis: that the sequence numbers are
unique. To show it, suppose that the sequence of x_1, ..., x_a is 0, then all
x_{a+1}, ..., x_{a+k} have sequence 1, and lastly all x_{a+k}, x_{a+k+1}, ...,
x_{n} have sequence 2. Then after the call to the server resequence, the
subsequence x_{a+1}, ..., x_{a+k} would have sequence numbers 1, 2,..., k.
Since the x_{a+k+1}, ..., x_{n} subsequence have 'sequence' value 1, this would
obviously be incorrect.

We fix the reordering of the data in the js frontend to be consistent with
what is done by the server.
Note that this is only pertinent for sequences of reordering moves, since
otherwise the data comes directly from the server.

opw 1867049
2018-08-24 17:25:54 +02:00
David Beguin 823958b69b [FIX] mass_mailing : remove duplicate from blacklist form view
As blacklist email is unique and email is the only attribute,
it's a non sens to allow to duplicate a record.
+ To avoid import error on blacklist if import a duplicated address,
the create return the existing record instead of creating a new one
(which is not possible due to unique email constraint).
The duplicate was only editing the existing record.
2018-08-24 17:11:47 +02:00
Pragya ladda 00ffef800a [FIX] project: fix sql injection in rating public controller 2018-08-24 17:08:22 +02:00
David Beguin bda6d60b8e [FIX] mass_mailing : renaming date GroupBy filters
Since modification on the group by filter on date field,
a forced value like group by date:month doesn't work anymore.
the user can choose the period but the forced value is not selected
when activating the group by.

Anyway, as it is more convenient to let the user choose the period
he wants to group by, the following groupby filters have been renamed
and the forced value have been removed.
In Mailings menu > Group By menu: rename 'Sent Month' by 'Sent Period'
In Mainling Lists > Group By menu: rename 'Creation Month' by 'Creation Period'
In Reporting > Group By menu: rename 'Scheduled Month' by 'Scheduled Perdiod'
2018-08-24 16:52:02 +02:00
qdp-odoo a86381831b [IMP] account: usability of journal
better names for journal sequences
2018-08-24 16:31:40 +02:00
Martin Trigaux bb86b51241 [FIX] do not translate useless t-attf-title
t-attf-title are translatable, do not use t-attf if a t-att is enough
2018-08-24 16:16:53 +02:00
qdp-odoo 97eff5b68b [FIX] account: usability in account.journal form view 2018-08-24 16:10:14 +02:00
Sébastien Theys 1addade28b [FIX] base: correctly detect when admin logs in
Since 2f7c03d9ca it's not possible to log in as user 1.

However, we reset the base url when the admin logs in, which is now user 2.

This fixes wkhtml2pdf was not able to load css file, and probably some other side effects.

PR: none
Task: 1879620
2018-08-24 15:55:46 +02:00
Odoo Translation Bot b8c6c24bbb [I18N] Update translation terms from Transifex 2018-08-24 15:54:45 +02:00
jem-odoo a7eb6832d1 [FIX] sale,pos_sale: use correct method to find conversion currency rate
Before this commit, if the currency of the company is not the reference
currency (rate = 1.0), then the conversion will be applied on the
sale.report with currency rate.
Example in the demo data:
- Company currency is USD
- Reference currency is EUR
- Currency rate of USD is 1.5289 (after 6th June)

Creating a SO with a total of 1000 USD in the USD company will
display 654.065 on the sale.report (= 1000 / 1.5289).

The applied computation simply takes the rate of the company currency,
which is not exactly correct. This does not cause many problem since,
usually, companies set their currency as the reference one
(with rate = 1).

Since e8f01b94ce, the rate is stored,
offering the possibility to make the code more robust and covering
the case detailled above.
The correct computation to apply is the one defined in `_convert`
method: 1000 * (1.5289 / 1.5289) (amount * (src rate / dst rate)).

This commit makes the correct convertion rate stored, instead of
the currency rate.
Doing that allow us to keep currency rates in the sale_timesheet
reporting test case.
2018-08-24 15:48:26 +02:00
Pragya Ladda 3c4c6b955d [FIX] project: use correct field name for rating
- Description of the issue
  Get internal server error while render page at website side

- Current behavior before commit:
  name field does not exist in rating.rating model
  because of this page should not be render and gives internal server error

- Desired behavior after commit:
  set proper field res_name which is already exist in rating.rating model

Closes #26568
2018-08-24 15:48:26 +02:00