Bug #87010

Copy, cut and paste function broken in weblist module

Added by Joerg Kummer about 1 year ago. Updated 9 months ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2018-11-26
Due date:
% Done:

100%

TYPO3 Version:
9
PHP Version:
7.2
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

Since solved issue #83334, copy and paste function do not work as before, in 9.5.2-dev for web list modul with default clipboard.

Given...
You are in backend modul web-list, and see some pages or content elements listed in main frame.
You can click the icon for copy an existing element to the clipboard, while you have selected the default clipboard (normal).

Expected
Main frame should reload, and the clicked icon for copy element should switch the icon, as marked as added to clipboard.
And below the clipboard section should show, that the copied element is added to default clipboard.

Actual
Page reloads, with no error in browser-console. But either icon switches or clipboard is filled with new element to copy.

It works, if you have selected the other clipboard presettings (multiple elements).
Copy and paste in page-tree frame works as expected. No other copy, cut, paste issues found currently.


Related issues

Related to TYPO3 Core - Task #83334: GeneralUtility::implodeArrayForUrl should always encode param-names Closed 2017-12-14

Associated revisions

Revision 6208be93 (diff)
Added by Anja Leichsenring about 1 year ago

[BUGFIX] Revert UrlEncoding for GeneralUtility functions

URLencoding applied to linkThisScript and linkThisUrl make
links fail that are only used and executed by Javascript.
Reverting to old behavior fixes the failure.

Change-Id: I992c9c362190bb7d4f6135bab059fd7ec058d2bb
Resolves: #87010
Relates: #83334
Releases: master
Reviewed-on: https://review.typo3.org/58956
Tested-by: TYPO3com <>
Tested-by: Joerg Kummer <>
Tested-by: Preben Rather Sørensen

Tested-by: Tim Karliczek <tkarliczek@3pc.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Reviewed-by: Mona Muzaffar <mona.muzaffar@gmx.de>
Reviewed-by: Richard Haeser <richard@maxserv.com>
Reviewed-by: Andreas Fernandez <a.fernandez@scripting-base.de>
Tested-by: Richard Haeser <richard@maxserv.com>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>

History

#1 Updated by Anja Leichsenring about 1 year ago

  • Related to Task #83334: GeneralUtility::implodeArrayForUrl should always encode param-names added

#2 Updated by Gerrit Code Review about 1 year ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58956

#3 Updated by Gerrit Code Review about 1 year ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58956

#4 Updated by Gerrit Code Review about 1 year ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58956

#5 Updated by Gerrit Code Review about 1 year ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58956

#6 Updated by Anonymous about 1 year ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#7 Updated by Benni Mack 9 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF