Emails are case sensitive???

grv

Found out today that the reason many people are getting login errors is the email address is case sensitive. If their email is Cool@xxx.com for example, if they try to login under cool@xxx.com it won't work.

Is there a fix for this???

svardaman

Find this on the source code:

$email=trim($_POST);

and use functions like strtolower() to fix it.

On most email servers email addresses are not case sensitive, and it generally considered good form to always use lower case.

http://email.about.com/od/netiquettetips/qt/et061705.htm(http://email.about.com/od/netiquettetips/qt/et061705.htm)

altjx

Anyone can show me how to change this? I have this problem as well and have been researching for awhile now.

consumedsoul

Anyone can show me how to change this? I have this problem as well and have been researching for awhile now.

Ditto - we'd like to get the emails not case-sensitive - please advise.

scottro

Ditto - we'd like to get the emails not case-sensitive - please advise.

Change:

$email=trim($_POST);

To:

$email=strtolower(trim($_POST));

ntozier

As a side note, you would also want to go through your database (specifically the ost_ticket table, and normalize the email field. So that all the emails currently stored are in lower case.

You could write a script to do this for you. Or if you can get to a mysql command line, you could simply run:

UPDATE ost_ticket SET email = LOWER(email);

scottro

As a side note, you would also want to go through your database (specifically the ost_ticket table, and normalize the email field. So that all the emails currently stored are in lower case.
You could write a script to do this for you. Or if you can get to a mysql command line, you could simply run:
UPDATE ost_ticket SET email = LOWER(email);

Ah yes! Good point! (We don't use the customer login page so I don't think too deeply about requests like these ;)

consumedsoul

login.php (in root of install directory) Line 28
Change:
$email=trim($_POST);
To:
$email=strtolower(trim($_POST));

Awesome, thanks!

ntozier

I was thinking about this some more over lunch. Wouldn't we want to go in and edit the include/client/open.inc.php also to ensure that data gets injected as lowercase?

scottro

I was thinking about this some more over lunch. Wouldn't we want to go in and edit the include/client/open.inc.php also to ensure that data gets injected as lowercase?

Yep, that makes sense - otherwise they'd have to enter it in exactly as they did when the created the ticket (if created via the webpage not an email).

freddyvangeel

Yep, that makes sense - otherwise they'd have to enter it in exactly as they did when the created the ticket (if created via the webpage not an email).

So how do I do this?

freddyvangeel

Found the solution.

In class.ticket.php find this code (twice):

',email='.db_input($var).

And replace with

',email='.db_input(strtolower($var)).

You have to do this twice, once for the "new" tickets and once for the "update function"

freddyvangeel

I was thinking about this some more over lunch. Wouldn't we want to go in and edit the include/client/open.inc.php also to ensure that data gets injected as lowercase?

just to complete this; it's not open.inc.php that needs to be edited..

you should do this in include/class.ticket.php

ntozier

Doh! I completely forgot to come back and post to this thread yesterday. I came up with basically the same thing.

edit

/include/client/open.inc.php

circa line 36 through line 38 change

<input type="hidden" name="email" size="25" value="<?=$email?>"><?=$email?>

<?}else {?>

<input type="text" name="email" size="25" value="<?=$info?>">

<input type="hidden" name="email" size="25" value="<?=strtolower($email)?>"><?=strtolower($email)?>

<?}else {?>

<input type="text" name="email" size="25" value="<?=strtolower($info)?>">

This makes sure that if they fill out the form wrong, that it echos back the lowercase email address.

Also edit

/include/class.ticket.php

circa line 1131 and 1320 change

',email='.db_input($var).

',email='.db_input(strtolower($var)).

This ensures that the email address is lower case when it is injected into the database.

Sorry for the delay.