Defect #32469
closed
Text copied from some applications such as MS Office and LibreOffice is pasted as an image in addition to plain text
Added by Takenori TAKAKI almost 5 years ago.
Updated over 4 years ago.
Description
When copying text from an Office product (e.g. MS Office, LibreOffice) and pasting it on redmine,
The image is pasted after the text. as shown below.
![](/attachments/download/24320/duplicate-pasting.png)
In this case, Clipboard contains multiple data (text and image), so both are pasted(attached) from clipboard.
To solve this problem, I propose the following change.
(Does not execute JS function ’copyImageFromClipboard’, if Clipboard contains text data)
index b856ff415..4a33daf55 100644
--- a/public/javascripts/attachments.js
+++ b/public/javascripts/attachments.js
@@ -255,6 +255,7 @@ function copyImageFromClipboard(e) {
if (!$(e.target).hasClass('wiki-edit')) { return; }
var clipboardData = e.clipboardData || e.originalEvent.clipboardData
if (!clipboardData) { return; }
+ if (clipboardData.types.some(t => /^text/.test(t))) { return; }
var items = clipboardData.items
for (var i = 0 ; i < items.length ; i++) {
Files
- Related to Feature #3816: Allow pasting screenshots from clipboard added
- Target version set to 4.1.0
The fix looks good to me.
- Subject changed from Duplicate pasting ("text" and "imaged text"). When copy and paste text from Office product (e.g. MS Office, LibreOffice). to Text copied from some applications such as MS Office and LibreOffice are pasted as an image in addition to plain text
- Subject changed from Text copied from some applications such as MS Office and LibreOffice are pasted as an image in addition to plain text to Text copied from some applications such as MS Office and LibreOffice is pasted as an image in addition to plain text
- Status changed from New to Closed
- Assignee set to Go MAEDA
- Target version deleted (
4.1.0)
- Resolution set to Fixed
Committed the patch as a part of #3816. Thank you for fixing the issue.
- Status changed from Closed to Reopened
Arrow function (=>) is not supported in IE, so my patch had a problem that an error was output to the browser console.
Therefore, reopen the ticket and write the improvement code below.
index 4a33daf55..af50cd8ba 100644
--- a/public/javascripts/attachments.js
+++ b/public/javascripts/attachments.js
@@ -255,7 +255,7 @@ function copyImageFromClipboard(e) {
if (!$(e.target).hasClass('wiki-edit')) { return; }
var clipboardData = e.clipboardData || e.originalEvent.clipboardData
if (!clipboardData) { return; }
- if (clipboardData.types.some(t => /^text/.test(t))) { return; }
+ if (clipboardData.types.some(function(t){ return /^text/.test(t); })) { return; }
var items = clipboardData.items
for (var i = 0 ; i < items.length ; i++) {
- Status changed from Reopened to Closed
Takenori TAKAKI wrote:
Arrow function (=>) is not supported in IE, so my patch had a problem that an error was output to the browser console.
Committed the fix in r19371. Thank you.
- Related to Defect #33639: Cannot paste image from clipboard when copying the image from web browsers or some apps added
Also available in: Atom
PDF