best_in_placeish 0.2.4 → 0.2.5

data/best_in_placeish.gemspec

@@ -1,27 +1,27 @@
-# -*- encoding: utf-8 -*-
-$:.push File.expand_path("../lib", __FILE__)
-require "best_in_placeish/version"
-
-Gem::Specification.new do |s|
-  s.name        = "best_in_placeish"
-  s.version     = BestInPlaceish::VERSION
-  s.platform    = Gem::Platform::RUBY
-  s.authors     = ["Bernat Farrero"]
-  s.email       = ["bernat@itnig.net"]
-  s.homepage    = "http://github.com/MilesRoberts/best_in_placeish"
-  s.summary     = %q{It makes any field in place editable by clicking on it, it works for inputs, textareas, select dropdowns and checkboxes}
-  s.description = %q{Forked from the rails-3.0 branch of best_in_place, this includes a couple of key enhancements for Rails 3.0.x users}
-  
-  s.rubyforge_project = 'bestinplaceish'
-
-  s.files         = `git ls-files`.split("\n")
-  s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
-  s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
-  s.require_paths = ["lib"]
-
-  s.add_dependency "rails", "~> 3.0.0"
-
-  s.add_development_dependency "rspec-rails", "~> 2.7.0"
-  s.add_development_dependency "nokogiri", ">= 1.5.0"
-  s.add_development_dependency "capybara", ">= 1.0.1"
-end
+# -*- encoding: utf-8 -*-
+$:.push File.expand_path("../lib", __FILE__)
+require "best_in_placeish/version"
+
+Gem::Specification.new do |s|
+  s.name        = "best_in_placeish"
+  s.version     = BestInPlaceish::VERSION
+  s.platform    = Gem::Platform::RUBY
+  s.authors     = ["Bernat Farrero"]
+  s.email       = ["bernat@itnig.net"]
+  s.homepage    = "http://github.com/MilesRoberts/best_in_placeish"
+  s.summary     = %q{It makes any field in place editable by clicking on it, it works for inputs, textareas, select dropdowns and checkboxes}
+  s.description = %q{Forked from the rails-3.0 branch of best_in_place, this includes a couple of key enhancements for Rails 3.0.x users}
+  
+  s.rubyforge_project = 'bestinplaceish'
+
+  s.files         = `git ls-files`.split("\n")
+  s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
+  s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
+  s.require_paths = ["lib"]
+
+  s.add_dependency "rails", "~> 3.0.0"
+
+  s.add_development_dependency "rspec-rails", "~> 2.7.0"
+  s.add_development_dependency "nokogiri", ">= 1.5.0"
+  s.add_development_dependency "capybara", ">= 1.0.1"
+end

data/lib/best_in_placeish.rb

@@ -1,8 +1,8 @@
-require "best_in_placeish/utils"
-require "best_in_placeish/helper"
-
-module BestInPlaceish
-  autoload :TestHelpers, "best_in_placeish/test_helpers"
-end
-
-ActionView::Base.send(:include, BestInPlaceish::BestInPlaceishHelpers)
+require "best_in_placeish/utils"
+require "best_in_placeish/helper"
+
+module BestInPlaceish
+  autoload :TestHelpers, "best_in_placeish/test_helpers"
+end
+
+ActionView::Base.send(:include, BestInPlaceish::BestInPlaceishHelpers)

data/lib/best_in_placeish/helper.rb

