Skip Menu |

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

Report information
The Basics
Id: 125142
Status: resolved
Priority: 0/
Queue: Net-SIP
People
Owner: Nobody in particular
Requestors: cpan [...] spidererrol.co.uk
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 0.814
Fixed in: (no value)

History Show all quoted text
  Fri Apr 20 05:58:36 2018 cpan [...] spidererrol.co.uk - Ticket created
Subject: Documentation for Net::SIP::Util::sip_uri2parts() is wrong.
sip_uri2parts is documented as returning (DOMAIN, USER, PROTO, DATA, PARAM) in array context but actually returns (DOMAIN, USER, PROTO, PARAM, DATA) - ie PARAM and DATA are swapped. In my opinion the documentation is wrong and the code is returning them in a more sensible order. Also updating the documentation will avoid breaking any code which is already relying on the current behaviour.
  Mon Apr 23 05:11:24 2018 Steffen_Ullrich [...] genua.de - Correspondence added
Show quoted text
> In my opinion the documentation is wrong and the code is returning > them in a more sensible order. Also updating the documentation will > avoid breaking any code which is already relying on the current > behaviour.
Thanks for reporting the issue. I've fixed the documentation in commit 234767c.
  Mon Apr 23 05:11:25 2018 The RT System itself - Status changed from 'new' to 'open'
  Mon Apr 23 05:11:25 2018 Steffen_Ullrich [...] genua.de - Status changed from 'open' to 'resolved'