diff --git a/app/presenters/post_presenter.rb b/app/presenters/post_presenter.rb
index 626b16a6f..9ebbf87ba 100644
--- a/app/presenters/post_presenter.rb
+++ b/app/presenters/post_presenter.rb
@@ -90,7 +90,7 @@ class PostPresenter < Presenter
def pool_html(template)
html = ["
"]
- if template.params[:pool_id]
+ if template.params[:pool_id].present?
pool = Pool.where(:id => template.params[:pool_id]).first
return if pool.nil?
html = pool_link_html(html, template, pool)
@@ -109,23 +109,31 @@ class PostPresenter < Presenter
end
def pool_link_html(html, template, pool)
- html << "- "
+ pool_html = ["
- "]
+ match_found = false
if pool.neighbors(@post).previous
- html << template.link_to("«prev".html_safe, template.post_path(pool.neighbors(@post).previous, :pool_id => pool.id), :rel => "prev")
+ pool_html << template.link_to("«prev".html_safe, template.post_path(pool.neighbors(@post).previous, :pool_id => pool.id), :rel => "prev")
+ match_found = true
else
- html << "«prev"
+ pool_html << "«prev"
end
if pool.neighbors(@post).next
- html << template.link_to("next»".html_safe, template.post_path(pool.neighbors(@post).next, :pool_id => pool.id), :rel => "next")
+ pool_html << template.link_to("next»".html_safe, template.post_path(pool.neighbors(@post).next, :pool_id => pool.id), :rel => "next")
+ match_found = true
else
- html << "next»"
+ pool_html << "next»"
end
- html << " "
- html << template.link_to(pool.pretty_name, template.pool_path(pool))
- html << "
"
+ pool_html << " "
+ pool_html << template.link_to(pool.pretty_name, template.pool_path(pool))
+ pool_html << ""
+
+ if match_found
+ html += pool_html
+ end
+
html
end
end