Commit Graph
140217 Commits
Author SHA1 Message Date
Antoine Guenet 4a4dfb8a40 [FIX] website: properly save popup id
closes odoo/odoo#63408

X-original-commit: 8ae2fee6e71fc8bde56a87ee6b21268e5b1d9bc9
Signed-off-by: David Monjoie (dmo) <dmo@odoo.com>
2020-12-15 14:50:35 +00:00
Sébastien Theys bd7aeae189 [FIX] mail: prevent messaging menu from hiding home menu in mobile
task-2330277

closes odoo/odoo#63405

X-original-commit: db7f4437a644b93a5227f663fa19dda00817c13c
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-12-15 14:33:23 +00:00
Nicolas Martinelli 9e1849d183 [FIX] stock_account: SVL for consumable
Stock Valuation Layers are used for the MRP Cost Analysis feature.
Therefore, they are useful for consumable.

opw-2412668

closes odoo/odoo#63404

X-original-commit: c68e572f7c6f5109f1bfc40f9a44fc051f315112
Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
2020-12-15 14:28:17 +00:00
Antoine Guenet e56e573dd9 [FIX] website: move sidebar out of theme
closes odoo/odoo#63399

X-original-commit: 3b604dcd7a74ff0c61d906b9e3db16176086486d
Signed-off-by: David Monjoie (dmo) <dmo@odoo.com>
2020-12-15 13:58:46 +00:00
Yannick Vaucher cb8b6db391 [FIX] l10n_ch: check reference format with supplier QR-IBAN
closes odoo/odoo#63396

X-original-commit: 0d80dbf8a530209679a65a5c62808be1589c5cef
Signed-off-by: Josse Colpaert <jco@openerp.com>
2020-12-15 13:34:19 +00:00
Rémy Voet (ryv) 93ed6754a5 [FIX] stock: fix in_date default
coming from aa7061e9bcbf10fa9dbeac4402d704a260644b66

closes odoo/odoo#63387

Signed-off-by: Arnold Moyaux <amoyaux@users.noreply.github.com>
2020-12-15 12:17:22 +00:00
Julien Giannone 0286a46c96 [IMP] mail: Rename temporary to uploading in attachments related content
This commit renames "temporary" into "uploading" in attachments content

task-2337268

closes odoo/odoo#62526

Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-12-15 12:48:21 +00:00
Gorash 1beea8c4f1 [FIX] web_editor: use withIntangibles to find ZoneNode
ZoneNodes are now intangible containers

closes odoo/odoo#63384

X-original-commit: b6164a683cdca4ebdbcd802166323a83be7c6fa8
Signed-off-by: David Monjoie (dmo) <dmo@odoo.com>
2020-12-15 12:04:19 +00:00
David Monjoie 4b8cbea36c [IMP] web_editor: update Jabberwock library to commit a20492ea
Changelog:
[IMP] BundleOdooWebsiteEditor: share withIntangibles object for odoo
[FIX] *: use withIntangibles walker to find intangible ZoneNodes
[REF] Layout: ZoneNode is now intangible
[REF] Core: refactor allowEmpty
[FIX] DomHelpers, DomLayout: ensure mutation on correct vdom match
[IMP] core: move browse the arch methods into a walker object
[IMP] core: add support for not tangible container
[ADD] Small: handle <small>
[ADD] Test: some new space tests

X-original-commit: 19661cf0bb8b5ec1fb832c6a935f816da4730b20
2020-12-15 12:04:18 +00:00
Rémy Voet (ryv) 47069ef023 [FIX] stock: fix performance of forecasted when lot of locations
In the DB with a warehouse with a lot of stock location.
The forecasted widget return a traceback due to a memory error due
to the `_get_domain_locations_new`
(`_get_report_lines`->`read(['qty_available'])`->`_compute_quantities`
->`_compute_quantities_dict`->`_get_domain_locations`)
how returns a extremely long domain (one expression
by location in the context) to compute child location.
But the locations (put in the context) contain already
children locations because of
`('id', 'child_of', warehouse.view_location_id.id)`.

TO SOLVE
Instead of adding location in the context
(for fix 1aa6a323e04232d24175593718637c6dac295eb9),
add the warehouse in the context before
calling the `_get_report_lines` in the `_compute_forecast_information`.

closes #61666

closes odoo/odoo#63377

X-original-commit: f8861e6a63fcd85295affdea4b525af753234b14
Signed-off-by: William Henrotin <Whenrow@users.noreply.github.com>
Signed-off-by: Rémy Voet <ryv-odoo@users.noreply.github.com>
2020-12-15 10:57:04 +00:00
Sébastien Theys 4816c7bf03 [FIX] mail: fix display of chatter topbar in multiline
When the topbar goes to multiple line due to too much text content to fit its
currently available space (depending for example on length of text after
translations), it will look bad before this commit because the "send message"
and "log note" have a visual effect when they are open that requires them to be
directly above the composer to look good.

Part of task-2413814

closes odoo/odoo#63351

X-original-commit: 008def3f878b63cb49e0b5ef09a2cc22c784c0a9
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-12-14 16:55:17 +00:00
William Henrotin 2794fee110 [IMP] stock: adds price on stock move
This commit adds some value on incoming stock moves created by the
`populate` generation. This makes transfers more realistic and allow
creating stock valuation if accounting is installed as well.

closes odoo/odoo#63139

Signed-off-by: William Henrotin <Whenrow@users.noreply.github.com>
2020-12-10 10:57:47 +00:00
wan 1023479af2 [FIX] account: compute used on account.account
The field is exportable but always returns false

opw-2409324

closes odoo/odoo#63340

X-original-commit: f5c1ba1556c7db2a882740c6572127b28d064b57
Signed-off-by: Quentin De Paoli (qdp) <qdp@openerp.com>
Signed-off-by: wan <william-andre@users.noreply.github.com>
2020-12-14 14:34:06 +00:00
Rémy Voet (ryv) e50dbe096d [REF] stock: _gather use now the ORM
Remove the feature of `NULL` order of stock.quant and put
the `in_date` required because it is always set (and should).

task-2413099

closes odoo/odoo#63165

Related: odoo/upgrade#2019
Signed-off-by: Arnold Moyaux <amoyaux@users.noreply.github.com>
2020-12-11 10:38:16 +00:00
Thomas (det) d0efcd8538 [FIX] account: Negative fixed taxes on 100% discount
This commit fix a previous fix : https://github.com/odoo/odoo/pull/62244

opw-23789781~[FIX] account: Negative fixed taxes on 100% discount

closes odoo/odoo#63349

X-original-commit: da6b1667ab3566a88765f38f812df6a20c0abb03
Signed-off-by: tdetry (det-odoo) <tdetry@users.noreply.github.com>
Signed-off-by: Laurent Smet <smetl@users.noreply.github.com>
2020-12-14 16:13:00 +00:00
oco-odoo addd40a09f [FIX] account: reconciliation models: fix auto-reconciliation result when the reconciliation doesn't cover the whole invoice
Before that, using an auto-validate reconciliation model to match an invoice with a statement line with a lesser amount resulted in the creation of an payment entry with which the invoice got fully reconciled, and the difference amount was fowarded to the suspsense account in an additional line. This wasn't correct, as we expect the invoice to be only partially paid in that case (with an amount equal to the statement line's).

closes odoo/odoo#63303

X-original-commit: 7393aa3ebcea7b80762b729315f1c890bc677efb
Signed-off-by: Laurent Smet <smetl@users.noreply.github.com>
2020-12-14 10:16:22 +00:00
nie bbf860ae97 [FIX] sale_coupon: allow salespeople to use coupons
Steps:
- Have a demo user with "User: Own Documents Only" rights on Sales.
- Connect as admin
- Go to Sales > Products > Coupon Program
- Click one of the programs
- Generate a Coupon
- Reconnect as demo user
- Create a Sales Order
- Apply the generated coupon

