Saturday, January 18, 2025
8 changes · master
Miscellaneous changes
[FIX] hr_contract_salary: Custom document field not working Steps: - Create a new file field on employee - Open salary package configurator / personal info - Create a new info with the created field - category: personal documents - display type: document - Open Employee/Abigail - Open its contract - Generate an offer - 17 - Send by email - From the generated template, copy the button link - Open the copied link in another tab - master
Original PR description
[FIX] hr_contract_salary: Custom document field not working
Steps:
- Create a new file field on employee
- Open salary package configurator / personal info
- Create a new info with the created field
- category: personal documents
- display type: document
- Open Employee/Abigail
- Open its contract
- Generate an offer
- 17
- Send by email
- From the generated template, copy the button link
- Open the copied link in another tab
- master
- Just open the Link field
Actual result:
- KeyError for the filename of the new file field
Expected result:
- No error
- Filename field is also used when using a binary field
- File input is well show in the website configurator
opw-4382086
Caused by https://github.com/odoo/enterprise/commit/8e50071471d194bc8b594a79a95fe951cd5055ab & https://github.com/odoo/enterprise/commit/6f40549b98e65f813456442f7df35e9349a7639c
Forward-Port-Of: odoo/enterprise#75868Steps to reproduce: - Create a spreadsheet - click on the the Share button - add a portal user (Joel Willis) to the allowed users - connect as the portal user and open his portal - go to the Documents section and try opening the spreadsheet => You are not allowed to access 'Spreadsheet Contributor' (spreadsheet.contributor) records. Task: 4453298 Forward-Port-Of: odoo/enterprise#77007 Forward-Port-Of: odoo/enterprise#76531
Original PR description
Steps to reproduce: - Create a spreadsheet - click on the the Share button - add a portal user (Joel Willis) to the allowed users - connect as the portal user and open his portal - go to the Documents section and try opening the spreadsheet => You are not allowed to access 'Spreadsheet Contributor' (spreadsheet.contributor) records. Task: 4453298 Forward-Port-Of: odoo/enterprise#77007 Forward-Port-Of: odoo/enterprise#76531
When we click on the waning the action does not show the correct list. - replace the `self.env['fleet.vehicle']` with `self.env['hr.employee']` as `employees_multiple_vehicles` is a list with employee_ids not vehicle_ids Task: 4458500 Forward-Port-Of: odoo/enterprise#77115 Forward-Port-Of: odoo/enterprise#76679
Original PR description
When we click on the waning the action does not show the correct list. - replace the `self.env['fleet.vehicle']` with `self.env['hr.employee']` as `employees_multiple_vehicles` is a list with employee_ids not vehicle_ids Task: 4458500 Forward-Port-Of: odoo/enterprise#77115 Forward-Port-Of: odoo/enterprise#76679
This commit's purpose is to clean up the graph view of the sla report analysis model. Some fields were added on the view by a generic improvement task to handle the widget 'float_time' with grap view. In this case, those fields are a nuisance for readability, so we're getting rid of them. target version 17.0 - master task - 4351830 Forward-Port-Of: odoo/enterprise#76237
Original PR description
This commit's purpose is to clean up the graph view of the sla report analysis model. Some fields were added on the view by a generic improvement task to handle the widget 'float_time' with grap view. In this case, those fields are a nuisance for readability, so we're getting rid of them. target version 17.0 - master task - 4351830 Forward-Port-Of: odoo/enterprise#76237
## Description Following 9a8777b4b96367597418cdc1c1ffc7ea3264113f, the criteria on the `move_type` of the invoices used in `sale.commission.report` was changed from `'out_invoice'` to `'out_invoice' AND 'out_refund'`, but the respective index `account_move_invoice_user_id_date_idx` partial clause hasn't been updated, leading to Postgres not being able to use the index, leading to a performance regression. We're updating both custom indexes partial clause to match the invoice `move_type` conditi
Original PR description
## Description Following 9a8777b4b96367597418cdc1c1ffc7ea3264113f, the criteria on the `move_type` of the invoices used in `sale.commission.report` was changed from `'out_invoice'` to `'out_invoice' AND 'out_refund'`, but the respective index `account_move_invoice_user_id_date_idx` partial clause hasn't been updated, leading to Postgres not being able to use the index, leading to a performance regression. We're updating both custom indexes partial clause to match the invoice `move_type` condition used in the reporting. ## Benchmark On a large database with millions of invoices and customers | Timings (hot) | Before | After | |---------------|--------|-------| | 1 read_group | 40s | 1s | | full load | 225s | 6s | ## Reference task-4448879 Forward-Port-Of: odoo/enterprise#76723
When a user create the first payment initiation with Ponto he need to do the KYC of Ponto. The issue is that today we initiate the payment and then start the process of KYC. Following that the first payment is always in error, because it was initiated before the KYC. In this commit, we create the payments only if the KYC is done. task-4476856 Forward-Port-Of: odoo/enterprise#77052
Original PR description
When a user create the first payment initiation with Ponto he need to do the KYC of Ponto. The issue is that today we initiate the payment and then start the process of KYC. Following that the first payment is always in error, because it was initiated before the KYC. In this commit, we create the payments only if the KYC is done. task-4476856 Forward-Port-Of: odoo/enterprise#77052
This commit fixes various issues with the context menu on the call participant: - Before this commit it wasn't possible to set the volume of a user to 0 due to the use of the `||` operator in the getVolume function. This commit fixes the issue by using the correct `??` operator. - Before this commit the volume slider in dark mode had the same background as the popover menu, making it difficult to see. This commit fixes the issue by making the slider darker than the background. - A volume ic
Original PR description
This commit fixes various issues with the context menu on the call participant: - Before this commit it wasn't possible to set the volume of a user to 0 due to the use of the `||` operator in the getVolume function. This commit fixes the issue by using the correct `??` operator. - Before this commit the volume slider in dark mode had the same background as the popover menu, making it difficult to see. This commit fixes the issue by making the slider darker than the background. - A volume icon was added to better indicate the slider purpose. - A volume percentage tooltip was added for more accurate use. - A chevron button was added to the participant card for better discoverability of the context menu. Forward-Port-Of: odoo/enterprise#77124
Before this commit, when editing a default value for a field, it was always setting th default on the company of the user (as set on the user), not on the current company the user is logged into. This was largely misleading. After this commit, we get and set the default on the current company. Some work need to be done in master to improve those flows. opw-4427384 Forward-Port-Of: odoo/enterprise#77186 Forward-Port-Of: odoo/enterprise#76819
Original PR description
Before this commit, when editing a default value for a field, it was always setting th default on the company of the user (as set on the user), not on the current company the user is logged into. This was largely misleading. After this commit, we get and set the default on the current company. Some work need to be done in master to improve those flows. opw-4427384 Forward-Port-Of: odoo/enterprise#77186 Forward-Port-Of: odoo/enterprise#76819