Skip Menu |

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>
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

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>
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>
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.