found another obscure bug where cache_bust values aren't being defaul…

…ted b/c of a 'false' setting accidentally being interpreted as 'nil'

Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    rack-less (2.0.0)
+    rack-less (2.0.1)
       less (~> 1.2)
       rack (~> 1.0)
 
@@ -43,8 +43,6 @@ PLATFORMS
 
 DEPENDENCIES
   bundler (~> 1.0)
-  less (~> 1.2)
-  rack (~> 1.0)
   rack-less!
   rack-test (>= 0.5.3)
   sinatra (>= 0.9.4)

lib/rack/less/config.rb

@@ -37,7 +37,7 @@ class Config
 
     def initialize(settings={})
       ATTRIBUTES.each do |a|
-        instance_variable_set("@#{a}", settings[a] || DEFAULTS[a])
+        instance_variable_set("@#{a}", settings[a].nil? ? DEFAULTS[a] : settings[a])
       end
     end
 
@@ -77,22 +77,27 @@ def stylesheet_filename(key)
       filename = key.strip
       filename += ".css" unless filename.include?('.css')
       if !filename.include?('?') && self.cache_bust != false
-        filename += "?"
-        filename += if self.cache_bust == true
-          Time.now.to_i
-        else
-          self.cache_bust ||= default_cache_bust
-        end.to_s
+        if !(cb = cache_bust_value).empty?
+          filename += "?#{cb}"
+        end
       end
       filename
     end
 
+    def cache_bust_value
+      if self.cache_bust == true
+        Time.now.to_i
+      else
+        self.cache_bust ||= default_cache_bust
+      end.to_s
+    end
+
     def default_cache_bust
       if defined?(::Sinatra) && defined?(::Sinatra::Application)
         app_root_cache_bust(::Sinatra::Application)
       elsif defined?(::Rails)
         app_root_cache_bust(::Rails)
-      end || false
+      end || ""
     end
 
     def app_root_cache_bust(app)

test/config_test.rb

@@ -10,7 +10,7 @@ class ConfigTest < Test::Unit::TestCase
     { :cache => false,
       :compress => false,
       :combinations => {},
-      :cache_bust => false
+      :cache_bust => nil
     }.each do |k,v|
       should "default #{k} correctly" do
         assert_equal v, @config.send(k)