Feature #16681
closed
C style TypoScript comments with start and end in one line lead to errors
100%
Description
Comments like this:
/******* Metatags **/
config.meta.keywords = foo, bar
in typoscript are treated like a beginning comment without end. Following code has no influence and is ignored.
(issue imported from #M4453)
Files
Updated by Jacob Floyd about 15 years ago
I can confirm this - I'm using PHP 5.2.6 with typo3 4.2.6
I couldn't figure out for quite a while why a section of my ts wasn't getting through - After moving it above a part that was working, I discovered that it disabled that too - after removing the multiline comment, everything worked as expected. Go figure.
Updated by Benni Mack over 13 years ago
Could you check if the attached patch solved this issue?
Updated by Alexander Opitz almost 11 years ago
- Status changed from Accepted to Needs Feedback
- Target version deleted (
0)
As this report is very old, is the handling in newer TYPO3 CMS Versions (like 6.0/6.1) more like you expect it?
Updated by Chris topher almost 11 years ago
- % Done changed from 0 to 100
Markus Bucher wrote:
Comments like this:
/******* Metatags **/
config.meta.keywords = foo, barin typoscript are treated like a beginning comment without end.
Sidenote: This is no bug, but exactly the documented behavior. Your comment block needs to be closed with "*/" signs at the beginning of a new line. Put the "*/" signs on a new line or - as you only have one line of comment anyway - use a normal one-line comment like "# Comment" and things should be working.
Updated by Alexander Opitz almost 11 years ago
- Status changed from Needs Feedback to Rejected
No bug as documented.
Updated by Ingo Renner almost 11 years ago
- Tracker changed from Bug to Feature
- Status changed from Rejected to New
- Assignee deleted (
Benni Mack)
re-opening. I think it's a valid issue and a low hanging fruit to ease live. If we support /* ... */, why not support it for single line comments? Also the patch on first sight looks like it would solve the issue.
Updated by Mathias Schreiber over 9 years ago
- Target version set to 7.2 (Frontend)
Updated by Benni Mack almost 9 years ago
- Target version changed from 7.2 (Frontend) to 7.4 (Backend)
Updated by Susanne Moog over 8 years ago
- Target version changed from 7.4 (Backend) to 7.5
Updated by Benni Mack over 8 years ago
- Target version changed from 7.5 to 8 LTS
Updated by Riccardo De Contardi about 7 years ago
- Target version changed from 8 LTS to 9 LTS
- PHP Version changed from 4.3 to 7.0
Updated by Gerrit Code Review over 5 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57763
Updated by Gerrit Code Review over 5 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57763
Updated by Gerrit Code Review over 5 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57763
Updated by Markus Klein over 5 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset df60fb73293d658211e9b0ebeeb5c4db8a969671.