problem creating clients in 3.1dev

Discussion in 'Developers' Forum' started by pssara, Jan 22, 2016.

  1. pssara

    pssara New Member

    Hi,
    I just installed the 3.1dev from git. it installed just fine, but it seems there is an error on creating new clients. I cannot create any. I always get the error "Please enter a valid email address" idependant of what email address I give.
    any ideas?
     
  2. till

    till Super Moderator Staff Member ISPConfig Developer

    Yes, that's possible. 3.1 dev is work in progress and not a software that can be used in production yet, so it can always be that functions are broken at the time you install it.
     
  3. dark alex

    dark alex New Member

    The problem lays here (client.tform.php):
    PHP:
            'paypal_email' => array (
                
    'datatype' => 'VARCHAR',
                
    'formtype' => 'TEXT',
                
    'filters'   => array( => array( 'event' => 'SAVE',
                        
    'type' => 'IDNTOASCII'),
                    
    => array( 'event' => 'SHOW',
                        
    'type' => 'IDNTOUTF8'),
                    
    => array( 'event' => 'SAVE',
                        
    'type' => 'TOLOWER')
                ),
                
    'validators' => array (
                    
    => array ( 'type' => 'ISEMAIL''errmsg'=> 'email_error_isemail'),
                ),
                
    'default' => '',
                
    'value'  => '',
                
    'separator' => '',
                
    'width'  => '30',
                
    'maxlength' => '255',
                
    'rows'  => '',
                
    'cols'  => '',
                
    'searchable' => 2
            
    ),
    Obviously that does not only check validity of the paypal email but also forces one to fill this field.
     
  4. till

    till Super Moderator Staff Member ISPConfig Developer

    Thank you for your findings. The global filter implementation has been changed recently, probably the email filter has been made amndatory now. I'll lookinto the code and change it back to its original behaviour.
     
  5. dark alex

    dark alex New Member

    Might I suggest to change the error message if the check fails for paypal so that the error also states that we are talking about paypal email?
    Probably that a bit confusing
     

Share This Page