[development] Re: [Drupal bug] Aggregator module using wrong unique ID for feed items?

[Steve Dondley]

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 at 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