? ? Success

User tests: Successful: Unsuccessful:

avatar wilsonge
wilsonge
19 Dec 2015

Previously we appeared to refrain from updating to 1.3 because of b/c issues. However 1.3.1 appeared to solve these from wordpress' experience (https://core.trac.wordpress.org/ticket/21183#comment:16).

Loading through composer also makes things easier for future updates (if there are any - the last release was 2012).

Honestly I don't know how to test this - it's going to rely on people who previously wrote simplepie specific scripts to test as all of core now uses the internal JFeed classes.

avatar wilsonge wilsonge - open - 19 Dec 2015
avatar wilsonge wilsonge - change - 19 Dec 2015
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 19 Dec 2015
Labels Added: ?
avatar brianteeman brianteeman - change - 19 Dec 2015
Category External Library
avatar mbabker
mbabker - comment - 19 Feb 2016

@wilsonge Rebase or close this. You're probably better off doing the latter, this ain't going anywhere anyway.

avatar Hackwar
Hackwar - comment - 23 Feb 2016

I think, I agree with @mbabker here. This might be a good improvement, but I actually would rather remove simplepie altogether instead of updating an unused class.

This would be a good situation to have separately managed extensions for libs like this.

avatar photodude
photodude - comment - 23 Feb 2016

@wilsonge I know there are other burning issues, but when you get a chance could you fix the conflicts so this could be tested?

avatar brianteeman brianteeman - change - 12 Mar 2016
Status Pending Needs Review
avatar ghazal
ghazal - comment - 7 May 2016

SimplePie has been updated to 1.4.
https://github.com/simplepie/simplepie/blob/master/library/SimplePie.php
Version 1.2 throws Deprecated errors (functions not made "public" in this version).
Is simplepie going to be removed from next joomla package as it is announced ?


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8735.

avatar joomla-cms-bot joomla-cms-bot - change - 7 May 2016
Labels Added: ?
avatar wilsonge
wilsonge - comment - 7 May 2016

Updated. I've kept it as 1.3.1 because I can't find any information as to what is contained in 1.4.0 anywhere - the website still lists 1.3.1 as the latest version (although obviously the github tag exists for 1.4). Until I'm convinced we arent going to hit b/c breaks i'm happier sitting on 1.3.1

avatar wilsonge
wilsonge - comment - 7 May 2016

But yes in J4 we will still remove simplepie

avatar roland-d roland-d - reference | 799ad27 - 7 May 16
avatar roland-d roland-d - merge - 7 May 2016
avatar roland-d roland-d - close - 7 May 2016
avatar roland-d roland-d - change - 7 May 2016
Status Needs Review Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-05-07 13:36:17
Closed_By roland-d
avatar roland-d
roland-d - comment - 7 May 2016

Merged on review.

avatar wilsonge wilsonge - head_ref_deleted - 7 May 2016
avatar mbabker
mbabker - comment - 7 May 2016

@wilsonge You need a PR updating the deleted files list.

Add a Comment

Login with GitHub to post a comment