[TYPO3-core] odd browser check in typo3/init.php

[Michael Stucki]

Hi Martin,

this patch changes the message slightly. I also had a look at 
t3lib_div::clientInfo().

In my opinion, this looks too much like a hack, so we should try to 
rewrite it from scratch again (anyone interested?)

- michael

Martin Kutschker wrote:
> Hi!
> 
> I found this 
> if (!$CLIENT['BROWSER'] && !(defined('TYPO3_cliMode') && TYPO3_cliMode)) {
> t3lib_BEfunc::typo3PrintError ('Browser error','You must use 4+ browsers with TYPO3!',0);
> exit;
> }
> 
> $CLIENT['BROWSER'] is set via t3lib_div::clientInfo() which seems to set $CLIENT['BROWSER'] only for version 4 or higher of IE and Netscape and allows all versions of Opera and Konqueror.
> 
> BTW, what is Safari?
> 
> IMHO the check is not very useful for modern Firefox/Netscape browsers. here a check for the Gecko rendering engine would make more sense.
> 
> And what's really need to be changed is the check in init.php. The version of the browser should be checked (only IE 6.0 is supported nowadays - flex forms!!!). And it would be great if the user would be presented with the results of the check. So in case of a wrong detection we can change the check.
> 
> Masi 


-- 
Use a newsreader! Check out
http://typo3.org/community/mailing-lists/use-a-news-reader/
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: browser_check.diff
Url: http://lists.netfielders.de/pipermail/typo3-team-core/attachments/20070121/1dd8a5b6/attachment.diff