Closed (fixed)
Project:
Social Insurance Number (CCK)
Version:
6.x-2.6
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
17 Sep 2009 at 20:31 UTC
Updated:
4 Jan 2010 at 16:30 UTC
When my own SIN number wouldn't validate as a default value, I knew something was up :-)
This patch fixes the simple error and adds some watchdog support as well to assist with stupid user tricks (only invalid numbers are written to the log, still probably ought to be an option as is a slight security problem, no time right now however.
In any case, the code currently sets $sinstring but uses $sin, simple fix was to set $sin instead.
! $sin = trim($sinstring);
+ $sinstring = trim($sinstring);
| Comment | File | Size | Author |
|---|---|---|---|
| sin.ca_.inc_.patch | 1.3 KB | Dave Kinchlea |
Comments
Comment #1
Dave Kinchlea commentedI think this status better reflects things
Comment #2
thierry_gd commentedFixed in 6.2.6 version