CS: Apply ternary_to_null_coalescing fixer

derrabus · derrabus · commit 2455a111174c · 2020-12-27T00:49:32.000+01:00
diff --git a/DataCollector/ConfigDataCollector.php b/DataCollector/ConfigDataCollector.php
@@ -219,7 +219,7 @@ public function getPhpVersion()
      */
     public function getPhpVersionExtra()
     {
-        return isset($this->data['php_version_extra']) ? $this->data['php_version_extra'] : null;
+        return $this->data['php_version_extra'] ?? null;
     }
 
     /**
diff --git a/DataCollector/LoggerDataCollector.php b/DataCollector/LoggerDataCollector.php
@@ -81,32 +81,32 @@ public function lateCollect()
 
     public function getLogs()
     {
-        return isset($this->data['logs']) ? $this->data['logs'] : [];
+        return $this->data['logs'] ?? [];
     }
 
     public function getPriorities()
     {
-        return isset($this->data['priorities']) ? $this->data['priorities'] : [];
+        return $this->data['priorities'] ?? [];
     }
 
     public function countErrors()
     {
-        return isset($this->data['error_count']) ? $this->data['error_count'] : 0;
+        return $this->data['error_count'] ?? 0;
     }
 
     public function countDeprecations()
     {
-        return isset($this->data['deprecation_count']) ? $this->data['deprecation_count'] : 0;
+        return $this->data['deprecation_count'] ?? 0;
     }
 
     public function countWarnings()
     {
-        return isset($this->data['warning_count']) ? $this->data['warning_count'] : 0;
+        return $this->data['warning_count'] ?? 0;
     }
 
     public function countScreams()
     {
-        return isset($this->data['scream_count']) ? $this->data['scream_count'] : 0;
+        return $this->data['scream_count'] ?? 0;
     }
 
     public function getCompilerLogs()
diff --git a/DataCollector/RequestDataCollector.php b/DataCollector/RequestDataCollector.php
@@ -88,7 +88,7 @@ public function collect(Request $request, Response $response/*, \Throwable $exce
             'format' => $request->getRequestFormat(),
             'content' => $content,
             'content_type' => $response->headers->get('Content-Type', 'text/html'),
-            'status_text' => isset(Response::$statusTexts[$statusCode]) ? Response::$statusTexts[$statusCode] : '',
+            'status_text' => Response::$statusTexts[$statusCode] ?? '',
             'status_code' => $statusCode,
             'request_query' => $request->query->all(),
             'request_request' => $request->request->all(),
@@ -339,12 +339,12 @@ public function getController()
      */
     public function getRedirect()
     {
-        return isset($this->data['redirect']) ? $this->data['redirect'] : false;
+        return $this->data['redirect'] ?? false;
     }
 
     public function getForwardToken()
     {
-        return isset($this->data['forward_token']) ? $this->data['forward_token'] : null;
+        return $this->data['forward_token'] ?? null;
     }
 
     /**
diff --git a/EventListener/RouterListener.php b/EventListener/RouterListener.php
@@ -116,7 +116,7 @@ public function onKernelRequest(GetResponseEvent $event)
 
             if (null !== $this->logger) {
                 $this->logger->info('Matched route "{route}".', [
-                    'route' => isset($parameters['_route']) ? $parameters['_route'] : 'n/a',
+                    'route' => $parameters['_route'] ?? 'n/a',
                     'route_parameters' => $parameters,
                     'request_uri' => $request->getUri(),
                     'method' => $request->getMethod(),
diff --git a/Fragment/AbstractSurrogateFragmentRenderer.php b/Fragment/AbstractSurrogateFragmentRenderer.php
@@ -71,12 +71,12 @@ public function render($uri, Request $request, array $options = [])
             $uri = $this->generateSignedFragmentUri($uri, $request);
         }
 
-        $alt = isset($options['alt']) ? $options['alt'] : null;
+        $alt = $options['alt'] ?? null;
         if ($alt instanceof ControllerReference) {
             $alt = $this->generateSignedFragmentUri($alt, $request);
         }
 
-        $tag = $this->surrogate->renderIncludeTag($uri, $alt, isset($options['ignore_errors']) ? $options['ignore_errors'] : false, isset($options['comment']) ? $options['comment'] : '');
+        $tag = $this->surrogate->renderIncludeTag($uri, $alt, $options['ignore_errors'] ?? false, $options['comment'] ?? '');
 
         return new Response($tag);
     }
diff --git a/Fragment/HIncludeFragmentRenderer.php b/Fragment/HIncludeFragmentRenderer.php
@@ -100,7 +100,7 @@ public function render($uri, Request $request, array $options = [])
         // We need to replace ampersands in the URI with the encoded form in order to return valid html/xml content.
         $uri = str_replace('&', '&amp;', $uri);
 
-        $template = isset($options['default']) ? $options['default'] : $this->globalDefaultTemplate;
+        $template = $options['default'] ?? $this->globalDefaultTemplate;
         if (null !== $this->templating && $template && $this->templateExists($template)) {
             $content = $this->templating->render($template);
         } else {
diff --git a/HttpCache/Esi.php b/HttpCache/Esi.php
@@ -97,7 +97,7 @@ public function process(Request $request, Response $response)
 
             $chunks[$i] = sprintf('<?php echo $this->surrogate->handle($this, %s, %s, %s) ?>'."\n",
                 var_export($options['src'], true),
-                var_export(isset($options['alt']) ? $options['alt'] : '', true),
+                var_export($options['alt'] ?? '', true),
                 isset($options['onerror']) && 'continue' === $options['onerror'] ? 'true' : 'false'
             );
             ++$i;
diff --git a/HttpCache/Store.php b/HttpCache/Store.php
@@ -277,8 +277,8 @@ private function requestsMatch(?string $vary, array $env1, array $env2): bool
 
         foreach (preg_split('/[\s,]+/', $vary) as $header) {
             $key = str_replace('_', '-', strtolower($header));
-            $v1 = isset($env1[$key]) ? $env1[$key] : null;
-            $v2 = isset($env2[$key]) ? $env2[$key] : null;
+            $v1 = $env1[$key] ?? null;
+            $v2 = $env2[$key] ?? null;
             if ($v1 !== $v2) {
                 return false;
             }
diff --git a/Log/Logger.php b/Log/Logger.php
@@ -43,7 +43,7 @@ public function __construct(string $minLevel = null, $output = null, callable $f
             $minLevel = null === $output || 'php://stdout' === $output || 'php://stderr' === $output ? LogLevel::ERROR : LogLevel::WARNING;
 
             if (isset($_ENV['SHELL_VERBOSITY']) || isset($_SERVER['SHELL_VERBOSITY'])) {
-                switch ((int) (isset($_ENV['SHELL_VERBOSITY']) ? $_ENV['SHELL_VERBOSITY'] : $_SERVER['SHELL_VERBOSITY'])) {
+                switch ((int) ($_ENV['SHELL_VERBOSITY'] ?? $_SERVER['SHELL_VERBOSITY'])) {
                     case -1: $minLevel = LogLevel::ERROR; break;
                     case 1: $minLevel = LogLevel::NOTICE; break;
                     case 2: $minLevel = LogLevel::INFO; break;
diff --git a/UriSigner.php b/UriSigner.php
@@ -93,12 +93,12 @@ private function buildUrl(array $url, array $params = []): string
         $url['query'] = http_build_query($params, '', '&');
 
         $scheme = isset($url['scheme']) ? $url['scheme'].'://' : '';
-        $host = isset($url['host']) ? $url['host'] : '';
+        $host = $url['host'] ?? '';
         $port = isset($url['port']) ? ':'.$url['port'] : '';
-        $user = isset($url['user']) ? $url['user'] : '';
+        $user = $url['user'] ?? '';
         $pass = isset($url['pass']) ? ':'.$url['pass'] : '';
         $pass = ($user || $pass) ? "$pass@" : '';
-        $path = isset($url['path']) ? $url['path'] : '';
+        $path = $url['path'] ?? '';
         $query = isset($url['query']) && $url['query'] ? '?'.$url['query'] : '';
         $fragment = isset($url['fragment']) ? '#'.$url['fragment'] : '';
 

Original file line number	Diff line number	Diff line change
`@@ -219,7 +219,7 @@ public function getPhpVersion()`
`219`	`219`	`*/`
`220`	`220`	`public function getPhpVersionExtra()`
`221`	`221`	`{`
`222`		`- return isset($this->data['php_version_extra']) ? $this->data['php_version_extra'] : null;`
	`222`	`+ return $this->data['php_version_extra'] ?? null;`
`223`	`223`	`}`
`224`	`224`
`225`	`225`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -81,32 +81,32 @@ public function lateCollect()`
`81`	`81`
`82`	`82`	`public function getLogs()`
`83`	`83`	`{`
`84`		`- return isset($this->data['logs']) ? $this->data['logs'] : [];`
	`84`	`+ return $this->data['logs'] ?? [];`
`85`	`85`	`}`
`86`	`86`
`87`	`87`	`public function getPriorities()`
`88`	`88`	`{`
`89`		`- return isset($this->data['priorities']) ? $this->data['priorities'] : [];`
	`89`	`+ return $this->data['priorities'] ?? [];`
`90`	`90`	`}`
`91`	`91`
`92`	`92`	`public function countErrors()`
`93`	`93`	`{`
`94`		`- return isset($this->data['error_count']) ? $this->data['error_count'] : 0;`
	`94`	`+ return $this->data['error_count'] ?? 0;`
`95`	`95`	`}`
`96`	`96`
`97`	`97`	`public function countDeprecations()`
`98`	`98`	`{`
`99`		`- return isset($this->data['deprecation_count']) ? $this->data['deprecation_count'] : 0;`
	`99`	`+ return $this->data['deprecation_count'] ?? 0;`
`100`	`100`	`}`
`101`	`101`
`102`	`102`	`public function countWarnings()`
`103`	`103`	`{`
`104`		`- return isset($this->data['warning_count']) ? $this->data['warning_count'] : 0;`
	`104`	`+ return $this->data['warning_count'] ?? 0;`
`105`	`105`	`}`
`106`	`106`
`107`	`107`	`public function countScreams()`
`108`	`108`	`{`
`109`		`- return isset($this->data['scream_count']) ? $this->data['scream_count'] : 0;`
	`109`	`+ return $this->data['scream_count'] ?? 0;`
`110`	`110`	`}`
`111`	`111`
`112`	`112`	`public function getCompilerLogs()`
Original file line number	Diff line number	Diff line change
`@@ -88,7 +88,7 @@ public function collect(Request $request, Response $response/*, \Throwable $exce`
`88`	`88`	`'format' => $request->getRequestFormat(),`
`89`	`89`	`'content' => $content,`
`90`	`90`	`'content_type' => $response->headers->get('Content-Type', 'text/html'),`
`91`		`- 'status_text' => isset(Response::$statusTexts[$statusCode]) ? Response::$statusTexts[$statusCode] : '',`
	`91`	`+ 'status_text' => Response::$statusTexts[$statusCode] ?? '',`
`92`	`92`	`'status_code' => $statusCode,`
`93`	`93`	`'request_query' => $request->query->all(),`
`94`	`94`	`'request_request' => $request->request->all(),`
`@@ -339,12 +339,12 @@ public function getController()`
`339`	`339`	`*/`
`340`	`340`	`public function getRedirect()`
`341`	`341`	`{`
`342`		`- return isset($this->data['redirect']) ? $this->data['redirect'] : false;`
	`342`	`+ return $this->data['redirect'] ?? false;`
`343`	`343`	`}`
`344`	`344`
`345`	`345`	`public function getForwardToken()`
`346`	`346`	`{`
`347`		`- return isset($this->data['forward_token']) ? $this->data['forward_token'] : null;`
	`347`	`+ return $this->data['forward_token'] ?? null;`
`348`	`348`	`}`
`349`	`349`
`350`	`350`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -71,12 +71,12 @@ public function render($uri, Request $request, array $options = [])`
`71`	`71`	`$uri = $this->generateSignedFragmentUri($uri, $request);`
`72`	`72`	`}`
`73`	`73`
`74`		`- $alt = isset($options['alt']) ? $options['alt'] : null;`
	`74`	`+ $alt = $options['alt'] ?? null;`
`75`	`75`	`if ($alt instanceof ControllerReference) {`
`76`	`76`	`$alt = $this->generateSignedFragmentUri($alt, $request);`
`77`	`77`	`}`
`78`	`78`
`79`		`- $tag = $this->surrogate->renderIncludeTag($uri, $alt, isset($options['ignore_errors']) ? $options['ignore_errors'] : false, isset($options['comment']) ? $options['comment'] : '');`
	`79`	`+ $tag = $this->surrogate->renderIncludeTag($uri, $alt, $options['ignore_errors'] ?? false, $options['comment'] ?? '');`
`80`	`80`
`81`	`81`	`return new Response($tag);`
`82`	`82`	`}`