#3236 closed defect (bug) (invalid)
Updating requirements for simplepie
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | General | Version: | 2.1 |
| Severity: | normal | Keywords: | |
| Cc: |
Description
Simplepie requires "PHP 4.3.x, 4.4.x, or any version of PHP5 greater than 5.0.3."
So we need to update the requirements on WP.org and in the code (http://trac.wordpress.org/browser/trunk/wp-settings.php#L49)
Also should we move http://trac.wordpress.org/browser/trunk/wp-includes/simplepie.php to http://trac.wordpress.org/browser/trunk/wp-includes/classes-simplepie.php to following the existing naming convention for included code.
Change History (7)
comment:1
markjaquith — 7 years ago
comment:2
Viper007Bond — 7 years ago
Well what functions does it need that could not be available? In all likelyhood, we could clone them.
- is_a() (which IIRC is already in compat)
- sha1()
- stream_get_meta_data()
Those are the only three I can think of.
How do you create a list, if what is in the WikiFormatting page doesn't work!?
comment:5
markjaquith — 7 years ago
You have to put a space before each asterisk or number one.
- test
- test
- test
- test
It's like retarded Textile, but you get used to it. Use the preview function!

Renaming now. Good call.
Regarding PHP version... we're using SimplePie (and were using MagpieRSS) for non-essential functionality at this point. I'm not opposed to upping WordPress' minimum PHP version (anyone using PHP versions that old is open to a myriad of security issues anyway), but I'm not sure that this is the change that should necessitate it. If they want to install it and do without the Dashboard feeds, why not let 'em?
Sounds like a good wp-hackers topic.