I think telling the user to lock to a x.x.0 is sufficient

Refactor post

.gitignore

@@ -42,3 +42,7 @@ nbproject
 
 # Capybara Bug
 capybara-*html
+
+# Future stuff
+Gemfile.lock
+spec/dummy

.travis.yml

@@ -0,0 +1,5 @@
+rvm:
+  - 1.9.2
+branches:
+  only:
+    - rails-3-1

app/helpers/blog_posts_helper.rb

@@ -1,10 +1,8 @@
 module BlogPostsHelper
   def blog_archive_list
-    posts = BlogPost.select('published_at').all_previous
+    posts = BlogPost.live.select('published_at').all_previous
     return nil if posts.blank?
-    html = '<section id="blog_archive_list"><h2>'
+    html = ''
-    html << t('archives', :scope => 'blog.shared')
-    html << '</h2><nav><ul>'
     links = []
     super_old_links = []
 
@@ -20,7 +18,7 @@ module BlogPostsHelper
     links.each do |l|
       year = l.split('/')[1]
       month = l.split('/')[0]
-      count = BlogPost.by_archive(Time.parse(l)).size
+      count = BlogPost.live.by_archive(Time.parse(l)).size
       text = t("date.month_names")[month.to_i] + " #{year} (#{count})"      
       html << "<li>"
       html << link_to(text, archive_blog_posts_path(:year => year, :month => month))
@@ -28,13 +26,12 @@ module BlogPostsHelper
     end
     super_old_links.each do |l|
       year = l.split('/')[1]
-      count = BlogPost.by_year(Time.parse(l)).size
+      count = BlogPost.live.by_year(Time.parse(l)).size
       text = "#{year} (#{count})"
       html << "<li>"
       html << link_to(text, archive_blog_posts_path(:year => year))
       html << "</li>"
     end
-    html << '</ul></nav></section>'
     html.html_safe
   end
 

app/views/admin/blog/posts/_form.html.erb

@@ -9,7 +9,12 @@
     <%= f.label :title -%>
     <%= f.text_field :title, :class => 'larger widest' -%>
   </div>
-
+  <div class="field">
+    <p>
+      <%= f.check_box :draft %>
+      <%= f.label :draft, t('.save_as_draft'), :class => "stripped" %>
+    </p>
+  </div>
   <div class='field'>
     <div id='page-tabs' class='clearfix ui-tabs ui-widget ui-widget-content ui-corner-all'>
       <ul id='page_parts'>
@@ -59,10 +64,7 @@
                   :id => 'toggle_advanced_options',
                   :title => t('.toggle_advanced_options') %>
     </p>
-    <span id='draft_field'>
+    
-      <%= f.check_box :draft %>
-      <%= f.label :draft, t('.save_as_draft'), :class => "stripped" %>
-    </span>
   </div>
   <div id='more_options' style="display:none;">
     <div class="hemisquare">

app/views/admin/blog/posts/_post.html.erb

@@ -6,6 +6,9 @@
       <%= " by #{post.author.username}" if post.author.present? %>
     </span>
   </span>
+  <% if post.draft? %>
+    <span class="label notice">Draft</span>
+  <% end %>
   <span class='actions'>
     <%= link_to refinery_icon_tag("application_go.png"), blog_post_url(post),
         :title => t('.view_live_html'),

app/views/blog/posts/_post.html.erb

@@ -1,31 +0,0 @@
-<% flash.each do |key, value| %>
- <div id='flash' class="flash flash_<%= key %>">
-   <%= value %>
- </div>
-<% end %>
-<article id="blog_post">
-  <header>
-    <h1><%= @blog_post.title %></h1>
-    <section class='details'>
-      <time datetime="<%=l @blog_post.published_at.to_date, :format => :default %>" class='posted_at'>
-        <%= t('created_at', :scope => 'blog.shared.posts', :when => l(@blog_post.published_at.to_date, :format => :short)) %>
-      </time>
-      <%= "#{t('by', :scope => 'blog.posts.show')} #{@blog_post.author.username}" if @blog_post.author.present? %>.
-      <% if (categories = @blog_post.categories).any? %>
-        <aside class='filed_in'>
-          <%= t('filed_in', :scope => 'blog.posts.show') %>
-          <% categories.each_with_index do |category, index| %>
-            <%= link_to category.title, blog_category_url(category) -%><%= ',' if index < ((categories.length) - 1) %>
-          <% end %>
-        </aside>
-      <% end %>
-    </section>
-  </header>
-  <%= @blog_post.body.html_safe %>
-
-  <% if BlogPost::ShareThis.enabled? %>
-    <span class="st_sharethis" displayText="ShareThis"></span>
-  <% end %>
-</article>
-<%= render :partial => '/shared/draft_page_message' unless @blog_post.nil? or @blog_post.live? -%>
-<%= render 'nav' if next_or_previous?(@blog_post) %>

app/views/blog/posts/archive.html.erb