Bug:
Access rights restriction on the coupon.

Sorry, you are not allowed to modify documents of type 'Sales Coupon' (sale.coupon). This operation is allowed for the groups:
	- Sales/Administrator - (Operation: write, User: 6)

Explanation:
When applying a coupon to the sales order, it tries to update the coupon's state and link it to the order. This needs to update the coupon record.
This commit also fixes confirming quotations and cancelling sales orders with coupons for salespeople.

opw:2409858

closes odoo/odoo#63239

X-original-commit: 873340ca98b9732132e056c1ca79c614c6255c1c
Signed-off-by: backspac <backspac@users.noreply.github.com>
Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
2020-12-11 12:38:12 +00:00
Achraf (abz) 4cc7c2bf00 [FIX] web: Load odoo script after other scripts
What are the steps to reproduce your issue ?

    1. Install website
    2. Connect on the website
    3. Choose Customize > HTML/CSS/JS Editor
    4. Choose "Web layout" (in "XML (HTML)" )
    5. Modify the HTML code by adding any <script> before the existing <script> (console.log whatever)
    6. Save
    7. Refresh

What is currently happening ?

    Traceback: TypeError: Cannot read property 'user_context' of undefined

How to fix the bug ?

    Target the odoo script by id in the xpath.

opw-2375490

closes odoo/odoo#63360

X-original-commit: 0b7ba571431d694dea7db3ba1da4c7e86d1013e5
Signed-off-by: Achraf <abz-odoo@users.noreply.github.com>
2020-12-15 07:04:04 +00:00
Achraf (abz) ae3a0dca96 [FIX] web: Select subfields without removing filter text on export
What are the steps to reproduce your issue ?

    1. Install "contacts"
    2. Select list view and select one or more contacts
    3. Select "Export" actions

What is currently happening ?

    When filtering fields with text, the system only displays the corresponding
    fields, but when clicking on sub-field the text written just before
    is deleted, which prevents us to return to all fields without filter

What are you expecting to happen ?

    Filter fields and select sub-field without deleting the filter text

opw-2411928

closes odoo/odoo#63339

X-original-commit: d947ea609b458ddbccabd661c22652d6c86a8167
Signed-off-by: Nicolas Lempereur (nle) <nle@odoo.com>
Signed-off-by: Achraf <abz-odoo@users.noreply.github.com>
2020-12-14 14:20:12 +00:00
Laurent Stukkens (LTU) 8c006367f9 [FIX] sale_timesheet: Non billable timesheets
Steps to reproduce the bug:

    - Create new project P with "Billable" unchecked
    - Create a new task T in P and log hours on the timesheet TS
    - Go to Timesheets > All Timesheets and filter for "Billable"

    Bug:

    TS was displayed even if P was not billable

opw:2394624

closes odoo/odoo#63348

X-original-commit: 78491ae8f6b7978444bf789d9ecdd71201afdebe
Signed-off-by: Simon Goffin (sig) <sig@openerp.com>
Co-authored-by: IT-Ideas <ltu@odoo.com>
2020-12-14 16:10:09 +00:00
Swapnesh Shah a18c8846f4 [FIX] account: readonly bank on posted payment
Field `partner_bank_id` should be readonly like other fields when Payment is not in `Draft` state.

closes odoo/odoo#63254

X-original-commit: 366bf51bb07aac6edca668635f49316b21bbb7be
Signed-off-by: Laurent Smet <smetl@users.noreply.github.com>
2020-12-11 14:58:25 +00:00
Swapnesh Shah 3e0d5e54f6 [FIX] addons: incorrect field help
Field's help doesn't match with field's Usage.

closes odoo/odoo#63249

X-original-commit: a48ba4672e90a7e9f3143de5ede0099a8e28d4cd
Signed-off-by: Laurent Smet <smetl@users.noreply.github.com>
2020-12-11 14:12:11 +00:00
Gorash 9a02c39977 [FIX] web_editor,website_mass_mailing: display the popup preview
closes odoo/odoo#63343

X-original-commit: a0470eed4e57f2df19d4306ab4e894b986971eef
Signed-off-by: David Monjoie (dmo) <dmo@odoo.com>
2020-12-14 14:40:22 +00:00
Gorash 720ea747f0 [FIX] web: dialog should not use field value footer items
Issue: display the footer tag from snippet in the modal and replace the
save and cancel button.
Steps to see: Email marketing -> mailing lists -> option icon of a
kanban -> Edit -> Website popups -> add a line

X-original-commit: 073699749401adadeec029de6534b4d205e41067
2020-12-14 14:40:21 +00:00
Aaron Bohy ae7aae7140 [FIX] web: update owl to last version
This commit updates owl from v1.0.13 to v1.2.0

See [1] and [2] for the list of changes.

[1] https://github.com/odoo/owl/commit/abb9d0b364da3a976915c092ab8d2696e5e524e5
[2] https://github.com/odoo/owl/commit/144b323d2bac7384114eb6b63670991f42943e67

closes odoo/odoo#63342

X-original-commit: cc77978c84d988a4b059abce97b46c72a9d3167d
Signed-off-by: Géry Debongnie (ged) <ged@openerp.com>
2020-12-14 14:35:50 +00:00
Raphael Reverdy 8983fd770e point_of_sale: fix typo in comment
closes odoo/odoo#63341

X-original-commit: 96a48d9d1af5abb7a28cc4648033bbf5ed72c129
Signed-off-by: pimodoo <pimodoo@users.noreply.github.com>
2020-12-14 14:34:33 +00:00
Thibault Delavallée 53021a0a8d [REF] mail: make activity_user_type not required on ir.actions.server model
PURPOSE

Required fields are annoying, notably when dealing with tests or xml-based data
to create in concurrent modules. For example ModuleA adds a required field on
Model. ModuleB that does not depend on ModuleA tries to create a data of
Model. Currently it crashes as field is required but default is not provided if
ModuleA is not loaded in registry.

SPECIFICATIONS

Field activity_user_type on ir.actions.server model in mail is currently
required. This field is however not really required as its use is limited
to actions linked to activities. Remove the hard required and put it only
in views.

If no value is given due to some bad configuration no responsible will be
set.

Followup of odoo/odoo@7309981426

LINKS

Task ID-2367655

closes odoo/odoo#61769

Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
2020-12-11 14:05:47 +00:00
Francois (fge) 815437df6b [FIX] Analytic: add groups to balance/debit/credit for 'account.analytic.account'
The balance/debit/credit compute fields need the user to be part of 'account.group_account_readonly' in order to be calculated. This will avoid having an access error when a user who is not in 'account.group_account_readonly' accesses a view of 'account.analytic.account'.

We need it for the task-2195019. This task adds the possibility to display a m2o('account.analytic.account') field that can access the tree view with the 'Search More...' button.

 Task-2195019

closes odoo/odoo#56554

Related: odoo/upgrade#1810
Related: odoo/enterprise#12680
Signed-off-by: Yannick Tivisse (yti) <yti@odoo.com>
2020-12-11 09:11:11 +00:00
Francois (fge) cc67988241 [IMP] web: added option model_field to the reference field.
model_field: name of the FieldMany2One('ir.model') containing the model of the records that can be selected. If its value is different from False, the select will not be displayed.

The purpose of adding this option is to allow the creation of a reference field having its model defined in another field (Many2One).

Task-2195019
2020-12-11 09:11:11 +00:00
Jacky (trj) 848c7eb464 [FIX]: point_of_sale: this is undefined in a non-arrow function
closes odoo/odoo#63331

X-original-commit: 3c374465b27a562c313dc65697741d7536b686ef
Signed-off-by: pimodoo <pimodoo@users.noreply.github.com>
2020-12-14 13:46:27 +00:00
qsm-odoo 89c1b47fbb [FIX] website: remove multi snippets zones in the header
task-2387598

closes odoo/odoo#63312

X-original-commit: 7c1b38ddbcd80b34bd0195b8900a52493eebea93
Signed-off-by: Quentin Smetz (qsm) <qsm@odoo.com>
2020-12-14 13:39:40 +00:00
qsm-odoo 859a11dc13 [IMP] base: remove useless XML condition
Introduced by mistake at [1].

[1]: https://github.com/odoo/odoo/commit/60d837044a028e578f804570361af128b40a12b6

closes odoo/odoo#61864

Signed-off-by: Quentin Smetz (qsm) <qsm@odoo.com>
2020-12-14 08:43:27 +00:00
Qiuyu (QHO) 8bfaedef29 [FIX] sale_project: fix usage of unavailable field "is_fsm"
Step to reproduce:
1. Install module "sale_project"
2. In Project > Configuration > Settings, turn on "Recurring Tasks"
3. Create a recurrent task
4. Set the "next_recurrence_date" for the recurrence ahead of today
5. Run scheduled action " Project: Create Recurring Tasks"
=> error: 'project.task' object has no attribute 'is_fsm'

Reason:
"is_fsm" is defined in the module "industry_fsm" and the module "sale_project" does not depend on that module.

opw-2412370

closes odoo/odoo#63325

X-original-commit: 36c3fb0e1d16f5cce21e3edcb01ee8a39a35caa8
Related: odoo/enterprise#15288
Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
2020-12-14 12:19:44 +00:00
Mathias Markl abea29fcd2 [FIX] mail: use correct badge color for messaging and activity menu
closes odoo/odoo#63323

X-original-commit: 8b419521124c47ac8eaf860fe70f6c0095fa8231
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-12-14 12:00:14 +00:00
Antoine Guenet 4e103b25f1 [FIX] web_editor: allow edition of branded nodes only
To be able to be saved, nodes have to provide information on where to
save them. This is done through the use of data attributes. Nodes that
do not have these data attributes or are not contained within nodes that
do, should therefore not be editable.
Recent changes in Jabberwock ensure that the nodes with these data
attributes are saved either as `OdooStructureNode` or as `OdooFieldNode`
so we can simply check for these.

closes odoo/odoo#63311

X-original-commit: adf778d4bb499c6beb6b54d8093ced4f81d2ac57
Signed-off-by: David Monjoie (dmo) <dmo@odoo.com>
2020-12-14 11:01:24 +00:00
Raphael Collet f2d4ba6890 [FIX] core: imports with one2many lines with dangling external ids
Consider a model M with a one2many field with comodel L, such that
deleting records in M cascade-deletes records in L.  Import a bunch of
records in M with one2many lines having specific external ids.  Delete
the imported records: the database cascade-deletes the corresponding
lines, but the lines' external ids are not deleted.  Now re-import the
same records: this crashes, because the import process finds the lines'
external ids and tries to update the deleted lines!

The fix consists in checking that an external id's record actually
exists.  The existence check was removed for performance reasons by
https://github.com/odoo/odoo/pull/26496.  We reintroduce it by resolving
the external id with a join on the model's name, which checks the
record's existence faster than an extra query.  This patch allows to
re-create the imported records' lines.

Another patch is necessary to actually update the external ids that
refer to deleted records, otherwise they remain dangling, and importing
the same records again duplicates the lines, because their external ids
do not match any existing record.

OPW 2409649

closes odoo/odoo#63318

