Moved double-click check up before we create the user, so we don't get the annoying error message in the error log. Also, got rid of the from the error we spit out if something does go wrong, since this variable is not defined, so we'd get a database error, then an internal server error.
We use $n_past-days instead of :n_past_days becasuse he pgdriverbind variable emulation puts single-quotes around the n_past_daysinteger. Postgresql tries to turn '-1' into a date datatypeso now()-'-1' fails but now()- -1 works fine.By Dave Bauer in the process of migrating the openacs.org site.