Your project uses discouraged functions to kill scripts 5

More information: https://insight.symfony.com/what-we-analyse/php.use_exit_function

in create_phar.php, line 16 
  2. declare(strict_types=1);
  4. if (!\class_exists('Phar')) {
  5.     echo "Enable Phar extension.\n";
  6.     exit(1);
    This line stops the execution flow, without explanation. If this is for debug, you should remove it. If this is to deal with an error, use exceptions instead.
    Time to fix: about 4 hours
    Read doc
                Open Issue
             Permalink
    
                                    Last edited  by Laurent Muller
                    
  7. }
  8. if (\ini_get('phar.readonly')) {
  9.     echo "Set directive 'phar.readonly=off'.\n";
  10.     exit(1);
  11. }
in create_phar.php, line 20 
  1.     echo "Enable Phar extension.\n";
  2.     exit(1);
  3. }
  4. if (\ini_get('phar.readonly')) {
  5.     echo "Set directive 'phar.readonly=off'.\n";
  6.     exit(1);
    This line stops the execution flow, without explanation. If this is for debug, you should remove it. If this is to deal with an error, use exceptions instead.
    Time to fix: about 4 hours
    Read doc
                Open Issue
             Permalink
    
                                    Last edited  by Laurent Muller
                    
  7. }
  9. function addFiles(Phar $phar, string $baseDirectory, string $sourceDirectory): void
  10. {
  11.     $offset = \strlen($baseDirectory) + 1;
in create_phar.php, line 49 
  1.     $buildDirectory = $baseDirectory . '/build';
  2.     $pharFile = $buildDirectory . '/makeFont.phar';
  4.     if (!\is_dir($buildDirectory) && !\mkdir($buildDirectory)) {
  5.         echo "Unable to create the output directory: $buildDirectory.\n";
  6.         exit(1);
    This line stops the execution flow, without explanation. If this is for debug, you should remove it. If this is to deal with an error, use exceptions instead.
    Time to fix: about 4 hours
    Read doc
                Open Issue
             Permalink
    
                                    Last edited  by Laurent Muller
                    
  7.     }
  9.     if (\is_file($pharFile) && !\unlink($pharFile)) {
  10.         echo "Unable to remove the old Phar: $pharFile.\n";
  11.         exit(1);
in create_phar.php, line 54 
  1.         exit(1);
  2.     }
  4.     if (\is_file($pharFile) && !\unlink($pharFile)) {
  5.         echo "Unable to remove the old Phar: $pharFile.\n";
  6.         exit(1);
    This line stops the execution flow, without explanation. If this is for debug, you should remove it. If this is to deal with an error, use exceptions instead.
    Time to fix: about 4 hours
    Read doc
                Open Issue
             Permalink
    
                                    Last edited  by Laurent Muller
                    
  7.     }
  9.     // create phar
  10.     $phar = new Phar($pharFile);
  11.     $phar->setStub("<?php
in create_phar.php, line 79 
  1.     \chmod($pharFile, 0o770);
  3.     echo "$pharFile successfully created.";
  4. } catch (Exception $e) {
  5.     echo 'Error: ' . $e->getMessage() . "\n";
  6.     exit(1);
    This line stops the execution flow, without explanation. If this is for debug, you should remove it. If this is to deal with an error, use exceptions instead.
    Time to fix: about 4 hours
    Read doc
                Open Issue
             Permalink
    
                                    Last edited  by Laurent Muller
                    
  7. }

Your project should not use global variables or functions

More information: https://insight.symfony.com/what-we-analyse/php.use_global_variable_or_function

in create_phar.php, line 23 
  1. if (\ini_get('phar.readonly')) {
  2.     echo "Set directive 'phar.readonly=off'.\n";
  3.     exit(1);
  4. }
  6. function addFiles(Phar $phar, string $baseDirectory, string $sourceDirectory): void
    addFiles() adds to the global scope. Prefer class properties or methods to let other developers know what this relates to.
    Time to fix: about 1 day
    Read doc
                Open Issue
             Permalink
    
                                    Last edited  by Laurent Muller
                    
  7. {
  8.     $offset = \strlen($baseDirectory) + 1;
  9.     $fullPath = $baseDirectory . $sourceDirectory;
  10.     $flags = FilesystemIterator::SKIP_DOTS | FilesystemIterator::UNIX_PATHS;
  11.     $directory = new RecursiveDirectoryIterator($fullPath, $flags);