Friday, February 7, 2025
2 changes · saas-17.2
Resolved issues and error corrections
Users can now sync appointment events to their calendar even when they do not have permission to view the related appointment type. This prevents missed calendar updates and keeps personal schedules accurate without changing broader access rights.
Original PR description
If a user has an appointment in an appointment type they may not read. They should still be able to sync it to their calendar. Related to this fix: e52b13c9fc9cb309351f527bfbe961fcb0fb961e task-4526059
Clicking an applicant from the recruitment documents list no longer causes an error. This keeps navigation from job position documents to applicant records working reliably for recruitment users.
Original PR description
Scenario: - install hr_recruitment but not documents (to have the right view) - go to Employee > Configuration > Recruitment | Job Positions - choose a job positions with documents and click on…
Scenario: - install hr_recruitment but not documents (to have the right view) - go to Employee > Configuration > Recruitment | Job Positions - choose a job positions with documents and click on Documents - click on Applicant column on one of the rows => traceback shown: OwlError: Invalid props for component 'FormController': 'resId' is not a number or boolean at Object.validateProps Reason: In saas-17.2 with a7586fdb4123481fd8794ef99fdc372e95de7459 merged in february 2024, many2one_reference was changed from a simple integer to either 0 if unset, or a object containing resId and displayName keys. Fix: Get the id with .resId and this should not happen, but test if the record is set with "if(res_id && ...)" in case res_id was 0. Note: without the fix, the added test fails by opening the view if res_id was 0, and if not the next click would fail with: OwlError: Invalid props for component 'FormController': 'resId' is not a number or boolean opw-4384139 PR note: the test could be run without mails models, but then we would need to mock a lot of models for services (like `mail.ChatHub`) or a method like `prepareRegistry` [from studio](https://github.com/odoo/enterprise/blob/f5b7b53963829b351446e9b6f71e00f21291bc58/web_studio/static/tests/view_editor_tests_utils.js#L123-L153) that would remove all unnecessary services for the test.