ereg deprecated in PHP 5.3, remove from password checking function

please confirm if you are using php5.3+ . seems like it is the cause of the deprecated warning.

Please downgrade php to 5.2.x. Read this for more information http://drupal.org/requirements

actually, i think we'll need to support PHP 5.3 for the 7.x version...

in any case, a deprecation warning is not critical -- things should function just fine.

ok, I just came across this http://drupal.org/node/360605 which show an work in progress patch for php5.3 compatibility.

according to http://devthought.com/tumble/2009/06/fix-ereg-is-deprecated-errors-in-ph... attached patch should fix the issue. any regex wizards want to confirm my work before i commit?

I just tried the patch, it seems to be working.
wondering if those two preg_match statements may be combined into one (line 1063-1064, logintoboggn.module).

i didn't originally write that section of code. if they can be combined, feel free to combine them and submit a patch :)

patch attached :)

i'm a little bit confused about this latest patch. it doesn't appear to address the same character sequences, and the [:graph:] and space characters seem to be missing. have these somehow been incorporated into the new character sequence?

Confirm patch makes the error go away on D7b2 & PHP5.3 with no apparent ill effects.

@sudhirporwal: i need an answer to my question in #12 before i can proceed with this fix.

[edited to make the explanation correct]
@hunmonk
while merging them into one preg sequence , I have aggregated everything into a +ve match, hence making [:graph:] character class unnecessary. The aggregated sequence only checks for the unnecessary charaters

To be honest , As per best of my knowledge this code works. I have tested the patch many times and can confirm that it does stop commonly used invalid characters in the passwords.

One unrelated question, even though I am member of the project, how come I don't get email notifications when an issue gets a reply. Please help me with this, as I assume I am doing something very silly here.

the [:graph:] character class does include the valid characters, so we need not to check for its inclusion

committed to HEAD, 6.x-1.x-dev, and 5.x-1.x-dev -- thanks!

@sudhirporwal: for issue emails, check your subscription settings for the project at http://drupal.org/project/issues/subscribe-mail/logintoboggan