X-original-commit: 0322febbea2cfd2d2fd7aedc42ef03632b17c967
Related: odoo/enterprise#15285
Signed-off-by: Raphael Collet (rco) <rco@openerp.com>
2020-12-14 11:29:21 +00:00
Christophe Monniez 0684219d59 [FIX] web: Allow accented uppercase characters in sign
Some characters are filtered out for an unknown reason, leading to
strange behavior when the font supports those characters.

closes odoo/odoo#61199

closes odoo/odoo#63290

X-original-commit: 429febb30bb3f5442cf310e836a5e0e6c4bc3e65
Signed-off-by: Olivier Dony (odo) <odo@openerp.com>
Signed-off-by: Christophe Monniez (moc) <moc@odoo.com>
2020-12-14 09:07:27 +00:00
Christophe Monniez 36cfe7cf75 [FIX] l10n_ch: use public domain ocrb otf file
X-original-commit: f882f20d8661549835b51244ae700446c27e172a
2020-12-14 09:06:45 +00:00
Christophe Monniez ba0105b1ce [FIX] website_hr_recruitment: replace images from odoo cdn
Some pictures are downloaded from odoo cdn, this is seen as a privacy
breach by Debian Lintian tests [1]

1: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=973605

X-original-commit: 5e7418fb52e83656963c6f0c2618e226bb95ef69
2020-12-14 09:06:05 +00:00
Christophe Monniez bc1a017bb8 [FIX] web: replace ttf fonts used to sign with better ones
Some ttf fonts used to sign are shipped does not come with a clear
licensing. Also some of them does not have accented characters.

With this commit, they are replaced with better ones and their
accompanying Open Font License.

Also, before this commit, any file in the font directory was read to be
rendered by the sign widget. With this commit, only `ttf`, `otf` and
'woff[2]' files are read, this allows to store the license file
alongside with the font.

X-original-commit: f4a9aec57b871806fb225bf91a941edb0fde5cbe
2020-12-14 09:05:09 +00:00
Christophe Monniez e9c16a0119 [FIX] survey: use better font for survey report
TheSecretRegular font license is not clear and moreover, does not
support accented characters.

With this commit, the font is replaced by one under the SIL Open Font
License.

X-original-commit: c455627fe0bf473c437e43831e88e907fc462845
2020-12-14 08:51:41 +00:00
Benjamin Vray 49e2a7136c [FIX] web_editor: fix css of active items
Before this commit, the css of the active items considered the items
to always be <a> elements. But a PR, coming after the one who
introduced this css code, changed the items generation system and now
the items are no longer <a> elements but <div> elements.

This caused a flickering effect when hovering over the last item in
the dropdown.

Part of #62971

task-2312878

closes odoo/odoo#63112

X-original-commit: b046fb31c34fd61086a492b2b8abf59a4e205dad
Signed-off-by: Jérémy Kersten (jke) <jke@openerp.com>
2020-12-09 17:49:12 +00:00
Benjamin Vray 5071390b8e [FIX] website: fix trigger of website_url_chosen
Before this commit, website_url_chosen was triggered when the dropdown
of the input closed instead of when an item was selected in the
list. This caused the following issue: you lost focus in the input
every time the list closed while typing.

After this commit, website_url_chosen is only triggered when an item
is selected in the results list of a urlpicker input.

Part of #62971

task-2312878

X-original-commit: a4cf182211de21970d5261713c5cbae3e609cfb1
2020-12-09 17:49:10 +00:00
Aurélien Warnon 08d978e173 [IMP] auth_signup: fix small design glitch in user form
This commit fixes a small design glitch in the user form by replacing a "X"
character to close an alert by an actual close icon.
Which looks better, the "X" really sticks out from the rest of the design.

Task n°2393814

closes odoo/odoo#62368

Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
2020-11-26 07:44:05 +00:00
jbw e723d0908f [IMP] account : defaults belgian company registry to vat number
In Belgium, the company registry and the VAT number (no country code) are identic.
The company registry field is not mandatory, but is needed to export the belgian Intrastat XML file.
Now:
- belgian company registry is set by default to vat number.
- missing company registry warning message redirects to company settings.

