apply cs fix to all dirs

rmikalkenas · Apr 23, 2019 · d88030d · d88030d
1 parent 5f10429
commit d88030d
Show file tree

Hide file tree

Showing 28 changed files with 86 additions and 65 deletions.
diff --git a/.php_cs b/.php_cs
@@ -1,8 +1,29 @@
 <?php
 // see https://github.com/FriendsOfPHP/PHP-CS-Fixer
 
+$dirs = [
+    __DIR__.'/Adapter',
+    __DIR__.'/Command',
+    __DIR__.'/DataCollector',
+    __DIR__.'/DependencyInjection',
+    __DIR__.'/Entity',
+    __DIR__.'/Event',
+    __DIR__.'/EventListener',
+    __DIR__.'/Exception',
+    __DIR__.'/Form',
+    __DIR__.'/Handler',
+    __DIR__.'/Injector',
+    __DIR__.'/Mapping',
+    __DIR__.'/Metadata',
+    __DIR__.'/Naming',
+    __DIR__.'/Storage',
+    __DIR__.'/Tests',
+    __DIR__.'/Twig',
+    __DIR__.'/Util',
+];
+
 $finder = PhpCsFixer\Finder::create()
-    ->in([__DIR__.'/Command', __DIR__.'/DependencyInjection', __DIR__.'/Form', __DIR__.'/Tests', __DIR__.'/Twig', __DIR__.'/Util'])
+    ->in($dirs)
 ;
 
 return PhpCsFixer\Config::create()

diff --git a/Adapter/ODM/MongoDB/MongoDBAdapter.php b/Adapter/ODM/MongoDB/MongoDBAdapter.php
@@ -28,7 +28,7 @@ public function recomputeChangeSet($event): void
 
         $dm = $event->getDocumentManager();
         $uow = $dm->getUnitOfWork();
-        $metadata = $dm->getClassMetadata(get_class($object));
+        $metadata = $dm->getClassMetadata(\get_class($object));
         $uow->recomputeSingleDocumentChangeSet($metadata, $object);
     }
 }
diff --git a/Adapter/ORM/DoctrineORMAdapter.php b/Adapter/ORM/DoctrineORMAdapter.php
@@ -28,7 +28,7 @@ public function recomputeChangeSet($event): void
 
         $em = $event->getEntityManager();
         $uow = $em->getUnitOfWork();
-        $metadata = $em->getClassMetadata(get_class($object));
+        $metadata = $em->getClassMetadata(\get_class($object));
         $uow->recomputeSingleEntityChangeSet($metadata, $object);
     }
 }
diff --git a/DataCollector/MappingCollector.php b/DataCollector/MappingCollector.php
@@ -2,9 +2,9 @@
 
 namespace Vich\UploaderBundle\DataCollector;
 
-use Symfony\Component\HttpKernel\DataCollector\DataCollector;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
+use Symfony\Component\HttpKernel\DataCollector\DataCollector;
 use Vich\UploaderBundle\Metadata\MetadataReader;
 
 final class MappingCollector extends DataCollector

diff --git a/EventListener/Doctrine/BaseListener.php b/EventListener/Doctrine/BaseListener.php
@@ -68,7 +68,7 @@ protected function getUploadableFields($object): array
     {
         $fields = $this->metadata->getUploadableFields(ClassUtils::getClass($object), $this->mapping);
 
-        return array_map(function ($data) {
+        return \array_map(function ($data) {
             return $data['propertyName'];
         }, $fields);
     }

diff --git a/EventListener/Propel/BaseListener.php b/EventListener/Propel/BaseListener.php
@@ -56,7 +56,7 @@ protected function getUploadableFields($object): array
     {
         $fields = $this->metadata->getUploadableFields(ClassUtils::getClass($object), $this->mapping);
 
-        return array_map(function ($data) {
+        return \array_map(function ($data) {
             return $data['propertyName'];
         }, $fields);
     }

diff --git a/Exception/MappingNotFoundException.php b/Exception/MappingNotFoundException.php
@@ -7,20 +7,20 @@ class MappingNotFoundException extends \RuntimeException
     public static function createNotFoundForClassAndField($mapping, $class, $field): self
     {
         return new self(
-            sprintf('Mapping "%s" does not exist. The configuration for the class "%s" is probably incorrect as the mapping to use for the field "%s" could not be found.', $mapping, $class, $field)
+            \sprintf('Mapping "%s" does not exist. The configuration for the class "%s" is probably incorrect as the mapping to use for the field "%s" could not be found.', $mapping, $class, $field)
         );
     }
 
     public static function createNotFoundForClass(string $mapping, string $class): self
     {
         if ('' === $mapping) {
             return new self(
-                sprintf('Mapping not found. The configuration for the class "%s" is probably incorrect.', $class)
+                \sprintf('Mapping not found. The configuration for the class "%s" is probably incorrect.', $class)
             );
         }
 
         return new self(
-            sprintf('Mapping "%s" does not exist. The configuration for the class "%s" is probably incorrect.', $mapping, $class)
+            \sprintf('Mapping "%s" does not exist. The configuration for the class "%s" is probably incorrect.', $mapping, $class)
         );
     }
 }
diff --git a/Handler/AbstractHandler.php b/Handler/AbstractHandler.php
@@ -42,7 +42,7 @@ protected function getMapping($obj, string $fieldName, ?string $className = null
         $mapping = $this->factory->fromField($obj, $fieldName, $className);
 
         if (null === $mapping) {
-            throw new MappingNotFoundException(sprintf('Mapping not found for field "%s"', $fieldName));
+            throw new MappingNotFoundException(\sprintf('Mapping not found for field "%s"', $fieldName));
         }
 
         return $mapping;

diff --git a/Handler/DownloadHandler.php b/Handler/DownloadHandler.php
@@ -34,7 +34,7 @@ public function downloadObject($object, string $field, ?string $className = null
         $stream = $this->storage->resolveStream($object, $field, $className);
 
         if (null === $stream) {
-            throw new NoFileFoundException(sprintf('No file found in field "%s".', $field));
+            throw new NoFileFoundException(\sprintf('No file found in field "%s".', $field));
         }
 
         if (true === $fileName) {
@@ -75,7 +75,7 @@ public function downloadObject($object, string $field, ?string $className = null
     private function createDownloadResponse($stream, string $filename, ?string $mimeType = 'application/octet-stream', bool $forceDownload = true): StreamedResponse
     {
         $response = new StreamedResponse(function () use ($stream): void {
-            stream_copy_to_stream($stream, fopen('php://output', 'wb'));
+            \stream_copy_to_stream($stream, \fopen('php://output', 'w'));
         });
 
         $disposition = $response->headers->makeDisposition(

diff --git a/Mapping/Annotation/UploadableField.php b/Mapping/Annotation/UploadableField.php
@@ -58,8 +58,8 @@ public function __construct(array $options)
         }
 
         foreach ($options as $property => $value) {
-            if (!property_exists($this, $property)) {
-                throw new \RuntimeException(sprintf('Unknown key "%s" for annotation "@%s".', $property, get_class($this)));
+            if (!\property_exists($this, $property)) {
+                throw new \RuntimeException(\sprintf('Unknown key "%s" for annotation "@%s".', $property, \get_class($this)));
             }
 
             $this->$property = $value;

diff --git a/Mapping/PropertyMapping.php b/Mapping/PropertyMapping.php
@@ -59,7 +59,7 @@ class PropertyMapping
      */
     public function __construct(string $filePropertyPath, string $fileNamePropertyPath, array $propertyPaths = [])
     {
-        $this->propertyPaths = array_merge(
+        $this->propertyPaths = \array_merge(
             $this->propertyPaths,
             ['file' => $filePropertyPath, 'name' => $fileNamePropertyPath],
             $propertyPaths
@@ -151,8 +151,8 @@ public function erase($obj): void
      */
     public function readProperty($obj, $property)
     {
-        if (!array_key_exists($property, $this->propertyPaths)) {
-            throw new \InvalidArgumentException(sprintf('Unknown property %s', $property));
+        if (!\array_key_exists($property, $this->propertyPaths)) {
+            throw new \InvalidArgumentException(\sprintf('Unknown property %s', $property));
         }
 
         if (!$this->propertyPaths[$property]) {
@@ -179,8 +179,8 @@ public function readProperty($obj, $property)
      */
     public function writeProperty($obj, string $property, $value): void
     {
-        if (!array_key_exists($property, $this->propertyPaths)) {
-            throw new \InvalidArgumentException(sprintf('Unknown property %s', $property));
+        if (!\array_key_exists($property, $this->propertyPaths)) {
+            throw new \InvalidArgumentException(\sprintf('Unknown property %s', $property));
         }
 
         if (!$this->propertyPaths[$property]) {
@@ -334,7 +334,7 @@ public function getUploadDir($obj): ?string
         $dir = $this->getDirectoryNamer()->directoryName($obj, $this);
 
         // strip the trailing directory separator if needed
-        $dir = $dir ? rtrim($dir, '/\\') : $dir;
+        $dir = $dir ? \rtrim($dir, '/\\') : $dir;
 
         return $dir;
     }
@@ -372,11 +372,11 @@ public function getUriPrefix(): string
      */
     protected function fixPropertyPath($object, string $propertyPath): string
     {
-        if (!is_array($object)) {
+        if (!\is_array($object)) {
             return $propertyPath;
         }
 
-        return '[' === $propertyPath[0] ? $propertyPath : sprintf('[%s]', $propertyPath);
+        return '[' === $propertyPath[0] ? $propertyPath : \sprintf('[%s]', $propertyPath);
     }
 
     protected function getAccessor(): PropertyAccessor

diff --git a/Mapping/PropertyMappingFactory.php b/Mapping/PropertyMappingFactory.php
@@ -122,7 +122,7 @@ public function fromField($obj, string $field, ?string $className = null): ?Prop
     protected function checkUploadable(string $class): void
     {
         if (!$this->metadata->isUploadable($class)) {
-            throw new NotUploadableException(sprintf('The class "%s" is not uploadable. If you use annotations to configure VichUploaderBundle, you probably just forgot to add `@Vich\Uploadable` on top of your entity. If you don\'t use annotations, check that the configuration files are in the right place. In both cases, clearing the cache can also solve the issue.', $class));
+            throw new NotUploadableException(\sprintf('The class "%s" is not uploadable. If you use annotations to configure VichUploaderBundle, you probably just forgot to add `@Vich\Uploadable` on top of your entity. If you don\'t use annotations, check that the configuration files are in the right place. In both cases, clearing the cache can also solve the issue.', $class));
         }
     }
 
@@ -140,7 +140,7 @@ protected function checkUploadable(string $class): void
      */
     protected function createMapping($obj, string $fieldName, array $mappingData): PropertyMapping
     {
-        if (!array_key_exists($mappingData['mapping'], $this->mappings)) {
+        if (!\array_key_exists($mappingData['mapping'], $this->mappings)) {
             throw MappingNotFoundException::createNotFoundForClassAndField($mappingData['mapping'], $this->getClassName($obj), $fieldName);
         }
 
@@ -158,7 +158,7 @@ protected function createMapping($obj, string $fieldName, array $mappingData): P
 
             if (!empty($namerConfig['options'])) {
                 if (!$namer instanceof ConfigurableInterface) {
-                    throw new \LogicException(sprintf('Namer %s can not receive options as it does not implement ConfigurableInterface.', $namerConfig['service']));
+                    throw new \LogicException(\sprintf('Namer %s can not receive options as it does not implement ConfigurableInterface.', $namerConfig['service']));
                 }
                 $namer->configure($namerConfig['options']);
             }
@@ -172,7 +172,7 @@ protected function createMapping($obj, string $fieldName, array $mappingData): P
 
             if (!empty($namerConfig['options'])) {
                 if (!$namer instanceof ConfigurableInterface) {
-                    throw new \LogicException(sprintf('Namer %s can not receive options as it does not implement ConfigurableInterface.', $namerConfig['service']));
+                    throw new \LogicException(\sprintf('Namer %s can not receive options as it does not implement ConfigurableInterface.', $namerConfig['service']));
                 }
                 $namer->configure($namerConfig['options']);
             }
@@ -199,7 +199,7 @@ protected function getClassName($object, ?string $className = null): string
             return $className;
         }
 
-        if (is_object($object)) {
+        if (\is_object($object)) {
             return ClassUtils::getClass($object);
         }
 

diff --git a/Metadata/ClassMetadata.php b/Metadata/ClassMetadata.php
@@ -10,7 +10,7 @@ class ClassMetadata extends BaseClassMetadata
 
     public function serialize(): string
     {
-        return serialize([
+        return \serialize([
             $this->fields,
             parent::serialize(),
         ]);
@@ -21,7 +21,7 @@ public function unserialize($str): void
         [
             $this->fields,
             $parentStr
-            ] = unserialize($str);
+            ] = \unserialize($str);
 
         parent::unserialize($parentStr);
     }

diff --git a/Metadata/MetadataReader.php b/Metadata/MetadataReader.php
@@ -90,11 +90,11 @@ public function getUploadableFields(string $class, ?string $mapping = null): arr
         $uploadableFields = [];
 
         foreach ($metadata->classMetadata as $classMetadata) {
-            $uploadableFields = array_merge($uploadableFields, $classMetadata->fields);
+            $uploadableFields = \array_merge($uploadableFields, $classMetadata->fields);
         }
 
         if (null !== $mapping) {
-            $uploadableFields = array_filter($uploadableFields, function (array $fieldMetadata) use ($mapping) {
+            $uploadableFields = \array_filter($uploadableFields, function (array $fieldMetadata) use ($mapping) {
                 return $fieldMetadata['mapping'] === $mapping;
             });
         }

diff --git a/Naming/Base64Namer.php b/Naming/Base64Namer.php
@@ -50,6 +50,6 @@ public function name($object, PropertyMapping $mapping): string
 
     protected function getRandomChar(): string
     {
-        return self::ALPHABET[random_int(0, 63)];
+        return self::ALPHABET[\random_int(0, 63)];
     }
 }
diff --git a/Naming/CurrentDateTimeDirectoryNamer.php b/Naming/CurrentDateTimeDirectoryNamer.php
@@ -47,7 +47,7 @@ public function __construct(DateTimeHelper $dateTimeHelper, ?PropertyAccessorInt
      */
     public function configure(array $options): void
     {
-        $options = array_merge(['date_time_format' => $this->dateTimeFormat], $options);
+        $options = \array_merge(['date_time_format' => $this->dateTimeFormat], $options);
 
         $this->dateTimeFormat = $options['date_time_format'];
 
@@ -70,6 +70,6 @@ public function directoryName($object, PropertyMapping $mapping): string
             $dateTime = $this->dateTimeHelper->getTimestamp();
         }
 
-        return date($this->dateTimeFormat, $dateTime);
+        return \date($this->dateTimeFormat, $dateTime);
     }
 }
diff --git a/Naming/CurrentDateTimeHelper.php b/Naming/CurrentDateTimeHelper.php
@@ -14,6 +14,6 @@ class CurrentDateTimeHelper implements DateTimeHelper
      */
     public function getTimestamp(): int
     {
-        return time();
+        return \time();
     }
 }
diff --git a/Naming/HashNamer.php b/Naming/HashNamer.php
@@ -24,7 +24,7 @@ class HashNamer implements NamerInterface, ConfigurableInterface
      */
     public function configure(array $options): void
     {
-        $options = array_merge(['algorithm' => $this->algorithm, 'length' => $this->length], $options);
+        $options = \array_merge(['algorithm' => $this->algorithm, 'length' => $this->length], $options);
 
         $this->algorithm = $options['algorithm'];
         $this->length = $options['length'];
@@ -34,20 +34,20 @@ public function name($object, PropertyMapping $mapping): string
     {
         $file = $mapping->getFile($object);
 
-        $name = hash($this->algorithm, $this->getRandomString());
+        $name = \hash($this->algorithm, $this->getRandomString());
         if (null !== $this->length) {
-            $name = substr($name, 0, $this->length);
+            $name = \substr($name, 0, $this->length);
         }
 
         if ($extension = $this->getExtension($file)) {
-            $name = sprintf('%s.%s', $name, $extension);
+            $name = \sprintf('%s.%s', $name, $extension);
         }
 
         return $name;
     }
 
     protected function getRandomString(): string
     {
-        return microtime(true).random_int(0, 9999999);
+        return \microtime(true).\random_int(0, 9999999);
     }
 }
diff --git a/Naming/OrignameNamer.php b/Naming/OrignameNamer.php
@@ -40,6 +40,6 @@ public function name($object, PropertyMapping $mapping): string
             $name = Transliterator::transliterate($name);
         }
 
-        return uniqid().'_'.$name;
+        return \uniqid().'_'.$name;
     }
 }
diff --git a/Naming/Polyfill/FileExtensionTrait.php b/Naming/Polyfill/FileExtensionTrait.php
@@ -17,7 +17,7 @@ private function getExtension(UploadedFile $file): ?string
     {
         $originalName = $file->getClientOriginalName();
 
-        if ($extension = pathinfo($originalName, PATHINFO_EXTENSION)) {
+        if ($extension = \pathinfo($originalName, PATHINFO_EXTENSION)) {
             return $extension;
         }
 

diff --git a/Naming/PropertyDirectoryNamer.php b/Naming/PropertyDirectoryNamer.php
@@ -62,11 +62,11 @@ public function directoryName($object, PropertyMapping $mapping): string
         try {
             $name = $this->propertyAccessor->getValue($object, $this->propertyPath);
         } catch (NoSuchPropertyException $e) {
-            throw new NameGenerationException(sprintf('Directory name could not be generated: property %s does not exist.', $this->propertyPath), $e->getCode(), $e);
+            throw new NameGenerationException(\sprintf('Directory name could not be generated: property %s does not exist.', $this->propertyPath), $e->getCode(), $e);
         }
 
         if (empty($name)) {
-            throw new NameGenerationException(sprintf('Directory name could not be generated: property %s is empty.', $this->propertyPath));
+            throw new NameGenerationException(\sprintf('Directory name could not be generated: property %s is empty.', $this->propertyPath));
         }
 
         if ($this->transliterate) {

diff --git a/Naming/PropertyNamer.php b/Naming/PropertyNamer.php
@@ -55,11 +55,11 @@ public function name($object, PropertyMapping $mapping): string
         try {
             $name = $this->getPropertyValue($object, $this->propertyPath);
         } catch (NoSuchPropertyException $e) {
-            throw new NameGenerationException(sprintf('File name could not be generated: property %s does not exist.', $this->propertyPath), $e->getCode(), $e);
+            throw new NameGenerationException(\sprintf('File name could not be generated: property %s does not exist.', $this->propertyPath), $e->getCode(), $e);
         }
 
         if (empty($name)) {
-            throw new NameGenerationException(sprintf('File name could not be generated: property %s is empty.', $this->propertyPath));
+            throw new NameGenerationException(\sprintf('File name could not be generated: property %s is empty.', $this->propertyPath));
         }
 
         if ($this->transliterate) {
@@ -68,7 +68,7 @@ public function name($object, PropertyMapping $mapping): string
 
         // append the file extension if there is one
         if ($extension = $this->getExtension($file)) {
-            $name = sprintf('%s.%s', $name, $extension);
+            $name = \sprintf('%s.%s', $name, $extension);
         }
 
         return $name;