Add Page method/field to taxonomy object

[jmooring]

We can get a reference to a term page with:

{{ $termPage := site.Taxonomies.tags.foo.Page }}

It would be convenient if we could get a reference to a taxonomy page with:

{{ $taxonomyPage := site.Taxonomies.tags.Page }}

That may not be possible, despite the fact we lowercase terms internally (e.g., there shouldn't be a collision between tags.page (a term) and tags.Page (a method/field).

The alternative has always seemed less elegant compared to how you can get a reference to a term page:

{{ site.GetPage "/tags" }}

[bep]

This is what site.Taxonomies.tags gets you:

type Taxonomy map[string]WeightedPages

• We cannot change the type (or: not until we get some better "struct iterator" support in Go templates)
• We cannot add state other than WeightedPages (e.g. tags)

What we in theory could do, but that would not work for empty taxonomies:

func (t Taxonomy) Page() page.Page {
   var first WeightedPages
   range _, v := t {
        first = v
        break
    }
    if first == nil {
         return nil
     }
     return first.Page().Parent()
}

[jmooring]

I tested with this:

// Page returns the taxonomy page or nil if the taxonomy has no terms.
func (i Taxonomy) Page() Page {
	var first WeightedPages
	for _, v := range i {
		first = v
		break
	}
	if first == nil {
		return nil
	}
	return first.Page().Parent()
}

and this:

{{ range $taxonomy, $terms := site.Taxonomies }}
  {{ with $terms }}
    <h2><a href="{{ .Page.RelPermalink }}">{{ .Page.LinkTitle }}</a></h2>
    {{ range $term, $weightedPages := . }}
      <h3><a href="{{ .Page.RelPermalink }}">{{ .Page.LinkTitle }}</a></h3>
      {{ range $weightedPages }}
        <a href="{{ .RelPermalink }}">{{ .LinkTitle }}</a>
      {{ end }}
    {{ end }}
  {{ end }}
{{ end }}

Works great.

I think this is OK as long as we document the "returns nil if no term pages" part.

And it's kind of sneaky, so I like it.

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.