I'm going to hold off on making patches for the database and update files until I'm sure this patch will be approved. On 5/3/06, Zen <development@drupal.org> wrote:
Issue status update for http://drupal.org/node/61433 Post a follow up: http://drupal.org/project/comments/add/61433
Project: Drupal Version: 4.7.0 Component: aggregator.module Category: bug reports Priority: normal Assigned to: Anonymous Reported by: Steve Dondley Updated by: Zen -Status: patch (code needs review) +Status: patch (code needs work)
No idea about the issue, but if this requires a DB change, the patch should incorporate the changes to database.mysql (4.0 and 4.1), database.pgsql and updates.inc.
Thanks, -K
Zen
Previous comments: ------------------------------------------------------------------------
Tue, 02 May 2006 15:12:28 +0000 : Steve Dondley
For background, please see: http://drupal.org/node/61427
I don't know much about RSS but I think the aggregator should be using the *guid* of the feed item as the unique identifier and not the *url* of the feed item.
------------------------------------------------------------------------
Tue, 02 May 2006 15:59:01 +0000 : Steve Dondley
Attachment: http://drupal.org/files/issues/aggregator_dup.patch (3.5 KB)
OK, here's a first stab at a fix.
------------------------------------------------------------------------
Wed, 03 May 2006 06:35:19 +0000 : Steve Dondley
One other thing:
In addition to applying the patch, you have to create a new field in the aggregator_item table called "guid". It's a varchar with a length of 255.
-- Dondley Communications http://www.dondleycommunications.com Communicate or Die: American Labor Unions and the Internet http://www.communicateordie.com