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);
CommentFileSizeAuthor
sin.ca_.inc_.patch1.3 KBDave Kinchlea

Comments

Dave Kinchlea’s picture

Status: Active » Needs review

I think this status better reflects things

thierry_gd’s picture

Status: Needs review » Fixed

Fixed in 6.2.6 version

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.