Always override respond_to_missing when overriding method_missing

README.textile

@@ -63,7 +63,7 @@ For an extensive usage in production, see the "wukong gem.":http://github.com/mr
 
 h2. Notice
 
-Configliere 4.x now has 100% spec coverage, more powerful commandline handling, zero required dependencies. However, it also strips out several obscure features and much magical code, which breaks said obscure features and magic-dependent code. See the "CHANGELOG.":CHANGELOG.textile for details as you upgrade.
+Configliere 4.x now has 100% spec coverage, more powerful commandline handling, and minimal required dependencies. However, it also strips out several obscure features and much magical code, which breaks said obscure features and magic-dependent code. See the "CHANGELOG.":CHANGELOG.textile for details as you upgrade.
 
 h2. Design goals:
 

lib/configliere.rb

@@ -1,6 +1,10 @@
 require 'date'                      # type conversion
 require 'time'                      # type conversion
 require 'fileutils'                 # so save! can mkdir
+
+require 'multi_json'
+require 'yaml'
+
 require 'configliere/deep_hash'     # magic hash for params
 require 'configliere/param'         # params container
 require 'configliere/define'        # define param behavior

lib/configliere/commandline.rb

@@ -70,7 +70,7 @@ def process_argv!
         # -a val
         when arg =~ /\A-(\w)\z/
           flag = find_param_for_flag($1)
-          unless flag then @unknown_argvs << flag ; next ; end
+          unless flag then @unknown_argvs << $1 ; next ; end
           if (not args.empty?) && (args.first !~ /\A-/)
             val = args.shift
           else
@@ -80,7 +80,7 @@ def process_argv!
         # -a=val
         when arg =~ /\A-(\w)=(.*)\z/
           flag, val = [find_param_for_flag($1), $2]
-          unless flag then @unknown_argvs << flag ; next ; end
+          unless flag then @unknown_argvs << $1 ; next ; end
           self[flag] = parse_value(val)
         else
           self.rest << arg

lib/configliere/config_file.rb

@@ -48,7 +48,6 @@ def read filename, options={}
     end
 
     def read_yaml yaml_str, options={}
-      require 'yaml'
       new_data = YAML.load(yaml_str) || {}
       # Extract the :env (production/development/etc)
       if options[:env]
@@ -58,11 +57,7 @@ def read_yaml yaml_str, options={}
       self
     end
 
-    #
-    # we depend on you to require some sort of JSON
-    #
     def read_json json_str, options={}
-      require 'multi_json'
       new_data = MultiJson.load(json_str) || {}
       # Extract the :env (production/development/etc)
       if options[:env]

lib/configliere/define.rb

@@ -173,6 +173,20 @@ def method_missing meth, *args
       else super ; end
     end
 
+    # Must override respond_to_missing? whenever overriding method_missing
+    #
+    # @example
+    #   Settings.respond_to? :foo
+    #   #=> false
+    #   Settings.define :foo
+    #   Settings.respond_to? :foo
+    #   #=> true
+    def respond_to_missing?(meth, include_private = false)
+      meth.to_s =~ /^(\w+)(=)?$/ or return super
+      name, setter = [$1.to_sym, $2]
+      has_definition?(name) ? true : super
+    end
+
   end
 
   # Define is included by default