refinerycms-news 1.2.0 → 2.0.0

data/db/migrate/20110817203706_remove_image_id_and_external_url_from_news.rb

@@ -0,0 +1,19 @@
+class RemoveImageIdAndExternalUrlFromNews < ActiveRecord::Migration
+  def up
+    if ::Refinery::News::Item.column_names.map(&:to_sym).include?(:external_url)
+      remove_column ::Refinery::News::Item.table_name, :external_url
+    end
+    if ::Refinery::News::Item.column_names.map(&:to_sym).include?(:image_id)
+      remove_column ::Refinery::News::Item.table_name, :image_id
+    end
+  end
+
+  def down
+    unless ::Refinery::News::Item.column_names.map(&:to_sym).include?(:external_url)
+      add_column ::Refinery::News::Item.table_name, :external_url, :string
+    end
+    unless ::Refinery::News::Item.column_names.map(&:to_sym).include?(:image_id)
+      add_column ::Refinery::News::Item.table_name, :image_id, :integer
+    end
+  end
+end

data/db/migrate/20120129230838_add_source_to_news_items.rb

@@ -0,0 +1,16 @@
+# This migration comes from refinery_news (originally 7)
+class AddSourceToNewsItems < ActiveRecord::Migration
+
+  def up
+    unless Refinery::News::Item.column_names.map(&:to_sym).include?(:source)
+      add_column Refinery::News::Item.table_name, :source, :string
+    end
+  end
+
+  def down
+    if Refinery::News::Item.column_names.map(&:to_sym).include?(:source)
+      remove_column Refinery::News::Item.table_name, :source
+    end
+  end
+
+end

data/db/migrate/20120129230839_translate_source.rb

@@ -0,0 +1,16 @@
+# This migration comes from refinery_news (originally 8)
+class TranslateSource < ActiveRecord::Migration
+
+  def up
+    unless Refinery::News::Item::Translation.column_names.map(&:to_sym).include?(:source)
+      add_column Refinery::News::Item::Translation.table_name, :source, :string
+    end
+  end
+
+  def down
+    if Refinery::News::Item::Translation.column_names.map(&:to_sym).include?(:source)
+      remove_column Refinery::News::Item::Translation.table_name, :source
+    end
+  end
+
+end

data/db/migrate/20120228150250_add_slug_to_news_items.rb

@@ -0,0 +1,5 @@
+class AddSlugToNewsItems < ActiveRecord::Migration
+  def change
+    add_column Refinery::News::Item.table_name, :slug, :string
+  end
+end

data/db/seeds.rb

@@ -0,0 +1,22 @@
+if defined?(::Refinery::User)
+  ::Refinery::User.all.each do |user|
+    if user.plugins.where(:name => 'refinerycms_news').blank?
+      user.plugins.create(:name => 'refinerycms_news')
+    end
+  end
+end
+
+if defined?(::Refinery::Page)
+  unless ::Refinery::Page.where(:menu_match => "^/news.*$").any?
+    page = ::Refinery::Page.create(
+      :title => "News",
+      :link_url => "/news",
+      :deletable => false,
+      :menu_match => "^/news.*$"
+    )
+
+    ::Refinery::Pages.default_parts.each do |default_page_part|
+      page.parts.create(:title => default_page_part, :body => nil)
+    end
+  end
+end

data/lib/generators/refinery/news_generator.rb

@@ -0,0 +1,19 @@
+module Refinery
+  class NewsGenerator < Rails::Generators::Base
+    def rake_db
+      rake("refinery_news:install:migrations")
+      rake("refinery_settings:install:migrations")
+    end
+
+    def append_load_seed_data
+      create_file 'db/seeds.rb' unless File.exists?(File.join(destination_root, 'db', 'seeds.rb'))
+      append_file 'db/seeds.rb', :verbose => true do
+        <<-EOH
+
+# Added by Refinery CMS News engine
+Refinery::News::Engine.load_seed
+        EOH
+      end
+    end
+  end
+end

data/lib/refinery/news.rb

@@ -0,0 +1,22 @@
+require 'refinerycms-core'
+require 'refinerycms-settings'
+
+module Refinery
+  autoload :NewsGenerator, 'generators/refinery/news_generator'
+
+  module News
+    require 'refinery/news/engine'
+
+    class << self
+      attr_writer :root
+
+      def root
+        @root ||= Pathname.new(File.expand_path('../../../', __FILE__))
+      end
+
+      def factory_paths
+        @factory_paths ||= [ root.join("spec/factories").to_s ]
+      end
+    end
+  end
+end

data/lib/refinery/news/engine.rb

@@ -0,0 +1,23 @@
+module Refinery
+  module News
+    class Engine < Rails::Engine
+      include Refinery::Engine
+
+      isolate_namespace Refinery::News
+
+      initializer "init plugin" do
+        Refinery::Plugin.register do |plugin|
+          plugin.pathname = root
+          plugin.name = "refinerycms_news"
+          plugin.menu_match = /refinery\/news(_items)?$/
+          plugin.url = {:controller => '/refinery/news/admin/items'}
+          plugin.activity = { :class_name => 'Refinery::News::Item' }
+        end
+      end
+
+      config.after_initialize do
+        Refinery.register_engine(Refinery::News)
+      end
+    end
+  end
+end

