From 80009b427fdc9ccac3d27218e41f0fa378087e87 Mon Sep 17 00:00:00 2001 From: Noah Campbell Date: Tue, 1 Oct 2013 14:42:08 -0700 Subject: [PATCH] Change the order of Apply to be more Unixy Typically the destination is on the left and the src is on the right. --- hugolib/site.go | 7 +++++-- transform/absurl.go | 2 +- transform/chain.go | 4 ++-- transform/nav.go | 2 +- transform/nav_test.go | 4 ++-- transform/post.go | 2 +- transform/posttrans_test.go | 2 +- 7 files changed, 13 insertions(+), 10 deletions(-) diff --git a/hugolib/site.go b/hugolib/site.go index 15e78d223..5119081b8 100644 --- a/hugolib/site.go +++ b/hugolib/site.go @@ -618,10 +618,13 @@ func (s *Site) WritePublic(path string, content io.Reader) (err error) { } if s.Transformer == nil { - s.Transformer = &transform.AbsURL{BaseURL: s.Config.BaseUrl} + s.Transformer = transform.NewChain( + &transform.AbsURL{BaseURL: s.Config.BaseUrl}, + &transform.NavActive{Section: "tbd"}, + ) } final := new(bytes.Buffer) - s.Transformer.Apply(content, final) + s.Transformer.Apply(final, content) return s.Target.Publish(path, final) } diff --git a/transform/absurl.go b/transform/absurl.go index 6e7eb125b..087aba5b1 100644 --- a/transform/absurl.go +++ b/transform/absurl.go @@ -10,7 +10,7 @@ type AbsURL struct { BaseURL string } -func (t *AbsURL) Apply(r io.Reader, w io.Writer) (err error) { +func (t *AbsURL) Apply(w io.Writer, r io.Reader) (err error) { var tr *htmltran.Transformer if tr, err = htmltran.NewFromReader(r); err != nil { diff --git a/transform/chain.go b/transform/chain.go index e408b01d2..b9a865fe9 100644 --- a/transform/chain.go +++ b/transform/chain.go @@ -13,11 +13,11 @@ func NewChain(trs ...Transformer) Transformer { return &chain{transformers: trs} } -func (c *chain) Apply(r io.Reader, w io.Writer) (err error) { +func (c *chain) Apply(w io.Writer, r io.Reader) (err error) { in := r for _, tr := range c.transformers { out := new(bytes.Buffer) - err = tr.Apply(in, out) + err = tr.Apply(out, in) if err != nil { return } diff --git a/transform/nav.go b/transform/nav.go index 2111ae89a..1c9ab1fb6 100644 --- a/transform/nav.go +++ b/transform/nav.go @@ -11,7 +11,7 @@ type NavActive struct { AttrName string } -func (n *NavActive) Apply(r io.Reader, w io.Writer) (err error) { +func (n *NavActive) Apply(w io.Writer, r io.Reader) (err error) { var tr *htmltran.Transformer if n.Section == "" { diff --git a/transform/nav_test.go b/transform/nav_test.go index 78e00f7df..4d578b87a 100644 --- a/transform/nav_test.go +++ b/transform/nav_test.go @@ -37,7 +37,7 @@ func TestDegenerateNoSectionSet(t *testing.T) { out = new(bytes.Buffer) ) - if err := tr.Apply(strings.NewReader(HTML_WITH_NAV), out); err != nil { + if err := tr.Apply(out, strings.NewReader(HTML_WITH_NAV)); err != nil { t.Errorf("Unexpected error in NavActive.Apply: %s", err) } @@ -49,7 +49,7 @@ func TestDegenerateNoSectionSet(t *testing.T) { func TestSetNav(t *testing.T) { tr := &NavActive{Section: "section_2"} out := new(bytes.Buffer) - if err := tr.Apply(strings.NewReader(HTML_WITH_NAV), out); err != nil { + if err := tr.Apply(out, strings.NewReader(HTML_WITH_NAV)); err != nil { t.Errorf("Unexpected error in Apply() for NavActive: %s", err) } diff --git a/transform/post.go b/transform/post.go index a23b48d2e..c9a292b56 100644 --- a/transform/post.go +++ b/transform/post.go @@ -5,5 +5,5 @@ import ( ) type Transformer interface { - Apply(io.Reader, io.Writer) error + Apply(io.Writer, io.Reader) error } diff --git a/transform/posttrans_test.go b/transform/posttrans_test.go index 3cac6e76b..8ec004e87 100644 --- a/transform/posttrans_test.go +++ b/transform/posttrans_test.go @@ -38,7 +38,7 @@ var abs_url_tests = []test { func apply(t *testing.T, tr Transformer, tests []test) { for _, test := range tests { out := new(bytes.Buffer) - err := tr.Apply(strings.NewReader(test.content), out) + err := tr.Apply(out, strings.NewReader(test.content)) if err != nil { t.Errorf("Unexpected error: %s", err) }