Slug tags for URL

Author: alfredxing

lib/jekyll-archives/archive.rb

@@ -2,7 +2,7 @@ module Jekyll
   class Archive
     include Convertible
 
-    attr_accessor :posts, :type, :name
+    attr_accessor :posts, :type, :name, :slug
     attr_accessor :data, :content, :output
     attr_accessor :path, :ext
     attr_accessor :site
@@ -27,6 +27,9 @@ def initialize(site, name, type, posts)
       @type  = type
       @name  = name
 
+      # Generate slug if tag or category (taken from jekyll/jekyll/features/support/env.rb)
+      @slug = name.downcase.gsub(/[^\w]/, " ").strip.gsub(/\s+/, '-') if name.is_a? String
+
       # Use ".html" for file extension and url for path
       @ext  = ".html"
       @path = url
@@ -50,7 +53,7 @@ def url_placeholders
       if @name.is_a? Hash
         @name.merge({ :type => @type })
       else
-        { :name => @name, :type => @type }
+        { :name => @slug, :type => @type }
       end
     end
 

test/source/_posts/2014-03-15-tagged-post.md

@@ -1,6 +1,6 @@
 ---
 title: Tagged post
-tags: [test, tagged]
+tags: ["Test Tag", tagged]
 ---
 
 This is a tagged post

test/test_jekyll_archives.rb

@@ -28,7 +28,7 @@ class TestJekyllArchives < Minitest::Test
 
     should "generate archive pages by tag" do
       @archives.generate(@site)
-      assert archive_exists? @site, "/tag/test/"
+      assert archive_exists? @site, "/tag/test-tag/"
       assert archive_exists? @site, "/tag/tagged/"
       assert archive_exists? @site, "/tag/new/"
     end
@@ -40,7 +40,7 @@ class TestJekyllArchives < Minitest::Test
 
     should "generate archive pages with a layout" do
       @site.process
-      assert_equal "Test", read_file("/tag/test/index.html")
+      assert_equal "Test", read_file("/tag/test-tag/index.html")
     end
   end
 
@@ -57,7 +57,7 @@ class TestJekyllArchives < Minitest::Test
 
     should "use custom layout" do
       @site.process
-      assert_equal "Test too", read_file("/tag/test/index.html")
+      assert_equal "Test too", read_file("/tag/test-tag/index.html")
     end
   end
 
@@ -78,7 +78,7 @@ class TestJekyllArchives < Minitest::Test
     should "use the right permalink" do
       assert archive_exists? @site, "/year/2014/"
       assert archive_exists? @site, "/year/2013/"
-      assert archive_exists? @site, "/tag-test.html"
+      assert archive_exists? @site, "/tag-test-tag.html"
       assert archive_exists? @site, "/tag-new.html"
       assert archive_exists? @site, "/category-plugins.html"
     end