The version management is already starting to get pretty complex, this particular function might be fine but this general practice is probably something we should stay away from:

We have a Versioned locator, and then we start assuming things elsewhere about those locators.  Later when we add new versions those assumptions may stop being true.  For example, that if the tag is a div then we use the class attribute.  We should have these assumptions contained in known places.