middleman-sitemap 0.0.11 → 0.0.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b6aec1a24ba9b08938486a49d644a42b2d72e4e4
-  data.tar.gz: ed96ebaa0ab1ef92c70ee3d0d347cd10d95b5c2d
+  metadata.gz: a94ace77445db68609654052b99b36b52dea329e
+  data.tar.gz: a4326897b2af686179153716b49d4c63aba21986
 SHA512:
-  metadata.gz: 3059dfa28a7cf0e66c102f88cb2345f73e70621b877fb052509f6b2e717fd0047c50d5e5ae86e8832facb9c16ee1d078a7a9a02466d6d45a2e5f529d5e2678d0
-  data.tar.gz: e0d81cc050cfa275040b2096daa6a278ac3fa8939f7b7d307141449f3d870ddfed8cc164693b228ee6eb30970efdbb2ec88baa032e5dcd9263336e8fd2f430f7
+  metadata.gz: e33afbff1c2ee244baf3491af460daf67559d8eab5c63a80b4d87ccdc7fae3beb5f806e43eb176977a20ea150004b433f710626711770c640970235d51440e78
+  data.tar.gz: f6cbe9370f5b0f15b5ffb98f6816ed8b74c83cdc09f5da67681f3cbda10fe2f852dbd9ea2db18cde7be0ac075d724e53b0b6315148d800e22dab491c4110a2f0

data/.travis.yml

@@ -5,4 +5,4 @@ rvm:
   - 2.1.1
   - jruby-19mode
 
-script: "bundle exec rake test"
+script: "bundle exec cucumber"

data/README.md

@@ -21,6 +21,8 @@ If you already have a Middleman project: Add `gem "middleman-sitemap"` to your `
 activate :sitemap
 ```
 
+By default, `middleman-sitemap` will create a sitemap with `loc1 elements. As per the [protocol](http://www.sitemaps.org/protocol.html), the `lastmod`, `changefreq`, and `priority` elements are optional. To include these elements in your sitemap, you MUST include them in the front matter of your page.
+
 ## Options
 
 ### GZIP

data/features/encoding.feature

@@ -3,9 +3,9 @@ Feature: Encode HTML entities
   Scenario: URLs in loc entities are HTML encoded
     Given a successfully built app at "weird-urls-app"
     When I cd to "build"
-    Then the file "sitemap.xml" should contain "http://www.example.com/this-&-that"
+    Then the file "sitemap.xml" should contain "<loc>http://www.example.com/this-&amp;-that.html</loc>"
 
   Scenario: Subdirectories
     Given a successfully built app at "subdirectories-app"
     When I cd to "build"
-    Then the file "sitemap.xml" should contain "http://www.example.com/error-pages/404.html"
+    Then the file "sitemap.xml" should contain "<loc>http://www.example.com/error-pages/404.html</loc>"

data/features/template.feature

@@ -0,0 +1,26 @@
+Feature: Default template
+
+  Scenario: Sitemap should have loc element
+    Given a successfully built app at "template-app"
+    When I cd to "build"
+    Then the file "sitemap.xml" should contain "<loc>http://www.example.com</loc>"
+
+  Scenario: Sitemap should not have lastmod element
+    Given a successfully built app at "template-app"
+    When I cd to "build"
+    Then the file "sitemap.xml" should not contain "<lastmod>"
+
+  Scenario: Sitemap should not have changefreq element
+    Given a successfully built app at "template-app"
+    When I cd to "build"
+    Then the file "sitemap.xml" should not contain "<changefreq>"
+
+  Scenario: Sitemap should not have priority element
+    Given a successfully built app at "template-app"
+    When I cd to "build"
+    Then the file "sitemap.xml" should not contain "<priority>"
+
+  Scenario: Page with relevant sitemap front matter should show up in sitemap
+    Given a successfully built app at "sitemap-frontmatter-app"
+    When I cd to "build"
+    Then the file "sitemap.xml" should contain "<priority>" 

data/fixtures/sitemap-frontmatter-app/config.rb

@@ -0,0 +1 @@
+activate :sitemap

data/fixtures/sitemap-frontmatter-app/source/index.html

@@ -0,0 +1,4 @@
+---
+priority: 0.3
+---
+Hello world!

data/fixtures/sitemap-frontmatter-app/source/something.html

@@ -0,0 +1,4 @@
+---
+changefreq: weekly
+---
+Something

data/fixtures/template-app/config.rb

@@ -0,0 +1 @@
+activate :sitemap

data/fixtures/template-app/source/index.html

@@ -0,0 +1 @@
+Hello world!

data/fixtures/template-app/source/something.html

@@ -0,0 +1 @@
+Something

data/lib/middleman-sitemap/extension.rb

@@ -54,7 +54,8 @@ class Sitemap < ::Middleman::Extension
   def build_sitemap(name, pages)
     @pages = pages
     @hostname = options.hostname
-    template = Tilt::ERBTemplate.new(File.expand_path(File.join("#{File.dirname(__FILE__)}", "../../templates/sitemap.xml.erb")))
+    templates_path = File.expand_path(File.join("#{File.dirname(__FILE__)}", "../../templates/sitemap.xml.erb"))
+    template = Tilt::ERBTemplate.new(templates_path, 0, :trim => '>')
     sitemap = template.render(self)
 
     outfile = File.join(app.build_dir, name)
@@ -91,7 +92,7 @@ class Sitemap < ::Middleman::Extension
   # Returns a URL with proper HTML entities
   def encode(path)
     str = path.split("/").map { |f| app.escape_html(f) }.join("/")
-    return "#{str}.html"
+    return str
   end
 
 end

data/middleman-sitemap.gemspec

@@ -3,7 +3,7 @@ $:.push File.expand_path("../lib", __FILE__)
 
 Gem::Specification.new do |s|
   s.name        = "middleman-sitemap"
-  s.version     = "0.0.11"
+  s.version     = "0.0.12"
   s.platform    = Gem::Platform::RUBY
   s.authors     = ["Larry Staton Jr."]
   s.email       = ["larry@larrystaton.com"]

data/templates/sitemap.xml.erb

@@ -3,9 +3,12 @@
 <% @pages.each do |p| %>
   <url>
     <loc><%= @hostname + encode(p.url).gsub(/\/$/,'') %></loc>
-    <lastmod><%= Date.today.to_time.iso8601 %></lastmod>
-    <changefreq><%= p.data.changefreq || 'weekly' %></changefreq>
-    <priority><%= p.data.priority || '0.5' %></priority>
+<% if p.data.lastmod %>    <lastmod><%= p.data.lastmod %></lastmod>
+<% end %>
+<% if p.data.changefreq %>    <changefreq><%= p.data.changefreq %></changefreq>
+<% end %>
+<% if p.data.priority %>    <priority><%= p.data.priority %></priority>
+<% end %>
   </url>
 <% end %>
 </urlset>

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: middleman-sitemap
 version: !ruby/object:Gem::Version
-  version: 0.0.11
+  version: 0.0.12
 platform: ruby
 authors:
 - Larry Staton Jr.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-04-03 00:00:00.000000000 Z
+date: 2014-04-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: middleman-core
@@ -41,6 +41,7 @@ files:
 - features/gzip.feature
 - features/multiple_sitemaps.feature
 - features/support/env.rb
+- features/template.feature
 - fixtures/gzip-app/config.rb
 - fixtures/gzip-app/source/index.html
 - fixtures/gzip-app/source/javascripts/test.js
@@ -51047,12 +51048,22 @@ files:
 - fixtures/large-site-app/source/page50999.html
 - fixtures/large-site-app/source/page51000.html
 - fixtures/large-site-app/source/stylesheets/test.css
+- fixtures/sitemap-frontmatter-app/config.rb
+- fixtures/sitemap-frontmatter-app/source/index.html
+- fixtures/sitemap-frontmatter-app/source/javascripts/test.js
+- fixtures/sitemap-frontmatter-app/source/something.html
+- fixtures/sitemap-frontmatter-app/source/stylesheets/test.css
 - fixtures/subdirectories-app/config.rb
 - fixtures/subdirectories-app/source/error-pages/404.html
 - fixtures/subdirectories-app/source/index.html
 - fixtures/subdirectories-app/source/javascripts/test.js
 - fixtures/subdirectories-app/source/stylesheets/test.css
 - fixtures/subdirectories-app/source/this-&-that.html
+- fixtures/template-app/config.rb
+- fixtures/template-app/source/index.html
+- fixtures/template-app/source/javascripts/test.js
+- fixtures/template-app/source/something.html
+- fixtures/template-app/source/stylesheets/test.css
 - fixtures/weird-urls-app/config.rb
 - fixtures/weird-urls-app/source/index.html
 - fixtures/weird-urls-app/source/javascripts/test.js
@@ -51093,4 +51104,5 @@ test_files:
 - features/gzip.feature
 - features/multiple_sitemaps.feature
 - features/support/env.rb
+- features/template.feature
 has_rdoc: