Project

General

Profile

Actions
Copy link

Feature #39578

closed

Add whitespace between two tags when using stripHtml stdWrap

Added by Ingo Pfennigstorf over 11 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Could have
Assignee:
-
Category:
TypoScript
Target version:
-
Start date:
2012-08-07
Due date:
% Done:

0%

Estimated time:
PHP Version:
Tags:
Complexity:
no-brainer
Sprint Focus:

Description

Having a block like this :

page = PAGE
page.10 = TEXT
page.10.value = <li>Hello</li><li>World</li>
page.10.stripHtml = 1

results in an output "HelloWorld"

I suggest to put a whitespace between the closing and opening tag to get the proper output "Hello World"

I will provide a patch for this feature

Actions
Copy link
 #1

Updated by Gerrit Code Review over 11 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13493

Actions
Copy link
 #2

Updated by Gerrit Code Review over 11 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13493

Actions
Copy link
 #3

Updated by Ingo Pfennigstorf over 11 years ago

If this breaks something, maybe it's smarter to add something like

page.10.stripHtml = 1
page.10.stripHtml.addWhitespaceBetweenTags = 1

and do not change the default behaviour.

What do you think?

Actions
Copy link
 #4

Updated by Oliver Klee over 11 years ago

Even if we make this feature non-default, we still need unit tests to make sure that this change does not break anything. (In other words: Breaking things is not okay even if the feature is optional. :-)

Actions
Copy link
 #5

Updated by Ingo Pfennigstorf over 11 years ago

I will surely add some unit tests that will be included in the next patchset

Actions
Copy link
 #6

Updated by Gerrit Code Review over 11 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13493

Actions
Copy link
 #7

Updated by Gerrit Code Review over 11 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13493

Actions
Copy link
 #8

Updated by Alexander Opitz almost 11 years ago

Hi Ingo,

do you like to update your patch?

Actions
Copy link
 #9

Updated by Chris topher almost 11 years ago

  • Status changed from Under Review to New

The patch was abandoned.

Actions
Copy link
 #10

Updated by Mathias Schreiber over 9 years ago

  • Assignee set to Mathias Schreiber
  • Priority changed from Should have to Could have
  • Target version set to 7.2 (Frontend)
Actions
Copy link
 #11

Updated by Benni Mack almost 9 years ago

  • Target version changed from 7.2 (Frontend) to 7.4 (Backend)
Actions
Copy link
 #12

Updated by Susanne Moog over 8 years ago

  • Target version changed from 7.4 (Backend) to 7.5
Actions
Copy link
 #13

Updated by Benni Mack over 8 years ago

  • Target version deleted (7.5)
Actions
Copy link
 #14

Updated by Susanne Moog over 4 years ago

  • Status changed from New to Closed
  • Assignee deleted (Mathias Schreiber)

Due to low demand / low priority I'm going to close this issue now. If anyone takes time to implement it, simply open a new issue please. Thanks.

Actions
Copy link

Also available in: Atom PDF