Project

General

Profile

Actions

Feature #86629

closed

Implement LinkHandler for telephone numbers

Added by Rune Piper about 6 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Could have
Assignee:
-
Category:
Backend User Interface
Target version:
Start date:
2018-10-11
Due date:
% Done:

100%

Estimated time:
PHP Version:
Tags:
Complexity:
easy
Sprint Focus:

Description

Currently the core supports 6 LinkHandlers:

'page'   => \TYPO3\CMS\Core\LinkHandling\PageLinkHandler::class,
'file'   => \TYPO3\CMS\Core\LinkHandling\FileLinkHandler::class,
'folder' => \TYPO3\CMS\Core\LinkHandling\FolderLinkHandler::class,
'url'    => \TYPO3\CMS\Core\LinkHandling\UrlLinkHandler::class,
'email'  => \TYPO3\CMS\Core\LinkHandling\EmailLinkHandler::class,
'record' => \TYPO3\CMS\Core\LinkHandling\RecordLinkHandler::class,

I suggest another LinkHandler for telephone numbers, similar to the email one.

Take a look at this extension for reference: https://github.com/monosize/typo3-ext-linkhandler-plus


Related issues 3 (0 open3 closed)

Related to TYPO3 Core - Bug #90004: Avoid showing telephone element for route link browserClosed2019-12-22

Actions
Related to TYPO3 Core - Bug #89532: Blind options folder,tel in redirectsClosedGeorg Ringer2019-10-28

Actions
Related to TYPO3 Core - Bug #90771: InputLinkElement does not support telefphone linksClosedGeorg Ringer2020-03-17

Actions
Actions #1

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

Actions #2

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/58674

Actions #3

Updated by Gerrit Code Review almost 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/58674

Actions #4

Updated by Gerrit Code Review almost 6 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/58674

Actions #5

Updated by Rune Piper almost 6 years ago

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

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions #7

Updated by Oliver Hader almost 5 years ago

  • Related to Bug #90004: Avoid showing telephone element for route link browser added
Actions #8

Updated by Oliver Hader almost 5 years ago

  • Related to Bug #89532: Blind options folder,tel in redirects added
Actions #9

Updated by Leonie Philine over 4 years ago

  • Category set to Backend User Interface
  • Status changed from Closed to New
  • Target version set to next-patchlevel

The "inputLink" backend form element renderType was overlooked:

\TYPO3\CMS\Backend\Form\Element\InputLinkElement::getLinkExplanation() does not handle Telephone links and returns $data['text'] == 'not implemented type telephone'

This is the situation at TYPO3 10.3.

To reproduce:

1. Create a TCA field like:

        'link'               => [
            'exclude'        => true,
            'label'          => 'LLL:EXT:frontend/Resources/Private/Language/locallang_ttc.xlf:header_link',
            'config'         => [
                'type'            => 'input',
                'renderType'      => 'inputLink',
                'size'            => 50,
                'max'             => 1024,
                'eval'            => 'trim,required',
                'autocomplete'    => true,
                'fieldControl'    => [
                    'linkPopup'   => [
                        'options' => [
                            'title'            => 'LLL:EXT:frontend/Resources/Private/Language/locallang_ttc.xlf:header_link_formlabel',
                            'blindLinkFields'  => 'class, params, title',
                            'blindLinkOptions' => '',
                        ],
                    ],
                ],
                'softref'    => 'typolink',
            ],
        ],

2. Create a record with above defined field in the backend.

3. Open the link popup ("BrowseLinks"), choose "Telephone" and enter a phone number.

4. Click "Set link" to apply and close the dialog window.

Actual result:
The link preview field displays: "not implemented type telephone" as returned by \TYPO3\CMS\Backend\Form\Element\InputLinkElement::getLinkExplanation().

Expected result:
The link preview field renders a phone icon and the phone number (as works perfectly for mailto links.).

Actions #10

Updated by Georg Ringer over 4 years ago

  • Related to Bug #90771: InputLinkElement does not support telefphone links added
Actions #11

Updated by Georg Ringer over 4 years ago

  • Status changed from New to Closed

dear Leonie Philine Bitto,

I am closing this issue again and opend a new one #90771.

Actions

Also available in: Atom PDF