@@ -1,57 +1,57 @@
-module BestInPlaceish
-  module BestInPlaceishHelpers
-
-    def best_in_placeish(object, field, opts = {})
-      opts[:type] ||= :input
-      opts[:collection] ||= []
-      field = field.to_s
-      if opts[:display_as]
-        value = object.send(opts[:display_as])
-      else
-        value = object.send(field).blank? ? "" : object.send(field)
-      end
-      collection = nil
-      if opts[:type] == :select && !opts[:collection].blank?
-        v = object.send(field)
-        value = Hash[opts[:collection]][!!(v =~ /^[0-9]+$/) ? v.to_i : v]
-        collection = opts[:collection].to_json
-      end
-      if opts[:type] == :checkbox
-        fieldValue = !!object.send(field)
-        if opts[:collection].blank? || opts[:collection].size != 2
-          opts[:collection] = ["No", "Yes"]
-        end
-        value = fieldValue ? opts[:collection][1] : opts[:collection][0]
-        collection = opts[:collection].to_json
-      end
-      out = "<span class='best_in_placeish'"
-      out << " id='#{BestInPlaceish::Utils.build_best_in_placeish_id(object, field)}'"
-      out << " data-url='#{opts[:path].blank? ? url_for(object).to_s : url_for(opts[:path])}'"
-      out << " data-object='#{object.class.to_s.gsub("::", "_").underscore}'"
-      out << " data-collection='#{collection}'" unless collection.blank?
-      out << " data-attribute='#{field}'"
-      out << " data-activator='#{opts[:activator]}'" unless opts[:activator].blank?
-      out << " data-nil='#{opts[:nil].to_s}'" unless opts[:nil].blank?
-      out << " data-type='#{opts[:type].to_s}'"
-      out << " data-inner-class='#{opts[:inner_class].to_s}'" if opts[:inner_class]
-      out << " data-html-attrs='#{opts[:html_attrs].to_json}'" unless opts[:html_attrs].blank?
-      if !opts[:sanitize].nil? && !opts[:sanitize]
-        out << " data-sanitize='false'>"
-        out << sanitize(value.to_s, :tags => %w(b i u s a strong em p h1 h2 h3 h4 h5 ul li ol hr pre span img br), :attributes => %w(id class href))
-      else
-        out << ">#{sanitize(value.to_s, :tags => nil, :attributes => nil)}"
-      end
-      out << "</span>"
-      raw out
-    end
-
-    def best_in_placeish_if(condition, object, field, opts={})
-      if condition
-        best_in_placeish(object, field, opts)
-      else
-        object.send field
-      end
-    end
-  end
-end
-
+module BestInPlaceish
+  module BestInPlaceishHelpers
+
+    def best_in_placeish(object, field, opts = {})
+      opts[:type] ||= :input
+      opts[:collection] ||= []
+      field = field.to_s
+      if opts[:display_as]
+        value = object.send(opts[:display_as])
+      else
+        value = object.send(field).blank? ? "" : object.send(field)
+      end
+      collection = nil
+      if opts[:type] == :select && !opts[:collection].blank?
+        v = object.send(field)
+        value = Hash[opts[:collection]][!!(v =~ /^[0-9]+$/) ? v.to_i : v]
+        collection = opts[:collection].to_json
+      end
+      if opts[:type] == :checkbox
+        fieldValue = !!object.send(field)
+        if opts[:collection].blank? || opts[:collection].size != 2
+          opts[:collection] = ["No", "Yes"]
+        end
+        value = fieldValue ? opts[:collection][1] : opts[:collection][0]
+        collection = opts[:collection].to_json
+      end
+      out = "<span class='best_in_placeish'"
+      out << " id='#{BestInPlaceish::Utils.build_best_in_placeish_id(object, field)}'"
+      out << " data-url='#{opts[:path].blank? ? url_for(object).to_s : url_for(opts[:path])}'"
+      out << " data-object='#{object.class.to_s.gsub("::", "_").underscore}'"
+      out << " data-collection='#{collection}'" unless collection.blank?
+      out << " data-attribute='#{field}'"
+      out << " data-activator='#{opts[:activator]}'" unless opts[:activator].blank?
+      out << " data-nil='#{opts[:nil].to_s}'" unless opts[:nil].blank?
+      out << " data-type='#{opts[:type].to_s}'"
+      out << " data-inner-class='#{opts[:inner_class].to_s}'" if opts[:inner_class]
+      out << " data-html-attrs='#{opts[:html_attrs].to_json}'" unless opts[:html_attrs].blank?
+      if !opts[:sanitize].nil? && !opts[:sanitize]
+        out << " data-sanitize='false'>"
+        out << sanitize(value.to_s, :tags => %w(b i u s a strong em p h1 h2 h3 h4 h5 ul li ol hr pre span img br), :attributes => %w(id class href))
+      else
+        out << ">#{sanitize(value.to_s, :tags => nil, :attributes => nil)}"
+      end
+      out << "</span>"
+      raw out
+    end
+
+    def best_in_placeish_if(condition, object, field, opts={})
+      if condition
+        best_in_placeish(object, field, opts)
+      else
+        object.send field
+      end
+    end
+  end
+end
+

data/lib/best_in_placeish/test_helpers.rb

@@ -1,39 +1,39 @@
-module BestInPlaceishish
-  module TestHelpers
-
-    def bip_area(model, attr, new_value)
-      id = BestInPlaceishish::Utils.build_best_in_placeish_id model, attr
-      page.execute_script <<-JS
-        $("##{id}").click();
-        $("##{id} form textarea").val('#{new_value}');
-        $("##{id} form textarea").blur();
-      JS
-    end
-
-    def bip_text(model, attr, new_value)
-      id = BestInPlaceish::Utils.build_best_in_placeish_id model, attr
-      page.execute_script <<-JS
-        $("##{id}").click();
-        $("##{id} input[name='#{attr}']").val('#{new_value}');
-        $("##{id} form").submit();
-      JS
-    end
-
-    def bip_bool(model, attr)
-      id = BestInPlaceish::Utils.build_best_in_placeish_id model, attr
-      page.execute_script("$('##{id}').click();")
-    end
-
-    def bip_select(model, attr, name)
-      id = BestInPlaceish::Utils.build_best_in_placeish_id model, attr
-      page.execute_script <<-JS
-        (function() {
-          $("##{id}").click();
-          var opt_value = $("##{id} select option:contains('#{name}')").attr('value');
-          $("##{id} select option[value='" + opt_value + "']").attr('selected', true);
-          $("##{id} select").change();
-        })();
-      JS
-    end
-  end
-end
+module BestInPlaceishish
+  module TestHelpers
+
+    def bip_area(model, attr, new_value)
+      id = BestInPlaceishish::Utils.build_best_in_placeish_id model, attr
+      page.execute_script <<-JS
+        $("##{id}").click();
+        $("##{id} form textarea").val('#{new_value}');
+        $("##{id} form textarea").blur();
+      JS
+    end
+
+    def bip_text(model, attr, new_value)
+      id = BestInPlaceish::Utils.build_best_in_placeish_id model, attr
+      page.execute_script <<-JS
+        $("##{id}").click();
+        $("##{id} input[name='#{attr}']").val('#{new_value}');
+        $("##{id} form").submit();
+      JS
+    end
+
+    def bip_bool(model, attr)
+      id = BestInPlaceish::Utils.build_best_in_placeish_id model, attr
+      page.execute_script("$('##{id}').click();")
+    end
+
+    def bip_select(model, attr, name)
+      id = BestInPlaceish::Utils.build_best_in_placeish_id model, attr
+      page.execute_script <<-JS
+        (function() {
+          $("##{id}").click();
+          var opt_value = $("##{id} select option:contains('#{name}')").attr('value');
+          $("##{id} select option[value='" + opt_value + "']").attr('selected', true);
+          $("##{id} select").change();
+        })();
+      JS
+    end
+  end
+end

data/lib/best_in_placeish/utils.rb

@@ -1,15 +1,15 @@
-module BestInPlaceish
-  class Utils
-
-    def self.build_best_in_placeish_id(object, field)
-      if object.is_a?(Symbol) || object.is_a?(String)
-        return "best_in_placeish_#{object}_#{field}"
-      end
-
-      id = "best_in_placeish_#{object.class.to_s.demodulize.underscore}"
-      id << "_#{object.id}" if object.class.ancestors.include?(ActiveRecord::Base)
-      id << "_#{field}"
-      id
-    end
-  end
-end
+module BestInPlaceish
+  class Utils
+
+    def self.build_best_in_placeish_id(object, field)
+      if object.is_a?(Symbol) || object.is_a?(String)
+        return "best_in_placeish_#{object}_#{field}"
+      end
+
+      id = "best_in_placeish_#{object.class.to_s.demodulize.underscore}"
+      id << "_#{object.id}" if object.class.ancestors.include?(ActiveRecord::Base)
+      id << "_#{field}"
+      id
+    end
+  end
+end