@@ -13,7 +13,7 @@
   <%= render :partial => "/blog/shared/categories" %>
   <%= render :partial => "/blog/shared/tags" %>
   <%= render :partial => "/blog/shared/rss_feed" %>
-  <%= blog_archive_list %>
+  <%= render :partial => "/blog/shared/archive_list" %>
 <% end %>
 
 <%= render :partial => "/shared/content_page" %>

app/views/blog/posts/index.html.erb

@@ -17,7 +17,7 @@
   <%= render :partial => "/blog/shared/categories" %>
   <%= render :partial => "/blog/shared/tags" %>
   <%= render :partial => "/blog/shared/rss_feed" %>
-  <%= blog_archive_list %>
+  <%= render :partial => "/blog/shared/archive_list" %>
 <% end %>
 
 <%= render :partial => "/shared/content_page" %>

app/views/blog/posts/show.html.erb

@@ -1,6 +1,37 @@
 <% content_for :body_content_left do %>
   <div id="show_blog_post">
-    <%= render 'post' %>
+    <% flash.each do |key, value| %>
+     <div id='flash' class="flash flash_<%= key %>">
+       <%= value %>
+     </div>
+    <% end %>
+    <article id="blog_post">
+      <header>
+        <h1><%= @blog_post.title %></h1>
+        <section class='details'>
+          <time datetime="<%=l @blog_post.published_at.to_date, :format => :default %>" class='posted_at'>
+            <%= t('created_at', :scope => 'blog.shared.posts', :when => l(@blog_post.published_at.to_date, :format => :short)) %>
+          </time>
+          <%= "#{t('by', :scope => 'blog.posts.show')} #{@blog_post.author.username}" if @blog_post.author.present? %>.
+          <% if (categories = @blog_post.categories).any? %>
+            <aside class='filed_in'>
+              <%= t('filed_in', :scope => 'blog.posts.show') %>
+              <% categories.each_with_index do |category, index| %>
+                <%= link_to category.title, blog_category_url(category) -%><%= ',' if index < ((categories.length) - 1) %>
+              <% end %>
+            </aside>
+          <% end %>
+        </section>
+      </header>
+      <%= @blog_post.body.html_safe %>
+
+      <% if BlogPost::ShareThis.enabled? %>
+        <span class="st_sharethis" displayText="ShareThis"></span>
+      <% end %>
+    </article>
+    <%= render :partial => '/shared/draft_page_message' unless @blog_post.nil? or @blog_post.live? -%>
+    <%= render 'nav' if next_or_previous?(@blog_post) %>
+    
   </div>
 
   <% if BlogPost.comments_allowed? %>
@@ -50,9 +81,9 @@
 <% content_for :body_content_right do %>
   <%= render :partial => "/blog/shared/categories" %>
   <%= render :partial => "/blog/shared/tags" %>
-  <%= render :partial => "/blog/shared/posts" %>
+  <%= render :partial => "/blog/shared/related_posts" %>
   <%= render :partial => "/blog/shared/rss_feed" %>
-  <%= blog_archive_list %>
+  <%= render :partial => "/blog/shared/archive_list" %>
 <% end %>
 
 <%= render :partial => "/shared/content_page", :locals => { :remove_automatic_sections => true }  %>

app/views/blog/posts/tagged.html.erb

@@ -15,7 +15,7 @@
   <%= render :partial => "/blog/shared/categories" %>
   <%= render :partial => "/blog/shared/tags" %>
   <%= render :partial => "/blog/shared/rss_feed" %>
-  <%= blog_archive_list %>
+  <%= render :partial => "/blog/shared/archive_list" %>
 <% end %>
 
 <%= render :partial => "/shared/content_page" %>

app/views/blog/shared/_archive_list.html.erb

@@ -0,0 +1,9 @@
+<%
+  archive_posts = blog_archive_list
+%>
+<% if archive_posts.present? %>
+  <h2><%= t('archives', :scope => 'blog.shared') %></h2>
+  <ul>
+    <%= archive_posts %>
+  </ul>
+<% end %>

changelog.md

