Add . to exception messages

Author: GromNaN

src/Doctrine/EntityRelation.php

@@ -34,7 +34,7 @@ public function __construct(
         private string $inverseClass,
     ) {
         if (!\in_array($type, self::getValidRelationTypes())) {
-            throw new \Exception(\sprintf('Invalid relation type "%s"', $type));
+            throw new \Exception(\sprintf('Invalid relation type "%s".', $type));
         }
 
         if (self::ONE_TO_MANY === $type) {

src/Maker/MakeEntity.php

@@ -301,7 +301,7 @@ public function generate(InputInterface $input, ConsoleStyle $io, Generator $gen
                 }
                 $currentFields[] = $newFieldName;
             } else {
-                throw new \Exception('Invalid value');
+                throw new \Exception('Invalid value.');
             }
 
             foreach ($fileManagerOperations as $path => $manipulatorOrMessage) {
@@ -827,7 +827,7 @@ private function askRelationType(ConsoleStyle $io, string $entityClass, string $
         $question->setAutocompleterValues(EntityRelation::getValidRelationTypes());
         $question->setValidator(static function ($type) {
             if (!\in_array($type, EntityRelation::getValidRelationTypes())) {
-                throw new \InvalidArgumentException(\sprintf('Invalid type: use one of: %s', implode(', ', EntityRelation::getValidRelationTypes())));
+                throw new \InvalidArgumentException(\sprintf('Invalid type, use one of: "%s"', implode('", "', EntityRelation::getValidRelationTypes())));
             }
 
             return $type;

src/Test/MakerTestRunner.php

@@ -114,7 +114,7 @@ public function modifyYamlFile(string $filename, \Closure $callback)
 
         $newData = $callback($manipulator->getData());
         if (!\is_array($newData)) {
-            throw new \Exception('The modifyYamlFile() callback must return the final array of data');
+            throw new \Exception('The modifyYamlFile() callback must return the final array of data.');
         }
         $manipulator->setData($newData);
 

src/Util/ClassSourceManipulator.php

@@ -433,7 +433,7 @@ private function addCustomGetter(string $propertyName, string $methodName, $retu
                     break;
                 default:
                     // implement other cases if/when the library needs them
-                    throw new \Exception('Not implemented');
+                    throw new \Exception('Not implemented.');
             }
         }
 
@@ -967,7 +967,7 @@ private function getClassNode(): Node\Stmt\Class_
         $node = $this->findFirstNode(static fn ($node) => $node instanceof Node\Stmt\Class_);
 
         if (!$node) {
-            throw new \Exception('Could not find class node');
+            throw new \Exception('Could not find class node.');
         }
 
         return $node;
@@ -978,7 +978,7 @@ private function getNamespaceNode(): Node\Stmt\Namespace_
         $node = $this->findFirstNode(static fn ($node) => $node instanceof Node\Stmt\Namespace_);
 
         if (!$node) {
-            throw new \Exception('Could not find namespace node');
+            throw new \Exception('Could not find namespace node.');
         }
 
         return $node;
@@ -1044,10 +1044,10 @@ private function createSingleLineCommentNode(string $comment, string $context):
         switch ($context) {
             case self::CONTEXT_OUTSIDE_CLASS:
                 // just not needed yet
-                throw new \Exception('not supported');
+                throw new \Exception('Not supported.');
             case self::CONTEXT_CLASS:
                 // just not needed yet
-                throw new \Exception('not supported');
+                throw new \Exception('Not supported.');
             case self::CONTEXT_CLASS_METHOD:
                 return BuilderHelpers::normalizeStmt(new Node\Expr\Variable(\sprintf('__COMMENT__VAR_%d', \count($this->pendingComments) - 1)));
             default:

src/Util/YamlSourceManipulator.php

@@ -1000,7 +1000,7 @@ private function guessNextArrayTypeAndAdvance(): string
     {
         while (true) {
             if ($this->isEOF()) {
-                throw new \LogicException('Could not determine array type');
+                throw new \LogicException('Could not determine array type.');
             }
 
             // get the next char & advance immediately
@@ -1029,7 +1029,7 @@ private function findPositionOfNextCharacter(array $chars)
         $currentPosition = $this->currentPosition;
         while (true) {
             if ($this->isEOF($currentPosition)) {
-                throw new \LogicException(\sprintf('Could not find any characters: %s', implode(', ', $chars)));
+                throw new \LogicException(\sprintf('Could not find any characters: "%s"', implode('", "', $chars)));
             }
 
             // get the next char & advance immediately