Bug #38717
strftime() expects parameter 2 to be long, string given
100%
Description
The typoscript rendering process with stdWrap does not handle empty values corrent:
Warning: strftime() expects parameter 2 to be long, string given in \typo3\sysext\cms\tslib\class.tslib_content.php on line 2571
A simple example to reproduce it:
10 = TEXT 10 { strftime = %d }
in the date stdWrap this was fixed, see #30931
Related issues
Associated revisions
BUGFIX] strftime expects parameter 2 to be long, string given
Any zero length string value is replaced with the current timestamp.
(Just like the default value for the second parameter of
strtime/gmstrftime.)
Change-Id: I7cc43a09f75d21ab8db84c0191fa0c1c5d67ddb0
Fixes: #38717
Releases: 6.0, 4.7, 4.6, 4.5
Reviewed-on: http://review.typo3.org/12713
Reviewed-by: Stefan Neufeind
Reviewed-by: Marcus Schwemer
Tested-by: Marcus Schwemer
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert
[BUGFIX] strftime expects parameter 2 to be long, string given
Any zero length string value is replaced with the current timestamp.
(Just like the default value for the second parameter of
strtime/gmstrftime.)
Change-Id: I6a6102927d1e8e858970eda25cd525c925c5e629
Fixes: #38717
Releases: 6.0, 4.7, 4.6, 4.5
Reviewed-on: http://review.typo3.org/14715
Reviewed-by: Steffen Ritter
Tested-by: Steffen Ritter
[BUGFIX] strftime expects parameter 2 to be long, string given
Any zero length string value is replaced with the current timestamp.
(Just like the default value for the second parameter of
strtime/gmstrftime.)
Change-Id: I64a92d72d5ae680a6288b3e0cd8063d581de8566
Fixes: #38717
Releases: 6.0, 4.7, 4.6, 4.5
Reviewed-on: http://review.typo3.org/14716
Reviewed-by: Steffen Ritter
Tested-by: Steffen Ritter
[BUGFIX] strftime expects parameter 2 to be long, string given
Any zero length string value is replaced with the current timestamp.
(Just like the default value for the second parameter of
strtime/gmstrftime.)
Change-Id: I58225b1604607685a3fd4ac4ee50b806c552d326
Fixes: #38717
Releases: 6.0, 4.7, 4.6, 4.5
Reviewed-on: http://review.typo3.org/14717
Reviewed-by: Steffen Ritter
Tested-by: Steffen Ritter
History
#1
Updated by Jigal van Hemert over 7 years ago
- Category set to TypoScript
- Status changed from New to Accepted
- Assignee set to Jigal van Hemert
- Target version set to 4.5.18
- Complexity set to easy
#2
Updated by Gerrit Code Review over 7 years ago
- Status changed from Accepted to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12713
#3
Updated by Simon Schaufelberger over 7 years ago
common guys, please merge that one. it has already been solved for the date function and this is just the same fix for the strftime function.
#4
Updated by Gerrit Code Review over 7 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12713
#5
Updated by Jigal van Hemert over 7 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset a0cf6ea5c3d95c4ce2b425af91ebc912305c8656.
#6
Updated by Riccardo De Contardi about 2 years ago
- Status changed from Resolved to Closed