Avoid reading nodes twice in WayController#full

[rails.git] / app / views / diary_entry / list.html.erb

diff --git a/app/views/diary_entry/list.html.erb b/app/views/diary_entry/list.html.erb
index 68c8c12668bb7134fee58e6430fd4e208b9522ab..09d8af2fdcfdee0a3f253b1c18096d234ee0f383 100644 (file)
--- a/app/views/diary_entry/list.html.erb
+++ b/app/views/diary_entry/list.html.erb
@@ -1,39 +1,56 @@
-<h2><%= h(@title) %></h2>
-
-<% if @this_user && @this_user.image %>
- <%= image_tag url_for_file_column(@this_user, "image") %>
-<% end %>
-
-
-<% if @this_user %>
-  <% if @user == @this_user %>
-    <%= link_to image_tag("new.png", :border=>0) + t('diary_entry.list.new'), {:controller => 'diary_entry', :action => 'new', :display_name => @user.display_name}, {:title => t('diary_entry.list.new_title')} %>
-  <% end %>
-<% else %>
-  <% if @user %>
-    <%= link_to image_tag("new.png", :border=>0) + t('diary_entry.list.new'), {:controller => 'diary_entry', :action => 'new', :display_name => @user.display_name}, {:title => t('diary_entry.list.new_title')} %>
-  <% end %>
+<% content_for :heading do %>
+  <div <% if @this_user %> id="userinformation"<% end %> >
+    <% if @this_user %>
+      <%= user_image @this_user %>
+    <% end %>
+    <h2><%= h(@title) %></h2>
+
+    <ul class='secondary-actions clearfix'>
+      <% unless params[:friends] or params[:nearby] -%>
+        <li><%= rss_link_to :action => 'rss', :language => params[:language] %></li>
+      <% end -%>
+
+      <% if @this_user %>
+        <%= if_user(@this_user) do %>
+          <li><%= link_to image_tag("new.png", :class => "small_icon", :border=>0) + t('diary_entry.list.new'), {:controller => 'diary_entry', :action => 'new'}, {:title => t('diary_entry.list.new_title')} %></li>
+        <% end %>
+      <% else %>
+        <%= if_logged_in do %>
+          <li><%= link_to image_tag("new.png", :class => "small_icon", :border=>0) + t('diary_entry.list.new'), {:controller => 'diary_entry', :action => 'new'}, {:title => t('diary_entry.list.new_title')} %></li>
+        <% end %>
+      <% end %>
+    </ul>
+  </div>

 <% end %>
 
 <% end %>
 

-

 <% if @entries.empty? %>
 <% if @entries.empty? %>

-  <p><%= t 'diary_entry.list.no_entries' %></p>
+  <h4><%= t 'diary_entry.list.no_entries' %></h4>

 <% else %>
 <% else %>

-  <p><%= t 'diary_entry.list.recent_entries' %></p>
-
-  <hr />
+  <h4><%= t 'diary_entry.list.recent_entries' %></h4>

 
 

-  <%= render :partial => 'diary_entry', :collection => @entries %>
-       
-  <%= link_to t('diary_entry.list.older_entries'), { :page => @entry_pages.current.next, :language => params[:language] } if @entry_pages.current.next %>
-  <% if @entry_pages.current.next and @entry_pages.current.previous %>|<% end %>
-  <%= link_to t('diary_entry.list.newer_entries'), { :page => @entry_pages.current.previous, :language => params[:language] } if @entry_pages.current.previous %>
+  <% if @this_user %>
+    <%= render :partial => 'diary_entry', :collection => @entries %>
+  <% else %>
+    <%= render :partial => 'diary_list_entry', :collection => @entries %>
+  <% end %>

 
 

-  <br />
-<% end %>
+  <% if @page > 1 -%>
+    <%= link_to t('diary_entry.list.older_entries'), params.merge(:page => @page - 1 ) %>
+  <% else -%>
+    <%= t('diary_entry.list.older_entries') %>
+  <% end -%>

 
 

-<%= rss_link_to :action => 'rss', :language => params[:language] %>
+  |

 
 

-<% content_for :head do %>
-<%= auto_discovery_link_tag :atom, :action => 'rss', :language => params[:language] %>
+  <% if @entries.size < @page_size -%>
+    <%= t('diary_entry.list.newer_entries') %>
+  <% else -%>
+    <%= link_to t('diary_entry.list.newer_entries'), params.merge(:page => @page + 1) %>
+  <% end -%>

 <% end %>
 <% end %>

+
+<% unless params[:friends] or params[:nearby] -%>
+  <% content_for :head do -%>
+  <%= auto_discovery_link_tag :atom, :action => 'rss', :language => params[:language] %>
+  <% end -%>
+<% end -%>