Logic issue in DocumentationProcessor

[finestructure]

I noticed this issue while working on the DocRoute changes. We're assigning the canonical url parameter and then testing it against itself:

This is from commit f63d212 in PR #2360

I don't think this is working as intended, @daveverwer , is it?

[daveverwer]

What's wrong with an occasional if true?

I'll figure out what I meant here when I fix up the canonicals on top of the new documentation routing.