Skip to content

Commit 0b64c71

Browse files
nicolas-grekasnicolas-grekas
committed
Merge branch '4.1'
* 4.1: Fix Clidumper tests Enable the fixer enforcing fully-qualified calls for compiler-optimized functions Apply fixers Disable the native_constant_invocation fixer until it can be scoped Update the list of excluded files for the CS fixer
2 parents 4a493bb + 84756c9 commit 0b64c71

16 files changed

+41
-41
lines changed

‎AppVariable.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ public function getUser()
8383
}
8484

8585
$user = $token->getUser();
86-
if (is_object($user)) {
86+
if (\is_object($user)) {
8787
return $user;
8888
}
8989
}
@@ -169,7 +169,7 @@ public function getFlashes($types = null)
169169
return $session->getFlashBag()->all();
170170
}
171171

172-
if (is_string($types)) {
172+
if (\is_string($types)) {
173173
return $session->getFlashBag()->get($types);
174174
}
175175

‎Command/DebugCommand.php

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -121,15 +121,15 @@ protected function execute(InputInterface $input, OutputInterface $output)
121121
$firstNamespace = true;
122122
$prevHasSeparator = false;
123123
foreach ($this->getLoaderPaths() as $namespace => $paths) {
124-
if (!$firstNamespace && !$prevHasSeparator && count($paths) > 1) {
124+
if (!$firstNamespace && !$prevHasSeparator && \count($paths) > 1) {
125125
$rows[] = array('', '');
126126
}
127127
$firstNamespace = false;
128128
foreach ($paths as $path) {
129129
$rows[] = array($namespace, $path.DIRECTORY_SEPARATOR);
130130
$namespace = '';
131131
}
132-
if (count($paths) > 1) {
132+
if (\count($paths) > 1) {
133133
$rows[] = array('', '');
134134
$prevHasSeparator = true;
135135
} else {
@@ -159,7 +159,7 @@ private function getLoaderPaths()
159159
foreach ($loader->getNamespaces() as $namespace) {
160160
$paths = array_map(function ($path) {
161161
if (null !== $this->projectDir && 0 === strpos($path, $this->projectDir)) {
162-
$path = ltrim(substr($path, strlen($this->projectDir)), DIRECTORY_SEPARATOR);
162+
$path = ltrim(substr($path, \strlen($this->projectDir)), DIRECTORY_SEPARATOR);
163163
}
164164

165165
return $path;
@@ -190,16 +190,16 @@ private function getMetadata($type, $entity)
190190
if (null === $cb) {
191191
return;
192192
}
193-
if (is_array($cb)) {
193+
if (\is_array($cb)) {
194194
if (!method_exists($cb[0], $cb[1])) {
195195
return;
196196
}
197197
$refl = new \ReflectionMethod($cb[0], $cb[1]);
198-
} elseif (is_object($cb) && method_exists($cb, '__invoke')) {
198+
} elseif (\is_object($cb) && method_exists($cb, '__invoke')) {
199199
$refl = new \ReflectionMethod($cb, '__invoke');
200-
} elseif (function_exists($cb)) {
200+
} elseif (\function_exists($cb)) {
201201
$refl = new \ReflectionFunction($cb);
202-
} elseif (is_string($cb) && preg_match('{^(.+)::(.+)$}', $cb, $m) && method_exists($m[1], $m[2])) {
202+
} elseif (\is_string($cb) && preg_match('{^(.+)::(.+)$}', $cb, $m) && method_exists($m[1], $m[2])) {
203203
$refl = new \ReflectionMethod($m[1], $m[2]);
204204
} else {
205205
throw new \UnexpectedValueException('Unsupported callback type');
@@ -249,8 +249,8 @@ private function getPrettyMetadata($type, $entity)
249249
}
250250

251251
if ('globals' === $type) {
252-
if (is_object($meta)) {
253-
return ' = object('.get_class($meta).')';
252+
if (\is_object($meta)) {
253+
return ' = object('.\get_class($meta).')';
254254
}
255255

256256
return ' = '.substr(@json_encode($meta), 0, 50);

‎Command/LintCommand.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
7777
$io = new SymfonyStyle($input, $output);
7878
$filenames = $input->getArgument('filename');
7979

80-
if (0 === count($filenames)) {
80+
if (0 === \count($filenames)) {
8181
if (0 !== ftell(STDIN)) {
8282
throw new RuntimeException('Please provide a filename or pipe template content to STDIN.');
8383
}
@@ -162,9 +162,9 @@ private function displayTxt(OutputInterface $output, SymfonyStyle $io, $filesInf
162162
}
163163

164164
if (0 === $errors) {
165-
$io->success(sprintf('All %d Twig files contain valid syntax.', count($filesInfo)));
165+
$io->success(sprintf('All %d Twig files contain valid syntax.', \count($filesInfo)));
166166
} else {
167-
$io->warning(sprintf('%d Twig files have valid syntax and %d contain errors.', count($filesInfo) - $errors, $errors));
167+
$io->warning(sprintf('%d Twig files have valid syntax and %d contain errors.', \count($filesInfo) - $errors, $errors));
168168
}
169169

170170
return min($errors, 1);
@@ -217,7 +217,7 @@ private function getContext($template, $line, $context = 3)
217217
$lines = explode("\n", $template);
218218

219219
$position = max(0, $line - $context);
220-
$max = min(count($lines), $line - 1 + $context);
220+
$max = min(\count($lines), $line - 1 + $context);
221221

222222
$result = array();
223223
while ($position < $max) {

‎Extension/CodeExtension.php

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ class CodeExtension extends AbstractExtension
3535
public function __construct($fileLinkFormat, string $rootDir, string $charset, string $projectDir = null)
3636
{
3737
$this->fileLinkFormat = $fileLinkFormat ?: ini_get('xdebug.file_link_format') ?: get_cfg_var('xdebug.file_link_format');
38-
$this->rootDir = str_replace('/', DIRECTORY_SEPARATOR, dirname($rootDir)).DIRECTORY_SEPARATOR;
38+
$this->rootDir = str_replace('/', DIRECTORY_SEPARATOR, \dirname($rootDir)).DIRECTORY_SEPARATOR;
3939
$this->charset = $charset;
4040
$this->projectDir = $projectDir;
4141
}
@@ -97,7 +97,7 @@ public function formatArgs($args)
9797
$short = array_pop($parts);
9898
$formattedValue = sprintf('<em>object</em>(<abbr title="%s">%s</abbr>)', $item[1], $short);
9999
} elseif ('array' === $item[0]) {
100-
$formattedValue = sprintf('<em>array</em>(%s)', is_array($item[1]) ? $this->formatArgs($item[1]) : $item[1]);
100+
$formattedValue = sprintf('<em>array</em>(%s)', \is_array($item[1]) ? $this->formatArgs($item[1]) : $item[1]);
101101
} elseif ('null' === $item[0]) {
102102
$formattedValue = '<em>null</em>';
103103
} elseif ('boolean' === $item[0]) {
@@ -108,7 +108,7 @@ public function formatArgs($args)
108108
$formattedValue = str_replace("\n", '', htmlspecialchars(var_export($item[1], true), ENT_COMPAT | ENT_SUBSTITUTE, $this->charset));
109109
}
110110

111-
$result[] = is_int($key) ? $formattedValue : sprintf("'%s' => %s", $key, $formattedValue);
111+
$result[] = \is_int($key) ? $formattedValue : sprintf("'%s' => %s", $key, $formattedValue);
112112
}
113113

114114
return implode(', ', $result);
@@ -151,10 +151,10 @@ public function fileExcerpt($file, $line, $srcContext = 3)
151151

152152
$lines = array();
153153
if (0 > $srcContext) {
154-
$srcContext = count($content);
154+
$srcContext = \count($content);
155155
}
156156

157-
for ($i = max($line - $srcContext, 1), $max = min($line + $srcContext, count($content)); $i <= $max; ++$i) {
157+
for ($i = max($line - $srcContext, 1), $max = min($line + $srcContext, \count($content)); $i <= $max; ++$i) {
158158
$lines[] = '<li'.($i == $line ? ' class="selected"' : '').'><a class="anchor" name="line'.$i.'"></a><code>'.self::fixCodeMarkup($content[$i - 1]).'</code></li>';
159159
}
160160

@@ -178,7 +178,7 @@ public function formatFile($file, $line, $text = null)
178178
if (null === $text) {
179179
$text = str_replace('/', DIRECTORY_SEPARATOR, $file);
180180
if (0 === strpos($text, $this->rootDir)) {
181-
$text = substr($text, strlen($this->rootDir));
181+
$text = substr($text, \strlen($this->rootDir));
182182
$text = explode(DIRECTORY_SEPARATOR, $text, 2);
183183
$text = sprintf('<abbr title="%s%2$s">%s</abbr>%s', $this->rootDir, $text[0], isset($text[1]) ? DIRECTORY_SEPARATOR.$text[1] : '');
184184
}
@@ -206,7 +206,7 @@ public function formatFile($file, $line, $text = null)
206206
public function getFileLink($file, $line)
207207
{
208208
if ($fmt = $this->fileLinkFormat) {
209-
return is_string($fmt) ? strtr($fmt, array('%f' => $file, '%l' => $line)) : $fmt->format($file, $line);
209+
return \is_string($fmt) ? strtr($fmt, array('%f' => $file, '%l' => $line)) : $fmt->format($file, $line);
210210
}
211211

212212
return false;

‎Extension/DumpExtension.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ public function dump(Environment $env, $context)
5858
return;
5959
}
6060

61-
if (2 === func_num_args()) {
61+
if (2 === \func_num_args()) {
6262
$vars = array();
6363
foreach ($context as $key => $value) {
6464
if (!$value instanceof Template) {
@@ -68,7 +68,7 @@ public function dump(Environment $env, $context)
6868

6969
$vars = array($vars);
7070
} else {
71-
$vars = func_get_args();
71+
$vars = \func_get_args();
7272
unset($vars[0], $vars[1]);
7373
}
7474

‎Extension/FormExtension.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -101,8 +101,8 @@ public function getName()
101101
*/
102102
function twig_is_selected_choice(ChoiceView $choice, $selectedValue)
103103
{
104-
if (is_array($selectedValue)) {
105-
return in_array($choice->value, $selectedValue, true);
104+
if (\is_array($selectedValue)) {
105+
return \in_array($choice->value, $selectedValue, true);
106106
}
107107

108108
return $choice->value === $selectedValue;

‎Extension/HttpFoundationExtension.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ public function generateAbsoluteUrl($path)
9797

9898
if (!$path || '/' !== $path[0]) {
9999
$prefix = $request->getPathInfo();
100-
$last = strlen($prefix) - 1;
100+
$last = \strlen($prefix) - 1;
101101
if ($last !== $pos = strrpos($prefix, '/')) {
102102
$prefix = substr($prefix, 0, $pos).'/';
103103
}

‎Extension/RoutingExtension.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ public function isUrlGenerationSafe(Node $argsNode)
100100
$argsNode->hasNode(1) ? $argsNode->getNode(1) : null
101101
);
102102

103-
if (null === $paramsNode || $paramsNode instanceof ArrayExpression && count($paramsNode) <= 2 &&
103+
if (null === $paramsNode || $paramsNode instanceof ArrayExpression && \count($paramsNode) <= 2 &&
104104
(!$paramsNode->hasNode(1) || $paramsNode->getNode(1) instanceof ConstantExpression)
105105
) {
106106
return array('html');

‎Extension/YamlExtension.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ public function encode($input, $inline = 0, $dumpObjects = 0)
4242
$dumper = new YamlDumper();
4343
}
4444

45-
if (defined('Symfony\Component\Yaml\Yaml::DUMP_OBJECT')) {
45+
if (\defined('Symfony\Component\Yaml\Yaml::DUMP_OBJECT')) {
4646
return $dumper->dump($input, $inline, 0, $dumpObjects);
4747
}
4848

@@ -51,12 +51,12 @@ public function encode($input, $inline = 0, $dumpObjects = 0)
5151

5252
public function dump($value, $inline = 0, $dumpObjects = false)
5353
{
54-
if (is_resource($value)) {
54+
if (\is_resource($value)) {
5555
return '%Resource%';
5656
}
5757

58-
if (is_array($value) || is_object($value)) {
59-
return '%'.gettype($value).'% '.$this->encode($value, $inline, $dumpObjects);
58+
if (\is_array($value) || \is_object($value)) {
59+
return '%'.\gettype($value).'% '.$this->encode($value, $inline, $dumpObjects);
6060
}
6161

6262
return $this->encode($value, $inline, $dumpObjects);

‎Form/TwigRendererEngine.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ protected function loadResourceForBlockName($cacheKey, FormView $view, $blockNam
9898

9999
// Check each theme whether it contains the searched block
100100
if (isset($this->themes[$cacheKey])) {
101-
for ($i = count($this->themes[$cacheKey]) - 1; $i >= 0; --$i) {
101+
for ($i = \count($this->themes[$cacheKey]) - 1; $i >= 0; --$i) {
102102
$this->loadResourcesFromTheme($cacheKey, $this->themes[$cacheKey][$i]);
103103
// CONTINUE LOADING (see doc comment)
104104
}
@@ -107,7 +107,7 @@ protected function loadResourceForBlockName($cacheKey, FormView $view, $blockNam
107107
// Check the default themes once we reach the root view without success
108108
if (!$view->parent) {
109109
if (!isset($this->useDefaultThemes[$cacheKey]) || $this->useDefaultThemes[$cacheKey]) {
110-
for ($i = count($this->defaultThemes) - 1; $i >= 0; --$i) {
110+
for ($i = \count($this->defaultThemes) - 1; $i >= 0; --$i) {
111111
$this->loadResourcesFromTheme($cacheKey, $this->defaultThemes[$i]);
112112
// CONTINUE LOADING (see doc comment)
113113
}

0 commit comments

Comments
 (0)