Feature #12442

Provide format.json ViewHelper

Added by Kevin Ulrich Moschallski almost 10 years ago. Updated over 8 years ago.

Status:
Resolved
Priority:
Could have
Assignee:
-
Category:
ViewHelpers
Start date:
2011-01-21
Due date:
% Done:

100%

Estimated time:
Has patch:
Yes

Description

Hi,

i suggest to add type json to the escape viewhelper.

Sometime you want to render json output with fluid and not with an own json view. The problem is that fluid also uses json like syntax, so if you want to add an object to a json string you run into a problem.

Imagine adding a link with the link viewhelper. It returns a sting with double quotes which need to be escaped for json.

I simply added type json to the escape viewhelper which returns the $value json_encode()'ed.

Tell me what you think.

Regards,
Kevin


Files

12442.patch (410 Bytes) 12442.patch Kevin Ulrich Moschallski, 2011-01-21 15:49
12442_v2.patch (785 Bytes) 12442_v2.patch Kevin Ulrich Moschallski, 2011-01-21 15:50
#2

Updated by Kevin Ulrich Moschallski almost 10 years ago

Description added.

#3

Updated by Sebastian Kurfuerst over 9 years ago

  • Category set to ViewHelpers
  • Priority changed from Should have to Could have
  • Has patch set to Yes
#4

Updated by Bastian Waidelich over 9 years ago

  • Subject changed from Add type json to escape viewhelper to Provide format.json ViewHelper

Note: The escape ViewHelper will be deprecated in the next version (see #27628) so this functionality should be put in it's own format.json ViewHelper. I changed the subject of the issue accordingly

#5

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

#6

Updated by Gerrit Code Review almost 9 years ago

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

#7

Updated by Gerrit Code Review almost 9 years ago

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

#8

Updated by Karsten Dambekalns almost 9 years ago

  • Target version set to 1.1
#9

Updated by Gerrit Code Review over 8 years ago

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

#10

Updated by Gerrit Code Review over 8 years ago

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

#11

Updated by Gerrit Code Review over 8 years ago

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

#12

Updated by Gerrit Code Review over 8 years ago

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

#13

Updated by Gerrit Code Review over 8 years ago

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

#14

Updated by Gerrit Code Review over 8 years ago

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

#15

Updated by Gerrit Code Review over 8 years ago

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

#16

Updated by Adrian Föder over 8 years ago

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

Also available in: Atom PDF