Project

General

Profile

Actions

Bug #82861

closed

linkPopup option 'allowedExtensions' in TCA inputLink field not considered

Added by Tobias Klepp about 7 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
FormEngine aka TCEforms
Target version:
Start date:
2017-10-26
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
8
PHP Version:
7.0
Tags:
tca, fieldControl, renderType, inputLink, linkPopup, option, allowedExtensions
Complexity:
Is Regression:
Sprint Focus:

Description

I setup the following TCA configuration:

        'link' => [
            'exclude' => 1,
            'label' => 'link',
            'config' => [
                'type' => 'input',
                'renderType' => 'inputLink',
                'size' => '50',
                'eval' => 'trim',
                'fieldControl' => [
                    'linkPopup' => [
                        'options' => [
                            'blindLinkOptions' => 'mail,page,spec,url,folder',
                            'blindLinkFields' => 'class,params,target,title',
                            'allowedExtensions' => 'html',
                        ],
                    ],
                ],
            ],
        ],

The allowedExtensions option has no effect in the Link browser popup. After some debugging, I found out, that the allowedExtensions parameter from the url is not available in $this->linkBrowser->getUrlParameters().

The effect is, that in FileLinkHandler.php on line 133 the allowed extensions are not present.

            if ($selectedFolder->checkActionPermission('read')) {
                $this->view->assign('selectedFolder', $selectedFolder);
                $parameters = $this->linkBrowser->getUrlParameters();
                $allowedExtensions = isset($parameters['allowedExtensions']) ? $parameters['allowedExtensions'] : '';
                $this->expandFolder($selectedFolder, $allowedExtensions);
            }
Actions #1

Updated by Christian Kuhn about 7 years ago

Added input_38 in ext:styleguide elements_basic as example:

https://github.com/TYPO3/styleguide/commit/c3cc88c6841a1f832778702f306d32e597d5f7c7

Actions #2

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

Actions #3

Updated by Susanne Moog about 7 years ago

  • Category set to FormEngine aka TCEforms
Actions #4

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

Actions #5

Updated by Christian Kuhn about 7 years ago

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

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF