Commit Graph
134375 Commits
Author SHA1 Message Date
jbm-odoo 5889dc3cc4 [FIX] fleet,hr_fleet: Fix user rights
Task ID: 2170625

closes odoo/odoo#47079

X-original-commit: ee13013808460c5100bec03ed415fc184ff031e5
Signed-off-by: Yannick Tivisse (yti) <yti@odoo.com>
2020-03-06 12:43:06 +00:00
Arnold Moyaux 1d1cdf3929 [FIX] stock_landed_costs: allow to create landed cost
The checkbox on product in order to define landed cost is under the
invoice tab. However it requires complex manipulation to be displayed.
Move the option under the purchase tab

closes odoo/odoo#47073

X-original-commit: f7567f10dd0cc936336066e7835c908fde163dc0
Signed-off-by: Simon Lejeune (sle) <sle@openerp.com>
2020-03-06 11:15:37 +00:00
Arnold Moyaux 871a925045 [FIX] stock_landed_cost: ensure landed cost product
On the landed cost form, the additional costs would allow
product that are not landed cost. It could lead to a misunderstanding
where the user thinks he should enter the picking's products.

X-original-commit: a0b8ebe9136890a950088c5a4195bfab576191a3
2020-03-06 11:15:36 +00:00
Hemali Bhimani 506c9764e0 [IMP] sale: opportunity to print a quote and change the status
currently button "print" does not exist so it is not possible to
reach the status "quotation sent" without sending it by email.
After this commit without "send by email" user can send
and/or print a quote and change the status of the quote to
"quotation sent" and also add the customer as follower

closes odoo/odoo#47008

Task: 2168504
Closes: #46813
X-original-commit: 0088e967afe4b8884e5bd482de855297d956207a
Signed-off-by: Yannick Tivisse (yti) <yti@odoo.com>
2020-03-05 15:23:02 +00:00
Odoo's Mergebot e6c406d8d6 [MERGE] survey: support conditional questions
Purpose
=======

This merge adds a new survey feature : conditional questions. A conditional
question is a question that will be displayed to the user only if the user
selects a specific answer from another previous question i.e. question B is
displayed if user answers 'Choice 1' at question A.

Specifications
==============

There are 2 different ways to handle conditional questions.

  * For one_page and page_per_section survey layout: conditional management is
    done mainly at frontend side. The idea is to be able to load all the
    questions of the section, hide the conditional questions and display them
    dynamically on answer check.
    E.g.: If user checks answer 'Choice 1' from question A, question B is
    displayed immediately.
    If question B is on section 2 and survey layout is set to section per page,
    question B will be displayed immediately on next page template load.
    Indeed, on each page load, server sends all the selected answers and the
    template displays the conditional question depending on the already
    elected answers;

  * For page_per_question survey layout: conditional management is done mainly
    at backend side. The idea is to find on each page the next question to
    display depending on:

    * navigation direction (next - previous)
    * already selected answers and conditional question dependance
    * survey questions sequence

    E.g.: If questions survey are Question A, B and C and question B must be
    displayed if answer 'Choice 1' from question A is selected.
    At Question A, if user select 'Choice 1' and goes next: display B
    At Question B, no matter the answer, if user goes next: display C
    At Question B, no matter the answer, if user goes prev: display A
    At Question A, if user select 'Choice 2' and goes next: display C
    At Question C, no matter the answer, if user goes next: End survey
    At Question C, if user has selected 'Choice 1' at question A and goes prev -> B
    At Question C, if user has selected 'Choice 2' at question A and goes prev -> A

In both modes answers on questions that should not have been displayed to the
user depending on the previous selected answers are cleared.
E.g. : User checks 'choice 1' from question A, then complete question B.
Then user unchecks 'choice 1' -> answer on question B is cleared.

Remove questions that should not have been displayed to the user (depending of
the made choices) in scoring computation and print layout.
--> Inactives questions are removed from predefined questions when user finish
the survey.
E.g.: If the user selects first path on 2, he will never see second path.
If the user answers correctly to all question of first path, he should have
100%, instead of 50%, even if questions of path 2 are mandatory and scored.

Specifications: sessions
========================

This merge makes the live session mode work with conditional question.
The point is to determine which question to display next. As multiple users
are 'voting' for answers of the same questions, the way to determine the
next question to display is to keep only the most voted answer.

The most voted answer will be the only one to trigger the linked conditional
question.

E.g.: considering
- following survey configuration:
    - Q1: A1, A2
    - Q2: triggered by Q1-A1
    - Q3: triggered by Q1-A2
- following user inputs:
    - U1: A1 selected
    - U2: A2 selected
    - U3: A2 selected
The next question displayed to ALL users will be Q3.

Note: If U4 had selected A1, the next question would have been Q2 as
when there is an equality, we select the first one.

Links
=====

Task ID 23509
Upgrade PR odoo/upgrade#906

closes odoo/odoo#44950

Related: odoo/upgrade#906
Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
2020-03-06 14:29:24 +01:00
Odoo's Mergebot fb846dc44a [MERGE] sms, mass_mailing(_sms): review contact list and composer usability
PURPOSE

Improve SMS composer usability, notably in single recipient mode. Allow to
edit and store number directly.

SPECIFICATIONS

  * the SMS button is visible next to a phone number in mass_mailing, and
    mass_mailing_sms;
  * in the mass_mailing contact list, the "email blacklisted" field has been
    moved next to the email field;
  * when mass_mailing_sms has been installed, the mass_mailing contact list
    also shows the phone number and whether this number has been blacklisted
    the mass_mailing_sms contact list also shows the email and whether this
    email has been blacklisted;
  * when sending an sms to a single contact, his phone number is now an
    editable field in the wizard and there is a warning if the number is
    invalid. Therefore the operator can modify the phone number directly
    and this new number is saved onto the record directly;

LINKS

Task ID 2088303

closes odoo/odoo#40482

Related: odoo/upgrade#873
Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
2020-03-06 14:29:10 +01:00
David Beguin 2d499c0236 [IMP] survey: integrate conditional questions in session mode
Purpose
=======

This commit makes the live session mode work with conditional question.
The point is to determine which question to display next. As multiple users
are 'voting' for answers of the same questions, the way to determine the
next question to display is to keep only the most voted answer.

The most voted answer will be the only one to trigger the linked conditional
question.

E.g.: considering
- following survey configuration:
    - Q1: A1, A2
    - Q2: triggered by Q1-A1
    - Q3: triggered by Q1-A2
- following user inputs:
    - U1: A1 selected
    - U2: A2 selected
    - U3: A2 selected
The next question displayed to ALL users will be Q3.

Note: If U4 had selected A1, the next question would have been Q2 as
when there is an equality, we select the first one.

Link
====

Task ID: 23509
PR #44950
2020-03-06 10:29:50 +00:00
David Beguin 3968bed239 [IMP] survey: add conditional questions demo data
Purpose
=======

This commit adds demo data to illustrate the conditional questions
configuration. Users will be able to choose among 4 themes. Depending on their
choice(s), the survey will continue by displaying only the questions related
to the selected theme(s).

Link
====

Task ID: 23509
PR #44950
2020-03-06 10:29:50 +00:00
David Beguin 0675d68afb [IMP] survey: add conditional display option on questions
Purpose
=======

This commit adds a new survey mode : conditional questions. A conditional
question is a question that will be displayed to the user only if the user
selects a specific answer from another previous question i.e. question B is
displayed if user answers 'Choice 1' at question A.

Specifications
==============

There are 2 different ways to handle conditional questions.

  * For one_page and page_per_section survey layout: conditional management is
    done mainly at frontend side. The idea is to be able to load all the
    questions of the section, hide the conditional questions and display them
    dynamically on answer check.
    E.g.: If user checks answer 'Choice 1' from question A, question B is
    displayed immediately.
    If question B is on section 2 and survey layout is set to section per page,
    question B will be displayed immediately on next page template load.
    Indeed, on each page load, server sends all the selected answers and the
    template displays the conditional question depending on the already
    elected answers;

  * For page_per_question survey layout: conditional management is done mainly
    at backend side. The idea is to find on each page the next question to
    display depending on:

    * navigation direction (next - previous)
    * already selected answers and conditional question dependance
    * survey questions sequence

	E.g.: If questions survey are Question A, B and C and question B must be
	displayed if answer 'Choice 1' from question A is selected.
	At Question A, if user select 'Choice 1' and goes next: display B
	At Question B, no matter the answer, if user goes next: display C
	At Question B, no matter the answer, if user goes prev: display A
	At Question A, if user select 'Choice 2' and goes next: display C
	At Question C, no matter the answer, if user goes next: End survey
	At Question C, if user has selected 'Choice 1' at question A and goes prev -> B
	At Question C, if user has selected 'Choice 2' at question A and goes prev -> A

In both modes answers on questions that should not have been displayed to the
user depending on the previous selected answers are cleared.
E.g. : User checks 'choice 1' from question A, then complete question B.
Then user unchecks 'choice 1' -> answer on question B is cleared.

Ignore conditional configuration :

  * during survey session, to avoid having different users answering different
    questions on the same session. That would be confusing for users to see
    questions on presenter screen that they didn't have the opportunity to
    answer;
  * if survey questions selection is random, because it will not display
    conditional questions if they are not kept by randomization. To avoid
    confusion, conditional configuration is simply ignored, not unset, so the
	user can still reactivate it without having to reconfigure everything.

Remove questions that should not have been displayed to the user (depending of
the made choices) in scoring computation and print layout.
--> Inactives questions are removed from predefined questions when user finish
the survey.
E.g.: If the user selects first path on 2, he will never see second path.
If the user answers correctly to all question of first path, he should have
100%, instead of 50%, even if questions of path 2 are mandatory and scored.

Link
====

Task ID: 23509
PR #44950
2020-03-06 10:29:50 +00:00
Victor Feyens 4d1009a275 [FIX] delivery: support multi for delivery_state compute
closes odoo/odoo#47017

X-original-commit: f778cd19be73fb4e6a1ed0d1b9194607578ab1d5
Signed-off-by: Victor Feyens (vfe) <vfe@odoo.com>
2020-03-06 10:21:56 +00:00
Victor Feyens 623d6a066b [FIX] delivery, sale: improve invoice_status performance.
* Avoid reading all lines invoice_status when the SO isn't confirmed.
* Do not use _default_product_id to discern down_payment lines, use is_downpayment instead
	(one ref = one query less by compute call)
* Do not consider display_type lines for SO invoice_status.

X-original-commit: 76a6d0475872f9ffa21cb917f13bc553c720eafd
2020-03-06 10:21:56 +00:00
Victor Feyens 20fec19fd0 [FIX] sale_stock: batch quantities computation correctly.
Available quantities computation is grouped by warehouse,date but using
datetime.now as fallback during the for loop ensures no batch
computation is done at all if order is not confirmed and has no
commitment_date.

X-original-commit: 563d0f73a8cea1933e57ae2b914f161ffeb5b7e4
2020-03-06 10:21:56 +00:00
Ankita Raval 5131572dab [FIX] mrp_workorder: Prevent to delete WO from Gantt
In this commit,Add a attribut "delete" in mrp workorder gantt view
to disable Remove button on "Planning by Workcenter" gantt view.

task-id:2206166

closes odoo/odoo#47064

X-original-commit: 669408f8b1e760a815e212d9e9fab8663e665dd6
Signed-off-by: Simon Lejeune (sle) <sle@openerp.com>
2020-03-06 10:20:08 +00:00
fja-odoo c730da7197 [FIX] website, *: fix placeholder translate
* = web_editor

InputUserValueWidget's data-placeholder where not translatable.

task-2210358

closes odoo/odoo#47029

