http://qs321.pair.com?node_id=702609


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

Replies are listed 'Best First'.
Re^2: Business::ISSN - where is is_valid_checksum
by toolic (Bishop) 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.