Improved: config_tag plugin is much more flexible now and can be used by other plugins directly through the config_tag method

imathis · imathis · commit 05db158fec6d · 2013-03-11T01:19:59.000-05:00
diff --git a/plugins/config_tag.rb b/plugins/config_tag.rb
@@ -3,27 +3,42 @@
 class ConfigTag < Liquid::Tag
   def initialize(tag_name, options, tokens)
     super
-    @options = options.split(' ').map {|i| i.strip }
-    @key = @options.first
-    @tag = (@options[1] || 'div')
+    options = options.split(' ').map {|i| i.strip }
+    @key = options.slice!(0)
+    @tag = nil
+    @classname = nil
+    options.each do |option|
+      @tag = $1 if option =~ /tag:(\S+)/ 
+      @classname = $1 if option =~ /classname:(\S+)/
+    end
   end
 
   def render(context)
-    config = context.registers[:site].config
-    options = @options.first.split('.').map { |k| config = config[k] }.last #reference objects with dot notation
-    keyclass = @key.sub(/_/, '-').sub(/\./, '-')
-    tag  = "<#{@tag} class='#{keyclass}'"
+    config_tag(context.registers[:site].config, @key, @tag, @classname)
+  end
+end
+
+def config_tag(config, key, tag=nil, classname=nil)
+  options     = key.split('.').map { |k| config[k] }.last #reference objects with dot notation
+  tag       ||= 'div'
+  classname ||= key.sub(/_/, '-').sub(/\./, '-')
+  output      = "<#{tag} class='#{classname}'"
+
+  if options.respond_to? 'keys'
     options.each do |k,v|
       unless v.nil?
         v = v.join ',' if v.respond_to? 'join'
         v = v.to_json if v.respond_to? 'keys'
-        tag += " data-#{k.sub'_','-'}='#{v}'"
+        output += " data-#{k.sub'_','-'}='#{v}'"
       end
     end
-    tag += "></#{@tag}>"
-    p tag
-    tag
+  elsif options.respond_to? 'join'
+    output += " data-value='#{config[key].join(',')}'"
+  else
+    output += " data-value='#{config[key]}'"
   end
+  output += "></#{tag}>"
 end
 
 Liquid::Template.register_tag('config_tag', ConfigTag)
+
