DB: playlists: make that 'insert' never raises

src/invidious/database/playlists.cr

@@ -94,17 +94,13 @@ module Invidious::Database::Playlists
   #  Salect
   # -------------------
 
-  def select(*, id : String, raise_on_fail : Bool = false) : InvidiousPlaylist?
+  def select(*, id : String) : InvidiousPlaylist?
     request = <<-SQL
       SELECT * FROM playlists
       WHERE id = $1
     SQL
 
-    if raise_on_fail
+    return PG_DB.query_one?(request, id, as: InvidiousPlaylist)
-      return PG_DB.query_one(request, id, as: InvidiousPlaylist)
-    else
-      return PG_DB.query_one?(request, id, as: InvidiousPlaylist)
-    end
   end
 
   def select_all(*, author : String) : Array(InvidiousPlaylist)

src/invidious/routes/playlists.cr

@@ -151,12 +151,8 @@ module Invidious::Routes::Playlists
     page = env.params.query["page"]?.try &.to_i?
     page ||= 1
 
-    begin
+    playlist = Invidious::Database::Playlists.select(id: plid)
-      playlist = Invidious::Database::Playlists.select(id: plid, raise_on_fail: true)
+    if !playlist || playlist.author != user.email
-      if !playlist || playlist.author != user.email
-        return env.redirect referer
-      end
-    rescue ex
       return env.redirect referer
     end
 
@@ -235,12 +231,8 @@ module Invidious::Routes::Playlists
     page = env.params.query["page"]?.try &.to_i?
     page ||= 1
 
-    begin
+    playlist = Invidious::Database::Playlists.select(id: plid)
-      playlist = Invidious::Database::Playlists.select(id: plid, raise_on_fail: true)
+    if !playlist || playlist.author != user.email
-      if !playlist || playlist.author != user.email
-        return env.redirect referer
-      end
-    rescue ex
       return env.redirect referer
     end