[TASK] Replace default value with placeholder in external url link handler

Use a placeholder attribute instead of a default value.

Resolves: #85689
Releases: master, 8.7
Change-Id: I99398125e16985ba78b3887150157da2b2c01905
Reviewed-on: https://review.typo3.org/57792
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Frank Naegler <frank.naegler@typo3.org>
Tested-by: Frank Naegler <frank.naegler@typo3.org>

Author: Joerg Boesche

typo3/sysext/core/Documentation/Changelog/8.7.x/Important-85689-ReplacedDefaultValueWithPlaceholderInExternalUrlLinkHandler.rst

@@ -0,0 +1,18 @@
+.. include:: ../../Includes.txt
+
+========================================================================================
+Important: #85689 - Replaced default value with placeholder in external url link handler
+========================================================================================
+
+See :issue:`85689`
+
+Description
+===========
+
+The :php:`UrlLinkHandler` in EXT:recordlist used a default input value "http://" for external links.
+This is not practical because editors often paste a link in the field. This caused broken links due
+to duplicate HTTP protocols, for example: `http://https://typo3.org/`
+
+A placeholder is now used instead of a default value. Editors can paste links directly and do not have to remove a default value first.
+
+.. index:: NotScanned

typo3/sysext/lang/Resources/Private/Language/locallang_browse_links.xlf

@@ -87,6 +87,9 @@
 			<trans-unit id="setLink">
 				<source>Set Link</source>
 			</trans-unit>
+			<trans-unit id="urlPlaceholder">
+				<source>https://...</source>
+			</trans-unit>
 			<trans-unit id="clickToRedrawFullSize">
 				<source>Image is larger than shown here! Click to redraw page with full sized images.</source>
 			</trans-unit>

typo3/sysext/recordlist/Classes/LinkHandler/UrlLinkHandler.php

@@ -88,7 +88,7 @@ public function render(ServerRequestInterface $request)
     {
         GeneralUtility::makeInstance(PageRenderer::class)->loadRequireJsModule('TYPO3/CMS/Recordlist/UrlLinkHandler');
 
-        $this->view->assign('url', !empty($this->linkParts) ? $this->linkParts['url'] : 'http://');
+        $this->view->assign('url', !empty($this->linkParts) ? $this->linkParts['url'] : '');
         return $this->view->render('Url');
     }
 

typo3/sysext/recordlist/Resources/Private/Templates/LinkBrowser/Url.html

@@ -6,7 +6,12 @@
 					<label class="col-xs-4 control-label">URL</label>
 					<div class="col-xs-8">
 						<div class="input-group">
-							<input type="text" name="lurl" size="30" class="form-control" value="{url}" />
+							<input type="text"
+								   name="lurl"
+								   size="30"
+								   class="form-control"
+								   value="{url}"
+								   placeholder="{f:translate(key: 'LLL:EXT:lang/Resources/Private/Language/locallang_browse_links.xlf:urlPlaceholder')}" />
 							<div class="input-group-btn">
 								<input class="btn btn-sm btn-default" type="submit"
 											value="{f:translate(key: 'LLL:EXT:lang/Resources/Private/Language/locallang_browse_links.xlf:setLink')}" />

typo3/sysext/recordlist/Resources/Public/JavaScript/UrlLinkHandler.js

@@ -33,8 +33,8 @@ define(['jquery', 'TYPO3/CMS/Recordlist/LinkBrowser'], function($, LinkBrowser)
     event.preventDefault();
 
     var value = $(this).find('[name="lurl"]').val();
-    if (value === "http://") {
-      return;
+    if (value === "") {
+       return;
     }
 
     LinkBrowser.finalizeFunction(value);