split out tag types in pots/show page

app/assets/stylesheets/specific/posts.css.scss

@@ -75,6 +75,10 @@ div#c-posts {
       color: #666;
     }
   }
+
+  aside#sidebar > section > ul {
+    margin-bottom: 1em;
+  }
   
   aside#sidebar > section > ul li {
     list-style-type: none;

app/presenters/post_presenter.rb

@@ -43,6 +43,11 @@ class PostPresenter < Presenter
     @tag_set_presenter.tag_list_html(template, options.merge(:show_extra_links => CurrentUser.user.is_privileged?))
   end
   
+  def split_tag_list_html(template, options = {})
+    @tag_set_presenter ||= TagSetPresenter.new(@post.tag_array)
+    @tag_set_presenter.split_tag_list_html(template, options.merge(:show_extra_links => CurrentUser.user.is_privileged?))
+  end
+  
   def post_footer_for_pool_html(template)
     if template.params[:pool_id]
       pool = Pool.where(:id => template.params[:pool_id]).first
@@ -91,7 +96,7 @@ class PostPresenter < Presenter
     end
     
     html << " "
-    html << template.link_to(pool.name, template.pool_path(pool))
+    html << template.link_to(pool.pretty_name, template.pool_path(pool))
     html << "</li>"
     html
   end

app/presenters/tag_set_presenter.rb

@@ -11,15 +11,76 @@ class TagSetPresenter < Presenter
   
   def tag_list_html(template, options = {})
     html = ""
-    html << "<ul>"
+    if @tags.present?
-    Array(@tags).each do |tag|
+      html << "<ul>"
-      html << build_list_item(tag, template, options)
+      @tags.each do |tag|
+        html << build_list_item(tag, template, options)
+      end
+      html << "</ul>"
     end
-    html << "</ul>"
+
+    html.html_safe
+  end
+  
+  def split_tag_list_html(template, options = {})
+    html = ""
+    
+    if copyright_tags.any?
+      html << '<h1>Copyrights</h1>'
+      html << "<ul>"
+      copyright_tags.keys.each do |tag|
+        html << build_list_item(tag, template, options)
+      end
+      html << "</ul>"
+    end
+    
+    if character_tags.any?
+      html << '<h1>Characters</h1>'
+      html << "<ul>"
+      character_tags.keys.each do |tag|
+        html << build_list_item(tag, template, options)
+      end
+      html << "</ul>"
+    end
+    
+    if artist_tags.any?
+      html << '<h1>Artist</h1>'
+      html << "<ul>"
+      artist_tags.keys.each do |tag|
+        html << build_list_item(tag, template, options)
+      end
+      html << "</ul>"
+    end
+    
+    if general_tags.any?
+      html << '<h1>Tags</h1>'
+      html << "<ul>"
+      general_tags.keys.each do |tag|
+        html << build_list_item(tag, template, options)
+      end
+      html << "</ul>"
+    end
+
     html.html_safe
   end
 
 private
+  def general_tags
+    @general_tags ||= categories.select {|k, v| v == Tag.categories.general}
+  end
+  
+  def copyright_tags
+    @copyright_tags ||= categories.select {|k, v| v == Tag.categories.copyright}
+  end
+  
+  def character_tags
+    @character_tags ||= categories.select {|k, v| v == Tag.categories.character}
+  end
+  
+  def artist_tags
+    @artist_tags ||= categories.select {|k, v| v == Tag.categories.artist}
+  end
+
   def categories
     @categories ||= Tag.categories_for(@tags)
   end
@@ -37,7 +98,7 @@ private
     current_query = template.params[:tags] || ""
     
     unless options[:name_only]
-      if categories[tag] == 1
+      if categories[tag] == Tag.categories.artist
         html << %{<a class="wiki-link" href="/artists/show_or_new?name=#{u(tag)}">?</a> }
       else
         html << %{<a class="wiki-link" href="/wiki_pages?title=#{u(tag)}">?</a> }

app/views/posts/partials/show/_edit.html.erb

@@ -5,7 +5,7 @@
 <% end %>
 
 <%= form_for(post, :html => {:class => "simple_form"}) do |f| %>
-  <%= f.hidden_field :old_tags, :value => post.tag_string %>
+  <%= f.hidden_field :old_tag_string, :value => post.tag_string %>
 
   <div class="input">
     <% if post.is_rating_locked? %>

app/views/posts/show.html.erb

@@ -11,8 +11,7 @@
       <% end %>
 
       <section>
-        <h1>Tags</h1>
+        <%= @post.presenter.split_tag_list_html(self) %>
-        <%= @post.presenter.tag_list_html(self) %>
       </section>
 
       <section>