produceRedirects should accept an array of aliases instead of the content catalog
To make the produceRedirects
function consistent / parallel with mapSite
, the second parameter should be an array of aliases (virtual files in the alias family) instead of the content catalog. This change will also make the function slightly more reusable. Currently, the function immediately retrieves the aliases from the content catalog and never uses it again. The content catalog abstraction here is largely unnecessary. If a custom implementation happens to need the content catalog, it can always retrieve it from the context (i.e., this.getVariables().contentCatalog
).
For backwards compatibility, the function should detect that the content catalog has been passed into this position and use it as it does today. Otherwise, it should assume that it has been passed an array of aliases.