data/lib/refinerycms-news.rb

@@ -1,23 +1 @@
-require 'refinery'
-require File.expand_path('../news', __FILE__)
-
-module Refinery
-  module News
-
-    class Engine < Rails::Engine
-      config.after_initialize do
-        Refinery::Plugin.register do |plugin|
-          plugin.name = "refinerycms_news"
-          plugin.menu_match = /(admin|refinery)\/news(_items)?$/
-          plugin.url = {:controller => '/admin/news_items', :action => 'index'}
-          plugin.activity = {
-            :class => NewsItem,
-            :title => 'title',
-            :url_prefix => 'edit'
-          }
-        end
-      end
-    end
-
-  end
-end
+require 'refinery/news'

data/readme.md

@@ -13,31 +13,33 @@ Key features:
 
 ## Requirements
 
-[RefineryCMS](http://refinerycms.com) "core" engine version 0.9.9.15 or later.
+[Refinery CMS](http://refinerycms.com) "core" engine version 2.0.0 or later.
 
 ### Gem Installation using Bundler (The very best way)
 
 Include the latest [gem](http://rubygems.org/gems/refinerycms-news) into your Refinery CMS application's Gemfile:
 
-    gem "refinerycms-news", '~> 1.1'
+    gem "refinerycms-news", '~> 2.0.0'
 
 Then type the following at command line inside your Refinery CMS application's root directory:
 
     bundle install
-    rails generate refinerycms_news
+    rails generate refinery:news
     rake db:migrate
+    rake db:seed
 
 ## Customising the views
 
 Type this command at your project root to override the default front end views:
 
-    $ rake refinery:override view=news_items/*
-    Copied view template file to app/views/news_items/_recent_posts.html.erb
-    Copied view template file to app/views/news_items/index.html.erb
-    Copied view template file to app/views/news_items/show.html.erb
+    $ rake refinery:override view=refinery/news/items/*
+    Copied view template file to app/views/refinery/news/items/_recent_posts.html.erb
+    Copied view template file to app/views/refinery/news/items/index.html.erb
+    Copied view template file to app/views/refinery/news/items/show.html.erb
+    etc.
 
 ## RSS (Really Simple Syndication)
 
 To get RSS for your entire site, insert this into the head section of your layout after installing:
 
-    <%= auto_discovery_link_tag(:rss, news_items_url(:format => 'rss')) %>
+    <%= auto_discovery_link_tag(:rss, refinery.news_items_url(:format => 'rss')) %>

data/refinerycms-news.gemspec

@@ -0,0 +1,20 @@
+# Encoding: UTF-8
+
+Gem::Specification.new do |s|
+  s.name              = %q{refinerycms-news}
+  s.version           = %q{2.0.0}
+  s.description       = %q{A really straightforward open source Ruby on Rails news engine designed for integration with Refinery CMS.}
+  s.date              = "#{Date.today.strftime("%Y-%m-%d")}"
+  s.summary           = %q{Ruby on Rails news engine for Refinery CMS.}
+  s.email             = %q{info@refinerycms.com}
+  s.homepage          = %q{http://refinerycms.com}
+  s.authors           = ["Philip Arndt", "Uģis Ozols"]
+  s.require_paths     = %w(lib)
+
+  s.files             = `git ls-files`.split("\n")
+  s.test_files        = `git ls-files -- spec/*`.split("\n")
+
+  s.add_dependency    'refinerycms-core',     '~> 2.0.1'
+  s.add_dependency    'refinerycms-settings', '~> 2.0.0'
+  s.add_dependency    'friendly_id',          '~> 4.0.1'
+end

data/spec/factories/news.rb

@@ -0,0 +1,7 @@
+FactoryGirl.define do
+  factory :news_item, :class => Refinery::News::Item do
+    title "Refinery CMS News Item"
+    content "Some random text ..."
+    publish_date Time.now - 5.minutes
+  end
+end

data/spec/models/refinery/news/item_spec.rb

@@ -0,0 +1,91 @@
+require 'spec_helper'
+
+module Refinery
+  module News
+    describe Item do
+
+      let(:time_now) { Time.now }
+      let(:news_item) { Factory(:news_item) }
+
+      describe "validations" do
+        subject do
+          news_item = Refinery::News::Item.create! :title => "Refinery CMS",
+                                                 :content => "Some random text ...",
+                                                 :publish_date => time_now
+          news_item
+        end
+
+        it { should be_valid }
+        its(:errors) { should be_empty }
+        its(:title) { should == "Refinery CMS" }
+        its(:content) { should == "Some random text ..." }
+        its(:publish_date) { should == time_now }
+      end
+
+      describe "attribute aliasing" do
+        subject { news_item }
+        its(:content) { should == news_item.body }
+      end
+
+      describe "default scope" do
+        it "orders by publish date in DESC order" do
+          news_item1 = Factory(:news_item, :publish_date => 1.hour.ago)
+          news_item2 = Factory(:news_item, :publish_date => 2.hours.ago)
+          news_items = Refinery::News::Item.all
+          news_items.first.should == news_item1
+          news_items.second.should == news_item2
+        end
+      end
+
+      describe ".not_expired" do
+        let!(:news_item) { Factory(:news_item) }
+
+        specify "expiration date not set" do
+          Refinery::News::Item.not_expired.count.should == 1
+        end
+
+        specify "expiration date set in future" do
+          news_item.expiration_date = Time.now + 1.hour
+          news_item.save!
+          Refinery::News::Item.not_expired.count.should == 1
+        end
+
+        specify "expiration date in past" do
+          news_item.expiration_date = Time.now - 1.hour
+          news_item.save!
+          Refinery::News::Item.not_expired.count.should == 0
+        end
+      end
+
+      describe ".published" do
+        it "returns only published news items" do
+          Factory(:news_item)
+          Factory(:news_item, :publish_date => Time.now + 1.hour)
+          Refinery::News::Item.published.count.should == 1
+        end
+      end
+
+      describe ".latest" do
+        it "returns 10 latest news items by default" do
+          5.times { Factory(:news_item) }
+          5.times { Factory(:news_item, :publish_date => Time.now + 1.hour) }
+          Refinery::News::Item.latest.count.should == 5
+          7.times { Factory(:news_item) }
+          Refinery::News::Item.latest.length.should == 10
+        end
+
+        it "returns latest n news items" do
+          4.times { Factory(:news_item) }
+          Refinery::News::Item.latest(3).length.should == 3
+        end
+      end
+
+      describe ".not_published?" do
+        it "returns not published news items" do
+          news_item = Factory(:news_item, :publish_date => Time.now + 1.hour)
+          news_item.not_published?.should be_true
+        end
+      end
+    end
+  end
+end

data/spec/requests/manage_news_items_spec.rb

@@ -0,0 +1,90 @@
+require "spec_helper"
+
+describe "manage news items" do
+  login_refinery_user
+
+  context "when no news items" do
+    it "invites to create one" do
+      visit refinery.news_admin_items_path
+      page.should have_content("There are no news items yet. Click \"Add News Item\" to add your first news item.")
+    end
+  end
+
+  describe "action links" do
+    it "shows add news item link" do
+      visit refinery.news_admin_items_path
+
+      within "#actions" do
+        page.should have_content("Add News Item")
+        page.should have_selector("a[href='/refinery/news/items/new']")
+      end
+    end
+  end
+
+  describe "new/create" do
+    it "allows to create news item" do
+      visit refinery.news_admin_items_path
+
+      click_link "Add News Item"
+
+      fill_in "Title", :with => "My first news item"
+      fill_in "Body", :with => "bla bla"
+      fill_in "Source", :with => "http://refinerycms.com"
+      click_button "Save"
+
+      page.should have_content("'My first news item' was successfully added.")
+      page.body.should =~ /Remove this news item forever/
+      page.body.should =~ /Edit this news item/
+      page.body.should =~ %r{/refinery/news/items/my-first-news-item/edit}
+      page.body.should =~ /View this news item live/
+      page.body.should =~ %r{/news/items/my-first-news-item}
+
+      Refinery::News::Item.count.should == 1
+    end
+  end
+
+  describe "edit/update" do
+    before(:each) { Factory(:news_item, :title => "Update me") }
+
+    it "updates news item" do
+      visit refinery.news_admin_items_path
+
+      page.should have_content("Update me")
+
+      click_link "Edit this news item"
+
+      fill_in "Title", :with => "Updated"
+      click_button "Save"
+
+      page.should have_content("'Updated' was successfully updated.")
+    end
+  end
+
+  describe "destroy" do
+    before(:each) { Factory(:news_item, :title => "Delete me") }
+
+    it "removes news item" do
+      visit refinery.news_admin_items_path
+
+      click_link "Remove this news item forever"
+
+      page.should have_content("'Delete me' was successfully removed.")
+
+      Refinery::News::Item.count.should == 0
+    end
+  end
+
+  context "duplicate news item titles" do
+    before(:each) { Factory(:news_item, :title => "I was here first") }
+
+    it "isn't a problem" do
+      visit refinery.new_news_admin_item_path
+
+      fill_in "Title", :with => "I was here first"
+      fill_in "Body", :with => "Doesn't matter"
+      click_button "Save"
+
+      Refinery::News::Item.count.should == 2
+    end
+  end
+end

data/spec/requests/news_archive.rb

@@ -0,0 +1,12 @@
+require "spec_helper"
+
+describe "manage news items" do
+  before do
+    date = "dec/2011"
+    @time = Time.parse(date)
+  end
+
+  it "should display proper date" do
+    @time.should == "dec/2011"
+  end
+end