2

Tenho um campo inteiro e um campo data que não são obrigatórios e estão programados para aceitar NULL, porém quando envio os dados do formulário sem preencher eles o meu controller recebe o valor vazio ao invés de ignorar o campo, o que gera o erro:

Incorrect integer value: '' for column vendedor_id

como posso resolver isso?

Migration:

 $table->integer('vendedor_id')->unsigned()->nullable();
 $table->date('data_venda')->nullable();

Controller:

$new = $this->repository->create($request->all());

View:

<div id="vendedor_id" class="form-group col-sm-4 opt">
    <label for="vendedor_id">Vendedor:</label>
    <select name="vendedor_id" class="form-control select2">
        <option value=""></option>
        <option value="1">User 1</option>
        <option value="2">User 2</option>
    </select>
</div>
<div id="data_venda" class="form-group col-sm-4">
    <label for="data_venda">Data:</label>
    <input type="date" name="data_venda" class="form-control">
</div>

No meu request tenho algumas validações para os campos obrigatórios, mas não inclui nada sobre esses 2 campos. Ex.:

public function rules()
{
    return [
        'cnpj' => 'numeric|required|cnpj|unique:empresas',
        'razao_social' => 'required',
    ];
}
7
  • Fornece a sua validação? e também aonde está fazendo esse código? acredito que tudo se resume na validação!!!
    – novic
    2/12/2016 às 12:53
  • Precisa postar o trecho do código na sua pergunta, para ficar mais claro. Podem ser dois problemas diferentes (ou mais). Precisamos de mais detalhes. 2/12/2016 às 12:58
  • Faça um submit sem enviar os valores obrigatórios. Debug com a função dd antes do insert. 2/12/2016 às 13:00
  • Esse formulário está sendo enviado via Ajax? Eu já tive problemas assim quando usei FormData. 2/12/2016 às 13:01
  • "vendedor_id" => "" "data_venda" => "" 2/12/2016 às 13:03

2 Respostas 2

5

Vou passar umas soluções abaixo que o próprio framework oferece:

Model Mutators

É possível definir um comportamento para a definição de um determinado atributo de um Model em Laravel.

Por exemplo, para você não ter que definir a todo o momento condições de verificações de um campo no seu controller, você pode criar um setter específico para esses campos no seu Model.

Veja:

  public function setVendedorIdAttribute($valor) {
      $this->attributes['vendedor_id'] = ctype_digit($valor) ? $valor : NULL;
  }

Assim, toda vez que um valor vazio fosse passado, seria inserido como NULL no banco.

Validação

De acordo coma documentação do Laravel 5.3, é possível, a partir dessa versão, definir uma validação nullable para determinado campo.

Array_Filter

Outra solução seria utilizar array_filter que remove os valores vazios de um array.

$data = array_filter($request->all());
$new = $this->repository->create($data);

Array_Map

Se você quiser apenas converter os valores vazios para NULL, você também pode usar array_map como solução:

$data = array_map($request->all(), function ($value) {
  return $value ?: NULL;
});
13
  • Eu pensei em dar também essa solução seria muito bom, o problema é que não sabemos o que o respository->create faz! se for fill vai funcionar
    – novic
    2/12/2016 às 13:13
  • Funcionou também dessa forma! Das 2 formas na verdade 2/12/2016 às 13:18
  • 1
    Acho que vou usar esse array_filter mesmo, fica tudo bem mais limpo e funciona perfeitamente 2/12/2016 às 13:22
  • 1
    @VirgilioNovic se não quiser setar o valor como NULL, os valores simplesmente não estarão no array. Então aquele campo não seria atualizado. O array_filter remove valores vazios do array, ele não define para Null. Quando um campo é nullable, só o fato de ser omitido já faz ele ser definido para NULL (mas no caso do create, quando é insert a omissão mantém o que já tem). 2/12/2016 às 13:24
  • 1
    Entendi, então se no update eu quiser voltar o valor pra Null não vai funcionar né 2/12/2016 às 13:28
5

Se não for escolhido valor naquele select ele está enviando "" ai mora o problema, precisa ser verificado antes de enviar para essa camada repository:

$data = $request->except('vendedor_id');
$data['vendedor_id'] = $request->input('vendedor_id', "") == "" 
                      ? null
                      : (int)$request->input('vendedor_id')
$new = $this->repository->create($data);
12
  • Isso resolve o problema, mas é meio gambiarra. Talvez o problema seja outro, mas o AP não descreveu direito. 2/12/2016 às 13:03
  • Exato, podia ir ainda mais longe, e colocar if(!is_numeric($request->input('vendedor_id')) { .... }, o erro é gerado porque a coluna é integer e o Raylan está a enviar uma string
    – Miguel
    2/12/2016 às 13:03
  • Então, o problema @WallaceMaxters está como ele está fazendo (talvez, porque não sei da intenção real), e não a minha resposta representar uma gambiarra. Gambiarra validar um campo que pode vir vazio ou então o número 1 ou 2 com um simples if não vejo como gambiarra.
    – novic
    2/12/2016 às 13:06
  • @VirgilioNovic eu fiz um teste aqui. Realmente parece que o Laravel "deu esse mole". Quando o valor do "select" está vazio, ele envia uma string vazia mesmo. 2/12/2016 às 13:07
  • 1
    @RaylanSoares é o seguinte, quando você manda um select e tem uma opção nele entre aspas duplas "", o Laravel entende que pode gravar um empty na tabela, mas, quando eu forço um NULL no campo ele entende que é pra gravar null é assim que ele trabalha! atualmente
    – novic
    2/12/2016 às 13:11

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .