Ticket #3236 (closed defect: invalid)

Opened 2 years ago

Last modified 1 year ago

Updating requirements for simplepie

Reported by: westi Assigned to: anonymous
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

10/13/06 10:15:39 changed by markjaquith

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.

10/13/06 10:30:22 changed by Viper007Bond

Well what functions does it need that could not be available? In all likelyhood, we could clone them.

10/13/06 14:21:27 changed by link92

* is_a() (which IIRC is already in compat) * sha1() * stream_get_meta_data()

Those are the only three I can think of.

10/13/06 14:22:17 changed by link92

How do you create a list, if what is in the WikiFormatting page doesn't work!?

10/13/06 14:26:20 changed by markjaquith

You have to put a space before each asterisk or number one.

  • test
  • test
  1. test
  2. test

It's like retarded Textile, but you get used to it. Use the preview function!

12/01/06 11:01:53 changed by matt

  • status changed from new to closed.
  • resolution set to invalid.

07/15/07 09:14:16 changed by Nazgul

  • milestone deleted.