odoo/odoo#204075
Created by fw-bot
Statuses:
- legal/cla: Contributor License Agreement check
- ci/runbot: Odoo Test Suite
- ci/upgrade_enterprise: Test upgrades for enterprise master
- ci/template: Contact runbot team on discord for help.
- ci/style: Optional style check. Ignore it only if strictly necessary.
- ci/security: Required security check. Can only be ignored by security team.
- label
- odoo-dev:saas-18.2-18.0-opw-4607020-fix_opening_closing_cropper_while_loading_image-wasa-425003-fw
- head
- 38baaddd6628236a4ea95a95ab444239efae29e2
- merged
- 7 months ago by Walid Sahli (wasa)
| odoo/odoo | |
|---|---|
| 18.0 | #200729 |
| saas-18.1 | #203102 |
| saas-18.2 | #204075 |
| saas-18.3 | |
| saas-18.4 | |
| 19.0 | |
| master | #204076 |
[FW][FIX] html_editor: fix cropper issue with large images
Problem:
Loading large images into the cropper takes time. Closing the cropper early causes this.imageRef.el to be null or this.initialSrc to be undefined, leading to a traceback.
What happens:
1. Add a large image.
2. Click "Crop".
- This triggers show.
- While loadImage is still unresolved due to image size.
3. Click "Discard" to close the cropper before image loads.
- Issue: this.imageRef.el is null and this.initialSrc is undefined as the component is destroyed but loadImage has not resolved yet, when it is resolved a traceback will be appear.
Solution:
Add a lifecycle flag as full mounting of the cropper might take time.
Steps to Reproduce:
1. Add a large image.
2. Open cropper.
3. Close cropper before image loads.
- Issue: Traceback.
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr