diff --git a/.travis.yml b/.travis.yml index f240b90..e8fb782 100644 --- a/.travis.yml +++ b/.travis.yml @@ -10,11 +10,9 @@ notifications: env: - DB=postgresql - DB=mysql -matrix: - allow_failures: - - rvm: rbx-19mode rvm: - 1.8.7 + - 1.9.2 - 1.9.3 - - rbx-19mode + - rbx - jruby diff --git a/Gemfile b/Gemfile index f9da5c3..b6ba6ac 100644 --- a/Gemfile +++ b/Gemfile @@ -2,14 +2,12 @@ source "http://rubygems.org" gemspec -gem 'refinerycms', :git => 'git://github.com/resolve/refinerycms.git' -gem 'refinerycms-i18n', :git => 'git://github.com/parndt/refinerycms-i18n.git' -gem 'refinerycms-settings', :git => 'git://github.com/parndt/refinerycms-settings.git' +gem 'refinerycms', '~> 2.0.0' group :development, :test do require 'rbconfig' - gem 'refinerycms-testing', :git => 'git://github.com/resolve/refinerycms.git' + gem 'refinerycms-testing', '~> 2.0.0' platforms :jruby do gem 'activerecord-jdbcsqlite3-adapter' diff --git a/app/controllers/refinery/blog/admin/posts_controller.rb b/app/controllers/refinery/blog/admin/posts_controller.rb index b379a24..f307a74 100644 --- a/app/controllers/refinery/blog/admin/posts_controller.rb +++ b/app/controllers/refinery/blog/admin/posts_controller.rb @@ -57,7 +57,7 @@ module Refinery unless request.xhr? redirect_to :back else - render "/shared/message" + render :partial => "/shared/message" end end else diff --git a/app/controllers/refinery/blog/posts_controller.rb b/app/controllers/refinery/blog/posts_controller.rb index caa46ec..5218493 100644 --- a/app/controllers/refinery/blog/posts_controller.rb +++ b/app/controllers/refinery/blog/posts_controller.rb @@ -2,7 +2,7 @@ module Refinery module Blog class PostsController < BlogController - caches_page :index, :unless => proc {|c| c.refinery_user_signed_in? || c.flash.any? || params[:page].present? } + caches_page :index, :unless => proc {|c| c.refinery_user_signed_in? || c.flash.any? } before_filter :find_all_blog_posts, :except => [:archive] before_filter :find_blog_post, :only => [:show, :comment, :update_nav] @@ -60,7 +60,7 @@ module Refinery date = "#{params[:month]}/#{params[:year]}" @archive_date = Time.parse(date) @date_title = @archive_date.strftime('%B %Y') - @posts = Post.live.by_month(@archive_date).page(params[:page]) + @posts = Post.live.by_archive(@archive_date).page(params[:page]) else date = "01/#{params[:year]}" @archive_date = Time.parse(date) diff --git a/app/mailers/refinery/blog/comment_mailer.rb b/app/mailers/refinery/blog/comment_mailer.rb index 69a7da4..d868354 100644 --- a/app/mailers/refinery/blog/comment_mailer.rb +++ b/app/mailers/refinery/blog/comment_mailer.rb @@ -5,7 +5,7 @@ module Refinery def notification(comment, request) @comment = comment mail :subject => Blog::Comment::Notification.subject, - :to => Blog::Comment::Notification.recipients, + :recipients => Blog::Comment::Notification.recipients, :from => "\"#{Refinery::Core.site_name}\" " end diff --git a/app/models/refinery/blog/category.rb b/app/models/refinery/blog/category.rb index 2935c89..6deb5c5 100644 --- a/app/models/refinery/blog/category.rb +++ b/app/models/refinery/blog/category.rb @@ -14,7 +14,7 @@ module Refinery attr_accessible :title def post_count - posts.live.count + posts.select(&:live?).count end # how many items to show per page diff --git a/app/models/refinery/blog/comment_mailer.rb b/app/models/refinery/blog/comment_mailer.rb new file mode 100644 index 0000000..7013355 --- /dev/null +++ b/app/models/refinery/blog/comment_mailer.rb @@ -0,0 +1 @@ +require File.expand_path('../../../../mailers/refinery/blog/comment_mailer', __FILE__) diff --git a/app/models/refinery/blog/post.rb b/app/models/refinery/blog/post.rb index c32b775..b078272 100644 --- a/app/models/refinery/blog/post.rb +++ b/app/models/refinery/blog/post.rb @@ -49,19 +49,14 @@ module Refinery end def friendly_id_source - custom_url.presence || title + custom_url.present? ? custom_url : title end class << self - def by_month(date) + def by_archive(date) where(:published_at => date.beginning_of_month..date.end_of_month) end - - def by_archive(date) - Refinery.deprecate("Refinery::Blog::Post.by_archive(date)", {:replacement => "Refinery::Blog::Post.by_month(date)", :when => 2.2 }) - by_month(date) - end - + def by_year(date) where(:published_at => date.beginning_of_year..date.end_of_year) end diff --git a/app/models/refinery/categorization.rb b/app/models/refinery/categorization.rb index ec51ea7..086c2d1 100644 --- a/app/models/refinery/categorization.rb +++ b/app/models/refinery/categorization.rb @@ -4,7 +4,6 @@ module Refinery self.table_name = 'refinery_blog_categories_blog_posts' belongs_to :blog_post, :class_name => 'Refinery::Blog::Post', :foreign_key => :blog_post_id belongs_to :blog_category, :class_name => 'Refinery::Blog::Category', :foreign_key => :blog_category_id - - attr_accessible :blog_category_id, :blog_post_id + end end \ No newline at end of file diff --git a/app/views/refinery/blog/admin/categories/edit.html.erb b/app/views/refinery/blog/admin/categories/edit.html.erb index e0f80e7..2872e82 100644 --- a/app/views/refinery/blog/admin/categories/edit.html.erb +++ b/app/views/refinery/blog/admin/categories/edit.html.erb @@ -1 +1 @@ -<%= render "form" %> +<%= render :partial => "form" %> diff --git a/app/views/refinery/blog/admin/categories/index.html.erb b/app/views/refinery/blog/admin/categories/index.html.erb index 451abc5..2e4313e 100644 --- a/app/views/refinery/blog/admin/categories/index.html.erb +++ b/app/views/refinery/blog/admin/categories/index.html.erb @@ -1,4 +1,4 @@ -<%= render '/refinery/blog/admin/submenu' %> +<%= render :partial => '/refinery/blog/admin/submenu' %>
<% if searching? %>

