Changeset 679

Show
Ignore:
Timestamp:
06/27/07 05:41:39 (18 months ago)
Author:
raphael
Message:

Fixed #435: Incorrect e-mail range or syntax! caused by new function names
Fixed #438: Incorrect e-mail range or syntax! when adding a reseller
Deleting from admin log everything older than 3 months wasn't supported by code
Corrected many spelling mistakes and made improvements on i18n

Location:
trunk
Files:
20 added
55 modified

Legend:

Unmodified
Added
Removed

  • trunk/CHANGELOG

    r678 r679  
    1111\_________________________________________________________________/ 
    1212 
     132007-06-26 Raphael Geissert 
     14        - GUI: 
     15                * Fixed #438: Incorrect e-mail range or syntax! when adding a reseller 
     16                * Deleting from admin log everything older than 3 months wasn't supported by code 
     17                * Corrected many spelling mistakes and made improvements on i18n 
     18 
    13192007-06-25 Raphael Geissert 
    1420        - SETUP: 
    … …  
    1723                * Fixed #432: Call to undefined function ispcp_domain_check() 
    1824                * Fixed #431: Undefined variable: data 
     25                * Fixed #435: Incorrect e-mail range or syntax! caused by new function names 
    1926 
    20272007-06-24 Raphael Geissert 

  • trunk/engine/ispcp_common_methods.pl

    r675 r679  
    11291129    if (!defined ($rwith)) { 
    11301130 
    1131         push_el(\@main::el, 'repl_tag()', "ERROR: Undefined input data, rwith: |$rwith| !"); 
     1131        push_el(\@main::el, 'repl_tag()', "ERROR: Undefined rwith!"); 
    11321132 
    11331133        return (-1, ''); 

  • trunk/engine/setup/ispcp-setup

    r678 r679  
    17451745 
    17461746        # 
    1747         # Configure the fastcgi.conf 
     1747        # Configure the fastcgi_ispcp.conf 
    17481748        # 
    17491749 

  • trunk/gui/admin/add_reseller.php

    r665 r679  
    447447        return false; 
    448448    } 
    449     if (chk_email(clean_input($_POST['email']))) { 
     449    if (!chk_email(clean_input($_POST['email']))) { 
    450450 
    451451        set_page_message( tr("Incorrect email range or syntax!")); 
    … …  
    541541        'TR_USERNAME' => tr('Username'), 
    542542        'TR_PASSWORD' => tr('Password'), 
    543         'TR_PASSWORD_REPEAT' => tr('Password repeat'), 
     543        'TR_PASSWORD_REPEAT' => tr('Repeat password'), 
    544544        'TR_EMAIL' => tr('Email'), 
    545545        'TR_UNLIMITED' => tr('unlimited'), 
    546         'TR_MAX_DOMAIN_COUNT' => tr('Max Domain count'), 
    547         'TR_MAX_SUBDOMAIN_COUNT' => tr('Max subdomain count'), 
    548         'TR_MAX_ALIASES_COUNT' => tr('Max aliases count'), 
    549         'TR_MAX_MAIL_USERS_COUNT' => tr('Max mail users count'), 
    550         'TR_MAX_FTP_USERS_COUNT' => tr('Max FTP users count'), 
    551         'TR_MAX_SQLDB_COUNT' => tr('Max SQL databases count'), 
    552         'TR_MAX_SQL_USERS_COUNT' => tr('Max SQL users count'), 
    553         'TR_MAX_TRAFFIC_AMOUNT' => tr('Max traffic amount [MB]'), 
    554         'TR_MAX_DISK_AMOUNT' => tr('Max disk amount [MB]'), 
     546        'TR_MAX_DOMAIN_COUNT' => tr('Domains limit'), 
     547        'TR_MAX_SUBDOMAIN_COUNT' => tr('Subdomains limit'), 
     548        'TR_MAX_ALIASES_COUNT' => tr('Aliases limit'), 
     549        'TR_MAX_MAIL_USERS_COUNT' => tr('Mail accounts limit'), 
     550        'TR_MAX_FTP_USERS_COUNT' => tr('FTP accounts limit'), 
     551        'TR_MAX_SQLDB_COUNT' => tr('SQL databases limit'), 
     552        'TR_MAX_SQL_USERS_COUNT' => tr('SQL users limit'), 
     553        'TR_MAX_TRAFFIC_AMOUNT' => tr('Traffic limit [MB]'), 
     554        'TR_MAX_DISK_AMOUNT' => tr('Disk limit [MB]'), 
    555555        'TR_PHP' => tr('PHP'), 
    556556        'TR_PERL_CGI' => tr('CGI / Perl'), 
    557557        'TR_JSP' => tr('JSP'), 
    558558        'TR_SSI' => tr('SSI'), 
    559         'TR_FRONTPAGE_EXT' => tr('Fronpage extensions'), 
     559        'TR_FRONTPAGE_EXT' => tr('Frontpage extensions'), 
    560560        'TR_BACKUP_RESTORE' => tr('Backup and restore'), 
    561561        'TR_CUSTOM_ERROR_PAGES' => tr('Custom error pages'), 

  • trunk/gui/admin/add_user.php

    r665 r679  
    231231        return false; 
    232232    } 
    233     if (chk_email($_POST['email'])) { 
     233    if (!chk_email($_POST['email'])) { 
    234234 
    235235        set_page_message( tr("Incorrect email range or syntax!")); 
    … …  
    276276                array( 
    277277                        'TR_EMPTY_OR_WORNG_DATA' => tr('Empty data or wrong field!'), 
    278                         'TR_PASSWORD_NOT_MATCH' => tr('Password not match!'), 
     278                        'TR_PASSWORD_NOT_MATCH' => tr("Passwords don't match!"), 
    279279                        'TR_ADD_ADMIN' => tr('Add admin'), 
    280280                        'TR_CORE_DATA' => tr('Core data'), 
    281281                        'TR_USERNAME' => tr('Username'), 
    282282                        'TR_PASSWORD' => tr('Password'), 
    283                         'TR_PASSWORD_REPEAT' => tr('Password repeat'), 
     283                        'TR_PASSWORD_REPEAT' => tr('Repeat password'), 
    284284                        'TR_EMAIL' => tr('Email'), 
    285285                        'TR_ADDITIONAL_DATA' => tr('Additional data'), 

  • trunk/gui/admin/admin_log.php

    r661 r679  
    204204    if (isset($_POST['uaction']) && $_POST['uaction'] === 'clear_log') { 
    205205 
     206        $query = null; 
     207 
    206208        switch ($_POST['uaction_clear']) { 
    207209 
    … …  
    240242 
    241243                        break; 
     244 
     245                case 12: 
     246                                $query = <<<SQL_QUERY 
     247            delete 
     248                from 
     249            log 
     250                where 
     251            DATE_SUB(CURDATE(), INTERVAL 3 MONTH) 
     252                        >= log_time 
     253SQL_QUERY; 
     254                        break; 
     255 
    242256                case 26: 
    243257                                $query = <<<SQL_QUERY 
    … …  
    262276 
    263277                        break; 
     278                        default: 
     279                            system_message(tr('Invalid time period!')); 
     280                            break; 
    264281 
    265282        } 

  • trunk/gui/admin/ahp.php

    r661 r679  
    5555                     'TR_HOSTING PLAN PROPS' => tr('Hosting plan properties'), 
    5656                     'TR_TEMPLATE_NAME' => tr('Template name'), 
    57                      'TR_MAX_SUBDOMAINS' => tr('Max subdomains<br><i>(-1 disabled, 0 unlimited)'), 
    58                      'TR_MAX_ALIASES' => tr('Max aliases<br><i>(-1 disabled, 0 unlimited)'), 
    59                      'TR_MAX_MAILACCOUNTS' => tr('Mail account limit<br><i>(-1 disabled, 0 unlimited)</i>'), 
    60                      'TR_MAX_FTP' => tr('FTP account limit<br><i>(-1 disabled, 0 unlimited)</i>'), 
     57                     'TR_MAX_SUBDOMAINS' => tr('Max subdomains<br><i>(-1 disabled, 0 unlimited)</i>'), 
     58                     'TR_MAX_ALIASES' => tr('Max aliases<br><i>(-1 disabled, 0 unlimited)</i>'), 
     59                     'TR_MAX_MAILACCOUNTS' => tr('Mail accounts limit<br><i>(-1 disabled, 0 unlimited)</i>'), 
     60                     'TR_MAX_FTP' => tr('FTP accounts limit<br><i>(-1 disabled, 0 unlimited)</i>'), 
    6161                     'TR_MAX_SQL' => tr('SQL databases limit<br><i>(-1 disabled, 0 unlimited)</i>'), 
    6262                     'TR_MAX_SQL_USERS' => tr('SQL users limit<br><i>(-1 disabled, 0 unlimited)</i>'), 
    63                      'TR_MAX_TRAFFIC' => tr('Traffic limit [Mb]<br><i>(0 unlimited)</i>'), 
    64                      'TR_DISK_LIMIT' => tr('Disk limit [Mb]<br><i>(0 unlimited)</i>'), 
     63                     'TR_MAX_TRAFFIC' => tr('Traffic limit [MB]<br><i>(0 unlimited)</i>'), 
     64                     'TR_DISK_LIMIT' => tr('Disk limit [MB]<br><i>(0 unlimited)</i>'), 
    6565                     'TR_PHP' => tr('PHP'), 
    6666                     'TR_CGI' => tr('CGI / Perl'), 

  • trunk/gui/admin/change_password.php

    r665 r679  
    137137                       'TR_PASSWORD_DATA' => tr('Password data'), 
    138138                       'TR_PASSWORD' => tr('Password'), 
    139                        'TR_PASSWORD_REPEAT' => tr('Password repeat'), 
     139                       'TR_PASSWORD_REPEAT' => tr('Repeat password'), 
    140140                       'TR_UPDATE_PASSWORD' => tr('Update password'), 
    141141                       'TR_CURR_PASSWORD' => tr('Current password') 

  • trunk/gui/admin/circular.php

    r661 r679  
    122122        $err_message = tr('Please specify a sender email!'); 
    123123 
    124     } else if (chk_email($sender_email)) { 
     124    } else if (!chk_email($sender_email)) { 
    125125 
    126126        set_page_message( tr("Incorrect email range or syntax!")); 

  • trunk/gui/admin/delete_ip.php

    r661 r679  
    5858    /* ERR - we have domain that use this ip */ 
    5959 
    60     set_page_message(tr('Error we have domain that use this IP!')); 
     60    set_page_message(tr('Error: we have a domain using this IP!')); 
    6161 
    6262    header( "Location: ip_manage.php" ); 
    … …  
    7777    while (($data = $res -> FetchRow())) { 
    7878      if (preg_match("/$delete_id;/", $data['reseller_ips'])) { 
    79         set_page_message(tr('Error we have reseller that use this IP!')); 
     79        set_page_message(tr('Error: we have a reseller using this IP!')); 
    8080        header( "Location: ip_manage.php" ); 
    8181        die(); 

  • trunk/gui/admin/edit_reseller.php

    r665 r679  
    201201        } 
    202202    } 
    203     if (chk_email($_POST['email'])) { 
     203    if (!chk_email($_POST['email'])) { 
    204204 
    205205        set_page_message( tr("Incorrect email range or syntax!")); 
    … …  
    468468        if ($data > 0) { 
    469469 
    470             $err = tr('This reseller has user(s) with unlimited rights for <b>').$obj.tr('</b> Service !<br>'); 
     470            $err = tr('This reseller has user(s) with unlimited rights for <b>%s</b> service !<br>', $obj); 
    471471 
    472472            $err .= tr('If you want to limit the reseller, fist you must limit its users !<br>'); 
    … …  
    862862    array( 
    863863        'TR_EMPTY_OR_WORNG_DATA' => tr('Empty data or wrong field!'), 
    864         'TR_PASSWORD_NOT_MATCH' => tr('Password not match!'), 
     864        'TR_PASSWORD_NOT_MATCH' => tr("Passwords don't match!"), 
    865865        'TR_EDIT_RESELLER' => tr('Edit reseller'), 
    866866        'TR_CORE_DATA' => tr('Core data'), 
    867867        'TR_USERNAME' => tr('Username'), 
    868868        'TR_PASSWORD' => tr('Password'), 
    869         'TR_PASSWORD_REPEAT' => tr('Password repeat'), 
     869        'TR_PASSWORD_REPEAT' => tr('Repeat password'), 
    870870        'TR_EMAIL' => tr('E-mail'), 
    871871        'TR_UNLIMITED' => tr('unlimited'), 
    … …  
    883883        'TR_JSP' => tr('JSP'), 
    884884        'TR_SSI' => tr('SSI'), 
    885         'TR_FRONTPAGE_EXT' => tr('Fronpage extensions'), 
     885        'TR_FRONTPAGE_EXT' => tr('Frontpage extensions'), 
    886886        'TR_BACKUP_RESTORE' => tr('Backup and restore'), 
    887887        'TR_CUSTOM_ERROR_PAGES' => tr('Custom error pages'), 

  • trunk/gui/admin/edit_user.php

    r665 r679  
    242242function check_user_data() 
    243243{ 
    244     if (chk_email($_POST['email'])) { 
     244    if (!chk_email($_POST['email'])) { 
    245245 
    246246        set_page_message( tr("Incorrect email range or syntax!")); 
    … …  
    310310                array( 
    311311                        'TR_EMPTY_OR_WORNG_DATA' => tr('Empty data or wrong field!'), 
    312                         'TR_PASSWORD_NOT_MATCH' => tr('Password not match!'), 
     312                        'TR_PASSWORD_NOT_MATCH' => tr("Passwords don't match!"), 
    313313                        'TR_EDIT_ADMIN' => tr('Edit admin'), 
    314314                        'TR_CORE_DATA' => tr('Core data'), 
    315315                        'TR_USERNAME' => tr('Username'), 
    316316                        'TR_PASSWORD' => tr('Password'), 
    317                         'TR_PASSWORD_REPEAT' => tr('Password repeat'), 
     317                        'TR_PASSWORD_REPEAT' => tr('Repeat password'), 
    318318                        'TR_EMAIL' => tr('Email'), 
    319319                        'TR_ADDITIONAL_DATA' => tr('Additional data'), 

  • trunk/gui/admin/ehp.php

    r665 r679  
    6161$tpl -> assign(array('TR_HOSTING PLAN PROPS' => tr('Hosting plan properties'), 
    6262                     'TR_TEMPLATE_NAME' => tr('Template name'), 
    63                      'TR_MAX_SUBDOMAINS' => tr('Max subdomains<br><i>(-1 disabled, 0 unlimited)'), 
    64                      'TR_MAX_ALIASES' => tr('Max aliases<br><i>(-1 disabled, 0 unlimited)'), 
    65                      'TR_MAX_MAILACCOUNTS' => tr('Mail account limit<br><i>(-1 disabled, 0 unlimited)</i>'), 
    66                      'TR_MAX_FTP' => tr('FTP account limit<br><i>(-1 disabled, 0 unlimited)</i>'), 
    67                      'TR_MAX_SQL' => tr('SQL databases Limit<br><i>(-1 disabled, 0 unlimited)</i>'), 
     63                     'TR_MAX_SUBDOMAINS' => tr('Max subdomains<br><i>(-1 disabled, 0 unlimited)</i>'), 
     64                     'TR_MAX_ALIASES' => tr('Max aliases<br><i>(-1 disabled, 0 unlimited)</i>'), 
     65                     'TR_MAX_MAILACCOUNTS' => tr('Mail accounts limit<br><i>(-1 disabled, 0 unlimited)</i>'), 
     66                     'TR_MAX_FTP' => tr('FTP accounts limit<br><i>(-1 disabled, 0 unlimited)</i>'), 
     67                     'TR_MAX_SQL' => tr('SQL databases limit<br><i>(-1 disabled, 0 unlimited)</i>'), 
    6868                     'TR_MAX_SQL_USERS' => tr('SQL users limit<br><i>(-1 disabled, 0 unlimited)</i>'), 
    6969                     'TR_MAX_TRAFFIC' => tr('Traffic limit [MB]<br><i>(0 unlimited)</i>'), 

  • trunk/gui/admin/manage_reseller_users.php

    r661 r679  
    484484    } else if ($dest_max > 0 && $src_max == 0 && $umax == 0) { 
    485485 
    486         $err = "<b>$uname</b>".tr('Has unlimited rights for a <b>').$obj.tr('</b> Service !<br>'); 
    487  
    488         $err .= tr('You can not move <b>').$uname.tr('</b> in a destination reseller,<br>'); 
    489  
    490         $err .= tr('which has limits for the <b>').$obj.tr('</b> service!'); 
     486        $err = tr('<b>%1$s</b> has unlimited rights for a <b>%2$s</b> Service !<br>', $uname, $obj); 
     487 
     488        $err .= tr('You can not move <b>%1$s</b> in a destination reseller,<br>which has limits for the <b>%2$s</b> service!', $uname, $obj); 
    491489 
    492490        return; 
    … …  
    496494        if ($dest + $umax > $dest_max) { 
    497495 
    498             $err = "<b>$uname</b>"." ".tr('is exceeding limits for a')." <b>".$obj.tr('</b>'); 
    499  
    500             $err .= tr('<br>service in destination reseller!<br>'); 
     496            $err = tr('<b>%1$s</b> is exceeding limits for a <b>%2$s</b><br>service in destination reseller!<br>', $uname, $obj); 
    501497 
    502498            $err .= tr('Moving aborted!'); 
    … …  
    526522        if ($dest + $umax > $dest_max) { 
    527523 
    528             $err = "<b>$uname</b>"." ".tr('is exceeding limits for a')." <b>".$obj.tr('</b><br>'); 
    529  
    530             $err .= tr('Service in destination reseller!<br>'); 
     524             $err = tr('<b>%1$s</b> is exceeding limits for a <b>%2$s</b><br>service in destination reseller!<br>', $uname, $obj); 
     525 
     526             $err .= tr('Moving aborted!'); 
     527 
     528        } else { 
     529 
     530            $src -= $umax; 
     531 
     532            $dest += $umax; 
     533 
     534        } 
     535 
     536        return; 
     537 
     538    } 
     539 
     540} 
     541 
     542 
     543function calculate_reseller_vals (&$dest, $dest_max, &$src, $src_max, $umax, &$err, $obj, $uname) { 
     544 
     545    if ($dest_max == 0 && $src_max == 0 && $umax == 0) { 
     546 
     547        return; 
     548 
     549    } else if ($dest_max == 0 && $src_max == 0 && $umax > 0) { 
     550 
     551        $src -= $umax; 
     552 
     553        $dest += $umax; 
     554 
     555        return; 
     556 
     557    } else if ($dest_max == 0 && $src_max > 0 && $umax == 0) { 
     558 
     559        // Impossible condition; 
     560 
     561        return; 
     562 
     563    } else if ($dest_max == 0 && $src_max > 0 && $umax > 0) { 
     564 
     565        $src -= $umax; 
     566 
     567        $dest += $umax; 
     568 
     569        return; 
     570 
     571    } else if ($dest_max > 0 && $src_max == 0 && $umax == 0) { 
     572 
     573        $err = tr('<b>%1$s</b> has unlimited rights for a <b>%2$s</b> service !<br>', $uname, $obj); 
     574 
     575        $err .= tr('You can not move <b>%1$s</b> in a destination reseller,<br>which has limits for the <b>%2$s</b> service!', $uname, $obj); 
     576 
     577        return; 
     578 
     579    } else if ($dest_max > 0 && $src_max == 0 && $umax > 0) { 
     580 
     581        if ($dest + $umax > $dest_max) { 
     582 
     583            $err = tr('<b>%1$s</b> is exceeding limits for a <b>%2$s</b><br>service in destination reseller!<br>', $uname, $obj); 
    531584 
    532585            $err .= tr('Moving aborted!'); 
    … …  
    542595        return; 
    543596 
    544     } 
    545  
    546 } 
    547  
    548  
    549 function calculate_reseller_vals (&$dest, $dest_max, &$src, $src_max, $umax, &$err, $obj, $uname) { 
    550  
    551     if ($dest_max == 0 && $src_max == 0 && $umax == 0) { 
    552  
    553         return; 
    554  
    555     } else if ($dest_max == 0 && $src_max == 0 && $umax > 0) { 
    556  
    557         $src -= $umax; 
    558  
    559         $dest += $umax; 
    560  
    561         return; 
    562  
    563     } else if ($dest_max == 0 && $src_max > 0 && $umax == 0) { 
     597    } else if ($dest_max > 0 && $src_max > 0 && $umax == 0) { 
    564598 
    565599        // Impossible condition; 
    … …  
    567601        return; 
    568602 
    569     } else if ($dest_max == 0 && $src_max > 0 && $umax > 0) { 
    570  
    571         $src -= $umax; 
    572  
    573         $dest += $umax; 
    574  
    575         return; 
    576  
    577     } else if ($dest_max > 0 && $src_max == 0 && $umax == 0) { 
    578  
    579         $err = "<b>$uname</b>".tr(' Has unlimited rights for a <b>').$obj.tr('</b> Service !<br>'); 
    580  
    581         $err .= tr('You can not move <b>').$uname.tr('</b> In a destination reseller,<br>'); 
    582  
    583         $err .= tr('Which has limits for the <b>').$obj.tr('</b> service!'); 
    584  
    585         return; 
    586  
    587     } else if ($dest_max > 0 && $src_max == 0 && $umax > 0) { 
     603    } else if ($dest_max > 0 && $src_max > 0 && $umax > 0) { 
    588604 
    589605        if ($dest + $umax > $dest_max) { 
    590606 
    591             $err = "<b>$uname</b>"." ".tr('is exceeding limits for a')." <b>".$obj.tr('</b>'); 
    592  
    593             $err .= tr('<br>service in destination reseller!<br>'); 
    594  
    595             $err .= tr('Moving aborted!'); 
    596  
    597         } else { 
    598  
    599             $src -= $umax; 
    600  
    601             $dest += $umax; 
    602  
    603         } 
    604  
    605         return; 
    606  
    607     } else if ($dest_max > 0 && $src_max > 0 && $umax == 0) { 
    608  
    609         // Impossible condition; 
    610  
    611         return; 
    612  
    613     } else if ($dest_max > 0 && $src_max > 0 && $umax > 0) { 
    614  
    615         if ($dest + $umax > $dest_max) { 
    616  
    617             $err = "<b>$uname</b>"." ".tr('is exceeding limits for a')." <b>".$obj.tr('</b><br>'); 
    618  
    619             $err .= tr('Service in destination reseller!<br>'); 
     607            $err = tr('<b>%1$s</b> is exceeding limits for a <b>%2$s</b><br>service in destination reseller!<br>', $uname, $obj); 
    620608 
    621609            $err .= tr('Moving aborted!'); 
    … …  
    661649        if (!preg_match("/$domain_ip_id;/", $dest)) { 
    662650 
    663             $err = "<b>$domain_name</b>".tr(' Has IP address that can not be managed from the destination reseller !<br>This user can not be moved!'); 
     651            $err = tr('<b>%s</b> has IP address that can not be managed from the destination reseller !<br>This user can not be moved!', $domain_name); 
    664652 
    665653            return false;