radiant-page_attachments-extension 1.0.0

data/app/models/page_attachments_interface.rb

@@ -0,0 +1,19 @@
+module PageAttachmentsInterface
+  def self.included(base)
+    base.class_eval {
+      before_filter :add_page_attachment_partials,
+                    :only => [:edit, :new]
+      include InstanceMethods
+    }
+  end
+
+  module InstanceMethods
+    def add_page_attachment_partials
+      @buttons_partials ||= []
+      @buttons_partials << "attachments_box"
+      include_javascript 'admin/dragdrop'
+      include_javascript 'admin/page_attachments'
+      include_stylesheet 'admin/page_attachments'
+    end
+  end
+end

data/app/views/admin/pages/_attachment.html.haml

@@ -0,0 +1,12 @@
+%li{:id => "attachment_#{attachment.id}", :class =>"attachment clearfix"}
+  = hidden_field_tag "page[attachments_attributes][#{attachment_counter}][id]", attachment.id
+  = hidden_field_tag "page[attachments_attributes][#{attachment_counter}][_delete]", "0"
+  = hidden_field_tag "page[attachments_attributes][#{attachment_counter}][position]", attachment.position
+  - unless attachment.first? && attachment.last?
+    = image_tag 'admin/drag_order.png', :alt => "Drag handle", :title => "Drag to change order", :class => 'drag_order'
+  %div= image_tag "admin/minus.png", :alt => "delete", :class => 'delete'
+  - unless attachment.thumbnails.empty?
+    = link_to image_tag(attachment.public_filename("icon")),  attachment.public_filename, :class => 'thumbnail'
+  - unless attachment.title.blank?
+    %p= h attachment.short_title
+  = link_to attachment.short_filename, attachment.public_filename

data/app/views/admin/pages/_attachments_box.html.haml

@@ -0,0 +1,12 @@
+#attachments
+  .toolbar
+    = image_tag('admin/plus.png', :alt => 'add')
+    = hidden_field_tag('attachment_index', @page.attachments.size, :id => 'attachment_index_field')
+  %h3
+    Attachments (
+    %span#attachment_count>= @page.attachments.count
+    )
+  - unless @page.attachments.count.zero?
+    %ol#attachment_list
+      = render :partial => 'attachment', :collection => @page.attachments
+    %div{:style => "clear:both;"}

data/db/migrate/001_create_page_attachments_extension_schema.rb

@@ -0,0 +1,22 @@
+class CreatePageAttachmentsExtensionSchema < ActiveRecord::Migration
+  def self.up
+    create_table "page_attachments" do |t|
+      t.column "content_type", :string
+      t.column "filename",     :string
+      t.column "size",         :integer
+      t.column "parent_id",    :integer
+      t.column "thumbnail",    :string
+      t.column "width",        :integer
+      t.column "height",       :integer
+      t.column "created_at",   :datetime
+      t.column "created_by",   :integer
+      t.column "updated_at",   :datetime
+      t.column "updated_by",   :integer
+      t.column "page_id",      :integer
+    end
+  end
+
+  def self.down
+    drop_table "page_attachments"
+  end
+end

data/db/migrate/002_add_page_attachments_fields.rb

@@ -0,0 +1,23 @@
+class AddPageAttachmentsFields < ActiveRecord::Migration
+  def self.up
+    add_column :page_attachments,	:title,				:string
+    add_column :page_attachments,	:description,		:string
+    add_column :page_attachments,	:position,			:integer
+    pages_with_attachments = PageAttachment.find_by_sql('select page_id from page_attachments where page_id is not null group by page_id')
+    pages_with_attachments.each do |pa|
+      attaches = PageAttachment.find(:all, :conditions => ['page_id = ?',pa.page_id], :order => :id)
+      i=1
+      attaches.each do |attach|
+        attach.position = i
+        attach.save
+        i = i + 1
+      end
+    end
+  end
+
+  def self.down
+    remove_column :page_attachments, :title
+    remove_column :page_attachments, :description
+    remove_column :page_attachments, :position
+  end
+end

data/features/attachments.feature

@@ -0,0 +1,42 @@
+Feature: attachments
+  As a content editor
+  I should be able to add, reorder, and delete attachments
+  
+  Background:
+    Given I am logged in as admin
+  
+  Scenario: attach a file
+    Given I have a page with no attachments
+    When I edit the page
+    And I click the plus icon
+    And I attach the Rails logo
+    And I save
+    Then the page should have a new attachment
+  
+  Scenario: don't change attachments
+    Given I have a page with 2 attachments
+    When I edit the page
+    And I save
+    Then the page should have 2 attachments
+  
+  Scenario: attach another file
+    Given I have a page with 2 attachments
+    When I edit the page
+    And I click the plus icon
+    And I attach the Rails logo
+    And I save
+    Then the page should have 3 attachments
+  
+  Scenario: delete a file
+    Given I have a page with 2 attachments
+    When I edit the page
+    And I delete the first attachment
+    And I save
+    Then the page should have 1 attachment
+  
+  Scenario: reorder attachments
+    Given I have a page with 2 attachments
+    When I edit the page
+    And I drag attachment 2 above attachment 1
+    And I save
+    Then attachment 2 should be in position 1

data/features/step_definitions/attachment_steps.rb

@@ -0,0 +1,61 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "support", "paths"))
+
+Given /^I am logged in as (.*)(?: user)?$/ do |username|
+  visit login_path
+  fill_in "Username", :with => username
+  fill_in "Password",  :with => "password"
+  click_button "Login"
+end
+
+Given /^I have a page$/ do
+  @page = pages(:home)
+end
+
+Given /^I have a page with (\d+) attachments$/ do |number|
+  Given "I have a page"
+  @page.attachments.length.should == number.to_i
+end
+
+Given /^I have a page with no attachments$/ do
+  @page = pages(:first)
+  @page.attachments.should be_empty
+end
+
+When /^I edit the page$/ do
+  visit edit_admin_page_path(@page)
+end
+
+When /^I click the plus icon$/ do
+  selenium.click "xpath=id('attachments')//img[@alt='Add']"
+end
+
+When /^I attach the Rails logo$/ do
+  attach_file "file_input", "#{PageAttachmentsExtension.root}/spec/fixtures/rails.png"
+end
+
+When /^I delete the first attachment$/ do
+  selenium.click "xpath=id('attachment_list')/li[1]//img[@alt='Delete']"
+end
+
+When /^I drag attachment 2 above attachment 1$/ do
+  @attachment = @page.attachments[1]
+  @attachment.position.should == 2
+  selenium.dragdrop("id=attachment_#{@attachment.id}", "0, -200")
+end
+
+When /^I save$/ do
+  click_button "Save"
+  response.should_not contain("errors")
+end
+
+Then /^the page should have a new attachment$/ do
+  @page.reload.attachments.should_not be_empty
+end
+
+Then /^the page should have (\d+) attachment(?:s)?$/ do |number|
+  @page.reload.attachments.length.should == number.to_i
+end
+
+Then /^attachment 2 should be in position 1$/ do
+  @attachment.reload.position.should == 1
+end

data/features/support/env.rb

@@ -0,0 +1,57 @@
+# Sets up the Rails environment for Cucumber
+ENV["RAILS_ENV"] = "test"
+require File.expand_path(File.dirname(__FILE__) + '/../../../../../config/environment')
+
+require 'cucumber/rails/world'
+require 'cucumber/formatter/unicode' # Comment out this line if you don't want Cucumber Unicode support
+require 'webrat'
+raise "Update the version of Webrat you specify in test.rb." if Webrat.const_defined?(:VERSION) && Webrat::VERSION < '0.5.1' # Webrat 0.4.4 won't work with selenium-client 1.2.16
+
+Webrat.configure do |config|
+  config.mode = :selenium
+  config.selenium_browser_key = "*chrome"
+end
+
+# this is necessary to have webrat "wait_for" the response body to be available
+# when writing steps that match against the response body returned by selenium
+World(Webrat::Selenium::Matchers)
+
+# Patch in attach_file functionality
+module Webrat
+  class SeleniumSession
+    def attach_file(field_locator, path, content_type = nil)
+      fill_in(field_locator, :with => path)
+    end
+  end
+end
+
+require 'cucumber/rails/rspec'
+require 'dataset'
+
+module Dataset
+  module Extensions # :nodoc:
+    # Selenium can't handle transactions, so we have to disable them in Dataset
+    module CucumberWorldNoTransactions # :nodoc:
+      def dataset(*datasets, &block)
+        add_dataset(*datasets, &block)
+
+        load = nil
+        $__cucumber_toplevel.Before do
+          Dataset::Database::Base.new.clear # Empty all tables
+          dataset_session.instance_variable_set("@load_stack", []) # notify load_stack that nothing's loaded
+          load = dataset_session.load_datasets_for(self.class)
+          extend_from_dataset_load(load)
+        end
+      end
+    end
+  end
+end
+ 
+Cucumber::Rails::World.class_eval do
+  include Dataset
+  self.extend Dataset::Extensions::CucumberWorldNoTransactions
+  datasets_directory "#{RADIANT_ROOT}/spec/datasets"
+  Dataset::Resolver.default << (File.dirname(__FILE__) + "/../../spec/datasets")
+  self.datasets_database_dump_path = "#{Rails.root}/tmp/dataset"
+  dataset :pages_with_layouts, :users, :page_attachments
+end

data/features/support/paths.rb

@@ -0,0 +1,14 @@
+def path_to(page_name)
+  case page_name
+  
+  when /the homepage/i
+    root_path
+  
+  when /edit the page/i
+    edit_admin_page_path(@page)
+  # Add more page name => path mappings here
+  
+  else
+    raise "Can't find mapping from \"#{page_name}\" to a path."
+  end
+end

data/lib/difference_test_helper.rb

@@ -0,0 +1,20 @@
+module DifferenceTestHelper
+  # http://project.ioni.st/post/217#post-217
+  #
+  #  def test_new_publication
+  #    assert_difference(Publication, :count) do
+  #      post :create, :publication => {...}
+  #      # ...
+  #    end
+  #  end
+  #
+  def assert_difference(object, method = nil, difference = 1)
+    initial_value = object.send(method)
+    yield
+    assert_equal initial_value + difference, object.send(method), "#{object}##{method}"
+  end
+
+  def assert_no_difference(object, method, &block)
+    assert_difference object, method, 0, &block
+  end
+end

data/lib/tasks/page_attachments_extension_tasks.rake

@@ -0,0 +1,28 @@
+namespace :radiant do
+  namespace :extensions do
+    namespace :page_attachments do
+
+      desc "Runs the migration of the Page Attachments extension"
+      task :migrate => :environment do
+        require 'radiant/extension_migrator'
+        if ENV["VERSION"]
+          PageAttachmentsExtension.migrator.migrate(ENV["VERSION"].to_i)
+        else
+          PageAttachmentsExtension.migrator.migrate
+        end
+      end
+
+      desc "Copies public assets of the Page Attachments to the instance public/ directory."
+      task :update => :environment do
+        is_svn_or_dir = proc {|path| path =~ /\.svn/ || File.directory?(path) }
+        puts "Copying assets from PageAttachmentsExtension"
+        Dir[PageAttachmentsExtension.root + "/public/**/*"].reject(&is_svn_or_dir).each do |file|
+          path = file.sub(PageAttachmentsExtension.root, '')
+          directory = File.dirname(path)
+          mkdir_p RAILS_ROOT + directory
+          cp file, RAILS_ROOT + path
+        end
+      end
+    end
+  end
+end

data/page_attachments_extension.rb

@@ -0,0 +1,27 @@
+require_dependency 'application_controller'
+# require File.dirname(__FILE__) + '/lib/geometry'
+# require 'tempfile'
+
+class PageAttachmentsExtension < Radiant::Extension
+  version "1.0"
+  description "Adds page-attachment-style asset management."
+  url "http://radiantcms.org"
+
+   define_routes do |map|
+     map.connect 'page_attachments/:action/:id', :controller => 'page_attachments'
+   end
+
+  def activate
+    # Regular page attachments stuff
+    Page.class_eval {
+      include PageAttachmentAssociations
+      include PageAttachmentTags
+    }
+    UserActionObserver.send :include, ObservePageAttachments
+    Admin::PagesController.send :include, PageAttachmentsInterface
+  end
+
+  def deactivate
+  end
+
+end

data/public/javascripts/admin/page_attachments.js

@@ -0,0 +1,30 @@
+Event.addBehavior({
+
+  '#attachment_list': function() {
+    Sortable.create('attachment_list', {
+      onUpdate: function(container) {
+        container.select(".attachment").each(function(e, i) {
+          e.down('input[name*="position"]').setValue(i+1);
+        });
+      }
+    });
+  },
+  
+  '#attachments:click': function(event) {
+    var target = $(event.target);
+    if (target.match('img[alt=add]')) {
+      var upload = '<div class="attachment_upload"><p class="title">Upload file</p><table><tr><th><label for="title_input">Title:</label></th><td><input id="title_input" size="60" name="page[attachments_attributes][][title]"></td></tr><tr><th><label for="description_input">Description:</label></th><td><input id="description_input" type="text" size="60"  name="page[attachments_attributes][][description]"></td></tr><tr><th><label for="file_input">File:</label></th><td><input id="file_input" type="file" size="60" name="page[attachments_attributes][][uploaded_data]" /><img src="/images/admin/minus.png" alt="cancel" /></td></tr></table></div>';
+      $('attachment_index_field').value = parseInt($('attachment_index_field').value) + 1;
+      $('attachments').insert(upload.gsub(/\[\]/, '[' + $('attachment_index_field').value + ']'));
+    } else if (target.match('img[alt=cancel]')) {
+      event.findElement('.attachment_upload').remove();
+      event.stop();
+    } else if (target.match('img[alt=delete]')) {
+      var attachment = event.findElement('.attachment');
+      attachment.addClassName('deleted');
+      attachment.insert("<em>Attachment will be deleted when page is saved.</em>");
+      attachment.down('input[name*="_delete"]').setValue('true');
+    }
+  }
+
+});

data/public/stylesheets/admin/page_attachments.css

@@ -0,0 +1,133 @@
+#attachments {
+  clear: both;
+  color: #333;
+  background: #f5f1e2 url(/images/admin/vertical_tan_gradient.png) repeat-x top;
+  margin: 10px 0;
+  border-radius: 8px;
+  -moz-border-radius: 8px;
+  -webkit-border-radius: 8px;
+  overflow: hidden;
+}
+#attachments .toolbar {
+  float: right;
+  padding: 6px 6px 0;
+  width: 50px;
+  text-align: right;
+}
+#attachments .toolbar img {
+  padding: 5px;
+  cursor: pointer;
+}
+#attachments h3 {
+  font-size: 16px;
+  padding: 7px 0;
+}
+#attachments h3, #attachments p.notice {
+  background: #eae3c5;
+  margin: 0;
+  padding: 0.5em;
+}
+#attachments .attachment_upload {
+  padding: 0 0.8em 0.8em 0.8em;
+}
+#attachments p.notice {
+  padding: .5em .8em;
+  font-style: italic;
+  font-size: 90%;
+  color: red;
+}
+#attachments .attachment_upload img,
+#attachments .attachment img.delete {
+  cursor: pointer;
+  padding: 2px;
+  margin-left:7px;
+}
+#attachments .attachment img.drag_order {
+  cursor: move;
+  float: right;
+  padding: 2px;
+  margin-right:6px;
+}
+#attachments ol {
+  list-style: none;
+  padding: 0 10px 0 10px;
+  margin: 0;
+}
+#attachments ol li {
+  margin-left: 0;
+}
+#attachments ol li div {
+  float: left;
+  width: 25px;
+}
+#attachments ol li a.thumbnail {
+  float: left;
+  margin: 0 10px 0 10px;
+}
+#attachments ol li p {
+  margin: 0 0 5px 0;
+}
+#attachments ol li {
+  border-bottom: 1px dashed #cccccc;
+  width: 100%;
+  padding: 8px 0;
+}
+#attachments ol li:last-child {
+  border-bottom: none;
+}
+#attachments .attachment .thumbnail img {
+  background: white;
+  border: 1px solid #d4cba4;
+  padding: 3px;
+  border-radius: 3px;
+  -moz-border-radius: 3px;
+  -webkit-border-radius: 3px;
+}
+#attachments p.title {
+  border-bottom: 1px #eae3c5 solid;
+  width: 66%;
+  font-size: 17px;
+  font-weight: bold;
+  margin: 1em 0 ;
+}
+#attachments table {
+  margin-top: 1em;
+}
+#attachments table th {
+  font-weight: bold;
+  text-align: right;
+  padding-right: 10px;
+  width: 120px;
+  padding-bottom: 5px;
+}
+#attachments table td {
+ padding-bottom: 5px;
+}
+#attachments li.attachment.deleted em {
+  color: #ff0000;
+  margin: 0 0 10px 20px;
+}
+#attachments li.attachment.deleted a {
+  opacity: 0.5;
+  text-decoration: line-through;
+}
+#attachments li.attachment.deleted img.delete {
+  visibility: hidden;
+}
+
+.clearfix:after {
+  content: ".";
+  display: block;
+  height: 0;
+  clear: both;
+  visibility: hidden;
+}
+.clearfix {
+  display:inline-block;
+}
+.clearfix {
+  display:block;
+}
+li.clearfix {
+  display: list-item;
+}