Evaluate how to handle errors in PlacesTransactions.batch()
Categories
(Toolkit :: Places, task, P3)
Tracking
()
People
(Reporter: mak, Unassigned)
References
Details
When we batch multiple transactions, one may fail.
What we currently do is just reporting the exception to the console and continuing... That "works" when for example we're pasting 100 bookmarks and 1 out of them fails, but not when a transaction depends on a previous one.
For example, failing to create a folder will likely also fail to insert any bookmarks into it.
How should we handle errors then?
For transactions that don't depend on previous ones, we could probably continue like today.
For depending transactions, we may want to undo all the already executed transactions in the batch.
Though, in both cases it may be worth notifying something to the user, as it would be very annoying to see nothing happen and try again and again because no error message was ever exposed.
Description
•