diff options
| author | Saku Ytti <saku@ytti.fi> | 2014-07-03 17:21:47 +0300 |
|---|---|---|
| committer | Saku Ytti <saku@ytti.fi> | 2014-07-03 17:21:47 +0300 |
| commit | c8b9e5af6d38e73ffa830c7abc137021447cb6bc (patch) | |
| tree | 4df9b143a1a99a7f114c141691624d289ad3f3f3 | |
| parent | 646794b7f1aca52914d76e95b4601d4b868c7217 (diff) | |
Fix class name resolution
In new method, where we no long explicitly set loaded class name for
later resolution we depend on file name being same as class name.
However for File output, this is not true, because I didn't want plain
File in code to resolve to Oxidized::File, and din't want to always
write ::File, File ouput has class name OxFile.
This change allows class names to start optionally with string
'Oxidized' if so, they still match.
| -rw-r--r-- | lib/oxidized/manager.rb | 1 | ||||
| -rw-r--r-- | lib/oxidized/output/file.rb | 2 | ||||
| -rw-r--r-- | oxidized.gemspec | 2 |
3 files changed, 3 insertions, 2 deletions
diff --git a/lib/oxidized/manager.rb b/lib/oxidized/manager.rb index ed49b7e..251f0cc 100644 --- a/lib/oxidized/manager.rb +++ b/lib/oxidized/manager.rb @@ -11,6 +11,7 @@ module Oxidized klass = nil [Oxidized, Object].each do |mod| klass = mod.constants.find { |const| const.to_s.downcase == file.downcase } + klass = mod.constants.find { |const| const.to_s.downcase == 'oxidized'+ file.downcase } unless klass klass = mod.const_get klass if klass break if klass end diff --git a/lib/oxidized/output/file.rb b/lib/oxidized/output/file.rb index f3443b4..2d08106 100644 --- a/lib/oxidized/output/file.rb +++ b/lib/oxidized/output/file.rb @@ -1,5 +1,5 @@ module Oxidized -class OxFile < Output +class OxidizedFile < Output require 'fileutils' def initialize diff --git a/oxidized.gemspec b/oxidized.gemspec index f8081cc..c0ef485 100644 --- a/oxidized.gemspec +++ b/oxidized.gemspec @@ -1,6 +1,6 @@ Gem::Specification.new do |s| s.name = 'oxidized' - s.version = '0.0.60' + s.version = '0.0.61' s.platform = Gem::Platform::RUBY s.authors = [ 'Saku Ytti' ] s.email = %w( saku@ytti.fi ) |