Skip Menu |

This queue is for tickets about the Net-STOMP-Client CPAN distribution.

Report information
The Basics
Id: 73060
Status: resolved
Priority: 0/
Queue: Net-STOMP-Client
People
Owner: Nobody in particular
Requestors: TJC [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 1.3
Fixed in: 1.4

History Show all quoted text
  Tue Dec 06 20:16:15 2011 TJC [...] cpan.org - Ticket created
Subject: Incorrect documentation for unsubscribe
In Net::STOMP::Client the POD gives this example of using unsubscribe: $stomp->unsubscribe(destination => "/queue/test"); However, unsubscribe actually takes an "id" parameter, not a destination parameter.
  Tue Dec 06 20:47:07 2011 TJC [...] cpan.org - Correspondence added
I should mention that the subscribe example seems incorrect too. It must include an 'id' parameter.
  Tue Dec 06 20:47:08 2011 TJC [...] cpan.org - Status changed from 'new' to 'open'
  Wed Dec 07 02:38:13 2011 LCONS [...] cpan.org - Correspondence added
Toby, This is not really incorrect since it works fine with STOMP 1.0. Only STOMP 1.1 requires the "id" header field. Anyway, I have modified the example so that the same code works fine with STOMP 1.0 and STOMP 1.1. The doc now contains: # subscribe to the given queue $stomp->subscribe( destination => "/queue/test", id => "testsub", # required in STOMP 1.1 ack => "client", # client side acknowledgment ); [...] $stomp->unsubscribe(id => "testsub"); This will be part of the next release. Thanks for your report.
  Wed Dec 07 02:38:14 2011 LCONS [...] cpan.org - Status changed from 'open' to 'patched'
  Fri Jan 27 02:16:17 2012 LCONS [...] cpan.org - Correspondence added
The patch is part of 1.4, just released in CPAN.
  Fri Jan 27 02:16:18 2012 The RT System itself - Status changed from 'patched' to 'open'
  Fri Jan 27 02:16:18 2012 LCONS [...] cpan.org - Status changed from 'open' to 'resolved'
  Fri Jan 27 07:58:42 2012 LCONS [...] cpan.org - Fixed in 1.4 added