Bug #18793

Default Javascript browser detection doesn't work while using GMENU

Added by Regis TEDONE over 13 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2008-05-15
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
4.2
PHP Version:
5.2
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

Roll over on GMENU doesn't work in Typo3 4.2 with PHP 5.2.
The javascript error occurs : version not defined.

It seems that the rendered Javascript has not a break return after the beginning comment :

<script type="text/javascript">
/*<![CDATA[*/
<!--var browserName=navigator.appName;
..........................................................
Or it should be :

<script type="text/javascript">
/*<![CDATA[*/
<!--
var browserName=navigator.appName;
..........................................................

To correct this, just modify the file : typo3//sysext/cms/tslib/class.tslib_pagegen.php

at line 723 :

- $_scriptCode = '
+ $_scriptCode = '\n
(issue imported from #M8421)


Related issues

Related to TYPO3 Core - Bug #21390: Missing JS linebreak crashes output within IE 8.0 (non non-compatibility mode) Closed2009-10-28

Actions
#1

Updated by Francois Suter about 11 years ago

Can you try the patch I just attached to 0012378? It includes your proposed changes, plus some specific to 0012378.

#2

Updated by Alexander Opitz over 8 years ago

  • Category deleted (Communication)
  • Status changed from New to Closed
  • Target version deleted (0)

No response in over one year => closed.

Also available in: Atom PDF