thor 0.18.0 → 0.18.1

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+## 0.18.1, release 2013-03-30
+* Revert regressions found in 0.18.0
+
 ## 0.18.0, release 2013-03-26
 * Remove rake2thor
 * Only display colors if output medium supports colors

data/lib/thor/actions/empty_directory.rb

@@ -97,28 +97,12 @@ class Thor
         #
         #   user.rb
         #
-        # The method referenced by %-string SHOULD be public. Otherwise you
-        # get the exception with the corresponding error message.
+        # The method referenced can be either public or private.
         #
         def convert_encoded_instructions(filename)
           filename.gsub(/%(.*?)%/) do |initial_string|
-            call_public_method($1.strip) or initial_string
-          end
-        end
-
-        # Calls `base`'s public method `sym`.
-        # Returns:: result of `base.sym` or `nil` if `sym` wasn't found in
-        #  `base`
-        # Raises::  Thor::PrivateMethodEncodedError if `sym` references
-        #  a private method.
-        def call_public_method(sym)
-          if base.respond_to?(sym)
-            base.send(sym)
-          elsif base.respond_to?(sym, true)
-            raise Thor::PrivateMethodEncodedError,
-              "Method #{base.class}##{sym} should be public, not private"
-          else
-            nil
+            method = $1.strip
+            base.respond_to?(method, true) ? base.send(method) : initial_string
           end
         end
 

data/lib/thor/base.rb

@@ -119,6 +119,18 @@ class Thor
     end
 
     module ClassMethods
+      def attr_reader(*) #:nodoc:
+        no_commands { super }
+      end
+
+      def attr_writer(*) #:nodoc:
+        no_commands { super }
+      end
+
+      def attr_accessor(*) #:nodoc:
+        no_commands { super }
+      end
+
       # If you want to raise an error for unknown options, call check_unknown_options!
       # This is disabled by default to allow dynamic invocations.
       def check_unknown_options!
@@ -578,9 +590,6 @@ class Thor
           # Return if it's not a public instance method
           return unless public_method_defined?(meth.to_sym)
 
-          # Return if attr_* added the method
-          return if caller.first.to_s[/`attr_(reader|writer|accessor)'/]
-
           return if @no_commands || !create_command(meth)
 
           is_thor_reserved_word?(meth, :command)

data/lib/thor/error.rb

@@ -25,8 +25,4 @@ class Thor
 
   class MalformattedArgumentError < InvocationError
   end
-
-  # Raised when a user tries to call a private method encoded in templated filename.
-  class PrivateMethodEncodedError < Error
-  end
 end

data/lib/thor/version.rb

@@ -1,3 +1,3 @@
 class Thor
-  VERSION = "0.18.0"
+  VERSION = "0.18.1"
 end

data/spec/actions/empty_directory_spec.rb

@@ -107,6 +107,16 @@ describe Thor::Actions::EmptyDirectory do
         expect(@action.send(:convert_encoded_instructions, "%file_name%.txt")).to eq("expected.txt")
       end
 
+      it "accepts and executes a private %\w+% encoded instruction" do
+        @action.base.extend Module.new {
+          private
+          def private_file_name
+            "expected"
+          end
+        }
+        expect(@action.send(:convert_encoded_instructions, "%private_file_name%.txt")).to eq("expected.txt")
+      end
+
       it "ignores an 'illegal' %\w+% encoded instruction" do
         expect(@action.send(:convert_encoded_instructions, "%some_name%.txt")).to eq("%some_name%.txt")
       end
@@ -114,17 +124,6 @@ describe Thor::Actions::EmptyDirectory do
       it "ignores incorrectly encoded instruction" do
         expect(@action.send(:convert_encoded_instructions, "%some.name%.txt")).to eq("%some.name%.txt")
       end
-
-      it "raises an error if the instruction refers to a private method" do
-        module PrivExt
-          private
-          def private_file_name
-            "something_hidden"
-          end
-        end
-        @action.base.extend(PrivExt)
-        expect { @action.send(:convert_encoded_instructions, "%private_file_name%.txt") }.to raise_error Thor::PrivateMethodEncodedError
-      end
     end
   end
 end

data/spec/base_spec.rb

@@ -287,9 +287,5 @@ describe Thor::Base do
       expect(capture(:stderr){ MyScript.start(["some_attribute"]) }).to match(/Could not find/)
       expect(capture(:stderr){ MyScript.start(["some_attribute=", "foo"]) }).to match(/Could not find/)
     end
-
-    it "respects visibility" do
-      expect(MyScript.public_instance_methods).to_not include(:private_attribute)
-    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: thor
 version: !ruby/object:Gem::Version
-  version: 0.18.0
+  version: 0.18.1
   prerelease: 
 platform: ruby
 authors:
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-03-26 00:00:00.000000000 Z
+date: 2013-03-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -132,9 +132,6 @@ required_ruby_version: !ruby/object:Gem::Requirement
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
-      segments:
-      - 0
-      hash: 1077930939435781644
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -196,3 +193,4 @@ test_files:
 - spec/subcommand_spec.rb
 - spec/thor_spec.rb
 - spec/util_spec.rb
+has_rdoc: