I am not really a programmer but I felt like seeing if I was able to port some modules... here is the patch that I came up with.

Known issues:

Note that the views integration is no longer working... at least with Views 6.x-2.0-rc1. I also noted the following errors:

* When visiting the node/add/update-status-aggregator-site page:

notice: Undefined property: stdClass::$drupal_key in /var/alternc/html/o/omar/drupaldemo/drupal-6.x-dev/sites/all/modules/update_status_aggregator/update_status_aggregator.module on line 443.

* When viewing admin/content/node after running cron:
notice: Undefined index: short_name in /var/alternc/html/o/omar/drupaldemo/drupal-6.x-dev/sites/all/modules/update_status_aggregator/update_status_notifier.module on line 78.

Comments

omar’s picture

... oh and I'd sure like to have the db schema stuff in the .install checked out... I'm not sure that it is quite right.

EgonO’s picture

i'm working on a complete port. the d6 notifier component works right now with a d5 aggregator site...

next to do is the aggregator part for d6 with views integration...

yrocq’s picture

Hi EgonO,

Thanks for your interest in this module.

I have begun to work on on a version 2 which won't use nodes to store sites and updates. You can check it in the CVS. It's a good thing if we can make the notifier of version 1 working on D6, but I don't think that we should try to port the version 1 of the aggregator to D6 because version 2 will replace it.

brianbrown’s picture

subscribing

EgonO’s picture

Status: Needs work » Needs review
StatusFileSize
new2.99 KB

Here is a working Patch for the notifier-part for drupal 6. Using it on 2 productions sites. aggregator still is unported.

Its against DRUPAL-5--2 Branch from CVS

j0nathan’s picture

subscribing

j0nathan’s picture

Hi,
Any news on this?

steven jones’s picture

Status: Needs review » Closed (won't fix)

There is a re-write for Drupal 6 that is supported.