Remove error handling in the parse_query_result method of

MangaDex::Job, and pass the possible exceptions to the frontend and
handle them there.

public/js/download-manager.js

@@ -91,6 +91,10 @@ const load = () => {
 		dataType: 'json'
 	})
 	.done(data => {
+		if (!data.success && data.error) {
+			alert('danger', `Failed to fetch download queue. Error: ${data.error}`);
+			return;
+		}
 		console.log(data);
 		const btnText = data.paused ? "Resume download" : "Pause download";
 		$('#pause-resume-btn').text(btnText);

src/mangadex/downloader.cr

@@ -33,7 +33,6 @@ module MangaDex
 		property time : Time
 
 		def parse_query_result(res : DB::ResultSet)
-			begin
 			@id = res.read String
 			@manga_id = res.read String
 			@title = res.read String
@@ -46,17 +45,13 @@ module MangaDex
 			time = res.read Int64
 			@status = JobStatus.new status
 			@time = Time.unix_ms time
-				return true
-			rescue e
-				puts e
-				return false
-			end
 		end
 
+		# Raises if the result set does not contain the correct set of columns
 		def self.from_query_result(res : DB::ResultSet)
 			job = Job.allocate
-			success = job.parse_query_result res
+			job.parse_query_result res
-			return success ? job : nil
+			return job
 		end
 
 		def initialize(@id, @manga_id, @title, @manga_title, @status, @time)

src/routes/api.cr

@@ -128,11 +128,19 @@ class APIRouter < Router
 		end
 
 		get "/api/admin/mangadex/queue" do |env|
+			begin
 				jobs = @context.queue.get_all
 				send_json env, {
 					"jobs" => jobs,
-				"paused" => @context.queue.paused?
+					"paused" => @context.queue.paused?,
+					"success" => true
 				}.to_json
+			rescue e
+				send_json env, {
+					"success" => false,
+					"error" => e.message
+				}.to_json
+			end
 		end
 
 		post "/api/admin/mangadex/queue/delete/:id" do |env|