Commits · 6088424b9b48c1fd81724115a4753229f7a762b7 · Coopdevs / Odoo / Coopdevs OCB mirror

Feb 09, 2023

ayushshawnfrost authored 2 years ago


closes odoo/odoo#112259

Signed-off-by: Martin Trigaux (mat) <mat@odoo.com>

6088424b

Feb 08, 2023

[FIX] website, *: fix some components in case of contrasted boxed layout · 14c985af

qsm-odoo authored 2 years ago

*: website_slides

In some cases, components had dark text over dark background (or light
text over light background) by mistake.

Example:
- Enter edit mode.
- In the theme tab, choose "boxed" as page layout.
- A color picker appears below to control the color behind the box.
- Set it to a dark color (if your box main color is light)
- Go to a course page (install website_slides)
- Check the mobile version
=> The bootstrap tab and its section uses the dark color you set up as
   body color instead of the expected boxed layout color.

Another example:
- Do the same thing (set up a dark color behind a boxed layout).
- Go to a shop / product page.
=> The inputs are dark with dark text.

This is because of bootstrap which uses `$body-bg` as default value for
other variables, such as `$nav-tabs-link-active-bg` in the first case
described above. It also uses the variable in the creation of CSS rules
not controlled by explicit variables.

In 16.0, bootstrap was updated to 5.1.3 with [1] and this actually
increased the problem: input backgrounds now default to `$body-bg`,
amongst other things. Since [2], `$body-bg` is also used as the default
color for range thumbs.

In previous versions, this fix focused on fixing a critical component:
nav-tabs, for which the fix was straightforward.
Starting from 16.0, this commit will fix everything at the small risk of
changing the `$body-bg` variable meaning in the case of boxed layouts.
Before this commit, its meaning was "the color used for the background
behind the boxed layout (the <body> background color)", so equal to the
Odoo value `o-color('body')`. After this commit, its meaning will be
"the color used for the background of the box itself", so equal to
`o-color('o-cc1-bg')`. The `<body>` background color will be forced by
using `o-color('body')` as the value for the related *CSS* variable
defined by bootstrap. This allows to have a correct CSS generation for
all components in case of boxed layouts: indeed, the components mix
their own color with `$body-bg` (or use it as it is) relying on the fact
this is the color which appears behind them... which was not right in
case of boxed layouts.

This commit actually fixes another bug that was found during adaptation.
It is 2-fold, and unfortunately, it does not make sense to fix one part
without the other as it would increase the problem without the other
part. The website_slides pages customize their default background color
to not be the one chosen by the user, but a mix of it with some
lightgray. Odoo default for the body being white, this makes it a
lightgray for website_slides pages. This is totally ok... but only in
"full" layout. In boxed layout, we have the 2-fold problem:

A. The mixed color is not applied to the boxed layout but on the
   background behind the box. So if you have a white box above a black
   background, in website_slides pages you won't have the black
   background you expected to keep but a lighter version of it and the
   website_slides box will not use the lightgray but stay white
   (creating other inconsistencies as the lightgray would also be used
   by other components like tabs, for that app only).

B. The mixed color is actually not mixing the right colors: it mixes
   the hardcoded lightgray with the color of the background behind the
   box, while it was intended to be the one of the content (the one of
   the box), like in "full" layout.

The changes explained above about `$body-bg` naturally fixes (B). Not
fixing (A) at the same time would result in a big change for the color
which is behind the box. This commit fixes it at the same time by now
applying the color to the right element. In previous version, this could
be fixed as well but would require a different fix (not relying on
`$body-bg`). So it makes sense to merge this first and backport+adapt.

[1]: https://github.com/odoo/odoo/commit/971e5a91aab96d36129a823e03f1f9f1b1293968
[2]: https://github.com/odoo/odoo/commit/46e53879749be7ba3d30338d0f25c0a68a88eb3c



opw-3151962

closes odoo/odoo#112136

X-original-commit: 03f238aa
Signed-off-by: Romain Derie (rde) <rde@odoo.com>
Signed-off-by: Quentin Smetz (qsm) <qsm@odoo.com>

14c985af

[REF] web_editor: prevent concurrency issue in resetEditor · 02900593

Nicolas Bayet authored 2 years ago


The PR #105579 introduce the call of `setupCollaboration` in
`resetEditor` but did not remove the instructions in `resetEditor`
that will be called in `setupCollaboration`.

`this._getNewPtp` should be called by `setupCollaboration` as it is
called after an asynchronous call.

Additionally, `this._peerToPeerLoading` has to be awaited to prevent
concurrency issues.

closes odoo/odoo#110150

Signed-off-by: David Monjoie (dmo) <dmo@odoo.com>

02900593

[FIX] web_editor: prevent having wrong connectors when an icon change · 93eeaf53

Guillaume (gdi) authored 2 years ago

Steps to reproduce the bug:
- Drop a block steps on a page
- Change one of the icons
- Change the color of the icon

=> The connectors are not correct anymore.

This is due to [this commit] changing the tag `<i class="fa...` to a
`<span class="fa...` when the user replaces an icon. The steps options
assume that `<i/>` tags will remain `<i/>` tags (as it was before
[this commit]). We should not change the tagName when changing the icon
of an `<i/>` tag. This is likely to break (s)css, options, public
widgets, ... This commit fixes this bug by ensuring that `<i/>` tags do
not become `<span>` tags when replacing the icon. Moreover it allows not
to recreate an editor when you change the icon.

[this commit]: https://github.com/odoo/odoo/commit/7fd0698cf765a79959566b51e33cb76bff83d344



task-3068834
opw-3123850

closes odoo/odoo#106316

Signed-off-by: Outagant Mehdi (mou) <mou@odoo.com>

93eeaf53

[REM] web_editor: remove useless _shouldDelayBlur fix · 0dd9da02

Nicolas Bayet authored 2 years ago


The fix that introduced `this._shouldDelayBlur = true` in
`openMediaDialog` was wrong as it was fixing a symptom rather than
the source of the problem. Later fix probably fixed the source of the
problem as the problem is not reproducible at the time of this commit.
If the symptom comes back, it ought to be fixed at it's source.

closes odoo/odoo#112216

Signed-off-by: David Monjoie (dmo) <dmo@odoo.com>

0dd9da02

[FIX] base: don't fail on empty groups attribute · 4379dce9

Alvaro Fuentes authored 2 years ago


In 15.0 this was supported. It may be also handy when editing views to
momentarily set the groups to `""`.

Steps to reproduce:
1. Install Odoo 15 locally
2. Edit or create a view with `groups=""` for some component
3. Upgrade to 16.
It fails.

Empty groups was allowed in 15.0 we want to ensure this is not broken
unintentionally anymore, such a new test was added.

Muted logged to hide the warning (also present in 15.0):
```
2023-02-08 11:09:18,697 506777 WARNING test_16_gr odoo.addons.base.models.ir_ui_view: The group '' defined in view does not exist!
View error context:
{'file': None,
 'line': 3,
 'name': 'foo',
 'view': ir.ui.view(242,),
 'view.model': 'res.partner',
 'view.parent': ir.ui.view(),
 'xmlid': ''}
 ```

closes odoo/odoo#112062

Signed-off-by: Denis Ledoux (dle) <dle@odoo.com>

4379dce9

[IMP] account: Remove 'move_id.state' dependency of _compute_amount_residual · 00aadcb6

Laurent Smet authored 2 years ago


The reconciliation fields of account.move.line should be independent of the move's state.

closes odoo/odoo#111962

Co-author: Habib (ayh) <ayh@odoo.com>
Related: odoo/enterprise#36815
Signed-off-by: Brice Bartoletti (bib) <bib@odoo.com>

00aadcb6

[FIX] account: revert commit · 63c9e9cf

Laurent Smet authored 2 years ago

This case of amls having a zero balance but an amount in foreign currency left is already managed by the new reconciliation in 16.0.
Furthermore:
-the test in POS is passing even with the revert
-the reverted commit introduces a performance issue on amount_residual by forcing the orm to fetch the whole m2m debit/credit_matched_ids.
-there is no test accounting-side depicting the issue (if there is really an issue fixed by the commit).

Part-of: odoo/odoo#111962

63c9e9cf

[IMP] google calendar: auto accept meeting when only creator is attendee · 3fcadfd9

Gabriel de Paula Felix (gdpf) authored 2 years ago

Before this commit, when an event containing only the creator was created in Google Calendar, the creator's attendee status in Odoo was marked as 'needsAction'. Now, when an event like this is created (in Google Calendar), the event is automatically accepted in Odoo.

A test was added to make sure that new events with only the creator are automatically accepted.

closes odoo/odoo#112179

Task: 2782961
Related-to: #68700
X-original-commit: 6d0ead3d73477614dc1bd97a95b695f80925fdd3
Signed-off-by: Arnaud Joset <arj@odoo.com>

3fcadfd9

[FIX] http: no rotate sid for unidentified user · d5d80d17

Florian Vranckx authored 2 years ago

This commit fixes a change in behavior between 15.2 and 15.3.

Previously, if an unidentified user tried to reach a route that had auth='user', it would simply redirect to the login page.

Currently, it redirects and invalidates the session_id.

This is an issue in the latest version of master after this PR https://github.com/odoo/enterprise/pull/36521

This commit changes the route of service-worker.js to auth='user'.

This route is called on the login page, which rotates the sid and therefore invalidates the csrf token. Making it impossible for a user to log in.

This is a race condition, meaning it would only appear if the user stayed on the login page for a few seconds, hence why the automated testing did not block the commit.

closes odoo/odoo#112169

Signed-off-by: Julien Castiaux (juc) <juc@odoo.com>
Co-authored-by: Julien Castiaux <juc@odoo.com>

d5d80d17

[FIX] account: show inactive warning only when currency is set · 1937e495

niyasraphy authored 2 years ago


before this commit, in active currency warning is shown in the account move form when the currency_field is empty.

after this commit, the inactive currency warning will be shown only if the currency is set and it is inactive.

forwardported manually

closes odoo/odoo#111863

X-original-commit: 0bb74026
Signed-off-by: Brice Bartoletti (bib) <bib@odoo.com>

1937e495

[FIX] point_of_sale: skip synchronize journal entries in PoS · 890b685e

pedrambiria authored 2 years ago

The problem happens when the company is Angelo Saxon, and
`l10n_eg_edi_eta` is installed. Also, the product should have a cost and
automatic inventory valuation.
https://github.com/odoo/odoo/commit/4b430f8e30efb15f06982e4673cd73531a2119b3


The problem is that with this commit, after calling the
`_set_next_sequence` function, it calls `flush_recordset`. It would cause
a write to be triggered and add an Automatic Balancing Line. While in
`_create_account_move` of PoS the stock output line will add later to it,
and make it unbalanced. The first Automatic Balancing Line is incorrect as
it will call `_create_stock_output_lines` later and it adds a line which
makes it balance. But with that Automatic Balancing Line as it's
unbalanced at the end it will call `_close_session_action` in PoS and
shows a pop-up to add another line to make it balance.

The solution is skip synchronize of journal entries.

opw-3137723

closes odoo/odoo#110956

Signed-off-by: Trinh Jacky (trj) <trj@odoo.com>

890b685e

[FIX] web: kanban view's quick create flickering · fa20b8de

Jorge Pinna Puissant authored 2 years ago


Before this commit, when creating a new record using the quick create on
a kanban view, a flickering was visible. We can see the quick create
form disappear, the record list goes up, the quick create form re-appear
and the record list goes down again.

