@@ -16,7 +16,7 @@ module ConfigFindAllAsTree # iterate through all path levels path.each do |path_element| - if path_element == path.last + if path_element.equal?(path.last) # We are at the end of the path, so set the settting object as the value current_level[path_element] = setting lib/config_find_all_as_tree.rb @@ -5,6 +5,7 @@ describe Radiant::Config, 'find_all_as_tree' do @title = Radiant::Config.create(:key => 'admin.title', :value => 'Radiant CMS') @parts = Radiant::Config.create(:key => 'defaults.page.parts', :value => 'body, extended') @status = Radiant::Config.create(:key => 'defaults.page.status', :value => 'draft') + @repetitive = Radiant::Config.create(:key => 'repetitive.setting.setting', :value => 'foo') end it "should find all settings and return a tree as a nested hash" do @@ -12,5 +13,6 @@ describe Radiant::Config, 'find_all_as_tree' do result['admin']['title'].should == @title result['defaults']['page']['parts'].should == @parts result['defaults']['page']['status'].should == @status + result['repetitive']['setting']['setting'].should == @repetitive end end \ No newline at end of file spec/models/config_spec.rb b4fe29d4ec456dcff0b42a199f5a0b173f616a97 Taylor Barstow taylorbarstow@gmail.com http://github.com/Squeegy/radiant-settings/commit/d93d9d78436960f5d40ec8d23067a92d105176bf d93d9d78436960f5d40ec8d23067a92d105176bf 2009-07-24T15:11:52-07:00 2009-07-14T08:23:16-07:00 avoid prematurely truncating paths with repeated key(s) Signed-off-by: Jim Gay <jim@saturnflyer.com> 948dfd49ffd4a3d0bdd4c55b74cd314e8aea3476 Jim Gay jim@saturnflyer.com