closes odoo/odoo#58047

Task: 2335088
Related: odoo/enterprise#15270
Signed-off-by: Laurent Smet <smetl@users.noreply.github.com>
2020-12-11 13:53:13 +00:00
Odoo's Mergebot d8c89be712 [FW][MERGE] mass_mailing(_sms / _sale), utm: provide various fixes
PURPOSE

Provide various fixes related to mass mailing (sms) and marketing automation
applications.

SPECIFICATIONS

See sub commits for more details.

[FIX] link_tracker: disable create and edit on clicks list view
[FIX] sale: prevent creating quotations from UTM campaign reporting buttons
[REF] mass_mailing: split mailing.trace.report sql to ease inheritance
[FIX] mass_mailing: let subscription_ids won over default_list_ids
[FIX] utm: fix naming of utm campaign card
[FIX] utm: show campaign tags by default by giving them a color

LINKS

Task ID-2300385
COM PR odoo/odoo#59872
ENT PR odoo/enterprise#14031

closes odoo/odoo#63271

Forward-port-of: odoo/odoo#59872
Related: odoo/enterprise#15278
Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
2020-12-14 09:26:52 +01:00
Thibault Delavallée 43e474e663 [FIX] utm: show campaign tags by default by giving them a color
Colorless tags are not displayed in kanban view of utm campaigns. When creating
tags on the fly currently they have no color and are therefore not displayed
once saving the campaign. This is not really intuitive for users.

We now set a default value to color field of tags. They are now displayed
by defauklt.

LINKS

Task ID-2300385
COM PR odoo/odoo#59872
ENT PR odoo/enterprise#14031

X-original-commit: 4f83be33d4cc53d50081050b3a9dee2892c562a4
2020-12-11 18:12:19 +00:00
Thibault Delavallée c0b7c78a8f [FIX] utm: fix naming of utm campaign card
LINKS

Task ID-2300385
COM PR odoo/odoo#59872
ENT PR odoo/enterprise#14031

X-original-commit: ef0a41893e5d9f04ad5db62198a1697d88d18599
2020-12-11 18:12:18 +00:00
Thibault Delavallée 576556d846 [FIX] mass_mailing: let subscription_ids won over default_list_ids
When creating new contact coming from a mailing list a default_list_ids
context key is set. It is used to give a default value to list belonging
and also to compute opt_out.

However subscription_ids field is displayed on contact form view, allowing
to add the contact to mailing lists. We may end up in situation where a list
is given twice to a contact: once through default_list_ids and once through
subscription_ids field manually completed by users.

In this commit we therefore merge those two values when creating contacts.

Also improve some wording while we are at it.

LINKS

Task ID-2300385
COM PR odoo/odoo#59872
ENT PR odoo/enterprise#14031

X-original-commit: 30bab79183b4b9f1a571d6ebd9bbc1f3c0692457
2020-12-11 18:12:17 +00:00
Thibault Delavallée 03b256214b [REF] mass_mailing: split mailing.trace.report sql to ease inheritance
Purpose is to ease inheritance of generated SQL report for mailing traces.
Notably we want to be able to filter out some mailings like those used
in marketing automation.

For that purpose we rework the hardcoded string to be a bit easier to
inherit. It is now build based on various sub methods returning sql items
(columns, group-by, ...) that are then aggregated to make the final SQL
query.

Functionally nothing changes. String manipulation is feasible as there is
no user input, meaning only valid terms will be accepted in final SQL.

LINKS

Task ID-2300385
COM PR odoo/odoo#59872
ENT PR odoo/enterprise#14031

X-original-commit: 15783a7dcfacc5904b3d3c823f175a4be5667129
2020-12-11 18:12:16 +00:00