mtils / formobject Goto Github PK

This makes if(!$form->wasSubmitted()){ $form->fillByArray()) redundant
Also implement Form::$ignoreFillIfFormSubmitted to switch that behaviour

In order to not manually split prefixes (like address__street) and retrieve fields by pefix this would make the using code much more simple to use

Inject UrlGenerator in FormObject\Support\Laravel\Http\CurrentActionUrlProvider
Inject Request in FormObject\Support\Laravel\Http\InputRequestProvider
Inject UrlGenerator and Router in FormObject\Support\Laravel\Http\ResourceActionUrlProvider

Support setReadOnly() or setDisabled() Methods which will trigger all internal work to make a field really disabled

Firstly in offsetUnset() the datafields are not updated.
The second thing is that if you use before() the key is removed from the keyOrder array before it is clear that the target $fieldName exists.

In order to remove duplicated code in controllers the standard usage in post methods should be instead of this:

if($this->form->wasSubmitted() as $this->form->getValidator()->passes()){
    $user->address->fill($this->form->getData('address'));
    $user->fill($this->form->getData(''));
    $user->save();
}
else{
    Redirect::to('getFoo');
}

it should be written as this:

try{
    $user->address->fill($this->form->getOrFail('address'));
    $user->fill($this->form->getOrFail(''));
    $user->save();
}
catch(ValidationException $e){
    Redirect::to('getFoo')->withInput()->withErrors($e);
}

The steps needed are:

1. implement ValidatorException (for Laravel implementing MessageProviderInterface)
2. implement AdapterFactoryInterface::createValidationException($validator)
3. implement Form::getOrFail which throws the exception

Laravel\ValidatorAdapter should look for Session::get('errors') and return them to fullfill the needs of hasErrors() and getMessages().
If the form wasnt submitted return an empty array

Used with laravel 5 there is an issue with the request object stored in InputRequestProvider. The request is missing the session store, causing a runtime exception during form rendering.

The problem is here that in any template Field::__toString() is called. This calls $this->form->autoFillIfNotDone(). This causes the subform to load the complete request into itself.
Then the editManyField gets wrong data.

The current implementation depends on valid keys. The iterate methods have to be forwarded to an array iterator