Merge pull request #71 from mattslack/allow-strings

Don't try to map Strings

lib/x-editable-rails/view_helpers.rb

@@ -72,7 +72,7 @@ def editable(object, method, options = {})
             })
 
             content_tag tag, html_options do
-              if %w(select checklist).include? data[:type].to_s
+              if %w(select checklist).include?(data[:type].to_s) && !source.is_a?(String)
                 source = normalize_source(source)
                 content = source.detect { |t| output_value == output_value_for(t[0]) }
                 content.present? ? content[1] : ""

test/view_helpers_test.rb

@@ -94,6 +94,14 @@ def initialize(attributes={})
                  "ViewHelpers#editable should generate content tag with the current value"
   end
 
+  test "editable should store the source url as a data attribute" do
+    subject = Subject.new
+    assert_match %r{<span[^>]+data-source="http://example.org"},
+                 editable(subject, :name, type: "select",
+                 source: "http://example.org"),
+                 "ViewHelpers#editable should generate content tag with url source as a data attribute"
+  end
+
   private
 
   def view_helpers_test_subject_path(subject)