Task #89033

Deprecate jumpToUrl

Added by Andreas Fernandez 3 months ago. Updated 3 months ago.

Status:
Resolved
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2019-08-28
Due date:
% Done:

100%

TYPO3 Version:
10
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

Currently, TYPO3 widely makes use of jumpToUrl() which in fact does a simple redirect. In some cases, the call isn't even necessary, since <a> tags support the href nowadays.


Related issues

Related to TYPO3 Core - Bug #89315: deleteRecord is not defined Resolved 2019-09-30

Associated revisions

Revision 04ea328d (diff)
Added by Andreas Fernandez 3 months ago

[TASK] Deprecate jumpToUrl()

This patch deprecated the well-known `jumpToUrl()` function which simply
does a redirect nowadays. Any occurence may be replaced by either using
`window.location.href` or using simple `<a>` tags.

Resolves: #89033
Releases: master
Change-Id: I213d89825b8ee4736e168653cddb1929ceb9844e
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/61389
Tested-by: TYPO3com <>
Tested-by: Richard Haeser <>
Tested-by: Susanne Moog <>
Reviewed-by: Daniel Goerz <>
Reviewed-by: Richard Haeser <>
Reviewed-by: Susanne Moog <>

Revision 50d24ac1 (diff)
Added by Andreas Fernandez about 2 months ago

[BUGFIX] Re-implement record deletion via EditDocumentController

The change of #89033 removed the callback for deleting records within
EditDocumentController, resulting in JavaScript errors.

This patch adds the delete action again, directly implemented in the
FormEngine module. The URL to delete the record is pre-built as well,
since any required information is already known.

Resolves: #89315
Related: #89033
Releases: master
Change-Id: I842d0119291b53be3cdc3b2d6373ccd474fc7146
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/61865
Tested-by: TYPO3com <>
Tested-by: Oliver Hader <>
Reviewed-by: Oliver Hader <>

History

#1 Updated by Gerrit Code Review 3 months ago

  • Status changed from In Progress to Under Review

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

#2 Updated by Gerrit Code Review 3 months ago

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

#3 Updated by Gerrit Code Review 3 months ago

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

#4 Updated by Gerrit Code Review 3 months ago

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

#5 Updated by Andreas Fernandez 3 months ago

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

#6 Updated by Andreas Fernandez about 2 months ago

  • Related to Bug #89315: deleteRecord is not defined added

Also available in: Atom PDF