Last modified: 2011-03-04 14:00:02 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T29781, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 27781 - Installer does not warn when webserver has PHP 5.1.x only
Installer does not warn when webserver has PHP 5.1.x only
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Installer (Other open bugs)
1.17.x
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks: 26676
  Show dependency treegraph
 
Reported: 2011-02-28 11:42 UTC by Raimond Spekking
Modified: 2011-03-04 14:00 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Raimond Spekking 2011-02-28 11:42:53 UTC
Installer does not warn when webserver has PHP 5.1.x only.

I run into this trap after a successfully installation of 1.17 (trunk) on a webserver with PHP 5.1.x only.

Fatal error: Call to undefined function array_fill_keys() in /var/www/html/kunden/ztd/w/includes/resourceloader/ResourceLoader.php on line 516

array_fill_keys() was introduced in PHP 5.2.0
Comment 1 Sam Reed (reedy) 2011-02-28 14:29:59 UTC
http://www.mediawiki.org/wiki/Installation says we support 5.1 or later (5.2 preferred). Looks like it needs to be 5.2 or later, or fix that method call
Comment 2 Raimond Spekking 2011-03-01 14:25:15 UTC
It seems easily to add an wrapper for this function, see the note in http://php.net/manual/en/function.array-fill-keys.php if we want to support PHP 5.1.x.
Comment 3 Chad H. 2011-03-01 14:34:02 UTC
I think we should support 5.1.x for the last time in 1.17, providing a back-compat version (or fixing the array_fill_keys functions to do something else).

For 1.18/trunk, I think we should bump the requirement to 5.2.x (5.3.x preferred, of course).
Comment 4 Chad H. 2011-03-04 14:00:02 UTC
Done in r83216, r83217.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links