Bugs item #2855430, was opened at 2009-09-09 21:36
Message generated for change (Comment added) made by christian_boltz
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=937964&aid=2855430&group_id=191583
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core
Group: None
>Status: Closed
>Resolution: Fixed
Priority: 6
Private: No
Submitted By: Christian Boltz (christian_boltz)
Assigned to: Nobody/Anonymous (nobody)
Summary: setup.php: session support checked broken / too late
Initial Comment:
|Mike| just told me on IRC that he got an error message "Call to undefined function session_start() in .../common.php" when calling setup.php. He uses FreeBSD 7.2, which has a separate php5-session package.
The problem: setup.php includes common.php at the beginning, which executes a session_start().
There's already a check for session support in setup.php, but it is done _after_ including common.php.
A possible solution is to "define('POSTFIXADMIN', 1);" in setup.php which would avoid the session_start() call.
The question is if this breaks something ;-) -> needs testing before commiting to SVN
----------------------------------------------------------------------
>Comment By: Christian Boltz (christian_boltz)
Date: 2009-10-20 21:27
Message:
A small section at the beginning of common.php is the only place where the
define(POSTFIXADMIN) in setup.php could break anything at all. Tested, does
not break ;-) (except a "already defined" warning, also fixed in the
commit)
Also tested that setup.php now works as expected be using php.ini
disable_function.
Commited to SVN r733.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=937964&aid=2855430&group_id=191583
|