adamsdesk
/
jekyll-seo-tag
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #14 from pathawks/strip_index 

Strip trailing /index.html from seo_url
This commit is contained in:
Ben Balter 2016-01-06 10:27:59 -05:00
parent 9f3da7e6fe 23e9b8ae21
commit c29c923dbb
2 changed files with 13 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
lib/template.html
View File

@ -41,8 +41,8 @@
{% endif %}
{% if seo_url %}
<link rel="canonical" href="{{ seo_url }}{{ page.url }}" itemprop="url" />
<meta property='og:url' content='{{ seo_url }}{{ page.url }}' />
<link rel="canonical" href="{{ seo_url }}{{ page.url | replace:'/index.html','/' }}" itemprop="url" />
<meta property='og:url' content='{{ seo_url }}{{ page.url | replace:'/index.html','/' }}' />
{% endif %}
{% if site.title %}
@ -64,10 +64,10 @@
{% if page.date %}
<meta property="og:type" content="article" />
{% if page.next.url %}
<link rel="next" href="{{ seo_url }}{{ page.next.url }}" title="{{ page.next.title | escape }}" />
<link rel="next" href="{{ seo_url }}{{ page.next.url | replace:'/index.html','/' }}" title="{{ page.next.title | escape }}" />
{% endif %}
{% if page.previous.url %}
<link rel="prev" href="{{ seo_url }}{{ page.previous.url }}" title="{{ page.previous.title | escape }}" />
<link rel="prev" href="{{ seo_url }}{{ page.previous.url | replace:'/index.html','/' }}" title="{{ page.previous.title | escape }}" />
{% endif %}
<script type="application/ld+json">
{

9
spec/jekyll_seo_tag_spec.rb
View File

@ -77,6 +77,15 @@ describe Jekyll::SeoTag do
expect(subject.render(context)).to match(expected)
end
it "uses replaces '/index.html' with '/'" do
page = page({ "permalink" => "/page/index.html" })
site = site({ "url" => "http://example.invalid" })
context = context({ :page => page, :site => site })
expected = %r!<link rel="canonical" href="http://example.invalid/page/" itemprop="url" />!
expected = %r!<meta property='og:url' content='http://example.invalid/page/' />!
expect(subject.render(context)).to match(expected)
end
it "outputs the site title meta" do
site = site({"title" => "Foo", "url" => "http://example.invalid"})
context = context({ :site => site })