Bug #66707

Fix usages of issueCommand

Added by Markus Klein over 4 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Must have
Assignee:
Category:
Backend API
Target version:
Start date:
2015-05-01
Due date:
% Done:

100%

TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
medium
Is Regression:
No
Sprint Focus:

Description

Using issueCommand in JavaScript context has to ensure the URL is properly quoted and escaped.


Related issues

Related to TYPO3 Core - Bug #67320: ReferenceError: T3_THIS_LOCATION is not defined` on Button "Copy default content element" Closed 2015-06-05

Associated revisions

Revision ede2d2cf (diff)
Added by Markus Klein over 4 years ago

[!!!][BUGFIX] Fix usage of issueCommand() and T3_THIS_LOCATION

Multiple usages of T3_THIS_LOCATION are removed as its usage
make only sense in JavaScript context.

issueCommand() takes care of properly quoting the URL when
using in JavaScript context.

Resolves: #66707
Releases: master
Change-Id: Ib0e89e9657fc83bd6514a2077013926f24434a37
Reviewed-on: http://review.typo3.org/39158
Reviewed-by: Wouter Wolters <>
Tested-by: Wouter Wolters <>
Reviewed-by: Helmut Hummel <>
Tested-by: Helmut Hummel <>

History

#1 Updated by Gerrit Code Review over 4 years ago

  • Status changed from Accepted 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 http://review.typo3.org/39158

#2 Updated by Gerrit Code Review over 4 years ago

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

#3 Updated by Anonymous over 4 years ago

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

#4 Updated by Anja Leichsenring about 4 years ago

  • Sprint Focus deleted (On Location Sprint)

#5 Updated by Riccardo De Contardi over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF