uspdev / chamados

sistema para atendimento de chamados por filas
23 stars 30 forks source link

se o formulário estiver vazio, ele não atualizará #326

Closed wgnann closed 2 years ago

wgnann commented 2 years ago

o problema vem de: https://github.com/uspdev/chamados/blob/8126eb6333912584e26e7ebdc98f7a2bc6c2e2fe/app/Http/Controllers/ChamadoController.php#L309

se não me engano, $chamado->extras nunca poderá ser NULL, pois é um JSON válido, então sempre será uma string.

outro problema é que a sentinela $atualiza_extras só é definida de forma condicional (subordinada ao condicional acima) https://github.com/uspdev/chamados/blob/8126eb6333912584e26e7ebdc98f7a2bc6c2e2fe/app/Http/Controllers/ChamadoController.php#L311

caso $chamado->extras efetivamente seja NULL, ela nunca será atualizada e haverá uma verificação que leva em consideração uma variável que não foi previamente declarada. o ideal seria movê-la para antes do condicional