#30 Email Validation Bug

Version 3.2.0
closed
Sam Snow
5
2006-12-25
2006-10-11
Gabriel
No

Hello. I am reporting a bug Onlinegrades. Although
this bug is in 3.2.2

The bug is something to do with email validation. It
does not allow .info extensions

Could you please fix this as soon as possible. Thanks.

Discussion

  • Sam Snow

    Sam Snow - 2006-10-12
    • labels: 724065 --> Administration Interface
     
  • Sam Snow

    Sam Snow - 2006-10-12

    Logged In: YES
    user_id=449920

    Reproduced!

    Further info:
    The php validation is fine. It is the javascript validation
    that needs updating.

    Sam

     
  • Sam Snow

    Sam Snow - 2006-10-13
    • assigned_to: nobody --> snowsam
    • status: open --> pending
     
  • Sam Snow

    Sam Snow - 2006-10-13

    Logged In: YES
    user_id=449920

    The javascript email validation was used in 8 locations:
    ---------- Find in Files ----------
    > Searching for the string
    '"(@.*@)|(\\.\\.)|(@\\.)|(\\.@)|(^\\.)";'...
    C:\www\webroot\onlinegrades\mailto.php
    C:\www\webroot\onlinegrades\admin\addadmin_f.php
    C:\www\webroot\onlinegrades\admin\addclient_f.php
    C:\www\webroot\onlinegrades\admin\add_student_f.php
    C:\www\webroot\onlinegrades\admin\admin_update.php
    C:\www\webroot\onlinegrades\admin\client_update.php
    C:\www\webroot\onlinegrades\admin\student_update.php
    C:\www\webroot\onlinegrades\parents\mailto.php
    > 8 occurrence(s) have been found.

    TO fix the issue by hand you have to edit each one of these
    and find the section that looks like:

    var str = field.value;
    if (window.RegExp) {
    var reg1str = "(@.*@)|(\\.\\.)|(@\\.)|(\\.@)|(^\\.)";
    var reg2str =
    "^.+\\@(\\[?)[a-zA-Z0-9\\-\\.]+\\.([a-zA-Z]{2,3}|[0-9]{1,3})(\\]?)$";
    var reg1 = new RegExp(reg1str);
    var reg2 = new RegExp(reg2str);
    if (!reg1.test(str) && reg2.test(str)) {
    return true;
    }

    and replace it with
    var str = field.value;
    if (window.RegExp) {
    var
    filter=/^([\w-]+(?:\.[\w-]+)*)@((?:[\w-]+\.)*\w[\w-]{0,66})\.([a-z]{2,6}(?:\.[a-z]{2})?)$/i
    if (filter.test(str)) {
    return true;
    }

    I have commited this change to CVS, so they will be
    available in the next released version > 3.2.2.

     
  • Gabriel

    Gabriel - 2006-10-13
    • status: pending --> open
     
  • Gabriel

    Gabriel - 2006-10-13

    Logged In: YES
    user_id=1600409

    Sam, thanks. Well, my web-host has broken hardware and is
    in the process of getting that fixed. I went through them
    all and changed them. So, hopefully when I get to upload
    the new files tonight, they will work. That is on my
    computer.

     
  • Sam Snow

    Sam Snow - 2006-12-10
    • status: open --> pending
     
  • SourceForge Robot

    • status: pending --> closed
     
  • SourceForge Robot

    Logged In: YES
    user_id=1312539
    Originator: NO

    This Tracker item was closed automatically by the system. It was
    previously set to a Pending status, and the original submitter
    did not respond within 14 days (the time period specified by
    the administrator of this Tracker).

     

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks