[Php70] Handle return ternary on IfIssetToCoalescingRector (#7683)

* [Php70] Handle return ternary on IfIssetToCoalescingRector

* fix

Author: samsonasik

rules-tests/Php70/Rector/StmtsAwareInterface/IfIssetToCoalescingRector/Fixture/with_ternary_return.php.inc

@@ -0,0 +1,37 @@
+<?php
+
+namespace Rector\Tests\Php70\Rector\StmtsAwareInterface\IfIssetToCoalescingRector\Fixture;
+
+class WithTernaryReturn
+{
+    private $values = [];
+    private $nullable = false;
+
+    public function resolve($param)
+    {
+        if (isset($this->values[$param])) {
+            return $this->values[$param];
+        }
+
+        return $this->nullable ? null : false;
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\Php70\Rector\StmtsAwareInterface\IfIssetToCoalescingRector\Fixture;
+
+class WithTernaryReturn
+{
+    private $values = [];
+    private $nullable = false;
+
+    public function resolve($param)
+    {
+        return $this->values[$param] ?? ($this->nullable ? null : false);
+    }
+}
+
+?>

rules/Php70/Rector/StmtsAwareInterface/IfIssetToCoalescingRector.php

@@ -8,10 +8,12 @@
 use PhpParser\Node\Expr;
 use PhpParser\Node\Expr\BinaryOp\Coalesce;
 use PhpParser\Node\Expr\Isset_;
+use PhpParser\Node\Expr\Ternary;
 use PhpParser\Node\Stmt;
 use PhpParser\Node\Stmt\Else_;
 use PhpParser\Node\Stmt\If_;
 use PhpParser\Node\Stmt\Return_;
+use Rector\NodeTypeResolver\Node\AttributeKey;
 use Rector\PhpParser\Enum\NodeGroup;
 use Rector\Rector\AbstractRector;
 use Rector\ValueObject\PhpVersionFeature;
@@ -113,6 +115,10 @@ public function refactor(Node $node): ?Node
 
             unset($node->stmts[$key - 1]);
 
+            if ($stmt->expr instanceof Ternary) {
+                $stmt->expr->setAttribute(AttributeKey::WRAPPED_IN_PARENTHESES, true);
+            }
+
             $stmt->expr = new Coalesce($ifOnlyStmt->expr, $stmt->expr);
             return $node;
         }