Skip to content

Commit e6d6d6f

Browse files
staabmclxmstaab
andauthored
simplify SyntaxError Rules (#77)
Co-authored-by: Markus Staab <[email protected]>
1 parent d187412 commit e6d6d6f

File tree

2 files changed

+11
-8
lines changed

2 files changed

+11
-8
lines changed

src/Rules/SyntaxErrorInQueryFunctionRule.php

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -71,19 +71,20 @@ public function processNode(Node $node, Scope $scope): array
7171
}
7272

7373
$args = $node->getArgs();
74-
$errors = [];
7574

7675
$queryReflection = new QueryReflection();
7776
$queryString = $queryReflection->resolveQueryString($args[$queryArgPosition]->value, $scope);
7877
if (null === $queryString) {
79-
return $errors;
78+
return [];
8079
}
8180

8281
$error = $queryReflection->validateQueryString($queryString);
8382
if (null !== $error) {
84-
$errors[] = RuleErrorBuilder::message('Query error: '.$error->getMessage().' ('.$error->getCode().').')->line($node->getLine())->build();
83+
return [
84+
RuleErrorBuilder::message('Query error: '.$error->getMessage().' ('.$error->getCode().').')->line($node->getLine())->build()
85+
];
8586
}
8687

87-
return $errors;
88+
return [];
8889
}
8990
}

src/Rules/SyntaxErrorInQueryMethodRule.php

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -63,19 +63,21 @@ public function processNode(Node $node, Scope $scope): array
6363
}
6464

6565
$args = $node->getArgs();
66-
$errors = [];
6766

6867
$queryReflection = new QueryReflection();
6968
$queryString = $queryReflection->resolveQueryString($args[$queryArgPosition]->value, $scope);
7069
if (null === $queryString) {
71-
return $errors;
70+
return [];
7271
}
7372

7473
$error = $queryReflection->validateQueryString($queryString);
7574
if (null !== $error) {
76-
$errors[] = RuleErrorBuilder::message('Query error: '.$error->getMessage().' ('.$error->getCode().').')->line($node->getLine())->build();
75+
return [
76+
RuleErrorBuilder::message('Query error: '.$error->getMessage().' ('.$error->getCode().').')->line($node->getLine())->build()
77+
];
78+
7779
}
7880

79-
return $errors;
81+
return [];
8082
}
8183
}

0 commit comments

Comments
 (0)