FAQ
Edit report at https://pear.php.net/bugs/bug.php?id=20236&edit=1

  ID: 20236
  Updated by: ken@linux.ie
  Reported By: 4od@moonraking.com
  Summary: PPS number not validaing correctly for new 9 char
                     numbers
-Status: Open
+Status: Bogus
  Type: Bug
  Package: Validate_IE
  Operating System: Linux
  Package Version: 1.1.0a1
  PHP Version: 5.3.28
  Roadmap Versions:
  New Comment:

-Status: Open
+Status: Bogus
Please do not submit the same bug more than once. An existing
bug report already describes this very problem. Even if you feel
that your issue is somewhat different, the resolution is likely
to be the same. Because of this, we hope you add your comments
to the existing bug instead.

Thank you for your interest in PEAR.

also submitted as #20237


Previous Comments:
------------------------------------------------------------------------

[2014-04-07 18:14:45] moonraking

Description:
------------
The checkmod32 function needs to be updated to take into account a 9
char pps number




Test script:
---------------
var_dump( Validate_IE::ppsn('1297188CA') );



Expected result:
----------------
bool(true)

Actual result:
--------------
bool(false)

its failing in checkMod32().
For my own use I have reimplemented (successfully) using algorithm
linked from the wikipedia page on Irish PPS's
http://pmhsilva.com/2013/02/12/validate-irish-ppsn-with-javascript-jquery/


I feel checkMod32 should be changed to getCheckCode($number_part,
$check_code_part) and return the checksum. That way getCheckCode can be
used in validation code to inform the user what the checkcode might be,
maybe they entered up an I instead of an L. Yes, they could have
entered a number wrong, but every little helps....

shane

------------------------------------------------------------------------

Search Discussions

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
grouppear-bugs @
categoriesphp
postedApr 8, '14 at 7:38p
activeApr 8, '14 at 7:38p
posts1
users1
websitepear.php.net

1 user in discussion

Ken: 1 post

People

Translate

site design / logo © 2022 Grokbase