Skip Menu |

This queue is for tickets about the DJabberd-Delivery-OfflineStorage CPAN distribution.

Report information
The Basics
Id: 30872
Status: open
Priority: 0/
Queue: DJabberd-Delivery-OfflineStorage

People
Owner: Nobody in particular
Requestors: misc [...] zarb.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 0.02
Fixed in: (no value)



Subject: InMemory plugin never remove message
It seems that DJabberd::Delivery::OfflineStorage::InMemory do not erase message after they were fetched on first login. Here is a patch to correct the problem.
Subject: djabberdofflinemessage.fix_inmemory.diff
--- lib/DJabberd/Delivery/OfflineStorage/InMemoryOnly.pm 2007-08-02 19:37:08.000000000 +0200 +++ lib/DJabberd/Delivery/OfflineStorage/InMemoryOnly.pm.new 2007-11-21 01:08:13.000000000 +0100 @@ -52,7 +52,7 @@ my $id = $self->{'offline_id'}++; $logger->info("InMemoryOnly OfflineStorage store for: $user/$id"); $self->{'offline'}{$user} ||= {}; - $self->{'offline'}{$id} = {'packet' => $packet, 'jid' => $user}; + $self->{'offline'}{$id} = {'id' => $id, 'packet' => $packet, 'jid' => $user}; $self->{'offline'}{$user}{$id} = 1; }
From: PIERS [...] cpan.org
On Tue Nov 20 19:14:47 2007, MISC wrote: Show quoted text
> It seems that DJabberd::Delivery::OfflineStorage::InMemory do not > erase message after they > were fetched on first login. > > Here is a patch to correct the problem.
Thanks - I have applied this and uploaded 0.03. Cheers.