From: Filipus K. <ch...@gm...> - 2010-11-10 00:11:19
|
It turned out the reason the check is "not working" is because nothing is working. tiki-install.php failed to parse with PHP 4, so people would get a parse error instead of the message saying requirements aren't met. I made the file parse in r30609, for Tiki 6.1, but as the change is not completely trivial, I'm not planning to backport to 5.x. On 2010-11-08 19:33, Marc Laporte wrote: > I will send you a site access with PHP4, you will see :-) > > > On Mon, Nov 8, 2010 at 6:59 PM, Filipus Klutiero<ch...@gm...> wrote: >> On 2010-11-06 12:00, Marc Laporte wrote: >>> Hi! >>> >>> I am working on this: >>> http://tiki.org/Testing+Tiki+installations+on+major+Shared+Hosting+companies >>> >>> And I tried to: >>> http://doc.tiki.org/1and1+install >>> >>> >>> Not a blocker for 6.0 but I would like to fix in 5.4 and 6.1 >>> >>> tiki-install.php has this code: >>> // Check that PHP version is at least 5 >>> if (version_compare(PHP_VERSION, '5.0.0', '<')) { >>> $title = 'PHP5 is required for Tiki 3.0'; >>> $content = '<p>Please contact your system administrator ( if you are >>> not the one ;) ).</p>'; >>> createPage($title, $content); >>> } >>> >>> >>> and tiki-setup.php >>> if (version_compare(PHP_VERSION, '5.0.0', '<')) { >>> header('location: tiki-install.php'); >>> exit; >>> } >>> >>> >>> But it's not working and this is leading to issues such as: >>> http://tiki.org/tiki-view_forum_thread.php?forumId=6&comments_parentId=35646 >>> >>> 1- How can this be fixed? >> Are you sure? I can't see why the one in tiki-setup.php wouldn't work, >> and I did a test that strongly suggests this is working. >>> 2- Can the proper version version number be used? (the minimum >>> requirement is not 5.0.0) >>> >>> Thanks! |