Release plugin.video.themoviedb.helper - Access to TheMovieDb API for Skinners
(2023-08-18, 17:36)jepsizofye Wrote:
(2023-08-18, 17:28)jurialmunkey Wrote:
(2023-08-18, 17:19)jepsizofye Wrote: if you know it is possible to get an error retrieving from the database then wouldn't it be pertinent to put in a "try" to handle any possible errors

just 1 opinion

No. The error should be raised because it indicates an problem. There should not be an incorrect DBID added.

i see, you prefer it to raise where i prefer to quietly log and fail to prevent user interruption

i will keep my opinions to myself

It is not an either/or type of thing.
Many errors I catch in a try block and simply log. Other errors should be raised because they are more significant.
Arctic Fuse - Alpha now available. Support me on Ko-fi.
Reply


Messages In This Thread
RE: TheMovieDB Helper - by jurialmunkey - 2019-07-31, 11:47
RE: plugin.video.themoviedb.helper - Access to TheMovieDb API for Skinners - by jurialmunkey - 2023-08-18, 17:52
Logout Mark Read Team Forum Stats Members Help
plugin.video.themoviedb.helper - Access to TheMovieDb API for Skinners2