data/lib/best_in_placeish/version.rb

@@ -1,3 +1,3 @@
-module BestInPlaceish
-  VERSION = "0.2.4"
-end
+module BestInPlaceish
+  VERSION = "0.2.5"
+end

data/lib/generators/best_in_placeish/setup_generator.rb

@@ -1,11 +1,11 @@
-module BestInPlaceish
-
-  class SetupGenerator < Rails::Generators::Base
-    source_root File.expand_path("../../../../public/javascripts", __FILE__)
-    desc "Copies best_in_placeish.js to the /public/javascripts folder of your app."
-
-    def copy_js
-      copy_file "best_in_placeish.js", "public/javascripts/best_in_placeish.js"
-    end
-  end
-end
+module BestInPlaceish
+
+  class SetupGenerator < Rails::Generators::Base
+    source_root File.expand_path("../../../../public/javascripts", __FILE__)
+    desc "Copies best_in_placeish.js to the /public/javascripts folder of your app."
+
+    def copy_js
+      copy_file "best_in_placeish.js", "public/javascripts/best_in_placeish.js"
+    end
+  end
+end

data/public/javascripts/best_in_placeish.js

@@ -258,6 +258,45 @@ BestInPlaceEditor.forms = {
       }
     }
   },
+  
+  "datepicker" : {
+    activateForm : function() {
+      var output = '<form class="form_in_place" action="javascript:void(0)" style="display:inline;">';
+      output += '<input type="text" name="'+ this.attributeName + '" id="'+ this.attributeName + '" value="' + this.sanitizeValue(this.oldValue) + '"';
+      if (this.inner_class != null) {
+        output += ' class="' + this.inner_class + '"';
+      }
+      output += '></form>'
+      this.element.html(output);
+      this.setHtmlAttributes();
+      $('input#' + this.attributeName).datepicker({ 
+        dateFormat: 'dd/M/yy',
+        onClose: function() { $('form.form_in_place').submit(); }
+      });
+      this.element.find('input')[0].select();
+      this.element.find("form").bind('submit', {editor: this}, BestInPlaceEditor.forms.datepicker.submitHandler);
+      //this.element.find("input").bind('blur',   {editor: this}, BestInPlaceEditor.forms.datepicker.inputBlurHandler);
+      this.element.find("input").bind('keyup', {editor: this}, BestInPlaceEditor.forms.datepicker.keyupHandler);
+    },
+
+    getValue :  function() {
+      return this.sanitizeValue(this.element.find("input").val());
+    },
+
+    inputBlurHandler : function(event) {
+      event.data.editor.update();
+    },
+
+    submitHandler : function(event) {
+      event.data.editor.update();
+    },
+
+    keyupHandler : function(event) {
+      if (event.keyCode == 27) {
+        event.data.editor.abort();
+      }
+    }
+  },
 
   "select" : {
     activateForm : function() {

data/spec/helpers/best_in_place_spec.rb

@@ -1,239 +1,239 @@
-# encoding: utf-8
-require "spec_helper"
-
-describe BestInPlace::BestInPlaceHelpers do
-  describe "#best_in_place" do
-    before do
-      @user = User.new :name => "Lucia",
-        :last_name => "Napoli",
-        :email => "lucianapoli@gmail.com",
-        :address => "Via Roma 99",
-        :zip => "25123",
-        :country => "2",
-        :receive_email => false,
-        :description => "Lorem ipsum dolor sit amet, consectetur adipiscing elit. Vivamus a lectus et lacus ultrices auctor. Morbi aliquet convallis tincidunt. Praesent enim libero, iaculis at commodo nec, fermentum a dolor. Quisque eget eros id felis lacinia faucibus feugiat et ante. Aenean justo nisi, aliquam vel egestas vel, porta in ligula. Etiam molestie, lacus eget tincidunt accumsan, elit justo rhoncus urna, nec pretium neque mi et lorem. Aliquam posuere, dolor quis pulvinar luctus, felis dolor tincidunt leo, eget pretium orci purus ac nibh. Ut enim sem, suscipit ac elementum vitae, sodales vel sem."
-    end
-
-    it "should generate a proper span" do
-      nk = Nokogiri::HTML.parse(helper.best_in_place @user, :name)
-      span = nk.css("span")
-      span.should_not be_empty
-    end
-
-    describe "general properties" do
-      before do
-        nk = Nokogiri::HTML.parse(helper.best_in_place @user, :name)
-        @span = nk.css("span")
-      end
-
-      context "when it's an ActiveRecord model" do
-        it "should have a proper id" do
-          @span.attribute("id").value.should == "best_in_place_user_#{@user.id}_name"
-        end
-      end
-
-      context "when it's not an AR model" do
-        it "shold generate an html id without any id" do
-          nk = Nokogiri::HTML.parse(helper.best_in_place [1,2,3], :first, :path => @user)
-          span = nk.css("span")
-          span.attribute("id").value.should == "best_in_place_array_first"
-        end
-      end
-
-      it "should have the best_in_place class" do
-        @span.attribute("class").value.should == "best_in_place"
-      end
-
-      it "should have the correct data-attribute" do
-        @span.attribute("data-attribute").value.should == "name"
-      end
-
-      it "should have the correct data-object" do
-        @span.attribute("data-object").value.should == "user"
-      end
-
-      it "should have no activator by default" do
-        @span.attribute("data-activator").should be_nil
-      end
-
-      it "should have no inner_class by default" do
-        @span.attribute("data-inner-class").should be_nil
-      end
-
-      describe "url generation" do
-        it "should have the correct default url" do
-          @user.save!
-          nk = Nokogiri::HTML.parse(helper.best_in_place @user, :name)
-          span = nk.css("span")
-          span.attribute("data-url").value.should == "/users/#{@user.id}"
-        end
-
-        it "should use the custom url specified in string format" do
-          out = helper.best_in_place @user, :name, :path => "/custom/path"
-          nk = Nokogiri::HTML.parse(out)
-          span = nk.css("span")
-          span.attribute("data-url").value.should == "/custom/path"
-        end
-
-        it "should use the path given in a named_path format" do
-          out = helper.best_in_place @user, :name, :path => helper.users_path
-          nk = Nokogiri::HTML.parse(out)
-          span = nk.css("span")
-          span.attribute("data-url").value.should == "/users"
-        end
-
-        it "should use the given path in a hash format" do
-          out = helper.best_in_place @user, :name, :path => {:controller => :users, :action => :edit, :id => 23}
-          nk = Nokogiri::HTML.parse(out)
-          span = nk.css("span")
-          span.attribute("data-url").value.should == "/users/23/edit"
-        end
-      end
-
-      describe "nil option" do
-        it "should have no nil data by default" do
-          @span.attribute("data-nil").should be_nil
-        end
-
-        it "should show '' if the object responds with nil for the passed attribute" do
-          @user.stub!(:name).and_return(nil)
-          nk = Nokogiri::HTML.parse(helper.best_in_place @user, :name)
-          span = nk.css("span")
-          span.text.should == ""
-        end
-
-        it "should show '' if the object responds with an empty string for the passed attribute" do
-          @user.stub!(:name).and_return("")
-          nk = Nokogiri::HTML.parse(helper.best_in_place @user, :name)
-          span = nk.css("span")
-          span.text.should == ""
-        end
-      end
-
-      it "should have the given inner_class" do
-        out = helper.best_in_place @user, :name, :inner_class => "awesome"
-        nk = Nokogiri::HTML.parse(out)
-        span = nk.css("span")
-        span.attribute("data-inner-class").value.should == "awesome"
-      end
-
-      it "should have the given activator" do
-        out = helper.best_in_place @user, :name, :activator => "awesome"
-        nk = Nokogiri::HTML.parse(out)
-        span = nk.css("span")
-        span.attribute("data-activator").value.should == "awesome"
-      end
-    end
-
-
-    context "with a text field attribute" do
-      before do
-        nk = Nokogiri::HTML.parse(helper.best_in_place @user, :name)
-        @span = nk.css("span")
-      end
-
-      it "should render the name as text" do
-        @span.text.should == "Lucia"
-      end
-
-      it "should have an input data-type" do
-        @span.attribute("data-type").value.should == "input"
-      end
-
-      it "should have no data-collection" do
-        @span.attribute("data-collection").should be_nil
-      end
-    end
-
-    context "with a boolean attribute" do
-      before do
-        nk = Nokogiri::HTML.parse(helper.best_in_place @user, :receive_email, :type => :checkbox)
-        @span = nk.css("span")
-      end
-
-      it "should have a checkbox data-type" do
-        @span.attribute("data-type").value.should == "checkbox"
-      end
-
-      it "should have the default data-collection" do
-        data = ["No", "Yes"]
-        @span.attribute("data-collection").value.should == data.to_json
-      end
-
-      it "should render the current option as No" do
-        @span.text.should == "No"
-      end
-
-      describe "custom collection" do
-        before do
-          nk = Nokogiri::HTML.parse(helper.best_in_place @user, :receive_email, :type => :checkbox, :collection => ["Nain", "Da"])
-          @span = nk.css("span")
-        end
-
-        it "should show the message with the custom values" do
-          @span.text.should == "Nain"
-        end
-
-        it "should render the proper data-collection" do
-          @span.attribute("data-collection").value.should == ["Nain", "Da"].to_json
-        end
-      end
-
-    end
-
-    context "with a select attribute" do
-      before do
-        @countries = COUNTRIES.to_a
-        nk = Nokogiri::HTML.parse(helper.best_in_place @user, :country, :type => :select, :collection => @countries)
-        @span = nk.css("span")
-      end
-
-      it "should have a select data-type" do
-        @span.attribute("data-type").value.should == "select"
-      end
-
-      it "should have a proper data collection" do
-        @span.attribute("data-collection").value.should == @countries.to_json
-      end
-
-      it "should show the current country" do
-        @span.text.should == "Italy"
-      end
-    end
-  end
-
-  describe "#best_in_place_if" do
-    context "when the parameters are valid" do
-      before(:each) do
-        @output = "Some Value"
-        @field = :somefield
-        @object = mock("object", @field => @output)
-        @options = mock("options")
-      end
-      context "when the condition is true" do
-        before {@condition = true}
-        context "when the options parameter is left off" do
-          it "should call best_in_place with the rest of the parameters and empty options" do
-            helper.should_receive(:best_in_place).with(@object, @field, {})
-            helper.best_in_place_if @condition, @object, @field
-          end
-        end
-        context "when the options parameter is included" do
-          it "should call best_in_place with the rest of the parameters" do
-            helper.should_receive(:best_in_place).with(@object, @field, @options)
-            helper.best_in_place_if @condition, @object, @field, @options
-          end
-        end
-      end
-      context "when the condition is false" do
-        before {@condition = false}
-        it "should return the value of the field when the options value is left off" do
-          helper.best_in_place_if(@condition, @object, @field).should eq @output
-        end
-        it "should return the value of the field when the options value is included" do
-          helper.best_in_place_if(@condition, @object, @field, @options).should eq @output
-        end
-      end
-    end
-  end
-end
+# encoding: utf-8
+require "spec_helper"
+
+describe BestInPlace::BestInPlaceHelpers do
+  describe "#best_in_place" do
+    before do
+      @user = User.new :name => "Lucia",
+        :last_name => "Napoli",
+        :email => "lucianapoli@gmail.com",
+        :address => "Via Roma 99",
+        :zip => "25123",
+        :country => "2",
+        :receive_email => false,
+        :description => "Lorem ipsum dolor sit amet, consectetur adipiscing elit. Vivamus a lectus et lacus ultrices auctor. Morbi aliquet convallis tincidunt. Praesent enim libero, iaculis at commodo nec, fermentum a dolor. Quisque eget eros id felis lacinia faucibus feugiat et ante. Aenean justo nisi, aliquam vel egestas vel, porta in ligula. Etiam molestie, lacus eget tincidunt accumsan, elit justo rhoncus urna, nec pretium neque mi et lorem. Aliquam posuere, dolor quis pulvinar luctus, felis dolor tincidunt leo, eget pretium orci purus ac nibh. Ut enim sem, suscipit ac elementum vitae, sodales vel sem."
+    end
+
+    it "should generate a proper span" do
+      nk = Nokogiri::HTML.parse(helper.best_in_place @user, :name)
+      span = nk.css("span")
+      span.should_not be_empty
+    end
+
+    describe "general properties" do
+      before do
+        nk = Nokogiri::HTML.parse(helper.best_in_place @user, :name)
+        @span = nk.css("span")
+      end
+
+      context "when it's an ActiveRecord model" do
+        it "should have a proper id" do
+          @span.attribute("id").value.should == "best_in_place_user_#{@user.id}_name"
+        end
+      end
+
+      context "when it's not an AR model" do
+        it "shold generate an html id without any id" do
+          nk = Nokogiri::HTML.parse(helper.best_in_place [1,2,3], :first, :path => @user)
+          span = nk.css("span")
+          span.attribute("id").value.should == "best_in_place_array_first"
+        end
+      end
+
+      it "should have the best_in_place class" do
+        @span.attribute("class").value.should == "best_in_place"
+      end
+
+      it "should have the correct data-attribute" do
+        @span.attribute("data-attribute").value.should == "name"
+      end
+
+      it "should have the correct data-object" do
+        @span.attribute("data-object").value.should == "user"
+      end
+
+      it "should have no activator by default" do
+        @span.attribute("data-activator").should be_nil
+      end
+
+      it "should have no inner_class by default" do
+        @span.attribute("data-inner-class").should be_nil
+      end
+
+      describe "url generation" do
+        it "should have the correct default url" do
+          @user.save!
+          nk = Nokogiri::HTML.parse(helper.best_in_place @user, :name)
+          span = nk.css("span")
+          span.attribute("data-url").value.should == "/users/#{@user.id}"
+        end
+
+        it "should use the custom url specified in string format" do
+          out = helper.best_in_place @user, :name, :path => "/custom/path"
+          nk = Nokogiri::HTML.parse(out)
+          span = nk.css("span")
+          span.attribute("data-url").value.should == "/custom/path"
+        end
+
+        it "should use the path given in a named_path format" do
+          out = helper.best_in_place @user, :name, :path => helper.users_path
+          nk = Nokogiri::HTML.parse(out)
+          span = nk.css("span")
+          span.attribute("data-url").value.should == "/users"
+        end
+
+        it "should use the given path in a hash format" do
+          out = helper.best_in_place @user, :name, :path => {:controller => :users, :action => :edit, :id => 23}
+          nk = Nokogiri::HTML.parse(out)
+          span = nk.css("span")
+          span.attribute("data-url").value.should == "/users/23/edit"
+        end
+      end
+
+      describe "nil option" do
+        it "should have no nil data by default" do
+          @span.attribute("data-nil").should be_nil
+        end
+
+        it "should show '' if the object responds with nil for the passed attribute" do
+          @user.stub!(:name).and_return(nil)
+          nk = Nokogiri::HTML.parse(helper.best_in_place @user, :name)
+          span = nk.css("span")
+          span.text.should == ""
+        end
+
+        it "should show '' if the object responds with an empty string for the passed attribute" do
+          @user.stub!(:name).and_return("")
+          nk = Nokogiri::HTML.parse(helper.best_in_place @user, :name)
+          span = nk.css("span")
+          span.text.should == ""
+        end
+      end
+
+      it "should have the given inner_class" do
+        out = helper.best_in_place @user, :name, :inner_class => "awesome"
+        nk = Nokogiri::HTML.parse(out)
+        span = nk.css("span")
+        span.attribute("data-inner-class").value.should == "awesome"
+      end
+
+      it "should have the given activator" do
+        out = helper.best_in_place @user, :name, :activator => "awesome"
+        nk = Nokogiri::HTML.parse(out)
+        span = nk.css("span")
+        span.attribute("data-activator").value.should == "awesome"
+      end
+    end
+
+
+    context "with a text field attribute" do
+      before do
+        nk = Nokogiri::HTML.parse(helper.best_in_place @user, :name)
+        @span = nk.css("span")
+      end
+
+      it "should render the name as text" do
+        @span.text.should == "Lucia"
+      end
+
+      it "should have an input data-type" do
+        @span.attribute("data-type").value.should == "input"
+      end
+
+      it "should have no data-collection" do
+        @span.attribute("data-collection").should be_nil
+      end
+    end
+
+    context "with a boolean attribute" do
+      before do
+        nk = Nokogiri::HTML.parse(helper.best_in_place @user, :receive_email, :type => :checkbox)
+        @span = nk.css("span")
+      end
+
+      it "should have a checkbox data-type" do
+        @span.attribute("data-type").value.should == "checkbox"
+      end
+
+      it "should have the default data-collection" do
+        data = ["No", "Yes"]
+        @span.attribute("data-collection").value.should == data.to_json
+      end
+
+      it "should render the current option as No" do
+        @span.text.should == "No"
+      end
+
+      describe "custom collection" do
+        before do
+          nk = Nokogiri::HTML.parse(helper.best_in_place @user, :receive_email, :type => :checkbox, :collection => ["Nain", "Da"])
+          @span = nk.css("span")
+        end
+
+        it "should show the message with the custom values" do
+          @span.text.should == "Nain"
+        end
+
+        it "should render the proper data-collection" do
+          @span.attribute("data-collection").value.should == ["Nain", "Da"].to_json
+        end
+      end
+
+    end
+
+    context "with a select attribute" do
+      before do
+        @countries = COUNTRIES.to_a
+        nk = Nokogiri::HTML.parse(helper.best_in_place @user, :country, :type => :select, :collection => @countries)
+        @span = nk.css("span")
+      end
+
+      it "should have a select data-type" do
+        @span.attribute("data-type").value.should == "select"
+      end
+
+      it "should have a proper data collection" do
+        @span.attribute("data-collection").value.should == @countries.to_json
+      end
+
+      it "should show the current country" do
+        @span.text.should == "Italy"
+      end
+    end
+  end
+
+  describe "#best_in_place_if" do
+    context "when the parameters are valid" do
+      before(:each) do
+        @output = "Some Value"
+        @field = :somefield
+        @object = mock("object", @field => @output)
+        @options = mock("options")
+      end
+      context "when the condition is true" do
+        before {@condition = true}
+        context "when the options parameter is left off" do
+          it "should call best_in_place with the rest of the parameters and empty options" do
+            helper.should_receive(:best_in_place).with(@object, @field, {})
+            helper.best_in_place_if @condition, @object, @field
+          end
+        end
+        context "when the options parameter is included" do
+          it "should call best_in_place with the rest of the parameters" do
+            helper.should_receive(:best_in_place).with(@object, @field, @options)
+            helper.best_in_place_if @condition, @object, @field, @options
+          end
+        end
+      end
+      context "when the condition is false" do
+        before {@condition = false}
+        it "should return the value of the field when the options value is left off" do
+          helper.best_in_place_if(@condition, @object, @field).should eq @output
+        end
+        it "should return the value of the field when the options value is included" do
+          helper.best_in_place_if(@condition, @object, @field, @options).should eq @output
+        end
+      end
+    end
+  end
+end