Posted by Dave Kinchlea on September 17, 2009 at 8:31pm
Jump to:
| Project: | Social Insurance Number (CCK) |
| Version: | 6.x-2.6 |
| Component: | Code |
| Category: | bug report |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
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);| Attachment | Size |
|---|---|
| sin.ca_.inc_.patch | 1.3 KB |
Comments
#1
I think this status better reflects things
#2
Fixed in 6.2.6 version
#3
Automatically closed -- issue fixed for 2 weeks with no activity.