I would rename the parameter so it better reflects which value it is actually holding. Right now it's too vague and when you check you see that it is not an element. So what about aUrlFragment?