The rename is to make the syntax look like the code in `getAllLocations`, even though I don't particularly agree with it.

`allCategories` is the class variable that holds the cache so I don't think we want to play with that when we are doing intermediary operations. In `getAllLocations` a locally scoped `allLocs` is used instead.