comment on

I find pc88mxer's astute discovery astonishing.

Navigating to CPAN and searching for Business::ISSN reveals that this module was uploaded nearly 10 years ago. However, there are no bugs reported against it. Well, perhaps no one (other than the author) has ever tried to use the module. This is reasonable since a person has to have the need for such a thing, then the person has to search and find it.

But, how on earth has it passed the rigors of CPAN testing? It has 46 passes and no fails on a variety of Perl versions and platforms.

So, I downloaded and installed this module myself. Sure enough, it passes its tests and installs cleanly. Curious.

Hmm, let me take a look at what is checked. The one and only exportable function is not checked!

CaMelRyder, please report this as a bug on CPAN. Perhaps someone came before you and did not take the time to do so.

Morals of this story:

Testing, testing, testing.
Scrutinize the source code.
Be a good citizen and report bugs.

In reply to Re: Business::ISSN - where is is_valid_checksum by toolic
in thread Business::ISSN - where is is_valid_checksum by CaMelRyder

Are you posting in the right place? Check out Where do I post X? to know for sure.
Posts may use any of the Perl Monks Approved HTML tags. Currently these include the following:
<code> <a> <b> <big> <blockquote> <br /> <dd> <dl> <dt> <em> <font> <h1> <h2> <h3> <h4> <h5> <h6> <hr /> <i> <li> <nbsp> <ol> <p> <small> <strike> <strong> <sub> <sup> <table> <td> <th> <tr> <tt> <u> <ul>
Snippets of code should be wrapped in <code> tags not <pre> tags. In fact, <pre> tags should generally be avoided. If they must be used, extreme care should be taken to ensure that their contents do not have long lines (<70 chars), in order to prevent horizontal scrolling (and possible janitor intervention).
Want more info? How to link or How to display code and escape characters are good places to start.


Welcome to the Monastery
	PerlMonks