#102 Error when saving album settings from the settings tab

Chiuso
aperto 2 anni fa da aheathershaw · 2 commenti
aheathershaw 2 anni fa ha commentato

When you edit an album’s settings within the Settings tab of the album itself, you get an error.

Changing the album through the album’s edit screen works fine.

When you edit an album’s settings within the Settings tab of the album itself, you get an error. Changing the album through the album’s edit screen works fine.
aheathershaw 2 anni fa auto-assegnato
aheathershaw added the
bug
label 2 anni fa
aheathershaw 2 anni fa ha commentato
Proprietario

This is due to the validation added in #58, allowing albums with the same name to exist under different parents.

The settings tab does not have the “parent” field that the Edit screen does, so it causes an undefined index trying to access the field’s value.

The validation needs to use the album’s current parent when this field is not present in the data.

This is due to the validation added in #58, allowing albums with the same name to exist under different parents. The settings tab does not have the “parent” field that the Edit screen does, so it causes an undefined index trying to access the field’s value. The validation needs to use the album’s current parent when this field is not present in the data.
aheathershaw aggiunta alle pietre miliari (rimosso) 2 anni fa
aheathershaw 1 anno fa ha commentato
Proprietario

This is also present in Cloud, needs fixing in 2.2.0 and merging across.

This is also present in Cloud, needs fixing in 2.2.0 and merging across.
aheathershaw pietra miliare modificata da (rimosso) a Version 2.2.0-beta.1 1 anno fa
aheathershaw added the
wip
label 1 anno fa
aheathershaw removed the
wip
label 1 anno fa
aheathershaw closed this issue 1 anno fa
Effettua l'accesso per partecipare alla conversazione.
Nessuna etichetta
bug
wip
Nessuna milestone
Nessuna assegnatario
1 Partecipanti
Notifiche
Data di scadenza

Nessuna data di scadenza impostata.

Dipendenze

Questo problema attualmente non ha alcuna dipendenza.

Caricamento…
Non ci sono ancora contenuti.