From 547a6721161dfcd3642aaa23c8b539f50c59bad5 Mon Sep 17 00:00:00 2001
From: mgsnova <github@mgsnova.de>
Date: Mon, 20 May 2013 22:00:08 +0200
Subject: [PATCH] add story listing for single feeds

---
 app/controllers/stories_controller.rb         | 21 ++++++++++++++-----
 app/repositories/story_repository.rb          |  6 +++++-
 app/views/index.erb                           |  6 +++---
 app/views/partials/_action_bar.erb            |  7 +++++--
 app/views/partials/_feed.erb                  |  2 +-
 app/views/partials/_mark_all_as_read_form.erb |  3 ++-
 app/views/partials/_tutorial_action_bar.erb   |  2 +-
 spec/controllers/stories_controller_spec.rb   | 11 +++++++++-
 8 files changed, 43 insertions(+), 15 deletions(-)

diff --git a/app/controllers/stories_controller.rb b/app/controllers/stories_controller.rb
index e7705c047..d5f208bb7 100644
--- a/app/controllers/stories_controller.rb
+++ b/app/controllers/stories_controller.rb
@@ -8,6 +8,13 @@ class Stringer < Sinatra::Base
     erb :index
   end
 
+  get "/news/:feed_id" do
+    @unread_stories = StoryRepository.unread_by_feed_id(params[:feed_id])
+    @feed_id = params[:feed_id]
+
+    erb :index
+  end
+
   get "/archive" do
     @read_stories = StoryRepository.read(params[:page])
 
@@ -16,17 +23,21 @@ class Stringer < Sinatra::Base
 
   put "/stories/:id" do
     json_params = JSON.parse(request.body.read, symbolize_names: true)
-    
+
     story = StoryRepository.fetch(params[:id])
     story.is_read = !!json_params[:is_read]
     story.keep_unread = !!json_params[:keep_unread]
-    
+
     StoryRepository.save(story)
   end
 
   post "/mark_all_as_read" do
     MarkAllAsRead.new(params[:story_ids]).mark_as_read
-    
-    redirect to("/news")
+
+    if params[:feed_id]
+      redirect to("/news/#{params[:feed_id]}")
+    else
+      redirect to("/news")
+    end
   end
-end
\ No newline at end of file
+end
diff --git a/app/repositories/story_repository.rb b/app/repositories/story_repository.rb
index 552ca2ae9..0ebac1a33 100644
--- a/app/repositories/story_repository.rb
+++ b/app/repositories/story_repository.rb
@@ -27,6 +27,10 @@ def self.unread
     Story.where(is_read: false).order("published desc") 
   end
 
+  def self.unread_by_feed_id(feed_id)
+    unread.where(feed_id: feed_id)
+  end
+
   def self.read(page = 1)
     Story.where(is_read: true).order("published desc").page(page).per_page(15)
   end
@@ -37,7 +41,7 @@ def self.read_count
 
   def self.extract_content(entry)
     sanitized_content = ""
-    
+
     if entry.content
       sanitized_content = entry.content.sanitize
     elsif entry.summary
diff --git a/app/views/index.erb b/app/views/index.erb
index 55a710b1b..bea1c9849 100644
--- a/app/views/index.erb
+++ b/app/views/index.erb
@@ -5,16 +5,16 @@
 <% end %>
 
 <div id="action-bar">
-  <%= render_partial :action_bar, { stories: @unread_stories } %>
+  <%= render_partial :action_bar, { stories: @unread_stories, feed_id: @feed_id } %>
 </div>
 
 <% if @unread_stories.empty? %>
   <%= render_partial :zen %>
 <% else %>
   <%= render_js :stories, { stories: @unread_stories } %>
-  
+
   <div id="stories">
     <ul id="story-list">
     </ul>
   </div>
-<% end %>
\ No newline at end of file
+<% end %>
diff --git a/app/views/partials/_action_bar.erb b/app/views/partials/_action_bar.erb
index 326fdf89d..5b2678e11 100644
--- a/app/views/partials/_action_bar.erb
+++ b/app/views/partials/_action_bar.erb
@@ -1,11 +1,14 @@
 <div class="row-fluid">
   <div class="pull-left">
