Beefy Boxes and Bandwidth Generously Provided by pair Networks
Problems? Is your data what you think it is?
 
PerlMonks  

Re: Business::ISSN - where is is_valid_checksum

by brian_d_foy (Abbot)
on Aug 06, 2008 at 11:39 UTC ( #702609=note: print w/ replies, xml ) Need Help??


in reply to Business::ISSN - where is is_valid_checksum

I've taken over the module (with Sami's permission), fixed the bug, and uploaded Business-ISSN-0.91.

The code is fully tested now, but if you find any more problems, please post them to RT as you did with this report.

Curiously, the ISSN code was derived from Business::ISBN, the second module I ever wrote. Looking at the code was going back in the time to see many of the worst practices I used to have, so updating the module gave me a chance to undo all of that and make amends. :)

Good luck :)

--
brian d foy <brian@stonehenge.com>
Subscribe to The Perl Review


Comment on Re: Business::ISSN - where is is_valid_checksum
Re^2: Business::ISSN - where is is_valid_checksum
by toolic (Chancellor) on Aug 07, 2008 at 17:23 UTC
    It is collaboration efforts like this that make Perl, in general, and PerlMonks, in particular, a great community.
    • pc88mxer for diagnosing the problem and providing a specific bug fix,
    • CaMelRyder for submitting the bug report,
    • and brian_d_foy for assuming ownership of the module, implementing the bug fix, creating the test to check the bug fix, creating more tests to check other things, and finally uploading the new version to CPAN.
    No one was under any obligation to do any of this. That's what's so cool about it.

    ++'s all around.

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://702609]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others about the Monastery: (15)
As of 2014-07-29 13:26 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    My favorite superfluous repetitious redundant duplicative phrase is:









    Results (217 votes), past polls