Bug #41327
closed
jsInline removes closing brackets
Added by Simon Schaufelberger about 12 years ago.
Updated about 2 years ago.
Description
this code does NOT work:
page.jsInline {
10 = TEXT
10.value (
jquery(document).ready(function($) {
$('.pagination li').hover(
function(){}
);
});
)
}
broken output:
jquery(document).ready(function($) {
$('.pagination li').hover(
function(){}
but this one does (notice the last line):
page.jsInline {
10 = TEXT
10.value (
jquery(document).ready(function($) {
$('.pagination li').hover(
function(){});});
)
}
i guess TYPO3 somehow interprets the last closing brackets as the closing bracket for the 10.value which is really strange.
- Status changed from New to Accepted
Just reproduced this example with the current master. Also if this can maybe fixed, I don' think that the parser can ever work perfectly for such constructs if there isn't something like a CDATA section in XML for such kind of inline code to indicate the parser to not parse the inner content for syntax elements except the ending marker.
- Target version set to 7.2 (Frontend)
- Complexity set to nightmare
- Is Regression set to No
- Status changed from Accepted to Needs Feedback
- Assignee set to Simon Schaufelberger
Hi Simon, I am looking at your initial javascript code, and I think there is an error: shouldn't it be:
jquery(document).ready(function($) {
$('.pagination li').hover(
function(){$(this).not('.current').find('em').stop().animate({left: "0px"}, 300)},
function(){$(this).not('.current').find('em').stop().animate({left: "-270px"}, 300)} /*<< here a "}" seems necessary */
);
});
? The second code has the same problem, I think.
- Status changed from Needs Feedback to Accepted
- Assignee changed from Simon Schaufelberger to Benni Mack
- TYPO3 Version changed from 4.7 to 6.2
The bug is still present in 6.2.12 and seems indipendent from the lack of the "}" I noticed.
- Status changed from Accepted 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 http://review.typo3.org/39596
- Target version changed from 7.2 (Frontend) to 7.4 (Backend)
- Status changed from Under Review to Accepted
- Target version changed from 7.4 (Backend) to 7 LTS
- Target version deleted (
7 LTS)
- Description updated (diff)
- Description updated (diff)
- Description updated (diff)
- Description updated (diff)
- Assignee changed from Benni Mack to Christian Kuhn
- Status changed from Accepted to Under Review
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
- Status changed from Resolved to Closed
Also available in: Atom
PDF