odoo/o-spreadsheet#6854

Created by BI, Adrien Minne (adrm)

label

odoo:saas-18.3-excel-copy-with-image-adrm

head

13b6d671e16e9d319aa88b3805ad037eb49d93e4

merged

3 weeks ago by BI, Rémi Rahir (rar)

	odoo/o-spreadsheet
saas-18.3	#6854
saas-18.4	#6960
master	#6963

[FIX] clipboard: don't paste both image and text content

Description

If we have a clipboard content that has bot an image and some text data, we would paste both inside the spreadsheet. There's a good chance that this is not what the user expects, as the iamge and the text are likely alternatives for the same content.

That's what happened when pasting from Excel (desktop). The user would end up with both the copied data, and an image of the copied data.

With this commit, we'll ignore the image data if we have text data in the clipboard.

Task: 4876682

review checklist

• [ ] feature is organized in plugin, or UI components
• [ ] support of duplicate sheet (deep copy)
• [ ] in model/core: ranges are Range object, and can be adapted (adaptRanges)
• [ ] in model/UI: ranges are strings (to show the user)
• [ ] undo-able commands (uses this.history.update)
• [ ] multiuser-able commands (has inverse commands and transformations where needed)
• [ ] new/updated/removed commands are documented
• [ ] exportable in excel
• [ ] translations (_t("qmsdf %s", abc))
• [ ] unit tested
• [ ] clean commented code
• [ ] track breaking changes
• [ ] doc is rebuild (npm run doc)
• [ ] status is correct in Odoo