Allow indexPagePath to be an array

[nezoomie]

Allows indexPagePath to be a string or an array:

indexPagePath: ["tags", "en/tags", "de/tags", "it/tags"]

This allows the generation of multiple, filtered localized tags pages using some logic in the tags' layout.

[rantecki]

Hi there. Interesting change. Having multiple tag index pages may have some implications for @getTagUrl() though (and consequently the tag cloud) as there are now multiple options. It looks as though @getTagUrl() will currently only return the index page from the first path in the list, and there is currently no mechanism to support finding the other pages. Perhaps additional optional query parameters could be passed to @getTagUrl()? What do you think?

[nezoomie]

good point. I think we can go a better way then. What do you think about:

  indexPagePath: 'tags',
  basePath: 'blog/',    // optional. Defaults to '/'
  context: ['en','de','it']     // string or array. Root context included by default

then using an optional context query:

  @getTagUrl({context: 'en'})

I'd throw in another problem which I'm currently facing, the tagCloud. Usign contexts, that may be solved all together using metadata:

---
title: 'My blog post'

tagsContext:
  - en

tags:
  - some
  - tags
---

then:

  @getTagCloud({context: 'en'})

as it comes almost for free, I'd support context to be an array as well:

  @getTagUrl({context: ['en','de']})
  @getTagCloud({context: ['en','de','it']})

[nezoomie]

Applied some changes.

• Dropped the basePath idea (didn't see the need for it)
• Dropped multiple indexPagePath in favor of an optional context array:

 {
    indexPagePath: 'blog/tags',
    context: ['en','de']
  }

• A context array can be specified in the document. If not available, the document will be available for context-less queries only
• @getTagUrl() now takes an optional context query:

@getTagUrl('news', context: 'en')

• @getTagCloud now takes an optional context query. Count and weighting are relative to the context:

@getTagCloud(context: 'en')

[rantecki]

Looks pretty good, thanks. Will review and merge in a couple of days. I'm a bit slammed at the moment!

[rantecki]

Just letting you know I haven't forgotten about this. I went to write some tests a while ago to cover the additional functionality, but I ran into some problems. Will look into it again in the next couple of days hopefully.

[nezoomie]

no problem 👍