X-original-commit: dfaf497de5eb95f1ba65e927631cc48d97ca6127
Signed-off-by: Quentin Smetz (qsm) <qsm@odoo.com>
2020-03-05 17:10:50 +00:00
Rémy Voet (ryv) 94b39d3ea3 [REF] stock,purchase,repair,mrp: clean ACL/report
For stock, purchase, repair and mrp modules, do a small clean of ACL:
Remove duplication and useless line.
Also improve the readability of the stock traceability report.

task-2172399

closes odoo/odoo#45055

Signed-off-by: Simon Lejeune (sle) <sle@openerp.com>
2020-03-06 09:16:16 +00:00
Xavier Morel 93ac430147 [REV] theme_*: rollback categories to just Theme
odoo/odoo#4c291e3f7049c3af859902a6f5857eab68be2f9e changed theme
categories from Theme to Website/Theme, however theme support has
technical dependencies on the category, so this breaks
design-theme (which was missed as design-themes was not yet under the
Not Rocket Science Rule).

Revert the change specifically for the theme modules.

closes odoo/odoo#47059
2020-03-06 09:20:01 +00:00
Loan (lse) 1c1d84255a [FIX] mass_mailing: remove wrong state default value from context
When testing to send an email of a campaign from the automation
marketing app, a traceback is thrown.

This is due to a context error. We define a default value which isn't
possible to be assigned for the variable.

opw-2190077

closes odoo/odoo#47039

X-original-commit: ff1d42e26bca3ce60697f9d4ee28f6aae5b7046b
Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
2020-03-05 18:41:12 +00:00
jvm-odoo 0e3b5c265b [FIX] mass_mailing: fix content not saved
Issue

	- Install Email Marketing
	- Create/Edit one of them
	- Save

	Changes not saved

	- Click edit & save

	Changes saved

Cause

	Before 30db768, the order of execution was:

	1. field_html > commitChanges (value = <p><br></p>)
	2. mass_mailing_widget > commitChanges (value = "test")

	After 30db768:

	1. mass_mailing_widget > commitChanges (value = "test")
	2. field_html > commitChanges (value = <p><br></p>)

Solution

	Use this.saveCroppedImages in mass_mailing widget to
	add the delay there too

OPW-2207107
OPW-2207817

closes odoo/odoo#47018

X-original-commit: 31e36fdc9b63fade99cef0c9ac4ed093b3936707
Signed-off-by: Jason Van Malder (jvm) <jvm@odoo.com>
2020-03-05 16:05:56 +00:00
Victor Feyens 917e7a1a9b [FIX] product: standard_price depends on company context
Product templates standard_price (cost) is company_dependent,
because the product_product standard_price is company dependent.

closes odoo/odoo#47041

Forward-port-of: odoo/odoo#46841
X-original-commit: 30ee9f1ac411aca9c2bf2158fca9be7db743616d
Signed-off-by: Victor Feyens (vfe) <vfe@odoo.com>
2020-03-05 19:17:17 +00:00
Xavier-Do 5ae6f02e45 [IMP] tests: scan upgrade path for additionnal tests
A future upgrades improvement will be to add tests in upgrades
modules. This small imp will scan in upgrade for some tests,
specific tooling will be added to upgrade repository in odoo/upgrade#878

closes odoo/odoo#47030

X-original-commit: 59b5a03bdf7ae1ad8851a67d278edc31a26a96a8
Signed-off-by: Xavier Dollé (xdo) <xdo@odoo.com>
2020-03-05 17:30:22 +00:00
fda-odoo 32c8922432 [FIX] web: eventBusDispatchToDOM set to true in pdf.js
In the module account_invoice_extract, the the code need to react to the 'pagerendered' event triggered by pdf.js. However in recent version of pdf.js, event are not visible outside of the library, except if the 'eventBusDispatchToDOM' has been set to true.

We tried to set this option from outside of the library but without success, as our pdf viewer is in an iframe.
There is no state of the iframe in which we can add an event listener to set the option.
pdf.js has an event used to signal when we can set settings, called 'webviewerloaded'.
This event is triggered in an EventListener attached to the 'DOMContentLoaded' event.
So, to list options we had, we could:
a) add an eventListener to the iframe document or window to react to 'webviewerloaded'. This doesn't work as document and windows are not the definitive ones and won't catche the event later.
b) add an eventListener to the iframe to react to 'DOMContentLoaded', which doens't work too as our listener will be called after the pdf.js one.

