Skip to content

fix(native forms): preserve selected Field Studio values when editing invoices - #7

Merged
vitormattos merged 2 commits into
mainfrom
fix/native-select-value-persistence-20261110
Nov 12, 2026
Merged

vitormattos merged 2 commits into
mainfrom
fix/native-select-value-persistence-20261110

Conversation

@vitormattos

Copy link
Copy Markdown
Member

Problema

O campo production_treatment da fatura CPD Municipal #2368 é gravado no Field Studio, mas a tela de edição volta a mostrar a seleção vazia. Uma gravação posterior da fatura pode apagar o valor: a auditoria mostra value.created e value.deleted em salvamentos da própria fatura. Essa falha afeta qualquer campo select do Field Studio nas faturas/contas a pagar.

Causa confirmada

O Form do Akaunting inicializa <select> usando form_element.getAttribute('value'), ignorando o atributo selected nas <option>. O componente Vue usa esse valor vazio no v-model e substitui a seleção existente.

Correção

  • Passar o valor já salvo no atributo value do <select> nativo para inicializar o modelo Vue corretamente; preserva os campos, nomes, tipos e a semântica PATCH existentes.
  • Ampliar o E2E da fatura para verificar carregamento de valor existente, alteração da seleção e nova edição/salvamento sem apagar a seleção.
  • Solução genérica para todos os campos select e documentos que reutilizam o componente, sem IDs ou regras específicas da CPD Municipal.

Evidências e testes

  • Smoke com código real de resources/assets/js/plugins/form.js do Akaunting: antes sem atributo value resulta em ""; depois com o atributo resulta em "Incluir mesmo sem pagamento" e "Ignorar na produção".
  • node --check tests/ui-e2e.js, php tests/i18n-check.php (176 mensagens/136 referências), git diff --check: aprovados.
  • Teste E2E do navegador ampliado, porém a execução completa não foi feita com credenciais de uma instância descartável. Não execute testes mutantes na produção.

Deploy

O módulo está instalado na Conceição. A alteração é exclusivamente de view Blade; não há mudança de schema nem de faturas, pagamentos ou impostos. Manter PR aberto para code review; nenhum merge solicitado.

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@vitormattos
vitormattos merged commit 769f887 into main Nov 12, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant