Skip to content

Commit d827907

Browse files
committed
Reaching PHPStan level 6
1 parent e5b5f7d commit d827907

22 files changed

+37
-36
lines changed

phpstan.neon

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
parameters:
2-
level: 5
2+
level: 6
33
inferPrivatePropertyTypeFromConstructor: true
44
ignoreErrors:
55
- "#Mouf\\\\MoufManager#"
6-
- "#Mouf\\\\MoufInstanceDescriptor#"
6+
- "#Mouf\\\\MoufInstanceDescriptor#"
7+
- '#Method Mouf\\Database\\QueryWriter\\Utils\\FindParametersService::findParameters\(\) should return array<string> but returns array<int, int\|string>#'

src/SQLParser/Node/AbstractManyInstancesOperator.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ public function toInstanceDescriptor(MoufManager $moufManager)
6464
* @param bool $extrapolateParameters
6565
* @return string
6666
*/
67-
public function toSql(array $parameters, AbstractPlatform $platform, $indent = 0, $conditionsMode = self::CONDITION_APPLY, bool $extrapolateParameters = true): ?string
67+
public function toSql(array $parameters, AbstractPlatform $platform, int $indent = 0, $conditionsMode = self::CONDITION_APPLY, bool $extrapolateParameters = true): ?string
6868
{
6969
$sqlOperands = array();
7070
foreach ($this->operands as $operand) {

src/SQLParser/Node/AbstractTwoOperandsOperator.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ public function toInstanceDescriptor(MoufManager $moufManager)
9797
* @param bool $extrapolateParameters
9898
* @return string
9999
*/
100-
public function toSql(array $parameters, AbstractPlatform $platform, $indent = 0, $conditionsMode = self::CONDITION_APPLY, bool $extrapolateParameters = true): ?string
100+
public function toSql(array $parameters, AbstractPlatform $platform, int $indent = 0, $conditionsMode = self::CONDITION_APPLY, bool $extrapolateParameters = true): ?string
101101
{
102102
if ($conditionsMode == self::CONDITION_GUESS) {
103103
$bypass = false;

src/SQLParser/Node/AggregateFunction.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,7 @@ public function toInstanceDescriptor(MoufManager $moufManager)
152152
* @param bool $extrapolateParameters
153153
* @return string
154154
*/
155-
public function toSql(array $parameters, AbstractPlatform $platform, $indent = 0, $conditionsMode = self::CONDITION_APPLY, bool $extrapolateParameters = true): ?string
155+
public function toSql(array $parameters, AbstractPlatform $platform, int $indent = 0, $conditionsMode = self::CONDITION_APPLY, bool $extrapolateParameters = true): ?string
156156
{
157157
$subTreeSql = NodeFactory::toSql($this->subTree, $platform, $parameters, ', ', false, $indent, $conditionsMode, $extrapolateParameters);
158158
if ($subTreeSql !== null) {

src/SQLParser/Node/Between.php

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -37,43 +37,43 @@ public function setLeftOperand($leftOperand)
3737
}
3838

3939
/**
40-
* @var string|NodeInterface|NodeInterface[]
40+
* @var NodeInterface
4141
*/
4242
private $minValueOperand;
4343

4444
/**
45-
* @var string|NodeInterface|NodeInterface[]
45+
* @var NodeInterface
4646
*/
4747
private $maxValueOperand;
4848

4949
/**
50-
* @return NodeInterface|NodeInterface[]|string
50+
* @return NodeInterface
5151
*/
5252
public function getMinValueOperand()
5353
{
5454
return $this->minValueOperand;
5555
}
5656

5757
/**
58-
* @param NodeInterface|NodeInterface[]|string $minValueOperand
58+
* @param NodeInterface $minValueOperand
5959
*/
60-
public function setMinValueOperand($minValueOperand)
60+
public function setMinValueOperand(NodeInterface $minValueOperand)
6161
{
6262
$this->minValueOperand = $minValueOperand;
6363
}
6464

6565
/**
66-
* @return NodeInterface|NodeInterface[]|string
66+
* @return NodeInterface
6767
*/
6868
public function getMaxValueOperand()
6969
{
7070
return $this->maxValueOperand;
7171
}
7272

7373
/**
74-
* @param NodeInterface|NodeInterface[]|string $maxValueOperand
74+
* @param NodeInterface $maxValueOperand
7575
*/
76-
public function setMaxValueOperand($maxValueOperand)
76+
public function setMaxValueOperand(NodeInterface $maxValueOperand)
7777
{
7878
$this->maxValueOperand = $maxValueOperand;
7979
}
@@ -154,7 +154,7 @@ public function toInstanceDescriptor(MoufManager $moufManager)
154154
* @param bool $extrapolateParameters
155155
* @return string
156156
*/
157-
public function toSql(array $parameters, AbstractPlatform $platform, $indent = 0, $conditionsMode = self::CONDITION_APPLY, bool $extrapolateParameters = true): ?string
157+
public function toSql(array $parameters, AbstractPlatform $platform, int $indent = 0, $conditionsMode = self::CONDITION_APPLY, bool $extrapolateParameters = true): ?string
158158
{
159159
$minBypass = false;
160160
$maxBypass = false;

src/SQLParser/Node/CaseOperation.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ public function toInstanceDescriptor(MoufManager $moufManager)
6060
* @param bool $extrapolateParameters
6161
* @return string
6262
*/
63-
public function toSql(array $parameters, AbstractPlatform $platform, $indent = 0, $conditionsMode = self::CONDITION_APPLY, bool $extrapolateParameters = true): ?string
63+
public function toSql(array $parameters, AbstractPlatform $platform, int $indent = 0, $conditionsMode = self::CONDITION_APPLY, bool $extrapolateParameters = true): ?string
6464
{
6565
$sql = 'CASE '.NodeFactory::toSql($this->operation, $platform, $parameters, ' ', false, $indent, $conditionsMode).' END';
6666

src/SQLParser/Node/ColRef.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -192,7 +192,7 @@ public function toInstanceDescriptor(MoufManager $moufManager)
192192
* @param bool $extrapolateParameters
193193
* @return string
194194
*/
195-
public function toSql(array $parameters, AbstractPlatform $platform, $indent = 0, $conditionsMode = self::CONDITION_APPLY, bool $extrapolateParameters = true): ?string
195+
public function toSql(array $parameters, AbstractPlatform $platform, int $indent = 0, $conditionsMode = self::CONDITION_APPLY, bool $extrapolateParameters = true): ?string
196196
{
197197
$sql = '';
198198
if ($this->database) {

src/SQLParser/Node/ConstNode.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ public function toInstanceDescriptor(MoufManager $moufManager)
106106
* @param bool $extrapolateParameters
107107
* @return string
108108
*/
109-
public function toSql(array $parameters, AbstractPlatform $platform, $indent = 0, $conditionsMode = self::CONDITION_APPLY, bool $extrapolateParameters = true): ?string
109+
public function toSql(array $parameters, AbstractPlatform $platform, int $indent = 0, $conditionsMode = self::CONDITION_APPLY, bool $extrapolateParameters = true): ?string
110110
{
111111
if ($this->value === null) {
112112
return 'NULL';

src/SQLParser/Node/Expression.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -203,7 +203,7 @@ public function toInstanceDescriptor(MoufManager $moufManager)
203203
* @param bool $extrapolateParameters
204204
* @return string|null
205205
*/
206-
public function toSql(array $parameters, AbstractPlatform $platform, $indent = 0, $conditionsMode = self::CONDITION_APPLY, bool $extrapolateParameters = true): ?string
206+
public function toSql(array $parameters, AbstractPlatform $platform, int $indent = 0, $conditionsMode = self::CONDITION_APPLY, bool $extrapolateParameters = true): ?string
207207
{
208208
if (empty($this->subTree)) {
209209
return $this->getBaseExpression();

src/SQLParser/Node/LimitNode.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -83,15 +83,15 @@ public function toInstanceDescriptor(MoufManager $moufManager)
8383
*
8484
* @param array $parameters
8585
* @param AbstractPlatform $platform
86-
* @param int|number $indent
86+
* @param int $indent
8787
* @param int $conditionsMode
8888
*
8989
* @param bool $extrapolateParameters
9090
* @return string
9191
*
9292
* @throws \Exception
9393
*/
94-
public function toSql(array $parameters, AbstractPlatform $platform, $indent = 0, $conditionsMode = self::CONDITION_APPLY, bool $extrapolateParameters = true): ?string
94+
public function toSql(array $parameters, AbstractPlatform $platform, int $indent = 0, $conditionsMode = self::CONDITION_APPLY, bool $extrapolateParameters = true): ?string
9595
{
9696
if ($this->value === null) {
9797
throw new \Exception('A limit parameter must be an integer');

0 commit comments

Comments
 (0)