Finally the option was choosed to modify the default value of this option directly in pdf.js as no hook worked in the
'account_invoice_extract' module.

closes odoo/odoo#47024

X-original-commit: 42e6f56c887c3175c76f0f84b8581e288ebc7daf
Related: odoo/enterprise#9044
Signed-off-by: Florian Daloze (fda) <fda@odoo.com>
2020-03-05 16:31:29 +00:00
Thomas Binsfeld 193ce3de97 [IMP] sale: get invoice grouping keys from an easy to override function
closes odoo/odoo#47023

X-original-commit: d7244f38c1e39d353eacad970a5e1099995238a7
Signed-off-by: Damien Bouvy (dbo) <dbo@odoo.com>
2020-03-05 16:31:06 +00:00
Nicolas Martinelli 4ca24a47de [FIX] mrp: default UoM when registering lot
- Create a products A
- Create a products B
  Tracked by lot
  UoM: kg
- Get some quantity in stock for B
- Create a BOM for A: 2g of B to produce A
- Create a MO for A, Produce

In the lot registration wizard, 2kg of B are suggested instead of 2g of
B.

In this case, the UoM of the product is taken instead of the move.
However, it should only be the case if the product is tracked by unique
S/N.

opw-2200263

closes odoo/odoo#47019

X-original-commit: 9d114eb7a48b82b76e9c900cce22dc5022206367
Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
2020-03-05 16:07:34 +00:00
Nicolas Martinelli 3c4c625a0a [FIX] sale_mrp: delivered qty in dropship
- Activate Dropshipping
- Create 3 products: KIT AB, A, B
  All have the Dropship route and a supplier set
- Create a BOM of type kit: KIT AB is composed of A & B
- Create a SO for 1 unit of KIT AB, validate
- Validate the created PO
- Deliver the products

The delivered quantity remains zero.

This happens because the `bom_line_id` was not written on the
`stock.move`. Indeed, the latter is created from a PO, not from a SO.
Since the PO doesn't have a `bom_line_id` field, the information is
lost.

As a solution, we apply the 'all or nothing' policy in case of
dropshipping.

opw-2207118

closes odoo/odoo#47014

X-original-commit: 6016476952f53097f8b04f34bdbaf09c12e431fe
Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
2020-03-05 16:02:38 +00:00
svs-odoo cdddd45a73 [IMP] stock: _get_without_quantities_error_message
Adds a method to be able to override the error message in the case where
the user try to validate a picking with no quantities done nor reserved.

task-2123726

closes odoo/odoo#45457

Related: odoo/enterprise#8364
Signed-off-by: Simon Lejeune (sle) <sle@openerp.com>
2020-03-05 13:32:57 +00:00
svs-odoo 3ed3ed183c [IMP] stock_picking_batch: confirm waiting picking
Before this commit, the batch action done processes only if all batch's
pickings are `reserved`, which cause batches with a waiting picking
(`confirmed` state) can not be validated.
As the picking itself can be validated even if it is in waiting state,
it seems logical that we can do the same from batch picking.

task-2123726
2020-03-05 13:32:57 +00:00
svs-odoo 339c40abe1 [IMP] stock: inventory line product_qty readonly
Moves the readonly condition for the `product_qty` field into the model
to have the same condition in the barcode app (enterprise).

task-2123726
2020-03-05 13:32:57 +00:00
Florimond Husquinet (fhu)andThibault Delavallée f685e70616 [IMP] sms: allow to edit number when sending an SMS to one recipient
After this commit

  * the SMS button is visible next to a phone number in mass_mailing, and
    mass_mailing_sms
  * when sending an sms to a single contact, its phone number is now an
    editable field in the wizard and there is a warning if the number is
    invalid;
  * if operator updated the recipient number, update record number according
    to the new number only in single recipient mode (aka, solving number
    issues directly from interface);

