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 bde3a9c..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 + 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] @@ -23,7 +23,7 @@ module Refinery @comment = Comment.new @canonical = url_for(:locale => ::Refinery::I18n.default_frontend_locale) if canonical? - + @post.increment!(:access_count, 1) respond_with (@post) do |format| @@ -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) @@ -76,6 +76,7 @@ module Refinery @posts = Post.tagged_with(@tag_name).page(params[:page]) end + protected def canonical? ::Refinery.i18n_enabled? && ::Refinery::I18n.default_frontend_locale != ::Refinery::I18n.current_frontend_locale end 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 05e6994..2935c89 100644 --- a/app/models/refinery/blog/category.rb +++ b/app/models/refinery/blog/category.rb @@ -11,8 +11,10 @@ module Refinery validates :title, :presence => true, :uniqueness => true + attr_accessible :title + def post_count - posts.select(&:live?).count + posts.live.count end # how many items to show per page @@ -20,4 +22,4 @@ module Refinery end end -end \ No newline at end of file +end 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/_form.html.erb b/app/views/refinery/blog/admin/posts/_form.html.erb index 9bfd42a..a977754 100644 --- a/app/views/refinery/blog/admin/posts/_form.html.erb +++ b/app/views/refinery/blog/admin/posts/_form.html.erb @@ -9,6 +9,13 @@ <%= f.label :title -%> <%= f.text_field :title, :class => 'larger widest' -%>
+ +
+

+ <%= f.check_box :draft %> + <%= f.label :draft, t('.save_as_draft'), :class => "stripped" %> +

+
@@ -50,10 +57,6 @@ :id => 'toggle_advanced_options', :title => t('.toggle_advanced_options') %>

- - <%= f.check_box :draft %> - <%= f.label :draft, t('.save_as_draft'), :class => "stripped" %> -