@@ -1,16 +1,29 @@
-## 1.6.3 [UNRELEASED]
+## 1.8.0 [UNRELEASED]
+* extraneous shared/_post partial was moved directly into posts/show [nicinabox](https://github.com/nicinabox)
+* blog archive list helper logic refactored (HTML exists in view template) [nicinabox](https://github.com/nicinabox)
+* pretty "Draft" label added to admin list of blog posts [nicinabox](https://github.com/nicinabox)
+* [See full list](https://github.com/resolve/refinerycms-blog/compare/1.7.0...1.8.0)
+
+## 1.7.0 [05 December 2011]
 * Bulgarian translations [mirosr](https://github.com/mirosr)
 * posts/tagged... params[:tag_name] is optional [joemsak](https://github.com/joemsak)
+* Bug Fix on Categorization relationship: destroying a post produced error because no primary key existed on join table [joemsak](https://github.com/joemsak)
+* Bug fix on archive listing, don't include draft posts [jgrevich](https://github.com/jgrevich)
+* [See full list](https://github.com/resolve/refinerycms-blog/compare/1.6.2...1.7.0)
 
 ## 1.6.2 [29 June 2011]
 * Custom teaser field to overwrite the automatic truncation of the body [wikyd](https://github.com/wikyd)
 * Tagging URL strategy updated again [joemsak](https://github.com/joemsak)
 * Tagging bug fixed [ruprict](https://github.com/ruprict)
 * Refactored specs [parndt](https://github.com/parndt)
+* * [See full list](https://github.com/resolve/refinerycms-blog/compare/1.6.1...1.6.2)
+
 
 ## 1.6.1 [21 June 2011]
 * Tagging URL strategy updated for reliability /blog/posts/tagged/ID-name-parameterized [joemsak](https://github.com/joemsak)
 * Heroku/PostgreSQL support for autocomplete tags [joemsak](https://github.com/joemsak)
+* [See full list](https://github.com/resolve/refinerycms-blog/compare/1.6...1.6.1)
+
 
 ## 1.6 [20 June 2011]
 * Category bug fixes and cleanup [wikyd](https://github.com/wikyd)
@@ -20,11 +33,14 @@
 * More testing [wakeless](https://github.com/wakeless)
 * Tag list autocomplete baked in [joemsak](https://github.com/joemsak)
 * Customize the URL of your blog post [wikyd](https://github.com/wikyd)
+* [See full list](https://github.com/resolve/refinerycms-blog/compare/1.5...1.6)
+
 
 ## 1.5 [28 May 2011]
 
 * Added Gravatar support. [parndt](https://github.com/parndt)
 * Added support for Refinery CMS 1.0.0 and above. [parndt](https://github.com/parndt)
+* [See full list](https://github.com/resolve/refinerycms-blog/compare/1.4...1.5)
 
 ## 1.4 [26 May 2011]
 

db/migrate/1_create_blog_structure.rb

@@ -30,7 +30,7 @@ class CreateBlogStructure < ActiveRecord::Migration
 
     add_index :blog_categories, :id
 
-    create_table :blog_categories_blog_posts, :id => true do |t|
+    create_table :blog_categories_blog_posts, :id => false do |t|
       t.integer :blog_category_id
       t.integer :blog_post_id
     end

lib/refinery/blog/version.rb

@@ -2,8 +2,8 @@ module Refinery
   module Blog
     class Version
       @major = 1
-      @minor = 6
+      @minor = 8
-      @tiny  = 2
+      @tiny  = 0
 
       class << self
         attr_reader :major, :minor, :tiny

readme.md

@@ -12,12 +12,13 @@ Options:
 ## Requirements
 
 Refinery CMS version 1.0.0 or above.
+Your Rails 3 application should not be called "blog"
 
 ## Install
 
 Open up your ``Gemfile`` and add at the bottom this line:
 
-    gem 'refinerycms-blog', '~> 1.6.1'
+    gem 'refinerycms-blog', '~> 1.8.0'
 
 Now, run ``bundle install``
 

refinerycms-blog.gemspec

@@ -1,8 +1,8 @@
 Gem::Specification.new do |s|
   s.name              = %q{refinerycms-blog}
-  s.version           = %q{1.6.2}
+  s.version           = %q{1.8.0}
   s.description       = %q{A really straightforward open source Ruby on Rails blog engine designed for integration with RefineryCMS.}
-  s.date              = %q{2011-06-29}
+  s.date              = %q{2011-12-08}
   s.summary           = %q{Ruby on Rails blogging engine for RefineryCMS.}
   s.email             = %q{info@refinerycms.com}
   s.homepage          = %q{http://refinerycms.com/blog}
@@ -99,12 +99,14 @@ Gem::Specification.new do |s|
     changelog.md
     config
     config/locales
+    config/locales/bg.yml
     config/locales/cs.yml
     config/locales/de.yml
     config/locales/en.yml
     config/locales/es.yml
     config/locales/fr.yml
     config/locales/it.yml
+    config/locales/ja.yml
     config/locales/nb.yml
     config/locales/nl.yml
     config/locales/pl.yml
@@ -122,6 +124,7 @@ Gem::Specification.new do |s|
     db/migrate/5_add_cached_slugs.rb
     db/migrate/6_add_custom_url_field_to_blog_posts.rb
     db/migrate/7_add_custom_teaser_field_to_blog_posts.rb
+    db/migrate/8_add_primary_key_to_categorizations.rb
     db/seeds
     db/seeds/refinerycms_blog.rb
     features
@@ -138,6 +141,8 @@ Gem::Specification.new do |s|
     features/support/step_definitions/category_steps.rb
     features/support/step_definitions/tags_steps.rb
     features/tags.feature
+    Gemfile
+    Gemfile.lock
     lib
     lib/gemspec.rb
     lib/generators
@@ -197,6 +202,7 @@ Gem::Specification.new do |s|
     spec/models/blog_category_spec.rb
     spec/models/blog_comment_spec.rb
     spec/models/blog_post_spec.rb
+    todo.md
   )
   
 end