<%= t('results_for', :scope => 'refinery.admin.search', @@ -13,7 +13,7 @@ <% if @categories.any? %> <%= will_paginate @categories %> - <%= render "sortable_list" %> + <%= render :partial => "sortable_list" %> <%= will_paginate @categories %> <% else %> diff --git a/app/views/refinery/blog/admin/categories/new.html.erb b/app/views/refinery/blog/admin/categories/new.html.erb index e0f80e7..2872e82 100644 --- a/app/views/refinery/blog/admin/categories/new.html.erb +++ b/app/views/refinery/blog/admin/categories/new.html.erb @@ -1 +1 @@ -<%= render "form" %> +<%= render :partial => "form" %> diff --git a/app/views/refinery/blog/admin/comments/index.html.erb b/app/views/refinery/blog/admin/comments/index.html.erb index 53c368e..dd84e9d 100644 --- a/app/views/refinery/blog/admin/comments/index.html.erb +++ b/app/views/refinery/blog/admin/comments/index.html.erb @@ -1,4 +1,4 @@ -<%= render '/refinery/blog/admin/submenu' %> +<%= render :partial => '/refinery/blog/admin/submenu' %>
<% if searching? %>

<%= t('results_for', :scope => 'shared.admin.search', :query => params[:search]) %>

@@ -18,7 +18,7 @@ <% if @comments.any? %> <%= will_paginate @comments %> - <%= render "sortable_list" %> + <%= render :partial => "sortable_list" %> <%= will_paginate @comments %> <% else %> diff --git a/app/views/refinery/blog/admin/posts/edit.html.erb b/app/views/refinery/blog/admin/posts/edit.html.erb index e0f80e7..2872e82 100644 --- a/app/views/refinery/blog/admin/posts/edit.html.erb +++ b/app/views/refinery/blog/admin/posts/edit.html.erb @@ -1 +1 @@ -<%= render "form" %> +<%= render :partial => "form" %> diff --git a/app/views/refinery/blog/admin/posts/index.html.erb b/app/views/refinery/blog/admin/posts/index.html.erb index 19c4ccf..4a4dc66 100644 --- a/app/views/refinery/blog/admin/posts/index.html.erb +++ b/app/views/refinery/blog/admin/posts/index.html.erb @@ -1,4 +1,4 @@ -<%= render '/refinery/blog/admin/submenu' %> +<%= render :partial => '/refinery/blog/admin/submenu' %>
<% if searching? %>

<%= t('results_for', :scope => 'refinery.admin.search', :query => params[:search]) %>

@@ -13,7 +13,7 @@ <% if @posts.any? %> <%= will_paginate @posts %> - <%= render "sortable_list" %> + <%= render :partial => "sortable_list" %> <%= will_paginate @posts %> <% else %> diff --git a/app/views/refinery/blog/admin/posts/new.html.erb b/app/views/refinery/blog/admin/posts/new.html.erb index e0f80e7..2872e82 100644 --- a/app/views/refinery/blog/admin/posts/new.html.erb +++ b/app/views/refinery/blog/admin/posts/new.html.erb @@ -1 +1 @@ -<%= render "form" %> +<%= render :partial => "form" %> diff --git a/app/views/refinery/blog/admin/posts/uncategorized.html.erb b/app/views/refinery/blog/admin/posts/uncategorized.html.erb index 21e7c04..1b3a512 100644 --- a/app/views/refinery/blog/admin/posts/uncategorized.html.erb +++ b/app/views/refinery/blog/admin/posts/uncategorized.html.erb @@ -1,4 +1,4 @@ -<%= render '/refinery/blog/admin/submenu' %> +<%= render :partial => '/refinery/blog/admin/submenu' %>
<% if searching? %>

<%= t('results_for', :scope => 'refinery.admin.search', @@ -13,7 +13,7 @@ <% if @posts.any? %> <%= will_paginate @posts %> - <%= render "sortable_list" %> + <%= render :partial => "sortable_list" %> <%= will_paginate @posts %> <% else %> diff --git a/app/views/refinery/blog/categories/show.html.erb b/app/views/refinery/blog/categories/show.html.erb index 3a4a617..aa4abaf 100644 --- a/app/views/refinery/blog/categories/show.html.erb +++ b/app/views/refinery/blog/categories/show.html.erb @@ -1,6 +1,6 @@ <% content_for :body_content_title, @category.title %> -<% content_for :body do %> +<% content_for :body_content_left do %> <% if @posts.any? %>
<%= render :partial => "/refinery/blog/shared/post", :collection => @posts %> @@ -13,9 +13,9 @@ <% end %> <% end %> -<% content_for :side_body do %> - <%= render "/refinery/blog/shared/categories" %> +<% content_for :body_content_right do %> + <%= render :partial => "/refinery/blog/shared/categories" %> <% end %> -<%= render "/refinery/content_page" %> +<%= render :partial => "/refinery/content_page" %> <% content_for :stylesheets, stylesheet_link_tag('refinery/blog/frontend') %> diff --git a/app/views/refinery/blog/posts/_post.html.erb b/app/views/refinery/blog/posts/_post.html.erb index 9811dd2..c3ae924 100644 --- a/app/views/refinery/blog/posts/_post.html.erb +++ b/app/views/refinery/blog/posts/_post.html.erb @@ -33,5 +33,5 @@ <% end %> -<%= render '/refinery/draft_page_message' unless @post.nil? or @post.live? -%> +<%= render :partial => '/refinery/draft_page_message' unless @post.nil? or @post.live? -%> <%= render 'nav' if next_or_previous?(@post) %> diff --git a/app/views/refinery/blog/posts/archive.html.erb b/app/views/refinery/blog/posts/archive.html.erb index 31b5c1e..bda6c45 100644 --- a/app/views/refinery/blog/posts/archive.html.erb +++ b/app/views/refinery/blog/posts/archive.html.erb @@ -1,6 +1,6 @@ <% content_for :title, "#{t('.blog_archive_for', :date => @archive_date.strftime('%B %Y'))}" %> -<% content_for :body do %> +<% content_for :body_content_left do %>

<%= t('.blog_archive_for', :date => @archive_date.strftime('%B %Y')) %>

<% if @posts.any? %>
@@ -11,7 +11,7 @@ <% end %> <% end %> -<%= render '/refinery/blog/shared/body_content_right' %> +<%= render :partial => '/refinery/blog/shared/body_content_right' %> -<%= render "/refinery/content_page" %> +<%= render :partial => "/refinery/content_page" %> <% content_for :stylesheets, stylesheet_link_tag('refinery/blog/frontend') %> diff --git a/app/views/refinery/blog/posts/index.html.erb b/app/views/refinery/blog/posts/index.html.erb index 772f2ec..2e8837c 100644 --- a/app/views/refinery/blog/posts/index.html.erb +++ b/app/views/refinery/blog/posts/index.html.erb @@ -1,4 +1,4 @@ -<% content_for :body do %> +<% content_for :body_content_left do %> <%= raw @page.content_for(Refinery::Pages.default_parts.first.to_sym) if Refinery::Pages.default_parts.any? %> <% if @posts.any? %> @@ -11,10 +11,10 @@ <% end %> <% end %> -<% content_for :side_body_prepend do -%> +<% content_for :body_content_right_prepend do -%> <%= raw @page.content_for(Refinery::Pages.default_parts.second.to_sym) %> <% end if Refinery::Pages.default_parts.many? -%> -<%= render '/refinery/blog/shared/body_content_right' %> +<%= render :partial => '/refinery/blog/shared/body_content_right' %> -<%= render "/refinery/content_page" %> +<%= render :partial => "/refinery/content_page" %> <% content_for :stylesheets, stylesheet_link_tag('refinery/blog/frontend') %> diff --git a/app/views/refinery/blog/posts/show.html.erb b/app/views/refinery/blog/posts/show.html.erb index d7abd1e..2ca168e 100644 --- a/app/views/refinery/blog/posts/show.html.erb +++ b/app/views/refinery/blog/posts/show.html.erb @@ -1,4 +1,4 @@ -<% content_for :body do %> +<% content_for :body_content_left do %>
<%= render 'post' %>
@@ -8,7 +8,7 @@ <% end %> <% end %> -<%= render '/refinery/blog/shared/body_content_right' %> +<%= render :partial => '/refinery/blog/shared/body_content_right' %> <%= render :partial => "/refinery/content_page", :locals => { :remove_automatic_sections => true } %> @@ -17,5 +17,5 @@ <%# enable AJAX'd post nav at your own risk until html5 history API implemented. %> <%#= javascript_include_tag('refinery/blog/frontend') %> - + <% end if Refinery::Blog::Post::ShareThis.enabled? %> diff --git a/app/views/refinery/blog/posts/tagged.html.erb b/app/views/refinery/blog/posts/tagged.html.erb index c22e55b..3a3ee75 100644 --- a/app/views/refinery/blog/posts/tagged.html.erb +++ b/app/views/refinery/blog/posts/tagged.html.erb @@ -2,7 +2,7 @@ <% content_for :body_content_title, "#{t('.posts_tagged')} “#{@tag_name.titleize}”".html_safe -%> -<% content_for :body do %> +<% content_for :body_content_left do %> <% if @posts.any? %>
<%= render :partial => "/refinery/blog/shared/post", :collection => @posts %> @@ -13,7 +13,7 @@ <% end %> <% end %> -<%= render '/refinery/blog/shared/body_content_right' %> +<%= render :partial => '/refinery/blog/shared/body_content_right' %> -<%= render "/refinery/content_page" %> +<%= render :partial => "/refinery/content_page" %> <% content_for :stylesheets, stylesheet_link_tag('refinery/blog/frontend') %> diff --git a/app/views/refinery/blog/shared/_body_content_right.html.erb b/app/views/refinery/blog/shared/_body_content_right.html.erb index b614dca..bd73d1c 100644 --- a/app/views/refinery/blog/shared/_body_content_right.html.erb +++ b/app/views/refinery/blog/shared/_body_content_right.html.erb @@ -1,9 +1,9 @@ -<% content_for :side_body do %> - <%= yield(:side_body_prepend) %> - <%= render "/refinery/blog/shared/rss_feed" %> - <%= render "/refinery/blog/shared/categories" %> - <%= render "/refinery/blog/shared/tags" %> - <%= render "/refinery/blog/shared/posts" %> +<% content_for :body_content_right do %> + <%= yield(:body_content_right_prepend) %> + <%= render :partial => "/refinery/blog/shared/rss_feed" %> + <%= render :partial => "/refinery/blog/shared/categories" %> + <%= render :partial => "/refinery/blog/shared/tags" %> + <%= render :partial => "/refinery/blog/shared/posts" %> <%= blog_archive_widget %> - <%= yield(:side_body_append) %> + <%= yield(:body_content_right_append) %> <% end %> diff --git a/app/views/refinery/blog/shared/_post.html.erb b/app/views/refinery/blog/shared/_post.html.erb index 5b90c8e..1a52fd4 100644 --- a/app/views/refinery/blog/shared/_post.html.erb +++ b/app/views/refinery/blog/shared/_post.html.erb @@ -30,9 +30,7 @@