hugolib: Simplify Page.Eq

2 files changed, 6 insertions, 12 deletions

diff --git a/hugolib/site_sections.go b/hugolib/site_sections.go
index 25731e9d5..1a42f7808 100644
--- a/hugolib/site_sections.go
+++ b/hugolib/site_sections.go
@@ -112,14 +112,13 @@ func (p *Page) IsAncestor(other interface{}) (bool, error) {
 // Eq returns whether the current page equals the given page.
 // Note that this is more accurate than doing `{{ if eq $page $otherPage }}`
 // since a Page can be embedded in another type.
-func (p *Page) Eq(other interface{}) (bool, error) {
+func (p *Page) Eq(other interface{}) bool {
 	pp, err := unwrapPage(other)
 	if err != nil {
-		// Do not return an error on type mismatch.
-		return false, nil
+		return false
 	}
 
-	return p == pp, nil
+	return p == pp
 }
 
 func unwrapPage(in interface{}) (*Page, error) {
diff --git a/hugolib/site_sections_test.go b/hugolib/site_sections_test.go
index 355b74bb6..37752467b 100644
--- a/hugolib/site_sections_test.go
+++ b/hugolib/site_sections_test.go
@@ -150,14 +150,9 @@ PAG|{{ .Title }}|{{ $sect.InSection . }}
 			assert.NotNil(d)
 			assert.Equal("T41_-1", d.Title)
 
-			equals, err := c.Eq(d)
-			assert.NoError(err)
-			assert.False(equals)
-			equals, err = c.Eq(c)
-			assert.NoError(err)
-			assert.True(equals)
-			_, err = c.Eq("asdf")
-			assert.Error(err)
+			assert.False(c.Eq(d))
+			assert.True(c.Eq(c))
+			assert.False(c.Eq("asdf"))
 
 		}},
 		{"empty3", func(p *Page) {