crummy 1.7.0 → 1.7.1

data/example/config/initializers/wrap_parameters.rb

@@ -0,0 +1,14 @@
+# Be sure to restart your server when you modify this file.
+#
+# This file contains settings for ActionController::ParamsWrapper which
+# is enabled by default.
+
+# Enable parameter wrapping for JSON. You can disable this by setting :format to an empty array.
+ActiveSupport.on_load(:action_controller) do
+  wrap_parameters :format => [:json]
+end
+
+# Disable root element in JSON by default.
+ActiveSupport.on_load(:active_record) do
+  self.include_root_in_json = false
+end

data/example/config/locales/en.yml

@@ -0,0 +1,5 @@
+# Sample localization file for English. Add more files in this directory for other locales.
+# See https://github.com/svenfuchs/rails-i18n/tree/master/rails%2Flocale for starting points.
+
+en:
+  hello: "Hello world"

data/example/config/routes.rb

@@ -0,0 +1,5 @@
+CrummyTest::Application.routes.draw do
+  resources :posts
+  
+  root :to => "pages#index"
+end

data/example/db/migrate/20111104103150_create_posts.rb

@@ -0,0 +1,10 @@
+class CreatePosts < ActiveRecord::Migration
+  def change
+    create_table :posts do |t|
+      t.string :title
+      t.string :slug
+
+      t.timestamps
+    end
+  end
+end

data/example/db/migrate/20111104103738_create_categories.rb

@@ -0,0 +1,12 @@
+class CreateCategories < ActiveRecord::Migration
+  def change
+    create_table :categories do |t|
+      t.string :title
+      t.string :slug
+      
+      t.string :category_id # Can have a category as a parent
+      
+      t.timestamps
+    end
+  end
+end

data/example/db/migrate/20111104104040_add_category_id_to_posts.rb

@@ -0,0 +1,5 @@
+class AddCategoryIdToPosts < ActiveRecord::Migration
+  def change
+    add_column :posts, :category_id, :integer
+  end
+end

data/example/db/schema.rb

@@ -0,0 +1,32 @@
+# encoding: UTF-8
+# This file is auto-generated from the current state of the database. Instead
+# of editing this file, please use the migrations feature of Active Record to
+# incrementally modify your database, and then regenerate this schema definition.
+#
+# Note that this schema.rb definition is the authoritative source for your
+# database schema. If you need to create the application database on another
+# system, you should be using db:schema:load, not running all the migrations
+# from scratch. The latter is a flawed and unsustainable approach (the more migrations
+# you'll amass, the slower it'll run and the greater likelihood for issues).
+#
+# It's strongly recommended to check this file into your version control system.
+
+ActiveRecord::Schema.define(:version => 20111104104040) do
+
+  create_table "categories", :force => true do |t|
+    t.string   "title"
+    t.string   "slug"
+    t.string   "category_id"
+    t.datetime "created_at"
+    t.datetime "updated_at"
+  end
+
+  create_table "posts", :force => true do |t|
+    t.string   "title"
+    t.string   "slug"
+    t.datetime "created_at"
+    t.datetime "updated_at"
+    t.integer  "category_id"
+  end
+
+end

data/example/db/seeds.rb

@@ -0,0 +1,15 @@
+# This file should contain all the record creation needed to seed the database with its default values.
+# The data can then be loaded with the rake db:seed (or created alongside the db with db:setup).
+#
+# Examples:
+#
+#   cities = City.create([{ name: 'Chicago' }, { name: 'Copenhagen' }])
+#   Mayor.create(name: 'Emanuel', city: cities.first)
+
+movies    = Category.create :title => "Movies",           :slug => "movies"
+star_wars = Category.create :title => "Star Wars",        :slug => "star-wars", :category => movies
+time      = Category.create :title => "What time Is It?", :slug => "time"
+
+Post.create :title => "A Long Long Time Ago", :slug => "a-long-long-time-ago",  :category => star_wars
+Post.create :title => "It's Hammer Time",     :slug => "its-hammer-time",       :category => time
+Post.create :title => "Test",                 :slug => "test"

