Commit Graph
167078 Commits
Author SHA1 Message Date
Alexandre Kühn bef68e577a [FIX] mail: center avatar with composer with a single line
Before this commit, the avatar next to composer in Discuss app
had the top aligned with the top of the text input of composer.
The avatar is smaller than the input, and the text in the input
is centered, so the alignment looks off.

This commit fixes the issue by offseting the avatar so that
when the composer has only 1 line, the avatar is vertically centered
with the input.

Note that this alignment should be fixed, i.e. if the input field has
more than 1 line, we want to keep the avatar in the same place, hence
why the avatar is just statically offset.

Also align composer avatar with message list, by removing the
`align-justify: self` that was moving avatar slightly towards the
discuss app sidebar.

closes odoo/odoo#140398

X-original-commit: 5fe4297f3d1a988ae65617c177046a223bc62890
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2023-10-30 21:50:48 +00:00
Xavier Luyckx (xlu) 733a93956f [IMP] purchase: rfq dashboard values background color
Prior to this commit, the Request for Quotations (RFQ) dashboard suffered
from a visibility issue. Specifically, the values displayed on the right
section of the dashboard were using a background color class that was not
correctly used leading to poor differentiation between the values and
their corresponding labels/titles.

This commit apply the correct background colors to these elements.

task-3573792

closes odoo/odoo#140395

X-original-commit: c46970dc773f8985acdb0d07ce9faa5e7d204641
Signed-off-by: Tiffany Chang (tic) <tic@odoo.com>
Signed-off-by: Xavier Luyckx (xlu) <xlu@odoo.com>
2023-10-30 21:50:47 +00:00
tsm-odoo 74f59fc59c [IMP] im_livechat: no 2 emoji icons in quick actions of message
Before this commit, messages in livechat from visitor POV show
2 emoji icons in the quick actions. This is confusing and comes
from 2 actions sharing this exact same icon: "Add a reaction"
and "View reactions".

This commit fixes the issue by reducing the number of quick message
actions to 2 for the embed live chat. In other contexts there's
"Reply-to" and "Mark as Favorite" in quick actions, so the
"View reactions" item is always in expand menu.

Note that this "dirty" fix come from 2 actions with same icon.
Eventually we should fix that by clearly making icons unique.
This solution is decent for the time being, giving us more time
to make a dedicate icon for one of the emoji actions.

task-3498446

closes odoo/odoo#140335

Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2023-10-30 21:50:46 +00:00
Romain Derie 709bffcb6d [FIX] web_editor: disable grid mode when inside navbar extra items
There is an exact case where having a grid mode block inside a mega menu
item will fully bug: all elements will be on top of each other.

In order to reach this case, you need to:
- Create a mega menu
- Make the block inside this mega menu use grid mode
- Find a screen size where this mega menu will be part of the extra menu
  items (hidden in the `+` entry)
- BUT the screen size should not toggle the mobile view

Point of attention when trying to replicate the issue:
When you enable grid mode, the result will depend on the available size.
So be sure to not enable the grid mode on a megamenu which is already
hidden in the extra items when you are in edit mode, in such a case,
there is no bug.

This commit simply disables the grid mode in such a case, it will thus
behave like the grid mode in mobile view and therefore be responsive
(disabled in mobile).

opw-3547405

closes odoo/odoo#139956

X-original-commit: fb209432049d9874308a2bdc3c9d2baa210ccced
Signed-off-by: Soukéina Bojabza (sobo) <sobo@odoo.com>
Signed-off-by: Romain Derie (rde) <rde@odoo.com>
2023-10-30 19:23:44 +00:00
fja-odoo c40492b0f5 [FIX] point_of_sale: block ui while syncing orders
Step to reproduce:

- Create a pos session
- Create a new order
- Process the payment (the syncing has to be long to notice the issue)

Issue:

You can still click on buttons in the interface payment screen and
trigger tracebacks

Cause:

The ui is not blocked while syncing orders

Solution:

Block the ui while syncing orders

closes odoo/odoo#140342

