Avoid doing work before its necessary in mutating scope.

Author: mad-briller

Diff for: src/Analyser/MutatingScope.php

@@ -1119,11 +1119,11 @@ private function resolveType(string $exprString, Expr $node): Type
 					return new ObjectType(Closure::class);
 				}
 
-				$classType = $this->resolveTypeByName($node->class);
 				if (!$node->name instanceof Node\Identifier) {
 					return new ObjectType(Closure::class);
 				}
 
+				$classType = $this->resolveTypeByName($node->class);
 				$methodName = $node->name->toString();
 				if (!$classType->hasMethod($methodName)->yes()) {
 					return new ObjectType(Closure::class);
@@ -2080,12 +2080,13 @@ private function issetCheckUndefined(Expr $expr): ?bool
 
 		if ($expr instanceof Node\Expr\ArrayDimFetch && $expr->dim !== null) {
 			$type = $this->getType($expr->var);
-			$dimType = $this->getType($expr->dim);
-			$hasOffsetValue = $type->hasOffsetValueType($dimType);
 			if (!$type->isOffsetAccessible()->yes()) {
 				return $this->issetCheckUndefined($expr->var);
 			}
 
+			$dimType = $this->getType($expr->dim);
+			$hasOffsetValue = $type->hasOffsetValueType($dimType);
+
 			if (!$hasOffsetValue->no()) {
 				return $this->issetCheckUndefined($expr->var);
 			}