cartographer 1.1.0 → 1.1.1

data/History.txt

@@ -1,3 +1,7 @@
+=== 1.1.1 / 2011-03-29
+
+* URI escaping for paths
+
 === 1.1.0 / 2011-03-28
 
 * from_xml takes sitemap.xml content and integrates it into the current Cartographer object. 

data/lib/cartographer.rb

@@ -24,7 +24,7 @@ require 'nokogiri'
 # See Cartographer and Cartographer::URL RDoc for more information.
 #
 class Cartographer
-  VERSION = '1.1.0'
+  VERSION = '1.1.1'
 
   ##
   # List of Cartographer::URL objects
@@ -91,7 +91,7 @@ class Cartographer
         end
       end
 
-      uri.path = new_path
+      uri.path = Cartographer.escape_path(new_path)
       add URL.new(:location => uri, :lastmod => stat.mtime, :changefreq => @default_changefreq)
     end
   end
@@ -132,6 +132,13 @@ class Cartographer
     urls.uniq!
   end
 
+  private
+
+  def self.escape_path(path)
+    require 'cgi'
+    path.split(%r!/!, -1).map { |x| CGI.escape(x) }.join("/")
+  end
+
   ##
   # Cartographer::URL is a structure to enforce the sitemap format.
   #

data/test/test_cartographer.rb

@@ -57,6 +57,18 @@ class TestCartographer < MiniTest::Unit::TestCase
     )
 
     FileUtils.rm_r("test/prune_me")
+
+    FileUtils.touch('test/with spaces')
+
+    sm = Cartographer.new("http://example.org")
+    sm.add_tree(File.expand_path("test")) 
+
+    assert_includes(
+      sm.urls.map(&:location),
+      URI.parse("http://example.org/with+spaces")
+    )
+
+    FileUtils.rm('test/with spaces')
   end
 
   def test_to_xml

metadata

@@ -2,7 +2,7 @@
 name: cartographer
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 1.1.0
+  version: 1.1.1
 platform: ruby
 authors: 
 - Erik Hollensbe
@@ -11,7 +11,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-03-28 00:00:00 -04:00
+date: 2011-03-29 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency