[Php71] Skip magic const __LINE__ on BinaryOpBetweenNumberAndStringRe…

…ctor (#200)
rectorphp · Jun 10, 2021 · 932458f · 932458f
1 parent 8675bcb
commit 932458f
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/...ector/BinaryOp/BinaryOpBetweenNumberAndStringRector/Fixture/skip_magic_const_line.php.inc b/...ector/BinaryOp/BinaryOpBetweenNumberAndStringRector/Fixture/skip_magic_const_line.php.inc
@@ -0,0 +1,12 @@
+<?php
+
+namespace Rector\Tests\Php71\Rector\BinaryOp\BinaryOpBetweenNumberAndStringRector\Fixture;
+
+class SkipMagicConstLine
+{
+    public function run()
+    {
+        $value = __LINE__ - 1;
+        $value = 1 - __LINE__;
+    }
+}
diff --git a/rules/Php71/Rector/BinaryOp/BinaryOpBetweenNumberAndStringRector.php b/rules/Php71/Rector/BinaryOp/BinaryOpBetweenNumberAndStringRector.php
@@ -12,6 +12,7 @@
 use PhpParser\Node\Expr\Variable;
 use PhpParser\Node\Scalar;
 use PhpParser\Node\Scalar\LNumber;
+use PhpParser\Node\Scalar\MagicConst\Line;
 use PhpParser\Node\Scalar\String_;
 use PHPStan\Type\Constant\ConstantStringType;
 use Rector\Core\Rector\AbstractRector;
@@ -102,6 +103,10 @@ private function isStringOrStaticNonNumbericString(Expr $expr): bool
             return false;
         }
 
+        if ($expr instanceof Line) {
+            return false;
+        }
+
         $value = null;
         $exprStaticType = $this->getStaticType($expr);