data/example/doc/README_FOR_APP

@@ -0,0 +1,2 @@
+Use this README file to introduce your application and point to useful places in the API for learning more.
+Run "rake doc:app" to generate API documentation for your models, controllers, helpers, and libraries.

data/example/public/404.html

@@ -0,0 +1,26 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>The page you were looking for doesn't exist (404)</title>
+  <style type="text/css">
+    body { background-color: #fff; color: #666; text-align: center; font-family: arial, sans-serif; }
+    div.dialog {
+      width: 25em;
+      padding: 0 4em;
+      margin: 4em auto 0 auto;
+      border: 1px solid #ccc;
+      border-right-color: #999;
+      border-bottom-color: #999;
+    }
+    h1 { font-size: 100%; color: #f00; line-height: 1.5em; }
+  </style>
+</head>
+
+<body>
+  <!-- This file lives in public/404.html -->
+  <div class="dialog">
+    <h1>The page you were looking for doesn't exist.</h1>
+    <p>You may have mistyped the address or the page may have moved.</p>
+  </div>
+</body>
+</html>

data/example/public/422.html

@@ -0,0 +1,26 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>The change you wanted was rejected (422)</title>
+  <style type="text/css">
+    body { background-color: #fff; color: #666; text-align: center; font-family: arial, sans-serif; }
+    div.dialog {
+      width: 25em;
+      padding: 0 4em;
+      margin: 4em auto 0 auto;
+      border: 1px solid #ccc;
+      border-right-color: #999;
+      border-bottom-color: #999;
+    }
+    h1 { font-size: 100%; color: #f00; line-height: 1.5em; }
+  </style>
+</head>
+
+<body>
+  <!-- This file lives in public/422.html -->
+  <div class="dialog">
+    <h1>The change you wanted was rejected.</h1>
+    <p>Maybe you tried to change something you didn't have access to.</p>
+  </div>
+</body>
+</html>

data/example/public/500.html

@@ -0,0 +1,26 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>We're sorry, but something went wrong (500)</title>
+  <style type="text/css">
+    body { background-color: #fff; color: #666; text-align: center; font-family: arial, sans-serif; }
+    div.dialog {
+      width: 25em;
+      padding: 0 4em;
+      margin: 4em auto 0 auto;
+      border: 1px solid #ccc;
+      border-right-color: #999;
+      border-bottom-color: #999;
+    }
+    h1 { font-size: 100%; color: #f00; line-height: 1.5em; }
+  </style>
+</head>
+
+<body>
+  <!-- This file lives in public/500.html -->
+  <div class="dialog">
+    <h1>We're sorry, but something went wrong.</h1>
+    <p>We've been notified about this issue and we'll take a look at it shortly.</p>
+  </div>
+</body>
+</html>

data/example/public/robots.txt

@@ -0,0 +1,5 @@
+# See http://www.robotstxt.org/wc/norobots.html for documentation on how to use the robots.txt file
+#
+# To ban all spiders from the entire site uncomment the next two lines:
+# User-Agent: *
+# Disallow: /

data/example/script/rails

@@ -0,0 +1,6 @@
+#!/usr/bin/env ruby
+# This command will automatically be run when you run "rails" with Rails 3 gems installed from the root of your application.
+
+APP_PATH = File.expand_path('../../config/application',  __FILE__)
+require File.expand_path('../../config/boot',  __FILE__)
+require 'rails/commands'

data/example/spec/controllers/posts_controller_spec.rb

@@ -0,0 +1,7 @@
+require "spec_helper"
+
+describe PostsController do
+  it "exists! (and RSpec works)" do
+    # nop
+  end
+end

data/example/spec/spec_helper.rb

@@ -0,0 +1,38 @@
+# This file is copied to spec/ when you run 'rails generate rspec:install'
+ENV["RAILS_ENV"] ||= 'test'
+require File.expand_path("../../config/environment", __FILE__)
+require 'rspec/rails'
+require 'rspec/autorun'
+
+# Requires supporting ruby files with custom matchers and macros, etc,
+# in spec/support/ and its subdirectories.
+Dir[Rails.root.join("spec/support/**/*.rb")].each {|f| require f}
+
+RSpec.configure do |config|
+  # ## Mock Framework
+  #
+  # If you prefer to use mocha, flexmock or RR, uncomment the appropriate line:
+  #
+  # config.mock_with :mocha
+  # config.mock_with :flexmock
+  # config.mock_with :rr
+
+  # Remove this line if you're not using ActiveRecord or ActiveRecord fixtures
+  config.fixture_path = "#{::Rails.root}/spec/fixtures"
+
+  # If you're not using ActiveRecord, or you'd prefer not to run each of your
+  # examples within a transaction, remove the following line or assign false
+  # instead of true.
+  config.use_transactional_fixtures = true
+
+  # If true, the base class of anonymous controllers will be inferred
+  # automatically. This will be the default behavior in future versions of
+  # rspec-rails.
+  config.infer_base_class_for_anonymous_controllers = false
+
+  # Run specs in random order to surface order dependencies. If you find an
+  # order dependency and want to debug it, you can fix the order by providing
+  # the seed, which is printed after each run.
+  #     --seed 1234
+  config.order = "random"
+end

data/gemfiles/rails3_2/Gemfile

@@ -0,0 +1,6 @@
+source "https://rubygems.org"
+
+gem 'activesupport', '>= 3.0', '< 4'
+gem 'actionpack', '>= 3.0', '< 4'
+
+gemspec path: File.join('..', '..')

data/gemfiles/rails4_0/Gemfile

@@ -0,0 +1,6 @@
+source "https://rubygems.org"
+
+gem 'activesupport', '>= 4.0.0.beta', '< 5'
+gem 'actionpack', '>= 4.0.0.beta', '< 5'
+
+gemspec path: File.join('..', '..')

data/lib/crummy.rb

@@ -20,8 +20,8 @@ module Crummy
     attr_accessor :ul_id
     attr_accessor :ul_class
     attr_accessor :li_class
-    attr_accessor :active_li_class
     attr_accessor :microdata
+    attr_accessor :last_crumb_linked
 
     def initialize
       @format = :html
@@ -35,8 +35,16 @@ module Crummy
       @ul_id = ''
       @ul_class = ''
       @li_class = ''
-      @active_li_class = ''
       @microdata = false
+      @last_crumb_linked = true
+    end
+
+    def active_li_class=(class_name)
+      puts "CRUMMY: The 'active_li_class' option is DEPRECATED and will be removed from a future version"
+    end
+
+    def active_li_class
+      puts "CRUMMY: The 'active_li_class' option is DEPRECATED and will be removed from a future version"
     end
   end
 

data/lib/crummy/standard_renderer.rb

@@ -22,7 +22,7 @@ module Crummy
     #   render_crumbs :format => :xml         #=> <crumb href="/">Home</crumb><crumb href="/businesses">Businesses</crumb>
     #   render_crumbs :format => :html_list   #=> <ul class="" id=""><li class=""><a href="/">Home</a></li><li class=""><a href="/">Businesses</a></li></ul>
     #   
-    # With :format => :html_list you can specify additional params: active_li_class, li_class, ul_class, ul_id
+    # With :format => :html_list you can specify additional params: li_class, ul_class, ul_id
     # The only argument is for the separator text. It does not assume you want spaces on either side so you must specify. Defaults to +&raquo;+
     #
     #   render_crumbs(" . ")  #=> <a href="/">Home</a> . <a href="/businesses">Businesses</a>
@@ -36,21 +36,21 @@ module Crummy
       options[:first_class] ||= Crummy.configuration.first_class
       options[:last_class] ||= Crummy.configuration.last_class
       options[:microdata] ||= Crummy.configuration.microdata
+      options[:last_crumb_linked] = Crummy.configuration.last_crumb_linked if options[:last_crumb_linked].nil?
 
       case options[:format]
       when :html
         crumb_string = crumbs.collect do |crumb|
-          crumb_to_html(crumb, options[:links], options[:first_class], options[:last_class], (crumb == crumbs.first), (crumb == crumbs.last), options[:microdata])
+          crumb_to_html(crumb, options[:links], options[:first_class], options[:last_class], (crumb == crumbs.first), (crumb == crumbs.last), options[:microdata], options[:last_crumb_linked])
         end.reduce { |memo, obj| memo << options[:separator] << obj }
         crumb_string
       when :html_list
         # Let's set values for special options of html_list format
-        options[:active_li_class] ||= Crummy.configuration.active_li_class
         options[:li_class] ||= Crummy.configuration.li_class
         options[:ul_class] ||= Crummy.configuration.ul_class
         options[:ul_id] ||= Crummy.configuration.ul_id
         crumb_string = crumbs.collect do |crumb|
-          crumb_to_html_list(crumb, options[:links], options[:li_class], options[:active_li_class], options[:first_class], options[:last_class], (crumb == crumbs.first), (crumb == crumbs.last), options[:microdata])
+          crumb_to_html_list(crumb, options[:links], options[:li_class], options[:first_class], options[:last_class], (crumb == crumbs.first), (crumb == crumbs.last), options[:microdata], options[:last_crumb_linked])
         end.reduce { |memo, obj| memo << options[:separator] << obj }
         crumb_string = content_tag(:ul, crumb_string, :class => options[:ul_class], :id => options[:ul_id])
         crumb_string
@@ -65,37 +65,38 @@ module Crummy
 
     private
 
-    def crumb_to_html(crumb, links, first_class, last_class, is_first, is_last, with_microdata)
+    def crumb_to_html(crumb, links, first_class, last_class, is_first, is_last, with_microdata, last_crumb_linked)
       html_classes = []
       html_classes << first_class if is_first
       html_classes << last_class if is_last
       name, url = crumb
-      html_content = url && links ? link_to(name, url) : content_tag(:span, name)
+      can_link = url && links && (!is_last || last_crumb_linked)
+      html_content = can_link ? link_to(name, url) : content_tag(:span, name)
       if with_microdata
         item_title = content_tag(:span, name, :itemprop => "title")
         html_options = {:itemscope => true, :itemtype => data_definition_url("Breadcrumb")}
-        html_content = url && links ? link_to(item_title, url, :class => html_classes, :itemprop => "url") : item_title
+        html_content = can_link ? link_to(item_title, url, :class => html_classes, :itemprop => "url") : item_title
         content_tag(:div, html_content, html_options)
       else
-        url && links ? link_to(name, url, :class => html_classes) : name
+        can_link ? link_to(name, url, :class => html_classes) : name
       end
     end
     
-    def crumb_to_html_list(crumb, links, li_class, active_li_class, first_class, last_class, is_first, is_last, with_microdata)
+    def crumb_to_html_list(crumb, links, li_class, first_class, last_class, is_first, is_last, with_microdata, last_crumb_linked)
       name, url = crumb
+      can_link = url && links && (!is_last || last_crumb_linked)
       html_classes = []
       html_classes << first_class if is_first
       html_classes << last_class if is_last
-      html_classes << active_li_class unless url && links
-      html_classes << li_class if !is_first && !is_last && url && links
+      html_classes << li_class
       html_options = {:class => html_classes.join(' ').strip}
       if with_microdata
         html_options[:itemscope] = true
         html_options[:itemtype]  = data_definition_url("Breadcrumb")
         item_title = content_tag(:span, name, :itemprop => "title")
-        html_content = url && links ? link_to(item_title, url, :itemprop => "url") : item_title
+        html_content = can_link ? link_to(item_title, url, :itemprop => "url") : item_title
       else
-        html_content = url && links ? link_to(name, url) : content_tag(:span, name)
+        html_content = can_link ? link_to(name, url) : content_tag(:span, name)
       end
       content_tag(:li, html_content, html_options)
     end

data/lib/crummy/version.rb

@@ -1,6 +1,6 @@
 module Crummy
   MAJOR = 1
   MINOR = 7
-  PATCH  = 0
+  PATCH  = 1
   VERSION = [MAJOR, MINOR, PATCH].join('.')
 end

data/test/standard_renderer_test.rb

@@ -22,9 +22,9 @@ class StandardRendererTest < Test::Unit::TestCase
 
     assert_equal('<a href="url1" class="first">name1</a> &raquo; <a href="url2" class="last">name2</a>',
                  renderer.render_crumbs([['name1', 'url1'], ['name2', 'url2']], :first_class => 'first', :last_class => 'last', :format => :html))
-    assert_equal('<ul class="" id=""><li class="first"><a href="url1">name1</a></li><li class="li_class"><a href="url2">name2</a></li><li class="last"><a href="url3">name3</a></li></ul>',
+    assert_equal('<ul class="" id=""><li class="first li_class"><a href="url1">name1</a></li><li class="li_class"><a href="url2">name2</a></li><li class="last li_class"><a href="url3">name3</a></li></ul>',
                  renderer.render_crumbs([['name1', 'url1'], ['name2', 'url2'], ['name3', 'url3']], :li_class => "li_class", :first_class => 'first', :last_class => 'last', :format => :html_list))
-    assert_equal('<ul class="" id=""><li class="first"><a href="url1">name1</a></li> / <li class="li_class"><a href="url2">name2</a></li> / <li class="last"><a href="url3">name3</a></li></ul>',
+    assert_equal('<ul class="" id=""><li class="first li_class"><a href="url1">name1</a></li> / <li class="li_class"><a href="url2">name2</a></li> / <li class="last li_class"><a href="url3">name3</a></li></ul>',
                  renderer.render_crumbs([['name1', 'url1'], ['name2', 'url2'], ['name3', 'url3']], :li_class => "li_class", :first_class => 'first', :last_class => 'last', :format => :html_list, :separator => " / "))
     assert_equal('<crumb href="url1">name1</crumb><crumb href="url2">name2</crumb>',
                  renderer.render_crumbs([['name1', 'url1'], ['name2', 'url2']], :first_class => 'first', :last_class => 'last', :format => :xml))
@@ -33,6 +33,34 @@ class StandardRendererTest < Test::Unit::TestCase
                  renderer.render_crumbs([['name', 'url']], :first_class => 'first', :last_class => 'last', :format => :html, :microdata => true))
     assert_equal('<ul class="" id=""><li class="first last" itemscope="itemscope" itemtype="http://data-vocabulary.org/Breadcrumb"><a href="url" itemprop="url"><span itemprop="title">name</span></a></li></ul>',
                  renderer.render_crumbs([['name', 'url']], :first_class => 'first', :last_class => 'last', :format => :html_list, :microdata => true))
+    assert_equal('<ul class="crumbclass" id="crumbid"><li class="liclass"><a href="url">name</a></li></ul>',
+                 renderer.render_crumbs([['name', 'url']], :format => :html_list, :ul_id => "crumbid", :ul_class => "crumbclass", :li_class => "liclass"))
+  end
+ 
+  def test_classes_last_crumb_not_linked
+    renderer = StandardRenderer.new
+    assert_equal('name',
+                 renderer.render_crumbs([['name', 'url']], :first_class => 'first', :last_class => 'last', :format => :html, :last_crumb_linked => false))  
+    assert_equal('<ul class="" id=""><li class="first last"><span>name</span></li></ul>',
+                 renderer.render_crumbs([['name', 'url']], :first_class => 'first', :last_class => 'last', :format => :html_list, :last_crumb_linked => false))
+    assert_equal('<crumb href="url">name</crumb>',
+                 renderer.render_crumbs([['name', 'url']], :first_class => 'first', :last_class => 'last', :format => :xml, :last_crumb_linked => false))
+
+    assert_equal('<a href="url1" class="first">name1</a> &raquo; name2',
+                 renderer.render_crumbs([['name1', 'url1'], ['name2', 'url2']], :first_class => 'first', :last_class => 'last', :format => :html, :last_crumb_linked => false))
+    assert_equal('<ul class="" id=""><li class="first li_class"><a href="url1">name1</a></li><li class="li_class"><a href="url2">name2</a></li><li class="last li_class"><span>name3</span></li></ul>',
+                 renderer.render_crumbs([['name1', 'url1'], ['name2', 'url2'], ['name3', 'url3']], :li_class => "li_class", :first_class => 'first', :last_class => 'last', :format => :html_list, :last_crumb_linked => false))
+    assert_equal('<ul class="" id=""><li class="first li_class"><a href="url1">name1</a></li> / <li class="li_class"><a href="url2">name2</a></li> / <li class="last li_class"><span>name3</span></li></ul>',
+                 renderer.render_crumbs([['name1', 'url1'], ['name2', 'url2'], ['name3', 'url3']], :li_class => "li_class", :first_class => 'first', :last_class => 'last', :format => :html_list, :separator => " / ", :last_crumb_linked => false))
+    assert_equal('<crumb href="url1">name1</crumb><crumb href="url2">name2</crumb>',
+                 renderer.render_crumbs([['name1', 'url1'], ['name2', 'url2']], :first_class => 'first', :last_class => 'last', :format => :xml, :last_crumb_linked => false))
+
+    assert_equal('<div itemscope="itemscope" itemtype="http://data-vocabulary.org/Breadcrumb"><span itemprop="title">name</span></div>',
+                 renderer.render_crumbs([['name', 'url']], :first_class => 'first', :last_class => 'last', :format => :html, :microdata => true, :last_crumb_linked => false))
+    assert_equal('<ul class="" id=""><li class="first last" itemscope="itemscope" itemtype="http://data-vocabulary.org/Breadcrumb"><span itemprop="title">name</span></li></ul>',
+                 renderer.render_crumbs([['name', 'url']], :first_class => 'first', :last_class => 'last', :format => :html_list, :microdata => true, :last_crumb_linked => false))
+    assert_equal('<ul class="crumbclass" id="crumbid"><li class="liclass"><span>name</span></li></ul>',
+                 renderer.render_crumbs([['name', 'url']], :format => :html_list, :ul_id => "crumbid", :ul_class => "crumbclass", :li_class => "liclass", :last_crumb_linked => false))
   end
 
   def test_configuration
@@ -67,5 +95,7 @@ class StandardRendererTest < Test::Unit::TestCase
     # using configured microdata setting
     assert_equal('<div itemscope="itemscope" itemtype="http://data-vocabulary.org/Breadcrumb"><a href="url" class="first last" itemprop="url"><span itemprop="title">name</span></a></div>',
                  renderer.render_crumbs([['name', 'url']], :first_class => 'first', :last_class => 'last', :format => :html))
-  end
+    # last crumb not linked
+    assert_equal('<div itemscope="itemscope" itemtype="http://data-vocabulary.org/Breadcrumb"><span itemprop="title">name</span></div>',
+                 renderer.render_crumbs([['name', 'url']], :first_class => 'first', :last_class => 'last', :format => :html, :last_crumb_linked => false))  end
 end