Kovah/LinkAce

New tags or lists vanish in case of a validation issue

Open

Aperta il 19 mar 2025

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)PHP (1761 star) (111 fork)batch import
BugHelp wanted

Descrizione

Bug Description

When adding a new tag or list during the link creation or editing, those will vanish if the validation fails.

Caused by https://github.com/Kovah/LinkAce/blob/2.x/resources/assets/js/components/TagsSelect.js#L52

How to reproduce

  • add a new link with the url https://example.com
  • open link creation form
  • set https://example.com as the url
  • add new list as a new list
  • try to submit the form, this should fail because the url already exists
  • new list vanished from the list input

Expected behavior

new list should stay in the list input

Logs

Screenshots

No response

LinkAce version

2.x

Setup Method

Docker

Operating System

Linux (Ubuntu, CentOS,...)

Client details

No response

Guida contributor

Tech stack
javascriptvuephplaravel
Dominio
frontend
Tipo issue
bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
under 1 hour
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
fresh
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
Vue.js basicsJavaScript debugging
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75
Direzione di ricerca
Il bug è che nuovi tag o liste scompaiono dopo un fallimento della validazione durante la creazione/modifica di un link. La causa principale è probabilmente nel componente TagsSelect in resources/assets/js/components/TagsSelect.js, riga 52, dove lo stato potrebbe essere ripristinato. Indaga come il componente gestisce lo stato locale durante l'invio del modulo e assicurati che le nuove voci persistano dopo una validazione fallita. Cerca eventuali test esistenti per il componente per comprenderne il comportamento previsto.