- Oct 22, 2020
-
-
Benjamin Vray authored
This commit align vertically the copyright text and the inline language switcher items. This commit also add a padding on list-inline-item in the language selector. Because before this commit, there was no space between lines if the list was to be displayed on multiple lines. Part of #59581 task-2312878 Signed-off-by:
Quentin Smetz (qsm) <qsm@odoo.com>
-
Benjamin Vray authored
Before this commit, the add language separator was visible at the end of the inline language switcher even if the add language link was not displayed. Part of #59581 task-2312878
-
- Oct 13, 2020
-
-
Bruno Zanotti (ADHOC) authored
closes odoo/odoo#59030 Signed-off-by:
Martin Trigaux (mat) <mat@odoo.com>
-
- Oct 22, 2020
-
-
wan authored
The fix c710c3a3 has been forgotten while applying https://github.com/odoo/odoo/pull/43768 closes odoo/odoo#60492 X-original-commit: c5d355444a3dde2635025e5452360e853afca099 Signed-off-by:
Laurent Smet <smetl@users.noreply.github.com> Signed-off-by:
wan <william-andre@users.noreply.github.com>
-
- Oct 21, 2020
-
-
IEL authored
TLDR: the videos doesn't work anyway, but nobody noticed it for years; at the same time there is no easy way to fix it STEPS: * create a record with html field, e.g. project.task * add an embedded video (File/Image icon) * save BEFORE: video doesn't work AFTER: there is no option to add embedded video anymore WHY: * iframe tags are removed before storing html in database, so after saving a record, embedded video doesn't work anymore * possible fixes look too tricky for stable branches: * new field attribute sanitize_iframe=False -- requires changes in core python framework which may have side effect * adding iframe on client side -- the problem is that the backend doesn't have mechanism to use animation in html fields like it's done in website: https://github.com/odoo/odoo/blob/ced500e9bcbb5c84033463689d135ad4c7dc4f39/addons/website/static/src/js/content/snippets.animation.js#L623-L632 * adding iframe on server side -- parsing html fields in _read... Doesn't look good either * double selector is required to make it work both in frontend and backend: $editable.closest('.o_editable, .note-editor') * we don't need hardcoded exception for mailing.mailing, mail.compose.message because those fields marked as sanitized (default value) https://github.com/odoo/odoo/blob/c92f058571b0656451f24e24154dd1bdb3bdb48a/odoo/fields.py#L1589-L1590 https://github.com/odoo/odoo/blob/c92f058571b0656451f24e24154dd1bdb3bdb48a/addons/mail/wizard/mail_compose_message.py#L115 https://github.com/odoo/odoo/blob/61734cdaf1cf6857125b1ece63928ab080359c6c/addons/mass_mailing/models/mailing.py#L77 * I don't apply this for Odoo 12, because js is refactored in Odoo 13 (e.g. field_html.js has another name in Odoo 12) --- opw-2353103 closes odoo/odoo#60474 X-original-commit: 2dcbac08effcedafbcf50610355752030902b16d Signed-off-by:
Nicolas Lempereur (nle) <nle@odoo.com> Signed-off-by:
Ivan Yelizariev // IEL <yelizariev@users.noreply.github.com>
-
Laurent Smet authored
A tax line could have something in the `tax_ids` field. Then, these condition needs to be inversed in order to not handle them as base lines. closes odoo/odoo#60469 X-original-commit: eada9f91f3361f8335c23b3a27161803cfda6608 Signed-off-by:
oco-odoo <oco-odoo@users.noreply.github.com> Signed-off-by:
Laurent Smet <smetl@users.noreply.github.com>
-
Swapnesh Shah authored
Before this commit, Going to Website --> Configurations --> Apps was displaying Modules related to Website only but the default focus on searchpanel was on "All". (https://tinyurl.com/y5tw5uob) Now, The default focus of searchpanel is on Website Category. (https://tinyurl.com/y6984hl7 ) closes odoo/odoo#60479 Signed-off-by:
Jérémy Kersten (jke) <jke@openerp.com>
-
Jeremy Kersten authored
Courtesy of sts closes odoo/odoo#60454 X-original-commit: bc2dc85c Signed-off-by:
Jérémy Kersten (jke) <jke@openerp.com>
-
Swapnesh Shah authored
Before this commit, Field "No of Views (visits)" was editable on multi_edit mode. Field "No of Views (visits)" should be readonly for Obvious reason. closes odoo/odoo#60483 Signed-off-by:
Jérémy Kersten (jke) <jke@openerp.com>
-
Nicolas Martinelli authored
- Create companies A & B - Create a Dropship route for B - Create a product P with route 'Dropship' - Set the website under company A - As a portal user, buy the product P on the website - Do the payment The SO is confirmed but the Dropship route for B is used despite the fact that the website is under company A. This happens because `_search_rule` is called as superuser: therefore, routes from all companies are retrieved. To prevent this, we add the company in the domain. opw-2349094 closes odoo/odoo#60475 X-original-commit: 51b7960f Signed-off-by:
Nicolas Martinelli (nim) <nim@odoo.com>
-
Nasreddin (bon) authored
Issue - Install "Website" app - Go to "Contact us" page on website - Edit Page, then click on submit button to edit form style - Alter 'Recipient Email' by selecting a value from 'autocomplete' values Other input text options are also altered. Cause Due to chrome autocompletition, others text input fields are also altered (like it will do with a form address). In this case, the error was obvious since it alter also the 'Labels Width' who break the layout. Solution Disable autocompletition on `<input type='text'/>` options elements. opw-2361169 closes odoo/odoo#60470 X-original-commit: 978788bbdf1a4a3d563fd70e2af89fdd1fd039ec Signed-off-by:
Nicolas Lempereur (nle) <nle@odoo.com> Signed-off-by:
bon-odoo <nboulif@users.noreply.github.com>
-
Victor Feyens authored
Recenlty, this method was changed to use set instead of recordsets to improve the performance of the code. But set have a non deterministic order, which means the order of the returned programs returned is not the same, modifying the resulting behavior as some programs cannot be applied together. To fix this problem, this commit replaces the set by lists to ensure the returned recordset keeps the same ordering. Also rename a potentially conflicting variable caused by the same commit. Finetuning of commit cc1bb922. closes odoo/odoo#60468 X-original-commit: 6d4da47f Signed-off-by:
Victor Feyens (vfe) <vfe@odoo.com>
-
Nasreddin (bon) authored
Issue - Install "Website" app - Go to "Contact us" page on website - Edit Page, then click on submit button to edit form style - Alter 'Recipient Email' then save - Edit Page and click on submit button 'Recipient Email' have default value. Cause If a field is required, the default value is set without checking if a value is already set. Solution Set current value if available, else default value. opw-2361169 closes odoo/odoo#60452 X-original-commit: e29d2818abf0e7902feb26c9e0b4809c9a3351c6 Signed-off-by:
Nicolas Lempereur (nle) <nle@odoo.com> Signed-off-by:
bon-odoo <nboulif@users.noreply.github.com>
-
- Oct 20, 2020
-
-
Achraf (abz) authored
Issue - Change country to Mexico - Install 'CRM' - Install 'l10n_mx_edi' - Try to open any lead in 'CRM' The same error is present in l10n_pe Cause 'l10n_mx_edi' and 'l10n_pe' use 'res.partner' in 'ir.ui.view' Solution Adding a condition to prevent '_fields_view_get_address' in 'format.address.mixin' from rewritting the model and let ir.ui.view determine it opw-2361875 closes odoo/odoo#60329 Signed-off-by:
Laurent Smet <smetl@users.noreply.github.com>
-
Antoine Prieels authored
The bottom of long receipts was missing when printing through the browser printing dialog. opw-2362522 closes odoo/odoo#60375 Signed-off-by:
pimodoo <pimodoo@users.noreply.github.com>
-
- Oct 21, 2020
-
-
Nicolas Martinelli authored
This reverts commit c382e8c8. closes odoo/odoo#60428 X-original-commit: 84ec7277 Signed-off-by:
Nicolas Martinelli (nim) <nim@odoo.com>
-
Nicolas Martinelli authored
- Create a partner outside Europe - Set a VAT number - Create an invoice for the partner - Post the invoice The `IdPaese` and `IdCodice` is obtained from the VAT number, but it is not correct for partners outside Europe: the VAT number should always be `OO99999999999`. A workaround is to set the VAT number of the partner to `XXOO99999999999`, where `XX` is the country code. However, in case of multi-company with shared partners, another company might need the proper VAT number. In case of a customer outside EU, we: - get the `IdPaese` from the country of the partner - set the `IdCodice` to `OO99999999999` We also add the `IdPaese` to customers without VAT. opw-2355842 closes odoo/odoo#60410 X-original-commit: 35267c59 Signed-off-by:
Nicolas Martinelli (nim) <nim@odoo.com>
-
Christophe Monniez authored
The package.py script is moving the addons in odoo/addons which is not more necessary for the Debian package as it's made at the Debian packaging level. So package.py will not do that anymore. closes odoo/odoo#60421 Signed-off-by:
Christophe Monniez (moc) <moc@odoo.com>
-
- Oct 20, 2020
-
-
Benjamin Vray authored
Before this commit, we executed the request even when creating a new badge. And it thrown a traceback. After this commit, we return default values before executing the request if this is a new badge. task-2312878 closes odoo/odoo#60347 Signed-off-by:
Jérémy Kersten (jke) <jke@openerp.com>
-
- Oct 16, 2020
-
-
Thanh Dodeur authored
This commit gives to o_Dialog the same z-index as modals, before this commit, the dialog could be hidden or partially obstructed by other elements. part of task-2171884 closes odoo/odoo#60162 Signed-off-by:
Alexandre Kühn (aku) <aku@odoo.com>
-
- Oct 21, 2020
-
-
Benjamin Vray authored
Before this commit, the width of carousel blockquotes was too small to display correctly on mobile device. Part of #55857 task-2318526 Signed-off-by:
Quentin Smetz (qsm) <qsm@odoo.com>
-
Benjamin Vray authored
Keep the original position of the background as a percentage. In this way, we avoid converting in pixel a direction that we have not changed. Part of #55857 Related to task-2318526
-
Benjamin Vray authored
-Remove arrows on mobile because it's too big and hide the content. User will use the touch screen to make the carousel move. -Bordered style: Reduce the borders on mobile. Part of #55857 task-2318526
-
Romain Derie authored
Depending of the configuration, on the portal, anchor do not consider the header, so a part of the text is hidden by the navbar. task-2290213 closes odoo/odoo#60408 X-original-commit: 2cd9a9d12f4cd7352430c166145a7488af0c120c Signed-off-by:
Jérémy Kersten (jke) <jke@openerp.com> Signed-off-by:
Romain Derie <rdeodoo@users.noreply.github.com>
-
Raphael Collet authored
The feature was actually not working as expected. It has been fixed, and a test now ensures it does work as expected. This commit also fixes some invalid parameters, hopefully nothing critical. closes odoo/odoo#60402 Related: odoo/enterprise#14273 Signed-off-by:
Raphael Collet (rco) <rco@openerp.com>
-
- Oct 20, 2020
-
-
Achraf (abz) authored
Issue - Install "Calender" - Install "Google Calendar" - Try to sync google in "Calendar" Every created events will trigger the sending mail to attendees even if the event date is past Cause 'events' was not filtered Solution Filter 'events' to get all the events that have not yet happened opw-2358123 closes odoo/odoo#60379 Signed-off-by:
Nicolas Martinelli (nim) <nim@odoo.com>
-
- Oct 21, 2020
-
-
Olivier Dony authored
There was a problem in the Transifex translations for Spanish and all translations were overwritten by mistake with english defaults. This commit reverts all modified es.po files to their previous version while we work to restore the translations on Transifex.
-
- Oct 20, 2020
-
-
Goffin Simon authored
State to reproduce the bug: - Let's consider a 2 steps delivery setting - Create a SO with a product P and a quantity Q and confirm it - Pick: Deliver a smaller quantity than Q, don't create a backorder but duplicate the Pick and validate it with the remaining quantity. - Out: the initial delivery order OUT P1 does not allow the reservation, so cancel it and duplicate it (let's call it P2). - Click on Mark to do on P2 Bug: The initial picking OUT P1 was changed in state Ready instead of keeping in state Cancelled opw:2339217 closes odoo/odoo#60390 X-original-commit: 61734cda Signed-off-by:
Simon Goffin (sig) <sig@openerp.com> Co-authored-by:
Whenrow <whe@odoo.com>
-
Benjamin Vray authored
Before this commit, when adding a new slide, the data-target of the new indicator was wrong because we were using the slide ID instead of the carousel ID. task-2318526 closes odoo/odoo#60381 X-original-commit: ac7a117ccc64df54cfd271ee4c4b62ef49dd71ff Signed-off-by:
Quentin Smetz (qsm) <qsm@odoo.com>
-
Luis González authored
There are currently some elements in views that are not reachable on a deterministic way when those views are inherited. That means, to reach them, positional selectors need to be used (e.g. `last()`). That causes inherited views to get broken when there are small changes in element position. A good example of this is [1]: <th class="text-right">Unit Price</th> Which can't be filtered by: - Class, because there are several `<th>` elements with the same class - Text content, because when content is translated, selector will fail To solve the above, this change provides IDs for several elements, to make possible/easier to reach them on a safely manner. [1] https://github.com/odoo/odoo/blob/54b40f1eb096/addons/sale_management/report/sale_report_templates.xml#L15 closes odoo/odoo#60378 X-original-commit: c4c45ff1 Signed-off-by:
Nicolas Martinelli (nim) <nim@odoo.com>
-
- Oct 22, 2020
-
-
Andrea Grazioso (agr-odoo) authored
Open POS Make some cash transaction Close POS Repeat for some days and close the session Cash statement lines will be saved with the closing date, even if the payment entries are saved with their payment date opw-2347645 closes odoo/odoo#60321 Signed-off-by:
agr-odoo <agr-odoo@users.noreply.github.com>
-
- Oct 20, 2020
-
-
nie authored
Steps to reproduce the bug: - Activate Margins in Sales > Settings - Go to the pivot view of quotations - In Measures, click Margin (%) Bug: Margin percentages are aggregated by computing the sum of the margin sub-percentages instead of using the data of the aggregated row (i.e. agg. margin / agg. total). Explanation: This is one of the flaws of the pivot view. The best way to solve this would be to create a custom aggregation in PostgreSQL. This commit is just hiding the measure for now. opw:2349896 closes odoo/odoo#60297 X-original-commit: 1e3fc6894b13d51a3f6327653afbf8d5a316a21d Signed-off-by:
backspac <backspac@users.noreply.github.com>
-
Benjamin Vray authored
Before this commit, the mouseup event was not detected outside the body element when resizing paddings. task-2312878 closes odoo/odoo#60356 X-original-commit: 127410f9 Signed-off-by:
Quentin Smetz (qsm) <qsm@odoo.com>
-
Antoine Prieels authored
Add support for Star Line Mode protocol, used by most Star receipt printers. closes odoo/odoo#60365 Taskid: 2365648 X-original-commit: 2cf0df13 Signed-off-by:
Quentin Lejeune (qle) <qle@odoo.com>
-
Odoo's Mergebot authored
# Purpose The wizard to test a mailing does not compute the jinja part. It does not detect syntax errors that would make cron crash # Specifications If there is at least one record in the mailing model then render template completely. Thow an error if rendering cannot be done. The downside is that we cannot test the jinja part if there is not yet any record in the mailing's model. Do it for mass_mailing and mass_mailing_sms. Add tests to ensure error detection in test wizard effectively works. Task ID-2312442 PR #55696 Signed-off-by:
Thibault Delavallee (tde) <tde@openerp.com>
-
Jérémy Hennecart authored
Like the mass_mailing app we want to raise an error in case there is a syntax error with jinja when we test the mailing and if there is at least one record in the mailing model. PR odoo/odoo#55696 Task ID-2312442
-
Jérémy Hennecart authored
Before when a user clicked on the button "Test", jinja syntax was not taken into account for the test mail sent. If an error was present clicking on the button did not raise an error before the cron was executed. Now, if there is at least one record in the mailing model, the template is correctly rendered and throws an error if there is a syntax error. If we don't have any record to render the template, we fallback on the raw content like before. PR odoo/odoo#55696 Task ID-2312442
-
Jérémy Hennecart authored
This test ensures that when using the test sending tool of mass mailing (sms) a wrong jinja content is detected if we have any record available to evaluate it. PR odoo/odoo#55696 Task ID-2312442
-
Nasreddin (bon) authored
Issue - Install "eCommerce" and "Inventoy" - Activate "Fedex" delivery connector in settings - Publish "Free Delivery" and "Fedex US" delivery method - Put the "FedEx" one above the "Free Delivery" - Go to shop and add an item to cart - Set an adress with no ZIP code and checkout - Select a payment methode and pay Order is generated without selecting a delivery method Same behavior happend when using only "Fedex" as delivery method. Cause The flow make the `payment.payment_form` trigger start after `website_sale_delivery.checkout` JS module. In 'start' function of `payment.payment_form`, the `disabled` attribut is removed from button if no checkbox_cgv is present and therefore break the `disabling` managemet since `disabledReasons` payButton data are not sync anymore. Solution Remove 'disabled' attribut only if has `disabledReasons` data on payButton (checkbox_cgv feature alter `disabledReasons`). opw-2355407 opw-2357605 closes odoo/odoo#60355 X-original-commit: 04e589e8 Signed-off-by:
bon-odoo <nboulif@users.noreply.github.com>
-
Benjamin Vray authored
This commit disable the pointer events on the overlay during the mouse wheel event. To prevent page scrolling from stopping when hovering over the overlay. task-2312878 closes odoo/odoo#60160 Signed-off-by:
Quentin Smetz (qsm) <qsm@odoo.com>
-