Opened 7 years ago

Closed 6 years ago

Last modified 6 years ago

#3236 closed defect (bug) (invalid)

Updating requirements for simplepie

Reported by: westi Owned by: 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 (7)

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.

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!?

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!

comment:6   matt6 years ago

  • Resolution set to invalid
  • Status changed from new to closed
  • Milestone 2.1 deleted
Note: See TracTickets for help on using tickets.