diff --git a/.travis.yml b/.travis.yml index e8fb782..f240b90 100644 --- a/.travis.yml +++ b/.travis.yml @@ -10,9 +10,11 @@ notifications: env: - DB=postgresql - DB=mysql +matrix: + allow_failures: + - rvm: rbx-19mode rvm: - 1.8.7 - - 1.9.2 - 1.9.3 - - rbx + - rbx-19mode - jruby diff --git a/Gemfile b/Gemfile index b6ba6ac..f9da5c3 100644 --- a/Gemfile +++ b/Gemfile @@ -2,12 +2,14 @@ source "http://rubygems.org" gemspec -gem 'refinerycms', '~> 2.0.0' +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' group :development, :test do require 'rbconfig' - gem 'refinerycms-testing', '~> 2.0.0' + gem 'refinerycms-testing', :git => 'git://github.com/resolve/refinerycms.git' 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 f307a74..b379a24 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 :partial => "/shared/message" + render "/shared/message" end end else diff --git a/app/controllers/refinery/blog/posts_controller.rb b/app/controllers/refinery/blog/posts_controller.rb index 5218493..caa46ec 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? } + caches_page :index, :unless => proc {|c| c.refinery_user_signed_in? || c.flash.any? || params[:page].present? } 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_archive(@archive_date).page(params[:page]) + @posts = Post.live.by_month(@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 d868354..69a7da4 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, - :recipients => Blog::Comment::Notification.recipients, + :to => 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 6deb5c5..2935c89 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.select(&:live?).count + posts.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 deleted file mode 100644 index 7013355..0000000 --- a/app/models/refinery/blog/comment_mailer.rb +++ /dev/null @@ -1 +0,0 @@ -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 b078272..c32b775 100644 --- a/app/models/refinery/blog/post.rb +++ b/app/models/refinery/blog/post.rb @@ -49,14 +49,19 @@ module Refinery end def friendly_id_source - custom_url.present? ? custom_url : title + custom_url.presence || title end class << self - def by_archive(date) + def by_month(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 086c2d1..ec51ea7 100644 --- a/app/models/refinery/categorization.rb +++ b/app/models/refinery/categorization.rb @@ -4,6 +4,7 @@ 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 2872e82..e0f80e7 100644 --- a/app/views/refinery/blog/admin/categories/edit.html.erb +++ b/app/views/refinery/blog/admin/categories/edit.html.erb @@ -1 +1 @@ -<%= render :partial => "form" %> +<%= render "form" %> diff --git a/app/views/refinery/blog/admin/categories/index.html.erb b/app/views/refinery/blog/admin/categories/index.html.erb index 2e4313e..451abc5 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 :partial => '/refinery/blog/admin/submenu' %> +<%= render '/refinery/blog/admin/submenu' %>
<% if searching? %>

<%= t('results_for', :scope => 'refinery.admin.search', @@ -13,7 +13,7 @@ <% if @categories.any? %> <%= will_paginate @categories %> - <%= render :partial => "sortable_list" %> + <%= render "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 2872e82..e0f80e7 100644 --- a/app/views/refinery/blog/admin/categories/new.html.erb +++ b/app/views/refinery/blog/admin/categories/new.html.erb @@ -1 +1 @@ -<%= render :partial => "form" %> +<%= render "form" %> diff --git a/app/views/refinery/blog/admin/comments/index.html.erb b/app/views/refinery/blog/admin/comments/index.html.erb index dd84e9d..53c368e 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 :partial => '/refinery/blog/admin/submenu' %> +<%= render '/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 :partial => "sortable_list" %> + <%= render "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 2872e82..e0f80e7 100644 --- a/app/views/refinery/blog/admin/posts/edit.html.erb +++ b/app/views/refinery/blog/admin/posts/edit.html.erb @@ -1 +1 @@ -<%= render :partial => "form" %> +<%= render "form" %> diff --git a/app/views/refinery/blog/admin/posts/index.html.erb b/app/views/refinery/blog/admin/posts/index.html.erb index 4a4dc66..19c4ccf 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 :partial => '/refinery/blog/admin/submenu' %> +<%= render '/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 :partial => "sortable_list" %> + <%= render "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 2872e82..e0f80e7 100644 --- a/app/views/refinery/blog/admin/posts/new.html.erb +++ b/app/views/refinery/blog/admin/posts/new.html.erb @@ -1 +1 @@ -<%= render :partial => "form" %> +<%= render "form" %> diff --git a/app/views/refinery/blog/admin/posts/uncategorized.html.erb b/app/views/refinery/blog/admin/posts/uncategorized.html.erb index 1b3a512..21e7c04 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 :partial => '/refinery/blog/admin/submenu' %> +<%= render '/refinery/blog/admin/submenu' %>
<% if searching? %>

<%= t('results_for', :scope => 'refinery.admin.search', @@ -13,7 +13,7 @@ <% if @posts.any? %> <%= will_paginate @posts %> - <%= render :partial => "sortable_list" %> + <%= render "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 aa4abaf..3a4a617 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_content_left do %> +<% content_for :body do %> <% if @posts.any? %>
<%= render :partial => "/refinery/blog/shared/post", :collection => @posts %> @@ -13,9 +13,9 @@ <% end %> <% end %> -<% content_for :body_content_right do %> - <%= render :partial => "/refinery/blog/shared/categories" %> +<% content_for :side_body do %> + <%= render "/refinery/blog/shared/categories" %> <% end %> -<%= render :partial => "/refinery/content_page" %> +<%= render "/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 c3ae924..9811dd2 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 :partial => '/refinery/draft_page_message' unless @post.nil? or @post.live? -%> +<%= render '/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 bda6c45..31b5c1e 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_content_left do %> +<% content_for :body do %>

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

<% if @posts.any? %>
@@ -11,7 +11,7 @@ <% end %> <% end %> -<%= render :partial => '/refinery/blog/shared/body_content_right' %> +<%= render '/refinery/blog/shared/body_content_right' %> -<%= render :partial => "/refinery/content_page" %> +<%= render "/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 2e8837c..772f2ec 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_content_left do %> +<% content_for :body 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 :body_content_right_prepend do -%> +<% content_for :side_body_prepend do -%> <%= raw @page.content_for(Refinery::Pages.default_parts.second.to_sym) %> <% end if Refinery::Pages.default_parts.many? -%> -<%= render :partial => '/refinery/blog/shared/body_content_right' %> +<%= render '/refinery/blog/shared/body_content_right' %> -<%= render :partial => "/refinery/content_page" %> +<%= render "/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 2ca168e..d7abd1e 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_content_left do %> +<% content_for :body do %>
<%= render 'post' %>
@@ -8,7 +8,7 @@ <% end %> <% end %> -<%= render :partial => '/refinery/blog/shared/body_content_right' %> +<%= render '/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 3a3ee75..c22e55b 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_content_left do %> +<% content_for :body do %> <% if @posts.any? %>
<%= render :partial => "/refinery/blog/shared/post", :collection => @posts %> @@ -13,7 +13,7 @@ <% end %> <% end %> -<%= render :partial => '/refinery/blog/shared/body_content_right' %> +<%= render '/refinery/blog/shared/body_content_right' %> -<%= render :partial => "/refinery/content_page" %> +<%= render "/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 bd73d1c..b614dca 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 :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" %> +<% 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" %> <%= blog_archive_widget %> - <%= yield(:body_content_right_append) %> + <%= yield(:side_body_append) %> <% end %> diff --git a/app/views/refinery/blog/shared/_post.html.erb b/app/views/refinery/blog/shared/_post.html.erb index 1a52fd4..5b90c8e 100644 --- a/app/views/refinery/blog/shared/_post.html.erb +++ b/app/views/refinery/blog/shared/_post.html.erb @@ -30,7 +30,9 @@