LINKS

Task ID 2088303
Community PR odoo/odoo#40482
Upgrade PR odoo/upgrade#873

Co-Authored-By: Florimond Husquinet <fhu@odoo.com>
Co-Authored-By: Thibault Delavallée <tde@odoo.com>
2020-03-05 11:27:01 +00:00
Thibault Delavallée e36661ecd9 [REF] sms: improve _sms_get_recipients returned information
Purpose is to improve information given back, adding notably

  * whether the value comes from the customer (partner_id field) or directly
    from the record we are looping on;
  * the actual field used when no specific field is enforced;

LINKS

Task ID 2088303
Community PR odoo/odoo#40482
2020-03-05 11:04:49 +00:00
Thibault DelavalléeandFlorimond Husquinet 3150541bed [REF] sms: composer model cleaning
In this commit we perform some cleaning in composer model before adding
number update feature:

  * remove partner_ids field that is not used anymore. It comes from early
    implementation of SMS composer and is now dead;
  * rename recipient_count to recipient_valid_count to better reflect its
    use;
  * rename recipient_description to recipient_single_description to better
    reflect it is used only in mono-recipient mode;
  * add a new single comment computed field, allowing to display some
    information only when sending an sms to a single given record;

We also fix computed fields by making them work in batch.

LINKS

Task ID
Community PR odoo/odoo#40482
Upgrade PR odoo/upgrade#873

Co-Authored-By: Florimond Husquinet <fhu@odoo.com>
Co-Authored-By: Thibault Delavallée <tde@odoo.com>
2020-03-05 11:04:49 +00:00
Thibault DelavalléeandFlorimond Husquinet 504566c595 [IMP] mass_mailing(sms): merge contact list views to have coherent views
SPECIFICATIONS

Merge contact list views used in mass_mailing and mass_mailing_sms

  * in the mass_mailing contact list, the "email blacklisted" field has been
    moved next to the email field;
  * when mass_mailing_sms has been installed, the mass_mailing contact list
    also shows the phone number and whether this number has been blacklisted;
  * the mass_mailing_sms contact list also shows the email and whether this
    email has been blacklisted;
  * enable sms option on mobile field of contact;

LINKS

Task ID 2088303
Community PR odoo/odoo#40482
Upgrade PR odoo/upgrade#873

Co-Authored-By: Florimond Husquinet <fhu@odoo.com>
Co-Authored-By: Thibault Delavallée <tde@odoo.com>
2020-03-05 11:04:49 +00:00
Aurélien Warnon d2dbb0da2e [IMP] mass_mailing: improve mailing.mailing kanban bottom block
This commit improves the bottom section of the mailing.mailing kanban view to
make it follow other kanban views structure.

It makes the user avatar on the mailing.mailing kanban display in a round shape
instead of a square (with border-radius).

LINKS

Task ID 2207856

closes odoo/odoo#46708

Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
2020-03-03 10:05:15 +00:00
Cocographique de06caa988 [FIX] website: enable options for slides of quotes carousel snippet
task-2209956

closes odoo/odoo#47000

X-original-commit: df959893cf9a0c763dd2f68bb31702a0ddcabfc5
Signed-off-by: Quentin Smetz (qsm) <qsm@odoo.com>
2020-03-05 15:10:22 +00:00
Samuel Degueldre 5033b870b6 [IMP] web_editor, website: improve border and border radius options
Previously, the border radius option was only available when there was a
border width, but in some cases you want no border, a background color,
and still have rounded corners. This commit makes the option available
when there is either a border OR a background color.

This commit also adds an option to choose the border style between
solid, dashed, dotted and double. The other styles have been left out as
they look very different browser to browser and are not very useful.

task-2195429

closes odoo/odoo#46625

Signed-off-by: Quentin Smetz (qsm) <qsm@odoo.com>
2020-03-04 13:15:01 +00:00
Yenthe666 cdde9be13b [IMP] website_sale: improve confirmation info
The sentence was a bit odd before this commit. After this commit it reads easier and it is more logical.

closes odoo/odoo#46617

Signed-off-by: Damien Bouvy (dbo) <dbo@odoo.com>
2020-03-02 13:52:01 +00:00
Yannick Tivisse 4c291e3f70 [IMP] base: Display searchpanel on ir.module.module views
Purpose
=======

The current kanban view is messy. It is difficult to identify which
apps are installed or not. The user can completely miss a module
that might have interested him. A search panel would make things way
more readable.

closes odoo/odoo#44401

Taskid: 2181557
Related: odoo/enterprise#8144
Related: odoo/upgrade#879
Signed-off-by: Yannick Tivisse (yti) <yti@odoo.com>
2020-03-05 14:03:45 +00:00
svs-odoo 4f1fa7e24c [FIX] delivery: put in pack package domain
Before this commit, on a picking, when we want to put in pack, we can't
select packages created with no package carrier type as they had a
confusion between `package_carrier_type` and `delivery_type` when we
pass the value in the context (value who is reused to the package
domain, making some package unfindable).

closes odoo/odoo#46992

X-original-commit: 21f054731e8635b9e3681e673a7cbf627ad49cca
Signed-off-by: pimodoo <pimodoo@users.noreply.github.com>
2020-03-05 14:37:09 +00:00
Lucas Lefèvre 7032dde60a [IMP] web: Add quick security access to debug menu
PURPOSE

Debugging access issues can quickly be a pain.
The goal is to allow to easily debug an access rights issue from the debug tools.

SPECIFICATION

In the debug tools, add the following two new items:
- View Access Rights
- View Record Rules

closes odoo/odoo#46906

Signed-off-by: Yannick Tivisse (yti) <yti@odoo.com>
2020-03-04 16:15:56 +00:00
Helly kapatel 01497474d7 [IMP] sale: rename mail notification template 'oezirhgoi'
In Sale, there is a view called oezirhgoi which adds the sign and pay
online buttons (xml id sale.mail_notification_paynow_online)

So rename it with the 'Quotation: Sign and Pay mail notification template'

closes odoo/odoo#46954

Taskid: 2196751
Closes: #46954
Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
2020-03-05 12:39:32 +00:00
Simon Lejeune 53e89e5006 [FIX] product: expand group on categories only when needed
Issue: on main product's kanban, if you group by categories, all
categories (even the empty ones) appear. This is not desired.

We only wanted this group_expand logic when using the products stat
button on the product category form view.

closes odoo/odoo#46966

X-original-commit: e3f1254e5b28aa801f137835d871427786fb3428
Signed-off-by: Simon Lejeune (sle) <sle@openerp.com>
2020-03-05 12:25:06 +00:00
Pierre Paridans a5dccc1586 [REV] mail: Move the attachment icon on the left
This revert of commit odoo/odoo@9cd3bcd440 is due to
the attachment's button needing to be kept on the right side of the
chatter's topbar.

Task ID: 2193950

closes odoo/odoo#46958

X-original-commit: d458aca28c25879ca9878b0628f2a6ea703504ef
Related: odoo/enterprise#9027
Signed-off-by: Pierre Paridans <pparidans@users.noreply.github.com>
2020-03-05 11:40:01 +00:00
Pierre Paridans 80c00af329 [FIX] mail: overflow in the attachment box
The chatter's attachment box has an horizontal overflow when adding at
least 1 attachment.

This is due to a negative margin applied on the attachment list/previews
not matching its container's padding (3px vs 5px).

This commit reuses the SCSS variable dedicated to this margin around the
attachment list/previews and apply it correctly to the chatter's
attachment box.

Task ID: 2194025

X-original-commit: 3a7eb3fe97541855ae893514f8b98cbfe6c1a7e4
2020-03-05 11:40:01 +00:00
Pierre Paridans 5d88e164bc [FIX] mail: chatter has horizontal scroll on medium screen sizes
A horizontal scroll appears on medium to large screen sizes in the
form's chatter sidebar (e.g. between 1534px and 1619px, when all modules
are installed).

This commit restores the padding set by Bootstrap for `.nav-link`
elements instead of a fraction of the grid gutter's (?) width then
applying slightly smaller padding.

Task ID: 2193950

X-original-commit: 7bc8b577d822b057d1bccf881b9c4b1ee127a728
2020-03-05 11:39:59 +00:00
Cocographique 43800cb046 [FIX] website: update path to author default image in blockquote snippet
task-2209946

closes odoo/odoo#46959

X-original-commit: a248391008caa8a0f04dcd2bc027fc8f0b7b0945
Signed-off-by: Quentin Smetz (qsm) <qsm@odoo.com>
2020-03-05 11:40:41 +00:00
Siddarth Gajjar 6a56b6c47b [IMP] crm: remove custom css on form view
Purpose of this commit is to get rid of custom css code and file and use
standard bootstrap classes.

TaskID 2182610
Closes #45472

closes odoo/odoo#46948

Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
2020-03-05 10:42:43 +00:00
Pierre Masereel b10841acbe [FIX] delivery: access rights when computing price rules
When a picking is validated with the delivery prices based on rules, you
can face an error, if you are not inventory manager or sales manager.

First, because you don't have access rights to see the sales orders used
to compute the price.

And then you don't don't have access rights on the delivery.price.rule
object to apply the formulas.

As the computation of the price of a delivery should not depends on
access rights other than the ones that let the user validate the
picking, as 'sudo' is called.

OPW-2209148

closes odoo/odoo#46947

X-original-commit: b68139bf2fec8fd2a79e1a578d126f260891c4f6
Signed-off-by: pimodoo <pimodoo@users.noreply.github.com>
2020-03-05 09:57:32 +00:00
Nicolas Martinelli 2e02be7404 [FIX] l10n_fr_pos_cert: decrease quantity
In a French configuration, the decimal separator is a comma.
Unfortunately, `parseFloat` doesn't handle this case and incorrectly
parses the string.

Similar issue and correction than
https://github.com/odoo/enterprise/commit/af8177612e569907bcb8fe4b9ce2d845de5ab2a0

opw-2207795

closes odoo/odoo#46945

X-original-commit: 3d87d79ecec92cafd7f964712ad0014add5cbb6a
Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
2020-03-05 09:49:15 +00:00
Harpritsinh Sisodiya d061852a97 [FIX] web: autocomplete dropdown: style of selected item
Before this commit, in the website, the background of the
highlighted item in an autocomplete dropdown didn't take full
width. Now it does.

closes odoo/odoo#46924

Task: 2066536
Closes: #36997
X-original-commit: e184e971790c0045b58db6ae588b1d1b14d3154c
Signed-off-by: Aaron Bohy (aab) <aab@odoo.com>
2020-03-04 16:44:06 +00:00
Adrian Torres 26680badef [FIX] base: handle custom non-stored m2m
Before this commit, one could create a custom, non-stored m2m field
through the web interface BUT the deletion of the custom field would be
impossible after saving the form.

This is due to the fact that the relation would not be stored in DB
since the field is store=False, therefore the value of the relation
would equal to None, and since the unlinking mechanism assumes that
the value is never None, it tries to drop a table of name None, which
does not exist.

closes odoo/odoo#46909

X-original-commit: cdd1439a380810569db80d984c8cb80d213b1bf3
Signed-off-by: Adrian Torres (adt) <adt@odoo.com>
2020-03-04 15:39:18 +00:00
Goffin Simon babf9e4f2c [FIX] account: Reference of invoices in payment receipt
Steps to reproduce the bug:

- Let's consider a supplier S
- Create two vendor bills for S B1 and B2
- B1 with the vendor reference R1
- B2 with the vendor reference R2
- Register a payment P for B1 and B2
- Print the payment receipt of P

Bug:

The reference of each vendor bill didn't appear in the payment receipt

opw:2190086

closes odoo/odoo#46360

X-original-commit: 068bc8b22b14c5a9756a19aed2f28a786265ad67
Signed-off-by: Simon Goffin (sig) <sig@openerp.com>
2020-02-26 15:06:10 +00:00