What is still missing (todo list for this week):

  • tests
  • overridable deduping
  • Add enclosure and namespace support to the parser
  • Add meaningful and helpful information and error messages
  • Kill hard-coded values
  • Create content-type at install stage with meaningful configuration

Comments

alex_b’s picture

Thank you.

Could you break down which tests there are to do and add an hour estimate to all items of your Todo list?

aron novak’s picture

One more: make drupal_execute better (remain silent, try it out that how it behaves in general, etc)
Tests detailed:
- test for feeds: tons of real-world test feeds (all of the types).Test cases should trigger the problematic points of the feed. The feeds should contain extremely big feeds too. Example: input: foo.xml, test: assertEqual($node>-feed->items[0]->namespaces['dc']['foo'], 'bar)
- tests for feed type autodetection Example: input: url1, test: assertEqual(detected_url, url2)
- test for the processors: function-based tests where possible (for example: input: feed item node, test: create it via aggregator_node's own function)
- test for the aggregator: general tests: creating a feed from 0 via a test (like feedapi does the testing)

aron novak’s picture

Updated todo list:

  • tests
  • overridable deduping
  • add enclosure and namespace support to the parser
  • add meaningful and helpful information and error messages
  • purging variables at uninstall
aron novak’s picture

Updated todo list:

  • Good default deduping method - done (existing aggregator 's method. But maybe replaced with even better...)
  • Implement two menu items: aggregator/rss and aggregator/opml - done
  • Make syndication_parser feed tests optional
  • Eliminate the warning when saving items with node processor and taxonomy inherit is turned on. (it works ok, but warnings appear)
  • Implement feed refresh frequency - done
aron novak’s picture

Category: task » bug
Status: Active » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.