Project

General

Profile

Actions

Bug #81903

closed

DatabaseRecordLinkBuilder is ignoring other settings

Added by René Pflamm almost 7 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
-
Target version:
-
Start date:
2017-07-18
Due date:
% Done:

100%

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

Description

The DatabaseRecordLinkBuilder is ignoring other settings make via typoscript.

So for example the ATagParams setting is ignored.

To Reproduce:

Make an Linkhandler for news, eg:

TCEMAIN.linkHandler {
  # Pressemitteilungen
  tx_news {
    handler = TYPO3\CMS\Recordlist\LinkHandler\RecordLinkHandler
    label = Pressemitteilungen

    configuration {
      table = tx_news_domain_model_news
      storagePid = 136
      hidePageTree = 1
    }

    scanAfter = page
  }
}

Now use typoscript to render the Link:

link = TEXT
link {
  value = Test
  typolink {
    parameter = t3://record?identifier=tx_news&uid=12046
    ATagParams = class="test" 
  }
}

You will see the added class will not be rendered.

My simple soloution: merge the Link Conf array with the typoScriptConfiguration for the linkHandler (see example Patch attached).


Files

DatabaseRecordLinkBuilder.patch (979 Bytes) DatabaseRecordLinkBuilder.patch René Pflamm, 2017-07-18 10:00

Related issues 3 (0 open3 closed)

Related to TYPO3 Core - Bug #81620: Linkhandler remove all attributes set in the link wizard and the templateClosed2017-06-19

Actions
Related to TYPO3 Core - Bug #84116: Linkhandler links not rendered in frontend when TSConfig configuration is applied using "registerPageTSConfigFile()" or "addPageTSConfig()"Closed2018-03-02

Actions
Related to TYPO3 Core - Bug #80896: TypolinkViewhelper with linkhandler ignores class attributeClosed2017-04-19

Actions
Actions #1

Updated by Mathias Brodala almost 7 years ago

  • Is duplicate of Bug #81620: Linkhandler remove all attributes set in the link wizard and the template added
Actions #2

Updated by Mathias Brodala almost 7 years ago

  • Status changed from New to Closed

Please continue in #81620.

Actions #3

Updated by René Pflamm almost 7 years ago

I think this are different bugs.

I applied the patch but the ATagParams is ignored again. The solution of #81620 don't work here because the $typoScriptConfiguration variable has only a ATagParams if this was configured for the LinkHandler.

Actions #4

Updated by Mathias Brodala almost 7 years ago

  • Is duplicate of deleted (Bug #81620: Linkhandler remove all attributes set in the link wizard and the template)
Actions #5

Updated by Mathias Brodala almost 7 years ago

  • Status changed from Closed to New
Actions #6

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

Actions #7

Updated by Christian Kuhn over 6 years ago

  • Related to Bug #81620: Linkhandler remove all attributes set in the link wizard and the template added
Actions #8

Updated by Gerrit Code Review about 6 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/55426

Actions #9

Updated by Gerrit Code Review about 6 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/55426

Actions #10

Updated by Gerrit Code Review about 6 years ago

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

Actions #11

Updated by René Pflamm about 6 years ago

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

Updated by Stefan Froemken about 6 years ago

  • Related to Bug #84116: Linkhandler links not rendered in frontend when TSConfig configuration is applied using "registerPageTSConfigFile()" or "addPageTSConfig()" added
Actions #13

Updated by Rémy DANIEL about 6 years ago

  • Related to Bug #80896: TypolinkViewhelper with linkhandler ignores class attribute added
Actions #14

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF