From adcefa4ffaf68bcba66130ef5c47a05a6079df29 Mon Sep 17 00:00:00 2001
From: Omar Roth <omarroth@hotmail.com>
Date: Tue, 13 Nov 2018 20:29:36 -0600
Subject: [PATCH] Add 'published - reverse' option to feed

---
 src/invidious.cr                    | 26 +++++++++++++++++++-------
 src/invidious/views/preferences.ecr |  2 +-
 2 files changed, 20 insertions(+), 8 deletions(-)

diff --git a/src/invidious.cr b/src/invidious.cr
index 0139d34a..8c558ccf 100644
--- a/src/invidious.cr
+++ b/src/invidious.cr
@@ -1541,6 +1541,12 @@ get "/feed/subscriptions" do |env|
       offset = (page - 1) * max_results
     end
 
+    if preferences.sort == "published - reverse"
+      sort = ""
+    else
+      sort = "DESC"
+    end
+
     notifications = PG_DB.query_one("SELECT notifications FROM users WHERE email = $1", user.email,
       as: Array(String))
     view_name = "subscriptions_#{sha256(user.email)[0..7]}"
@@ -1549,7 +1555,7 @@ get "/feed/subscriptions" do |env|
       args = arg_array(notifications)
 
       notifications = PG_DB.query_all("SELECT * FROM channel_videos WHERE id IN (#{args})
-      ORDER BY published DESC", notifications, as: ChannelVideo)
+      ORDER BY published #{sort}", notifications, as: ChannelVideo)
       videos = [] of ChannelVideo
 
       notifications.sort_by! { |video| video.published }.reverse!
@@ -1574,11 +1580,11 @@ get "/feed/subscriptions" do |env|
           end
 
           videos = PG_DB.query_all("SELECT DISTINCT ON (ucid) * FROM #{view_name} WHERE \
-            id NOT IN (#{watched}) ORDER BY ucid, published DESC",
+          id NOT IN (#{watched}) ORDER BY published, ucid #{sort}",
             user.watched, as: ChannelVideo)
         else
           videos = PG_DB.query_all("SELECT DISTINCT ON (ucid) * FROM #{view_name} \
-          ORDER BY ucid, published DESC", as: ChannelVideo)
+          ORDER BY published, ucid #{sort}", as: ChannelVideo)
         end
 
         videos.sort_by! { |video| video.published }.reverse!
@@ -1591,11 +1597,11 @@ get "/feed/subscriptions" do |env|
           end
 
           videos = PG_DB.query_all("SELECT * FROM #{view_name} WHERE \
-          id NOT IN (#{watched}) LIMIT $1 OFFSET $2",
+          id NOT IN (#{watched}) ORDER BY published #{sort} LIMIT $1 OFFSET $2",
             [limit, offset] + user.watched, as: ChannelVideo)
         else
           videos = PG_DB.query_all("SELECT * FROM #{view_name} \
-          ORDER BY published DESC LIMIT $1 OFFSET $2", limit, offset, as: ChannelVideo)
+          ORDER BY published #{sort} LIMIT $1 OFFSET $2", limit, offset, as: ChannelVideo)
         end
       end
 
@@ -1746,14 +1752,20 @@ get "/feed/private" do |env|
   latest_only ||= 0
   latest_only = latest_only == 1
 
+  if user.preferences.sort == "published - reverse"
+    sort = ""
+  else
+    sort = "DESC"
+  end
+
   view_name = "subscriptions_#{sha256(user.email)[0..7]}"
 
   if latest_only
-    videos = PG_DB.query_all("SELECT DISTINCT ON (ucid) * FROM #{view_name} ORDER BY ucid, published DESC", as: ChannelVideo)
+    videos = PG_DB.query_all("SELECT DISTINCT ON (ucid) * FROM #{view_name} ORDER BY ucid, published #{sort}", as: ChannelVideo)
     videos.sort_by! { |video| video.published }.reverse!
   else
     videos = PG_DB.query_all("SELECT * FROM #{view_name} \
-    ORDER BY published DESC LIMIT $1 OFFSET $2", limit, offset, as: ChannelVideo)
+    ORDER BY published #{sort} LIMIT $1 OFFSET $2", limit, offset, as: ChannelVideo)
   end
 
   sort = env.params.query["sort"]?
diff --git a/src/invidious/views/preferences.ecr b/src/invidious/views/preferences.ecr
index f7da7540..f34c40aa 100644
--- a/src/invidious/views/preferences.ecr
+++ b/src/invidious/views/preferences.ecr
@@ -131,7 +131,7 @@ function update_value(element) {
             <div class="pure-control-group">
                 <label for="sort">Sort videos by: </label>
                 <select name="sort" id="sort">
-                <% ["published", "alphabetically", "alphabetically - reverse", "channel name", "channel name - reverse"].each do |option| %>
+                <% {"published", "published - reverse", "alphabetically", "alphabetically - reverse", "channel name", "channel name - reverse"}.each do |option| %>
                     <option <% if user.preferences.sort == option %> selected <% end %>><%= option %></option>
                 <% end %>
                 </select>