Port for Drupal 6
| Project: | Update status aggregator |
| Version: | 5.x-1.2 |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs review |
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.
| Attachment | Size |
|---|---|
| update_status_aggregator.patch | 16.5 KB |
#1
... 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.
#2
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...
#3
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.
#4
subscribing
#5
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
#6
subscribing