Bug #12777

Empty a tag ends with /> instead of </a>

Added by Xavier Perseguers over 4 years ago. Updated about 4 years ago.

Status:Resolved Start date:2011-02-04
Priority:Must have Due date:
Assigned To:Sebastian Kurfuerst % Done:

100%

Category:ViewHelpers
Target version:TYPO3 Flow Base Distribution - 1.0 beta 1
Has patch:Yes Affected Flow version:

Description

When creating a link with no content, the result tag is

<a href="http:///"/>

instead of

<a href="http:///"></a>

or even better (if no url is provided):

<a href=""></a>

InvalidATag.diff Magnifier (1.7 kB) Xavier Perseguers, 2011-02-04 14:50


Related issues

related to TYPO3.Fluid - Bug #10401: f:link.external viewHelper should prepend http:// when ne... Resolved 2010-10-22

Associated revisions

Revision bd18b4f1
Added by Sebastian Kurfuerst about 4 years ago

[+BUGFIX] Empty a tag ends with /> instead of </a>

Change-Id: Icab2204f464f43716e0e52e7f2dae2133c6404c1
Resolves: #12777

History

#1 Updated by Georg Ringer over 4 years ago

but this is not always the case, e.g. img or meta dont need a closing tag. so there needs to be a special config

#2 Updated by Xavier Perseguers over 4 years ago

Hi Georg,

Where do you see a "special" configuration? I only set an existing attribute for all link tags, that's all. Of course meta or img should not be changed but I don't change them with my patch.

#3 Updated by Andreas Kiessling over 4 years ago

+1 on reading
Had that error in the process of localizing an extension, but some labels were still missing.

#4 Updated by Sebastian Kurfuerst about 4 years ago

  • Target version set to 1.0 beta 1

#5 Updated by Sebastian Kurfuerst about 4 years ago

  • Has patch set to Yes

#6 Updated by Sebastian Kurfuerst about 4 years ago

  • Parent task set to #26665

#7 Updated by Sebastian Kurfuerst about 4 years ago

  • Tracker changed from Task to Bug
  • Parent task deleted (#26665)

#8 Updated by Sebastian Kurfuerst about 4 years ago

  • Status changed from New to Under Review
  • Assigned To set to Sebastian Kurfuerst

#9 Updated by Mr. Hudson about 4 years ago

Patch set 1 of change Icab2204f464f43716e0e52e7f2dae2133c6404c1 has been pushed to the review server.
It is available at http://review.typo3.org/1940

#10 Updated by Sebastian Kurfuerst about 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 90 to 100

Also available in: Atom PDF