This occurs because, the record list contains an empty record that is
used on the quick create form. When clicking on the add button on the
quick create form, this record is directly saved, and as is already on
the list (the quick create form disappear and the record list goes up),
only after this action, a new empty record is created and added to the
list (the quick create form re-appear and the record list goes down
again).

Now, an empty record is created independently of the list, so when we
click on the add button on the quick create form, the record is saved,
a new empty record is created (the quick create form is emptied), and
the saved record is added to the list (the record list got the new saved
record), avoiding the flickering.

task-id=3085247

closes odoo/odoo#111818

Signed-off-by: Aaron Bohy (aab) <aab@odoo.com>

fa20b8de

[FIX] web_editor: set recovery connection to 3s · 43f6ea6c

Nicolas Bayet authored 2 years ago

When trying to recover peer to peer connection, the timeout of 500ms or
1000ms might be too short and 15000ms in unnecessary too long.

If the timeout is too short, it could remove the connection before the
peer to peer had the time to reconnect to it. When we call
`_recoverConnection` on `disconnect` in `oniceconnectionstatechange`
or `onconnectionstatechange`, the signal `disconnect` does not
necessarily mean that it will remain disconnected. Doing nothing could
be enough to reconnect.
(see: https://w3c.github.io/webrtc-pc/#dom-rtcicetransportstate-disconnected

)

closes odoo/odoo#112170

X-original-commit: 3647374c
Signed-off-by: David Monjoie (dmo) <dmo@odoo.com>

43f6ea6c

[FIX] web_editor: properly add iceCandidateBuffer · 376bc44b

Nicolas Bayet authored 2 years ago

Before this commit, the iceCandidateBuffer was emptied before finishing
the loop. This is wrong as we must add all candidates from the buffer.

X-original-commit: 520a8805
Part-of: odoo/odoo#112170

376bc44b

[FIX] core: avoid losing translations during module upgrade · 1e6e482a

Chong Wang (cwg) authored 2 years ago


Consider field F in module X with parameter translate=False, and F is
overridden in module Y with parameter translate=True.  During the
upgrade of module X, module Y hasn't been loaded yet, and the ORM
considers the field to be `translate=False`.  Therefore it converts its
database column from type jsonb to varchar, and accidentally drops
non-en_US values:

    {"en_US": "English value", "fr_FR": "French value"} (jsonb)
        -> 'English value' (varchar)

As a result, translations are lost after upgrade.

This commit fixes the bug by checking whether the field is translated in
database and patches the field accordingly when loading the registry.
This avoids the ORM considering the field as non-translated while
upgrading modules.

In order to "force" translated fields to become non-translated ones, at
the end of the loading process the patch above is discarded, and fields
are checked again.  We then adapt the schema of models that have such
fields.  This extra step handles the uninstallation of modules like
module Y in the example above.

The patching of the fields has one potential issue.  While upgrading
module X, field F is patched with translate=True.  If module Y actually
overrides F with translate=xml_translate or so, this may cause the
behavior of the upgrade to be slightly incorrect.  Because of the
complexity, we have chosen to not support this case.

closes odoo/odoo#110572

Signed-off-by: Raphael Collet <rco@odoo.com>
Co-authored-by: Raphael Collet <rco@odoo.com>

1e6e482a

[FIX] website: add website field in ir_asset list view · aa9805d4

Romain Derie authored 2 years ago

For some reason the `website_id` field was added in the form view of the
`ir.asset` model in a website module overide but it was not done for the
list view where it matters equally (if not most regarding the flow).

Indeed, those views / this model is mainly accessed for debugging
purpose in which case you are most likely looking for a specific asset.
In the website case, it's most of the time to find the custom asset
that was created following a scss customization in the right panel of
the website builder.
In such a case, it will have a website_id and will be easy to find in
the list view.

It's also the case for all the records having a `website_id`, we show
that field in both form and list view, it's always important when
managing / debugging DBs in multi-website environment.

See [1] for introduction of `ir.asset`.

[1]: https://github.com/odoo/odoo/commit/8cc066173dfb61bd95b8e1f0716f71f4e251810a



closes odoo/odoo#112139

X-original-commit: 1e6ea1a1
Signed-off-by: Quentin Smetz (qsm) <qsm@odoo.com>

aa9805d4

[FIX] pos_restaurant: giftcard and rewards · d5a64bb6

David Monnom (moda) authored 2 years ago


Previously, order lines containing gift cards or rewards disappeared when the order was closed and reopened in the pos_restaurant module

The following adaptations have been applied:

- Added a backend function in the pos_loyalty module to return active coupons (gift cards) to the frontend
- Added a frontend function in the pos_loyalty module to remap coupon_id to couponPointChanges and order lines
- Added this.partner in the init_from_json function in the pos_loyalty module to avoid deleting reward order lines

Now, command lines don't disappear when switching from one command to another.

closes odoo/odoo#111864

Signed-off-by: Trinh Jacky (trj) <trj@odoo.com>

d5a64bb6

[IMP] hr: simplify and correct the default data · dcc8bbf0

Rob Coekaerts authored 2 years ago


Some changes were made to the default/demo/test data to be more
consistent:
  - Duplicates of "Paid Time Off" time off types are consolidated into
    one type (inluding year specific versions, ig "Paid Time Off 2019"
  - Annual Time Off is renamed back to Paid Time Off (both for the
    work entry type as the time off type) to be consistent everywhere.
  - All mentions of years in work entry types and time off types have
    been removed, as this is no longer relevant with the new allocation
    rules.
  - Time off types in the default data have been explicitely made
    company agnostic, in order for them to be available to all companies
    and not just the one company that was select when installing
    hr_holidays. This was already the case for the be_payroll data, but
    not for the standard hr_holidays ones.
  - Various small cosmetic / functional fixes and simplifications
    (eg deduplication of data)
  - expense_other_input has been made country agnostic, in order for it
    to be available in all countries.

task-2978513

closes odoo/odoo#112173

Related: odoo/enterprise#36796
Signed-off-by: Kevin Baptiste <kba@odoo.com>

dcc8bbf0

[FIX] spreadsheet,web: use dot or comma in date format · 5f17d897

MerlinGuillaume authored 2 years ago


Date fields are changing format if you type it in instead of using the
date selector calendar popup

Steps to reproduce:
1. Install Time Off
2. Open the current language and change the date format to `%d.%m.%Y`
3. Go to Time Off > Approvals > Allocations
4. Create a new allocation
5. Change the validity period to 10.03.2023 (by typing it in, not using
   the datepicker) and click out of the field
6. The date displayed is changed to 2010/03/20 or 20.03.2010 (if the
   datepicker was opened)

Solution:
Add dot and comma as a possible character for static format
Also deduplicated function isValidStaticFormat so we have a single
definition

Problem:
Formats using dots were not considered as valid static format so the
value entered was parsed with the format `yyyy/MM/dd` instead

opw-3081268

closes odoo/odoo#111862

Signed-off-by: Rémi Rahir (rar) <rar@odoo.com>

5f17d897

[FIX] web: oe_title + oe_avatar alignment in Form view · 45ebcbf3

Pierre Paridans authored 2 years ago


This commit fixes a broken selector when an avatar image (`.oe_avatar`)
is used next to a title element (`.oe_title`) to put them side-by-side.

Also it goes beyond that by ensuring the `.oe_title` element doesn't
take the whole width (cf. max 75%) to make a visual distinction between
it and the other form's fields, both in editable and non-editable form.

In small screen, as space is limited, the whole width is kept to make it
more userfriendly, only allowing a thin gap between the `.oe_title` and
the `.oe_avatar`.

Steps to reproduce:
- Open Apps
- Choose an App and, in the dropdown, click on Module Info
=> there is a huge gap at the top of the title

closes odoo/odoo#110431

Signed-off-by: Adrien Dieudonné (adr) <adr@odoo.com>

45ebcbf3

[FIX] web_editor: join peer to peer on focus in resetEditor · 12a4fbe1

Nicolas Bayet authored 2 years ago


The PR #110241 inadvertently re-introduce the attempt to connect
in collaboration when the editor was reset. The attempt should only
be made when the user focus on the editable.

The line
`this.$editable[0].addEventListener('focus', this._joinPeerToPeer);`
in `resetEditor` will handle that case.

closes odoo/odoo#112167

Signed-off-by: David Monjoie (dmo) <dmo@odoo.com>

12a4fbe1

[FIX] web: display "invite" button label on one line · fa258ab9

Audric Onockx (auon) authored 2 years ago


Settings > General Settings > Section Users:
Button Invite behaves in a weird way.

In dutch, e.g., "Nodig uit" splits into 2 lines,
whereas it could expand, eating out a little bit of the
input's space.

closes odoo/odoo#112161

Signed-off-by: Jorge Pinna Puissant (jpp) <jpp@odoo.com>

fa258ab9

[FIX] repair: prevent cancelling a completed repair order · d8fa6d0c

Touati Djamel (otd) authored 2 years ago


Steps to reproduce the bug:
- Create a repair:
    - Add any product to repair
    - Add any other product as part of the repair
    - Confirm the repair
    - Start the repair
    - End repair

Problem:
The cancel button becomes visible and clicking on it won't cancel the
moves, so it doesn't make sense to cancel a finished repair.

opw-3146606

closes odoo/odoo#112119

X-original-commit: b21bbcef
Signed-off-by: William Henrotin (whe) <whe@odoo.com>
Signed-off-by: Djamel Touati (otd) <otd@odoo.com>

d8fa6d0c

[FIX] hw_*: Modify the default user for iot in windows · 36c03128

lejeune quentin authored 2 years ago


currently there is an error in the configuration
of the default user for the iot windows
it is configured for "local service"
which is the French version of the local user.
Except if the pc window is not in French,
the user is not configured correctly.
We have to use the default local user name
of windows which is "LOCALSERVICE"

closes odoo/odoo#112129

Signed-off-by: Quentin Lejeune (qle) <qle@odoo.com>

36c03128

Feb 07, 2023

[FIX] web_editor: write with string in update_checklist · 18f25070

Nicolas Bayet authored 2 years ago


Before this commit, when the user tried to update a checklist, there
was a traceback in `handle_history_divergence` because the `value` is a
byte sequence instead of a string.

opw-3158660
task-3165844

closes odoo/odoo#112098

X-original-commit: e3900dfe
Signed-off-by: David Monjoie (dmo) <dmo@odoo.com>

18f25070

[FIX] mail: fix y overflow in chatter · 3a40fa55

Adrien Minne (adrm) authored 2 years ago


For some font families, there was an y overflow and a scrollbar in the
chatter when inserting an @mention. This was because the font height +
the padding of `a.o_mail_redirect` was greater than the line height.

Added an `overflow-y: hidden;` in the css to fix the issue.

closes odoo/odoo#112096

Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>

3a40fa55

[FIX] web_editor: force custom style in ui on non-custom link with style · 7656e16f

Antoine Guenet authored 2 years ago

This reverts commit [1] so as to avoid updating the DOM in two separate
places of the code for link edition. Said commit intended to prevent the
removal of custom styles on a button on which styles were applied
(erroneously) without applying a custom button class. This fixes that
issue by adapting the UI to show the button is custom if there are
custom styles applied. This way, if any change is applied, when
modifying the DOM, the custom class will be applied as well. This is the
order in which such corrections are usually applied with the snippets
bar.

[1]: https://github.com/odoo/odoo/commit/eb4edac560227b46efdcfe958e3b93e0b88def64



closes odoo/odoo#112086

Signed-off-by: Quentin Smetz (qsm) <qsm@odoo.com>
Signed-off-by: Antoine Guenet (age) <age@odoo.com>

7656e16f

[FIX] mass_mailing: properly disable wrong link color option · 3e4409fc

Antoine Guenet authored 2 years ago

The background color snippet option contains a superfluous selector. To
make sure the option doesn't actually appear for that selector, without
modifying the XML file in stable, commit [1] hid it in those cases. But
we want the option not to even be instantiated, which this commit
achieves by modifying the selector when computing the snippet templates.

[1]: https://github.com/odoo/odoo/commit/925e73e02c5bc054d06fc2097acc9e4244cabe38

Part-of: odoo/odoo#112086

3e4409fc

[FIX] website_sale: show the list of products from the price range selected · 2d34cd6f

Eteil Djoumatchoua (etdj) authored 2 years ago

Steps to reproduce:
1. Go to website shop page
2. Change the price sliders so no products are shown
3. Increase the range of the sliders

Issue:
Traceback

```python
UncaughtPromiseError > TypeError
Uncaught Promise > Cannot read properties of null (reading 'classList')
TypeError: Cannot read properties of null (reading 'classList')
at Class._onPriceRangeSelected (https://23198922-16-0-all.runbot173.odoo.com/web/assets/debug/1/web.assets_frontend_lazy.js:121254:70) (/website_sale/static/src/js/website_sale.js:1200)
at HTMLInputElement.<anonymous> (https://23198922-16-0-all.runbot173.odoo.com/web/assets/debug/1/web.assets_frontend_lazy.js:72834:23) (/web/static/src/legacy/js/core/mixins.js:277)
at HTMLDivElement.dispatch (https://23198922-16-0-all.runbot173.odoo.com/web/assets/debug/1/web.assets_frontend_lazy.js:26934:27) (/web/static/lib/jquery/jquery.js:5183)
at elemData.handle (https://23198922-16-0-all.runbot173.odoo.com/web/assets/debug/1/web.assets_frontend_lazy.js:26742:28) (/web/static/lib/jquery/jquery.js:4991)
at Multirange.dispatchNewValueEvent (https://23198922-16-0-all.runbot173.odoo.com/web/assets/debug/1/web.assets_frontend_lazy.js:122060:24

) (/website/static/lib/multirange/multirange_custom.js:258)
```

Cause:
When the list of products is empty, the div containing it is no longer present. So when we try to get it with querySelector() it returns null.

Solution:
Ensure the div is present in the DOM to modify it

opw-3144545

closes odoo/odoo#111486

Signed-off-by: Yannick Tivisse (yti) <yti@odoo.com>

2d34cd6f

[FIX] l10n_fr_pos_cert: default timezone · 694de9d9

asno-odoo authored 2 years ago


When mocking views, if there is no timezone set in context and for user,
providing Boolean value to pytz.timezone() functions results in error.
It doesn't happen in UI as UI takes value from session timezone.

closes odoo/odoo#111329

Signed-off-by: William André (wan) <wan@odoo.com>

694de9d9

[FIX] web: add DomainSelector debugValue default value · bec8da1d

Stefan-Calin Crainiciuc (stcc) authored 2 years ago


Steps to reproduce:

- Install Events
- Activate debug mode
- Events > Config > Lead generation > Create

Traceback is raised.

The `debugValue` prop added in a29691bf should be a string but doesn't
have a default value, and it is evaluated as `false` at runtime.

Solution:

Add default value for `debugValue`.

opw-3056391

closes odoo/odoo#109238

Signed-off-by: Michaël Mattiello <mcm@odoo.com>

bec8da1d

[FIX] web_editor,project,note: ensure same history · 1b8d663c

Nicolas Bayet authored 2 years ago


This commit add a mechanism to ensure that someone could never save
changes from an history that diverge (in case there is a partition in
the RTC network or a person A was disconnected while another person B
saved changes that were not transmitted to person A).

task-3002163

closes odoo/odoo#110241

Signed-off-by: David Monjoie (dmo) <dmo@odoo.com>

1b8d663c

[FIX] website_hr_recruitment: add slug to jobs' website_url · 32a3de4b

qsm-odoo authored 2 years ago

This follows the merge of [1].

Model implementing the `_compute_website_url` method sometimes adds the
"base url" part which is probably not necessary and sometimes forgot to
add the "slug" part. This commit only fixes the website_hr_recruitment
job's one to add the slub. The others will be fixed later, this should
probably be part of a deeper refactoring to use routes with Model
validation rather than accepting any int and reviewing when the base
URL is actually necessary.

Notice that the main point of this change is that the `website_url`
field as some models now shown in the "Site" menu of the website app.
In that case, when clicking on a record, the user is redirected to the
iframe preview, with the right website and domain without the need for
it to be part of the object's URL.

[1]: https://github.com/odoo/odoo/commit/da3f4c2aff92cca23ca2d7a87fa0eb5b1d7ce5fe



closes odoo/odoo#99959

Signed-off-by: Romain Derie (rde) <rde@odoo.com>

32a3de4b

[FIX] website_event : check ticket id · 149d96a2

Florian Vranckx authored 2 years ago


There was previously no check that the ticket id belong to the selected event.

For better data integrity, this should be the case

closes odoo/odoo#111744

X-original-commit: e3ff6991c78b8e35834492a9d9a883cd99c11a3b
Signed-off-by: Vranckx Florian (flvr) <flvr@odoo.com>

149d96a2

[FIX] account: can't get previous move · 5b953364

Florent de Labarre authored 2 years ago


In case of a user have no access to all account.move (restrict by ir.rules). An exception can raise during move posting.

Add an index to speed up this function in large database.

OPW #3159258

closes odoo/odoo#112027

X-original-commit: b2e3c58a
Signed-off-by: Nicolas Viseur (vin) <vin@odoo.com>

5b953364

[FIX] account: prevent change restrict mode hash · 0004a86b

Florent de Labarre authored 2 years ago

In case of an user can edit account.journal and have no access of all account.move of this journal, the user can set to false this field : restrict_mode_hash_table.

closes odoo/odoo#112021

X-original-commit: c1bcf20a
Signed-off-by: Nicolas Viseur (vin) <vin@odoo.com>

0004a86b

[FIX] l10n_be: tax report: fix wrong carryover for grids 82, 83, 86, 87 and 88 · 92fa3f28

oco-odoo authored 2 years ago

Those lines were badly configured. The proper setup is the one already used on grid 81. Without this, we're also considering entries from the previous periods (because of the date_scope), and just summing everything while we actually only want the last one.

closes odoo/odoo#112017

Signed-off-by: John Laterre (jol) <jol@odoo.com>

92fa3f28

[FIX] web: update moment.js to 2.19.3 · 11f3e735

Jorge Pinna Puissant authored 2 years ago


This version of moment.js is required to avoid CVE-2017-18214

task-2291793

closes odoo/odoo#111986

Signed-off-by: Aaron Bohy (aab) <aab@odoo.com>

11f3e735

[FIX] pos_adyen: prevent having the same Adyen in multi-company · 8a9d7ddf

pedrambiria authored 2 years ago


Before this commit: it was possible to create two PoS terminals in two
companies with the same `adyen_terminal_identifier`. So it's possible,
after each payment, it would add the response to the wrong payment
method's `adyen_latest_response`.

The solution is to bypass multi-company record rule in
`_check_adyen_terminal_identifier`.

opw-3131814

closes odoo/odoo#111982

X-original-commit: 5af4ffa9
Signed-off-by: Trinh Jacky (trj) <trj@odoo.com>
Signed-off-by: Pedram Bi Ria (pebr) <pebr@odoo.com>

8a9d7ddf