X-original-commit: 61dd356e48e7d0bfe1c8758c3835b858e2aa8de0
Signed-off-by: Joseph Caburnay (jcb) <jcb@odoo.com>
Signed-off-by: fja-odoo <fja@odoo.com>
2023-10-30 17:48:47 +00:00
Antoine Vandevenne (anv) f16da3bfdc [FIX] payment_stripe: fix payment method code mapping for SEPA
closes odoo/odoo#140340

Signed-off-by: Antoine Vandevenne (anv) <anv@odoo.com>
2023-10-30 17:48:46 +00:00
Arnold Moyaux 715f877eab [FIX] repair: pick moves automaticaly in repair
Since PR #137864
We do not have quantity done anymore and we use the picked field
to know if something is picked or not. But we remove the immediate
transfer wizard and we consider instead than if nothing is mark as
pick, everything is pick. We will do the same for repair because it
would confusing to have a different behavior

closes odoo/odoo#140322

Signed-off-by: William Henrotin (whe) <whe@odoo.com>
2023-10-30 17:48:45 +00:00
Arthur Detroux (ard) 57ed8bc0bf [REF] website, *: replace legacy dialog with OWL Dialog
*: test_website, web_editor, website_sale

The goal of this commit is to replace all Dialog in the Website Editor
flow with OWL Dialog. This uses the new `bindService` and `call` methods
available on legacy widgets. The replacements mainly use the
ConfirmationDialog component, but some had to have custom-made dialogs.

closes odoo/odoo#138947

Signed-off-by: Francois Georis (fge) <fge@odoo.com>
2023-10-30 17:48:42 +00:00
Arthur Detroux (ard) 0997b0a401 [REF] website: remove company_data SnippetOption
This code was use in 14.0 to allow users to edit the Company Information
visible on the /contactus page. It is no longer needed as the page is
now fully editable and the t-field has been replaced with static text.

Part-of: odoo/odoo#138947
2023-10-30 17:48:42 +00:00
Mahdi Cheikh Rouhou (macr) 4bb1bfe25b [FIX] website_slides_survey: compute left attempts correctly
Issue:
======
- When a user attempt a certification it will count as 1st attempt and
then do retry directly it will count as second attempt : fine.
- When a user attempt a certification it will count as 1st attempt and
  then exit the page and go to the certification again it will count as
  1st attempt too. So the user can have unlimited number of attempts
  like this.

Steps to reproduce the issue:
=============================
- Install survey , elearning
- Go to a course with a certification and do the certification, exit the
  page and enter to it again.
- Go to the user page and see the certifications you will see they both
  counted as 1st attempt.

Expected behavior:
==================
- when a user leaves a course (willingly or not) we remove the
  relationship between the user_input and slide_partner_id.
- To get the remaining attempts we just get the user_input_ids of the
  slide_partner because the old ones are removed.

opw-3498054

closes odoo/odoo#140305

X-original-commit: 74d2b551ae392fa20744fa22582b5b38b3babff7
Signed-off-by: Warnon Aurélien (awa) <awa@odoo.com>
Signed-off-by: Mahdi Cheikh Rouhou (macr) <macr@odoo.com>
2023-10-30 16:41:52 +00:00
Alexandre Kühn a74691949c [FIX] mail: same discuss badge color as systray in dark theme
Dark-mode colors have been adapted from MILK redesign in 16.3 [1].
Discuss badge colors were changed in 16.4 to match systray color [2].
Commit [1] adapted them to use primary color instead of intended
sytray color. Removing the override of style fixes the issue, as the
base style in `core.scss` (white theme) works in both themes.

[1]: https://github.com/odoo/odoo/pull/130991
[2]: https://github.com/odoo/odoo/pull/122946

closes odoo/odoo#140300

X-original-commit: b46bded27349acd311682846c0a0416ce4898882
Signed-off-by: Didier Debondt (did) <did@odoo.com>
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2023-10-30 16:41:51 +00:00
sofiagvaladze 20d87404db [FIX] hr_holidays: align buttons on time off popup
Correct the view. Prior to this commit the buttons on the time off
popup are not alligned.

closes odoo/odoo#140284

Signed-off-by: Bertrand Dossogne (bedo) <bedo@odoo.com>
2023-10-30 16:41:49 +00:00
Louis (wil) 48ee45d6ce [FIX] *: fix typos
Corrects typos found when translating Odoo 17.

* : hr_fleet, payment

closes odoo/odoo#140282

Related: odoo/enterprise#49831
Signed-off-by: Louis Wicket (wil) <wil@odoo.com>
2023-10-30 16:41:49 +00:00
Sébastien Theys ad612321bc [IMP] mail: improve debug views of channel (member)
Make partner_id and guest_id properly exclusive.
Allow creating a channel member from its form view.

closes odoo/odoo#140272

Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2023-10-30 16:41:47 +00:00
Alexandre Kühn 7cf86921b4 [FIX] mail: click on messaging menu item should not rename thread
Before this commit, when opening a thread in the Discuss app and
changing to another active thread from the Messaging Meu in the
systray, the new thread was mistakenly renamed to the previous
active thread name.

This happens because the component `AutoresizeInput`, which is used
to show the thread name in the header of Discuss app, also allows to
rename the thread. The code to trigger rename was too naïve, in that
a click away was considered as a rename operation on the current
thread.

This commit fixes the issue by triggering the editing of the value
in the `AutoresizeInput` component only when the click away happens
while the input had focus. The click in messaging menu will not
trigger it as the input is not focused.

Task-3570377

closes odoo/odoo#140208

Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2023-10-30 16:41:46 +00:00
Ricardo Gomes Rodrigues 8c4e855a1b [FIX] account_reports: warn user if some accounts are deprecated while generating the grouped deferral entries
Currently, if a user has an active deferred entry whose account has been deprecated in the mean time,
we create a move line containing this deprecated account which raises an unclear error to the user.

Now, instead, we let the user create deferral entries for deprecated accounts.

We also add a test to check that old moves are not taken into account when generating deferral entries.

task-id 3544877

closes odoo/odoo#140155

X-original-commit: 47f97fb208ee6c6fc4ce522bc7ad648ea6bbe07c
Related: odoo/enterprise#49758
Signed-off-by: William André (wan) <wan@odoo.com>
Signed-off-by: Ricardo Gomes Rodrigues (rigr) <rigr@odoo.com>
2023-10-30 16:41:45 +00:00
David Monnom (moda) f54c572152 [IMP] pos_self_order: add tests
This commit adds tests for the pos_self_order module. The tests are
written in javascript and use the odoo testing framework. The tests
cover the following scenarios:
- attribute selection
- combo selection
- kiosk mode
- mobile mode

closes odoo/odoo#140138

Signed-off-by: Adrien Guilliams (adgu) <adgu@odoo.com>
2023-10-30 16:41:42 +00:00
“moda” 569016bf63 [FIX] point_of_sale: error handling
Before, error handling didn't work in point_of_sale when offline.
This was due to poor error handling.

Now, the "limited functionnalities" popup is displayed correctly.

closes odoo/odoo#140037

Related: odoo/enterprise#49705
Signed-off-by: Joseph Caburnay (jcb) <jcb@odoo.com>
2023-10-30 16:41:42 +00:00
Gauthier Wala (gawa) ffc001b4c3 [FIX] l10n_pl: name_get deprecated
Since https://github.com/odoo/odoo/commit/3c62ca1eb96d571b2b686b5caee370324c589ab4, name_get is deprecated.
We should replace it with a _compute_display_name.

Linked to runbot error 25698

closes odoo/odoo#139082

Signed-off-by: Laurent Smet (las) <las@odoo.com>
2023-10-30 16:41:37 +00:00
rhe-odoo 384c4e2f21 [FIX] point_of_sale: Flush order not null date order
The field date_order cas removed in a previous commit.
Thus, this line needs to be removed as well.

closes odoo/odoo#133654

Related: odoo/enterprise#46580
Signed-off-by: Joseph Caburnay (jcb) <jcb@odoo.com>
2023-10-30 16:41:34 +00:00
rhe-odoo a1a77d4ffa [FIX] point_of_sale: vat letter missing
When the pr of the skip receipt was merged, the vat letter feature was missing.

This commit put it back to the point of sale module so we can inherit it again in other modules.

Signed-off-by: rhe-odoo <rhe@odoo.com>

m

Part-of: odoo/odoo#133654
2023-10-30 16:41:34 +00:00
rhe-odoo 97f1f80da0 [IMP] point_of_sale: error handling blackbox
This commit remove the logic of error handling of the black boxes.
It'll be done in the specific module.

Part-of: odoo/odoo#133654
2023-10-30 16:41:34 +00:00
Arnold Moyaux bd5601cf59 [FIX] stock: error message for put in pack
closes odoo/odoo#140307

Signed-off-by: William Henrotin (whe) <whe@odoo.com>
2023-10-30 15:18:04 +00:00
Arnold Moyaux 8327a83f38 [FIX] stock: avoid warning for location_dest
It's not needed anymore since the location dest on stock.move.line
are updated accordingly.

Part-of: odoo/odoo#140307
2023-10-30 15:18:04 +00:00
Arnold Moyaux caa33c4f14 [FIX] mrp_subcontracting: rename record component button
There is a button in the header to register subcontracting process
(when needed). And there is an additional button in the view with
the purpose to correct data later after the encoding. However we
can't set button with optional="hide". We rename it with an edit
icon

Part-of: odoo/odoo#140307
2023-10-30 15:18:04 +00:00
FrancoisGe 92daf621b5 [REF] stock: don't use "extension" mode for custom component
It's bad practice to use the "extension" inheritance mode for a custom
component. In this case, inheritance is only necessary for the
"StockMoveX2ManyField" custom field.
In this commit, we will therefore use primary mode to generate a custom
template used only by "StockMoveX2ManyField".

closes odoo/odoo#140301

Signed-off-by: William Henrotin (whe) <whe@odoo.com>
2023-10-30 15:18:03 +00:00
FrancoisGe f534d1cccf [FIX] web: x2m editable in line and dialog
Purpose:
-------
The "StockMoveX2ManyField" custom field has given rise to a new need,
that of being able to edit a record in an x2many directly or from a dialog.
This is the first use case to require the ability to do both at the same time.

This particular use case highlighted a bug preventing a change from being saved.

How to reproduce:
- Go to a form view with an x2m using a custom widget
- Open an x2m record in dialog mode
- Close the dialog
- No changes have been made to the record, so the save button is not displayed
- Edit a field in the same record

Before this commit:
    The save button is not visible

After this commit:
    The save button is visible.

closes odoo/odoo#140298

Signed-off-by: Pierre Rousseau (pro) <pro@odoo.com>
2023-10-30 15:18:02 +00:00
Michael (mcm) 89bf62b0c1 [IMP] web,*: remove legacy/utils.js from backend
This commit removes useless function from legacy/utils.js and removes it
from several backend bundles. The file is moved in wysiwyg and frontend
bundles as it is still used in these bundles.

task id: 3439226

closes odoo/odoo#140260

Related: odoo/enterprise#49812
Signed-off-by: Francois Georis (fge) <fge@odoo.com>
2023-10-30 15:18:00 +00:00
Michael (mcm) d8c0373f33 [IMP] web: link popover together
Before this commit, cliking on a popover that was open inside another
popover closed both of them.
After this commit, the popovers are not closed. them will close only if
we click out of them and the popovers they opened.

closes odoo/odoo#140258

Signed-off-by: Francois Georis (fge) <fge@odoo.com>
2023-10-30 15:17:59 +00:00
FrancoisGe 64037d03d0 [FIX] web: FormStatusIndicator flickering
Purpose:
--------
Before this commit, if you edit a char field with an onchange in
an x2many, the FormStatusIndicator buttons will flicker.

How to reproduce:
-----------------
- Go to a form view with an x2many field in list mode
- Type a char in the char field of a record
- The FormStatusIndicator buttons are visible
- Click outside the char field
- An onchange is triggered

Before this commit:
    The FormStatusIndicator buttons will disappear until the onchange returns.

After this commit:
    The FormStatusIndicator buttons remain visible.

closes odoo/odoo#140179

Signed-off-by: Pierre Rousseau (pro) <pro@odoo.com>
2023-10-30 13:56:01 +00:00
Theo VINCENT (thvi) c9ebb4de41 [FIX] pos_self_order: receipt payment lines
Fixes the receipt payment lines name (replacing "Bank" by the payment method name).

closes odoo/odoo#140098

Task-id: 3572422
Signed-off-by: Joseph Caburnay (jcb) <jcb@odoo.com>
2023-10-30 13:55:59 +00:00
Theo VINCENT (thvi) 1627d929f5 [FIX] pos_self_order: categories display
Fixes the display of categories in the self-order UI.

closes odoo/odoo#140095

Task-id: 3572141
Signed-off-by: Joseph Caburnay (jcb) <jcb@odoo.com>
2023-10-30 13:55:58 +00:00
Adrien Guilliams (adgu) a97cb5ae11 [IMP] pos_self_order, point_of_sale: service type on ticket
In this commit, we add the mention of the service type on the ticket and
when the service is "table", we add the table stand number.

We also fix a problem with stripe and adyen where the ticket does not
print the order but an empty order with the kiosk.

closes odoo/odoo#140047

Signed-off-by: David Monnom (moda) <moda@odoo.com>
2023-10-30 13:55:57 +00:00
Adrien Minne (adrm) eba33ff004 [IMP] spreadsheet: change global filter icon
Change the global filter icon from the `fa-search` to something more
filter-like.

Also use the same icon in shared dashboard.

closes odoo/odoo#139164

Task: 3560817
Related: odoo/enterprise#49166
Signed-off-by: Lucas Lefèvre (lul) <lul@odoo.com>
2023-10-30 13:55:56 +00:00
FrancoisGe 6b045f540a [REM] web, *, remove few dom.js functions
In this commit, we will remove "closestScrollable", "compensateScrollbar"
and the last usage of "scrollFixedOffset" outside of dom.js.

closes odoo/odoo#140181

Part-of-task: 3439226
Signed-off-by: Jorge Pinna Puissant (jpp) <jpp@odoo.com>
2023-10-30 12:05:31 +00:00
Jorge Pinna Puissant 77c89b3316 [IMP] web: remove hasScrollableContent function from dom.js
Part-of-task: 3439226
Part-of: odoo/odoo#140181
2023-10-30 12:05:31 +00:00
Jorge Pinna Puissant 95fa59533d [IMP] web: remove areColliding function from dom.js
Part-of-task: 3439226
Part-of: odoo/odoo#140181
2023-10-30 12:05:31 +00:00
Jorge Pinna Puissant e2b0d3efe4 [IMP] web: remove non used dom.js functions
Part-of-task: 3439226
Part-of: odoo/odoo#140181
2023-10-30 12:05:31 +00:00
Jorge Pinna Puissant 3de178ed42 [IMP] web: remove non used safeConfirm from legacy Dialog
Part-of-task: 3439226
Part-of: odoo/odoo#140181
2023-10-30 12:05:31 +00:00
Rob Coekaerts 80c5e8986e [FIX] hr_holidays: fix employee_id(s) domain
Before this commit, the domain on ``employee_id`` and ``employee_ids``
would be computed in a computed method ``_compute_allowed_employee_ids``.
However, due this compute method not triggering for new records, the
domain was not correctly set.

Additionally, this way of computing the domain was not very efficient.
With this change, the domain is computed correctly and efficiently.

closes odoo/odoo#140146

Signed-off-by: Sofie Gvaladze (sgv) <sgv@odoo.com>
2023-10-30 12:05:28 +00:00
William Henrotin 27c95fa2c0 [FIX] stock: remove 'set' and 'clear' quantities buttons
Thanks to ref https://github.com/odoo/odoo/pull/137864, the quantity on
stock move can be emptied or fill with the unreserve/reserve mechanism.
Those two buttons becomes useless

closes odoo/odoo#140116

Signed-off-by: Arnold Moyaux (arm) <arm@odoo.com>
2023-10-30 12:05:28 +00:00
Hamza (hisl) c3289f0734 [FIX] web: set pointer events none for ribbon
The forms where we have ribbons do not provide a great user experience when the internal links are right below the ribbon. Because of the shape and size of the ribbon, the internal link button on the topmost fields gets behind the ribbon. Because the rectangular part of the ribbon is transparent, we can see the button, but cannot click or hover over it, because the ribbon element takes over. To resolve this, I set the pointer-events of ribbon to none. The ribbons already do not have any click events so this solution seems okay.

Steps to Reproduce:
1. Move a helpdesk ticket to the archive.
2. Try to move to the customer page using the internal link.

Current Behavior:
The link button cannot be hovered over and clicked.

Expected Behavior:
The link button should be above other elements and clickable.

OPW-3468153

closes odoo/odoo#140099

X-original-commit: c9aa5fb7645470c5a63784e58fdb4ee9acf7d5e8
Signed-off-by: Bastien Fafchamps (bafa) <bafa@odoo.com>
Signed-off-by: Simon Goffin (sig) <sig@odoo.com>
2023-10-30 12:05:26 +00:00
Florian Damhaut 9f70f90df7 [FIX] sale_management: use _can_be_edited_on_portal() when needed
add values missed in 060e13ad6d

PR : odoo/odoo#140051
task-id : 3571692

Related: odoo/enterprise#49713
Signed-off-by: Arnaud Joset (arj) <arj@odoo.com>
2023-10-30 12:05:25 +00:00
Florian Damhaut 4b2f989c71 [FIX] sale: hide cancelled quote alert
PR : odoo/odoo#140051
task-id : 3571692
2023-10-30 12:05:25 +00:00
Florian Damhaut 6e13d715f9 [FIX] account: fix portal cards count
PR : odoo/odoo#140051
task-id : 3571692
2023-10-30 12:05:25 +00:00
Adrien Guilliams (adgu) 4f39f7cb29 [FIX] pos_loyalty, l10n_in_pos: add partner to order receipt
In this commit, we are adding partner to the order receipt as it is
needed for the receipt in india and for the loyalty module.

closes odoo/odoo#140046

Signed-off-by: Robin Heinz (rhe) <rhe@odoo.com>
2023-10-30 12:05:23 +00:00
Thomas Becquevort (thbe) 14879c2623 [IMP] account: Splitting Amount to settle menu
Currently, the "Amounts to settle" button in the "Vendors" drop-down menu
opens a view showing all the account.move.line that are to be reconciled.
Also, they should only be shown when their residual is less than zero.

This commit makes it now so that when clicking on that button, only the amls
linked to a purchase journal are shown and another "Amounts to Settle" button
has been added in the "Customers" drop-down menu to show the amls linked to
a sale journal

task-3572482

closes odoo/odoo#140040

Signed-off-by: William André (wan) <wan@odoo.com>
2023-10-30 12:05:22 +00:00
Pierre Pulinckx (pipu) e179fce25c [FIX] web: graph: sort tooltip content
Tooltip items are not correctly sorted anymore after migration to last
Chart.js version in the commit eb530b83b06a057f96.
This feature is fixed in this commit.

task-3552948

closes odoo/odoo#139676

Signed-off-by: Mathieu Duckerts-Antoine (dam) <dam@odoo.com>
2023-10-30 12:05:20 +00:00
Tiffany Chang (tic) 7df50314b5 [FIX] mrp: use correct qty variable/term
A "quantity_done" removal was missed in the MO Overview report, which
made it so the report would throw an error when opened for a done MO.

Also replace "Reserved" with "Quantity" in the overview so that
it is more consistent with pickings + is more intuitive since it is
currently confusing that when the qty produced is updated, then it
appears that the qty reserved changes (rather than a qty being done)

closes odoo/odoo#140246

Signed-off-by: William Henrotin (whe) <whe@odoo.com>
2023-10-30 09:01:05 +00:00
oco-odoo ef055095df [FIX] account: don't loop forever when trying to expand a sum_children aggregation
This happened when auditing an aggregation line with a sum_children formula. A test was added in enterprise to prevent that to occur again in the future.

closes odoo/odoo#140200

X-original-commit: 1b24098676c1334666fc11b9973f76f4d200fa27
Related: odoo/enterprise#49783
Signed-off-by: Olivier Colson (oco) <oco@odoo.com>
Signed-off-by: John Laterre (jol) <jol@odoo.com>
2023-10-30 09:01:03 +00:00