Skip Menu |
 

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Validation-Class CPAN distribution.

Report information
The Basics
Id: 69893
Status: open
Priority: 0/
Queue: Validation-Class

People
Owner: Nobody in particular
Requestors: carnil [...] debian.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Small spelling error in manpage
Date: Fri, 29 Jul 2011 18:07:21 +0200
To: bug-validation-class [...] rt.cpan.org
From: Salvatore Bonaccorso <carnil [...] debian.org>
Download (untitled) / with headers
text/plain 302b
In Debian we are currently applying the attached patch to Validation-Class. We thought you might be interested in it, too. It is really small. Description: Fixing man page Author: Fabrizio Regalli <fabreg@fabreg.it> Last-Update: 2011-07-07 Thanks in advance, Salvatore Bonaccorso, Debian Perl Group
Download spelling-error.diff
text/x-diff 584b

Message body is not shown because sender requested not to inline it.

Subject: Re: [rt.cpan.org #69893] Small spelling error in manpage
Date: Fri, 29 Jul 2011 15:16:41 -0400
To: bug-Validation-Class [...] rt.cpan.org
From: "Al Newkirk & Associates" <we [...] ana.im>
Download (untitled) / with headers
text/plain 1.9k
why? also, why change .. # you should note that if the params dont exist they will return undef into ... # you should note that if the params don't exist they will return undef and leave ... # meaning you should check that it exists before checking its value instead of ... # meaning you should check that it exists before checking it's value unless of-course, you missed it like I did. I suck at sPlLng, especially in code documentation because I'm usually rushing but thank you and I will try to do better. -Al On Fri, Jul 29, 2011 at 12:07 PM, Salvatore Bonaccorso via RT < bug-Validation-Class@rt.cpan.org> wrote: Show quoted text
> Fri Jul 29 12:07:57 2011: Request 69893 was acted upon. > Transaction: Ticket created by carnil@debian.org > Queue: Validation-Class > Subject: Small spelling error in manpage > Broken in: (no value) > Severity: (no value) > Owner: Nobody > Requestors: carnil@debian.org > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=69893 > > > > In Debian we are currently applying the attached patch to Validation-Class. > We thought you might be interested in it, too. It is really small. > > Description: Fixing man page > Author: Fabrizio Regalli <fabreg@fabreg.it> > Last-Update: 2011-07-07 > > > Thanks in advance, > Salvatore Bonaccorso, Debian Perl Group > > > Description: Fixing man page > Author: Fabrizio Regalli <fabreg@fabreg.it> > Last-Update: 2011-07-07 > Forwarded: no > > --- a/lib/Validation/Class.pm > +++ b/lib/Validation/Class.pm > @@ -1732,7 +1732,7 @@ > my ($name, $email, $login, $password) = > $self->get_params(qw/name email login password/); > > - # you should note that if the params dont exist they will return > undef > + # you should note that if the params don't exist they will return > undef > # ... meaning you should check that it exists before checking its > value > # e.g. > > >
-- Al Newkirk & Associates Internet Business Consultants http://ana.im we@ana.im 215 469 1262
Subject: Re: [rt.cpan.org #69893] Small spelling error in manpage
Date: Fri, 29 Jul 2011 21:37:17 +0200
To: Al Newkirk & Associates via RT <bug-Validation-Class [...] rt.cpan.org>
From: Salvatore Bonaccorso <salvatore.bonaccorso [...] gmail.com>
Download (untitled) / with headers
text/plain 1.4k
Hi Al first of all thanks to reply so quickly to my report. I'm not a native english speaker so ... On Fri, Jul 29, 2011 at 03:16:58PM -0400, Al Newkirk & Associates via RT wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=69893 > > > why? > > also, why change .. # you should note that if the params dont exist they > will return undef > into ... # you should note that if the params don't exist they will return > undef > and leave ... # meaning you should check that it exists before checking its > value > instead of ... # meaning you should check that it exists before checking > it's value > > unless of-course, you missed it like I did. I suck at sPlLng, especially in > code > documentation because I'm usually rushing but thank you and I will try to do > better.
As far as I know "don't" is correct, or "do not", but not "dont". I agree this is really small, but I wanted to report anyway so if you are interested, you can change it. For the second -- I have not looked at it -- there is a difference in "it's" vs "its". It's is a contraction of "it is" or "it has." Its is the possessive form of "it." See http://www.elearnenglishlanguage.com/difficulties/its.html The idea was not to be "nit-picking" but only reporting, as we applied that to our packaging and forwarding back to you. Many thanks for your work on Validation-Class and other modules! Regards Salvatore p.s.: keep the warning in mind that I'm not a native english speaker.
Download signature.asc
application/pgp-signature 836b

Message body not shown because it is not plain text.



This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.