A small bug at session_start.php

nomad · Aug 19th, 2009, 10:33 PM

Hi Ben,
just let you know there is a small bug at sessions_start.php I found after upgrade to last package version v2.0.0-beta-20090815.

This two lines must be on very top position of code:

$folder = dirname(__FILE__);
require_once("$folder/library.php");

Otherwise users will get php notice:
Notice: Undefined variable: g_session_type in ... session_start.php ... line 3 on each page.

Regards,

Pavel

***Ben*** · Aug 21st, 2009, 6:25 PM

Hi Pavel,

Thanks! Hmm.. odd, though. I don't get that notice myself, but looking at the code it seems like I should. Also, shifting the order can cause problems on some systems.

I'll put this down as a bug to investigate. Thanks!

- Ben

***Ben*** · Aug 26th, 2009, 8:52 PM

Well, I've looked into this and I can't see ANY reason why I coded it in that order... kinda worries me!

I'm releasing an update tonight - I'll include the fix to place the require_once("library.php") first, as it should be. We'll have to see if it causes any problems on other people's systems.

Erk! This is a strange one.

FORUMS