Opened 3 years ago
Closed 3 years ago
#1564 closed Defect (fixed)
consider making 5.2 minimum php version
| Reported by: | ewinslow | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | Elgg 1.8.0 |
| Component: | Installation | Version: | 1.6 |
| Severity: | minor | Keywords: | |
| Cc: | brettp, ewinslow | Difficulty: |
Description
For installations running older versions of php, this means export/api functionality for json can't work without editing core. Solution would be to include a third party json encoding library such as Zend in the /vendors/ folder and use that function instead. Zend in particular defaults to the native php encoder/decorder when available, so there should be no performance hit for php 5.3 enabled installations.
Change History (4)
comment:1 Changed 3 years ago by cash
- Severity changed from major to minor
- Summary changed from json_encode only available in php 5.3 to consider making 5.2 minimum php version
comment:2 Changed 3 years ago by ewinslow
Well check that out, you're right! We're farther behind than I thought...
comment:3 Changed 3 years ago by cash
- Component changed from Core to Installation
http://github.com/cash/Elgg makes 5.2 the minimum version and does a check for json
comment:4 Changed 3 years ago by cash
- Resolution set to fixed
- Status changed from new to closed
New installer checks for PHP 5.2 and above
Note: See
TracTickets for help on using
tickets.
json_encode has been available since php 5.2. The Elgg installer used to require that php be >= 5.2. Maybe that should be the default threshold again?
For version of php that pass the current installer but do not have json, it can be installed from pecl: pecl install json