Incorrect scope handling in ScopedASTVisitor

[jenrik]

ScopedASTVisitor doesn't open and close scopes at the same point as SymbolChecker does. Notably in the visit function for ConditionalBlock expr is visited outside the scope in SymbolChecker and inside the scope in ScopedASTVisitor.