+    <a class="btn" id="back-to-feeds" href="/feeds" title="<%= t('partials.action_bar.view_feeds') %>">
+      <i class="icon-reply"></i>
+    </a>
     <a class="btn" id="mark-all" title="<%= t('partials.action_bar.mark_all') %>">
       <i class="icon-ok"></i>
       <%= render_partial :mark_all_as_read_form,
-                        {stories: stories} %>
+                        {stories: stories, feed_id: feed_id} %>
     </a>
-    <a class="btn" href="/" id="refresh" title="<%= t('partials.action_bar.refresh') %>">
+    <a class="btn" href="/news<%= @feed_id ? ('/' + @feed_id) : '' %>" id="refresh" title="<%= t('partials.action_bar.refresh') %>">
       <i class="icon-repeat"></i>
     </a>
   </div>
diff --git a/app/views/partials/_feed.erb b/app/views/partials/_feed.erb
index 117547826..1c0e5bb39 100644
--- a/app/views/partials/_feed.erb
+++ b/app/views/partials/_feed.erb
@@ -3,7 +3,7 @@
     <div class="span7 feed-title-container">
       <p class="feed-title">
         <i class="icon-circle status <%= feed.status_bubble %>" data-toggle="tooltip" title="<%= t("partials.feed.status_bubble.#{feed.status_bubble}") if feed.status_bubble %>" data-placement="left"></i>
-        <%= feed.name %>
+        <a href="news/<%= feed.id %>"><%= feed.name %></a>
         <span class="feed-url-link">
           <a class="feed-url" href="<%= feed.url %>">
             <i class="icon-external-link"></i>
diff --git a/app/views/partials/_mark_all_as_read_form.erb b/app/views/partials/_mark_all_as_read_form.erb
index 8e34745cd..c03ed414c 100644
--- a/app/views/partials/_mark_all_as_read_form.erb
+++ b/app/views/partials/_mark_all_as_read_form.erb
@@ -2,6 +2,7 @@
   <form id="mark-all-as-read" method="POST" action="/mark_all_as_read">
     <% stories.each do |story| %>
       <input type="hidden" name="story_ids[]" value="<%= story.id %>" />
+      <input type="hidden" name="feed_id" value="<%= feed_id %>" />
     <% end %>
   </form>
-</div>
\ No newline at end of file
+</div>
diff --git a/app/views/partials/_tutorial_action_bar.erb b/app/views/partials/_tutorial_action_bar.erb
index f5ed6d893..a99829b66 100644
--- a/app/views/partials/_tutorial_action_bar.erb
+++ b/app/views/partials/_tutorial_action_bar.erb
@@ -3,7 +3,7 @@
     <a class="btn" id="mark-all" title="<%= t('partials.action_bar.mark_all') %>">
       <i class="icon-ok"></i>
       <%= render_partial :mark_all_as_read_form,
-                        {stories: stories} %>
+                        {stories: stories, feed_id: nil} %>
     </a>
     <a class="btn" href="/" id="refresh" title="<%= t('partials.action_bar.refresh') %>">
       <i class="icon-repeat"></i>
diff --git a/spec/controllers/stories_controller_spec.rb b/spec/controllers/stories_controller_spec.rb
index 8e7c36be2..63adabed5 100644
--- a/spec/controllers/stories_controller_spec.rb
+++ b/spec/controllers/stories_controller_spec.rb
@@ -123,5 +123,14 @@
       last_response.status.should be 302
       URI::parse(last_response.location).path.should eq "/news"
     end
+
+    it "marks all unread stories of a feed as read and reload the page" do
+      MarkAllAsRead.any_instance.should_receive(:mark_as_read).once
+
+      post "/mark_all_as_read", story_ids: ["1", "2", "3"], feed_id: 4
+
+      last_response.status.should be 302
+      URI::parse(last_response.location).path.should eq "/news/4"
+    end
   end
-end
\ No newline at end of file
+end