I will remove the setViewVar() function for now. I was going to propose that ViewVarTrait::set() be renamed to setViewVar(), so left this function in there just to raise the point. But it should be handled in a separate RFC. Especially if this so called upgrade shell thing can refactor it automatically :).