Fixed: Don't Fail if User Tries Adding Existing Movie from Discovery

pull/4938/head
Qstick 4 years ago
parent 87bf010c67
commit e45f0ad5fc

@ -521,11 +521,13 @@ export const actionHandlers = handleThunks({
// Make sure we have a selected movie and // Make sure we have a selected movie and
// the same movie hasn't been added yet. // the same movie hasn't been added yet.
if (selectedMovie && !acc.some((a) => a.tmdbId === selectedMovie.tmdbId)) { if (selectedMovie && !acc.some((a) => a.tmdbId === selectedMovie.tmdbId)) {
const newMovie = getNewMovie(_.cloneDeep(selectedMovie), addOptions); if (!selectedMovie.isExisting) {
newMovie.id = 0; const newMovie = getNewMovie(_.cloneDeep(selectedMovie), addOptions);
newMovie.id = 0;
addedIds.push(id); addedIds.push(id);
acc.push(newMovie); acc.push(newMovie);
}
} }
return acc; return acc;
@ -548,24 +550,19 @@ export const actionHandlers = handleThunks({
...data.map((movie) => updateItem({ section: 'movies', ...movie })), ...data.map((movie) => updateItem({ section: 'movies', ...movie })),
...addedIds.map((id) => removeItem({ section, id })) ...addedIds.map((id) => (items.find((i) => i.id === id).lists.length === 0 ? removeItem({ section, id }) : updateItem({ section, id, isExisting: true })))
])); ]));
}); });
promise.fail((xhr) => { promise.fail((xhr) => {
dispatch(batchActions( dispatch(
set({ set({
section, section,
isImporting: false, isAdding: false,
isImported: true isAdded: true
}), })
);
addedIds.map((id) => updateItem({
section,
id,
importError: xhr
}))
));
}); });
}, },

Loading…
Cancel
Save