Project

General

Profile

Actions

Bug #80373

closed

Epic #78916: ext:rte_ckeditor

href contains t3:// protocol

Added by Robert Vock over 7 years ago. Updated about 6 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 1 (0 open1 closed)

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

Actions
Actions #1

Updated by Riccardo De Contardi over 7 years ago

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

Updated by Riccardo De Contardi over 7 years ago

  • Parent task set to #78916
Actions #3

Updated by Benni Mack over 7 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

Actions #4

Updated by Gerrit Code Review over 7 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

Actions #5

Updated by Gerrit Code Review over 7 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

Actions #6

Updated by Gerrit Code Review over 7 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

Actions #7

Updated by Gerrit Code Review over 7 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

Actions #8

Updated by Benni Mack over 7 years ago

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

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF