Bug #1017

[PATCH] Refactoring Aggregation Sidebars (Flickr, 43Things, etc)

Added by tim-digital-achievement-com - 839 days ago. Updated 465 days ago.

Status :Closed Start :
Priority :Low Due date :
Assigned to :scott - % Done :

0%

Category :sidebars
Target version :release2
Resolution :

invalid


Description

Every aggregating sidebar had the same code repeated over and over, and no aggregating sidebars handle errors in an intuitive manner. This patch adds an Aggregator class that encapsulates all of the feed handling, including (bad) Atom support and decent error reporting. Other feed based sidebar models can use the Aggregator as a superclass to reduce code duplication.

This patch additionally provides an AggregationController sidebar that will render any arbitrary RSS/Atom feed.

The Flickr and 43Things sidebar models were converted to use the Aggregator as a superclass because they had unit tests, and they were easily tested in a running Typo instance by the patch author.

Errors should be reported in the admin console, but getting them into the application log is a start.

History

08/24/2007 02:02 PM - neuro -

  • Status changed from New to Closed
  • Resolution set to invalid

Patch has been lost long ago in a trac migration.

Closed

Also available in: Atom PDF