Pending

User tests: Successful: Unsuccessful:

avatar elinw
elinw
12 Aug 2012
avatar elinw elinw - open - 12 Aug 2012
avatar elinw
elinw - comment - 12 Aug 2012

Replaces #301

joomla/joomla-platform#1431 deals with some notices that show if you have error reporting on and put in a bad url.

Feature tracker:
http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=28905

avatar realityking
realityking - comment - 12 Aug 2012

JFeed will need some love to get this ready, just from a casual look I saw it's using JXMLElement everywhere which is deprecated. The alternative is SimpleXMLElement. Also we have to change the docblocks to make this a CMS library.

avatar elinw
elinw - comment - 12 Aug 2012

JFeed is actually working fine, I've done a lot of testing and it does the
simple tasks required of it in com_newsfeed and mod_feed as well as
simplepie. You're right about the docblocks though, forget to do them this
round. Will send an update in a few hours.

On Sun, Aug 12, 2012 at 12:13 PM, Rouven Weßling
notifications@github.comwrote:

JFeed will need some love to get this ready, just from a casual look I saw
it's using JXMLElement everywhere which is deprecated. The alternative is
SimpleXMLElement. Also we have to change the docblocks to make this a CMS
library.


Reply to this email directly or view it on GitHub#306 (comment).

avatar mbabker
mbabker - comment - 12 Aug 2012

It's not that the code doesn't work, we're just trying to not introduce any deprecated code usage into the CMS. Switching from JXMLElement to SimpleXMLElement is as simple as a find/replace action.

avatar elinw
elinw - comment - 13 Aug 2012

@realityking It would be very helpful to have the needed fix to JHttpTransportSocket that has been sitting in the platform tracker for a while merged.

Ok updated and should be working

avatar elinw
elinw - comment - 13 Aug 2012

oops I thought I pushed that 12 hours ago

avatar elinw
elinw - comment - 25 Aug 2012

merge

avatar elinw elinw - close - 25 Aug 2012
avatar elinw elinw - head_ref_deleted - 17 Jan 2013

Add a Comment

Login with GitHub to post a comment