maybe a bug on forms Form::MAX_LENGTH rule

hi jan here you have

public function signUp()
    {
        $this->form->addText('username', 'Username')
            ->setRequired('Username field is required')
            ->addRule(Form::MAX_LENGTH, 'The max length allowed is %d',10)
            ->addRule(Form::PATTERN, 'Use only lowercase letters and numbers','[a-z0-9]+');

        $this->form->addText('email', 'Email')
            ->setRequired('Email field is required')
            ->addRule(Form::MAX_LENGTH, 'The max length allowed is %d',200)
            ->addRule(Form::EMAIL, 'Please write a valid email address');

        $this->form->addPassword('password', 'Password')
            ->setRequired('Password field is required')
            ->addRule(Form::MAX_LENGTH, 'The max length allowed is %d',200)
            ->addRule(Form::PATTERN, 'the next characters are allowed @#%!*','[a-zA-Z0-9@#%!*]+');


        $this->form->addSubmit('send', 'Signup');

        $this->form->onSuccess[] = callback($this, 'processSignUp');
        return $this->form;
    }

    public function processSignUp()
    {
        if($this->form->isValid())
        {
			dump($this->form->getErrors(),$this);
        }

hi jan, im using the forms outside from presenters. All is working good, except Form::MAX_LENGTH server side validation witch does not validate. mmmm i will share you the code, but ONLY the MAX_LENGTH does NOT WORK on server side

BaseForm.php

<?php namespace App\Common\Forms;
/**
 * @author Leonardo Allende <leonardoap@yandex.com> 2015
 */
use \Nette\Object,
    Nette\Application\UI\Form;

abstract class BaseForm extends Object{

    protected $form;

    public function __construct()
    {
        $this->form = new Form;
        $this->form->addProtection('Form token expired');
    }

}

signUpForm.php

<?php namespace App\Common\Forms\Src;
/**
 * @author Leonardo Allende <leonardoap@yandex.com> 2015
 */
use \Nette\Application\UI\Form;
use App\Common\Forms\BaseForm;

class SignUpForm extends BaseForm{

    public function signUp()
    {
        $this->form->addText('username', 'Username')
            ->setRequired('Username field is required')
            ->addRule(Form::MAX_LENGTH, 'The max length allowed is %d',10)
            ->addRule(Form::PATTERN, 'Use only lowercase letters and numbers','[a-z0-9]+');

        $this->form->addText('email', 'Email')
            ->setRequired('Email field is required')
            ->addRule(Form::MAX_LENGTH, 'The max length allowed is %d',200)
            ->addRule(Form::EMAIL, 'Please write a valid email address');

        $this->form->addPassword('password', 'Password')
            ->setRequired('Password field is required')
            ->addRule(Form::MAX_LENGTH, 'The max length allowed is %d',200)
            ->addRule(Form::PATTERN, 'the next characters are allowed @#%!*','[a-zA-Z0-9@#%!*]+');


        $this->form->addSubmit('send', 'Signup');

        $this->form->onSuccess[] = callback($this, 'processSignUp');
        return $this->form;
    }

    public function processSignUp()
    {
        if($this->form->isValid())
        {
            $values = $this->form->getValues();
            $new_sign_up = $this->form->getPresenter()->context->models->getService('users');
            $values['created_at'] = $new_sign_up->timestamp();
            $values['updated_at'] = $new_sign_up->timestamp();
            //$values['password_hash'] = $new_sign_up;
            dump($this->form->getErrors(),$this); // here does not show FORM::MAX_LENGTH validate if this has been altered. by example editing  html whit chrome tools changing username length field from 10 to 20 and sending 20 characters pressing the submit Button
        }

    }

}

just in case here you have the serviceFActory and FormsContainer

serviceFactory

<?php namespace App\Common\Classes;
/**
 * @author Leonardo Allende <leonardoap@yandex.com> 2015
 */
use \Nette\DI\Container;
use \App\Common\Classes\ModelsContainer;
use \App\Common\Classes\FormsContainer;
/**
 * Factory of services
 */
class ServiceFactory {

    public static function serviceModels(Container $container)
    {
        $models = new ModelsContainer();
	    $models->addService('container', $container);
	    return $models;
    }

    public static function serviceForms(Container $container)
    {
        $forms = new FormsContainer();
        $forms->addService('container', $container);
        return $forms;
    }
}

formsContainer

<?php namespace App\Common\Classes;
/**
 * @author Leonardo Allende <leonardoap@yandex.com> 2015
 */
use \Nette\DI\Container;
use App\Common\Forms\Src\SignUpForm;

/**
 * Container to find all forms
 */
class FormsContainer extends Container{

    protected function createServiceSignUp()
    {
        //$presenter = $this->getService('container')->getService('application')->getPresenter()->getPresenter();
        return new SignUpForm();
    }
}

and finally here the presenter calling

protected function createComponentSignUp()
 {
     $signUpForm = $this->context->forms->getService('signUp');
     //$signUpForm->signUp()->setTranslator($this->translator);
     return $signUpForm->signUp();
 }

regards

Last edited by leonardoap (2015-02-05 04:43)

It is ok, MAX_LENGTH silently truncates string, without error message.

It was changed in master.