Bug #80373

Epic #78916: ext:rte_ckeditor

href contains t3:// protocol

Added by Robert Vock almost 4 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
RTE (rtehtmlarea + ckeditor)
Target version:
-
Start date:
2017-03-21
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Is Regression:
Yes
Sprint Focus:

Description

This is a followup for #79896

After updating to latest master (4e4beb), I get two href-attributes when outputting links using parseFunc_RTE:

<a href="/leistungen/" href="t3://page?uid=16" class="internal-link">

This happens because of

ATagParams.data = parameters:allParams

parameters:allParameters also contains the original href.


Related issues

Follows TYPO3 Core - Bug #79896: All <a> tag parameters are dropped on FE rendering due to typo in lib.parseFuncClosedBenni Mack2017-02-18

Actions
#1

Updated by Riccardo De Contardi almost 4 years ago

  • Category set to RTE (rtehtmlarea + ckeditor)
#2

Updated by Riccardo De Contardi almost 4 years ago

  • Parent task set to #78916
#3

Updated by Benni Mack almost 4 years ago

Example:

lib.parseFunc_RTE.tags {
        a = TEXT
        a {
            current = 1
            typolink {
                parameter.data = parameters:href
                title.data = parameters:title
                ATagParams.data = parameters:allParams
                target.data = parameters:target
                extTarget = {$styles.content.links.extTarget}
                extTarget.override.data = parameters:target
            }
        }
}

page.3 = TEXT
page.3.value = Nice <a href="t3://page?uid=87">Overview</a> from here.
page.3.parseFunc < lib.parseFunc_RTE

#4

Updated by Gerrit Code Review almost 4 years 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/52323

#5

Updated by Gerrit Code Review almost 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 https://review.typo3.org/52323

#6

Updated by Gerrit Code Review almost 4 years 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/52323

#7

Updated by Gerrit Code Review almost 4 years 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/52323

#8

Updated by Benni Mack almost 4 years ago

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

Updated by Benni Mack over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF