odoo/o-spreadsheet#6960

Created by fw-bot
Merged at 1c099dc42298ad39c3762b72c4fbe6d7a68a897d

Statuses:

label
odoo:saas-18.4-saas-18.3-excel-copy-with-image-adrm-452720-fw
head
0269d21e78526d16605ddac7da13f9bd35c413b5
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

Forward-Port-Of: #6854