Merge pull request #999 from lcharette/develop-feature-updatedDependencies

Update to the latest Laravel & Composer Packages

Author: lcharette

CHANGELOG.md

@@ -10,6 +10,17 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 ### Changed Requirements
 - Changed minimum PHP Version to 7.1
 
+### Changed Dependencies
+- Updated Laravel Illuminate packages to 5.8
+- Updated Twig to 2.11
+- Updated PHPUnit to 7.5
+- Updated Mockery to 1.2
+- Updated nikic/php-parser to 4.2.2
+- Updated PHPMailer/PHPMailer to 6.0.7
+- Updated league/csv to 9.2.1
+- Updated symfony/console to 4.3
+- Updated vlucas/phpdotenv to 3.4.0
+
 ### Added
 - Separated `BakeCommand` class into multiple methods to make it easier for sprinkle to add custom command to the `bake` command.
 

app/sprinkles/account/composer.json

@@ -13,7 +13,7 @@
     ],
     "require": {
         "birke/rememberme" : "^2.0",
-        "nikic/php-parser" : "^1"
+        "nikic/php-parser" : "^4.2.2"
     },
     "autoload": {
         "psr-4": {

app/sprinkles/account/src/Authorize/AccessConditionExpression.php

@@ -12,9 +12,8 @@
 
 use Monolog\Logger;
 use PhpParser\Error as PhpParserException;
-use PhpParser\Lexer\Emulative as EmulativeLexer;
 use PhpParser\NodeTraverser;
-use PhpParser\Parser as Parser;
+use PhpParser\ParserFactory as Parser;
 use PhpParser\PrettyPrinter\Standard as StandardPrettyPrinter;
 use UserFrosting\Sprinkle\Account\Database\Models\Interfaces\UserInterface;
 
@@ -75,7 +74,7 @@ public function __construct(ParserNodeFunctionEvaluator $nodeVisitor, UserInterf
     {
         $this->nodeVisitor = $nodeVisitor;
         $this->user = $user;
-        $this->parser = new Parser(new EmulativeLexer());
+        $this->parser = (new Parser())->create(Parser::ONLY_PHP7);
         $this->traverser = new NodeTraverser();
         $this->traverser->addVisitor($nodeVisitor);
         $this->prettyPrinter = new StandardPrettyPrinter();
@@ -119,7 +118,7 @@ public function evaluateCondition($condition, $params)
             $stmts = $this->traverser->traverse($stmts);
 
             // Evaluate boolean statement.  It is safe to use eval() here, because our expression has been reduced entirely to a boolean expression.
-            $expr = $this->prettyPrinter->prettyPrintExpr($stmts[0]);
+            $expr = $this->prettyPrinter->prettyPrintExpr($stmts[0]->expr);
             $expr_eval = 'return ' . $expr . ";\n";
             $result = eval($expr_eval);
 

app/sprinkles/account/src/Authorize/ParserNodeFunctionEvaluator.php

@@ -71,7 +71,7 @@ public function leaveNode(Node $node)
     {
         // Look for function calls
         if ($node instanceof \PhpParser\Node\Expr\FuncCall) {
-            $eval = new \PhpParser\Node\Scalar\LNumber();
+            $eval = new \PhpParser\Node\Scalar\LNumber(0);
 
             // Get the method name
             $callbackName = $node->name->toString();

app/sprinkles/core/composer.json

@@ -32,28 +32,27 @@
         "illuminate/database": "5.8.*",
         "illuminate/events": "5.8.*",
         "illuminate/filesystem": "5.8.*",
-        "league/csv": "^8.1",
+        "league/csv": "^9.2.1",
         "league/flysystem": "~1.0",
         "league/flysystem-aws-s3-v3": "~1.0",
         "league/flysystem-cached-adapter": "~1.0",
         "league/flysystem-rackspace": "~1.0",
         "league/flysystem-sftp": "~1.0",
         "monolog/monolog": "^1",
-        "phpmailer/phpmailer": "5.2.10",
-        "rockettheme/toolbox": "1.3.1",
+        "phpmailer/phpmailer": "^6.0.7",
         "slim/csrf": "^0.8",
         "slim/slim": "^3",
-        "slim/twig-view": "^1.2",
+        "slim/twig-view": "^2.5",
         "symfony/http-foundation": "*",
-        "twig/twig": "^1.18",
-        "userfrosting/assets": "^5.0.4",
-        "userfrosting/config": "~4.2.0",
-        "userfrosting/cache": "~4.2.0",
-        "userfrosting/fortress": "~4.2.0",
-        "userfrosting/i18n": "~4.2.0",
-        "userfrosting/session": "~4.2.2",
-        "userfrosting/support": "~4.2.0",
-        "vlucas/phpdotenv": "^2"
+        "twig/twig": "^2.11",
+        "userfrosting/assets": "^6.0.0",
+        "userfrosting/config": "~4.3.0",
+        "userfrosting/cache": "~4.3.0",
+        "userfrosting/fortress": "~4.3.0",
+        "userfrosting/i18n": "~4.3.0",
+        "userfrosting/session": "~4.3.0",
+        "userfrosting/support": "~4.3.0",
+        "vlucas/phpdotenv": "^3.4.0"
     },
     "autoload": {
         "files" : [

app/sprinkles/core/src/Bakery/MigrateCommand.php

@@ -14,6 +14,7 @@
 use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
 use UserFrosting\Sprinkle\Core\Bakery\Helper\ConfirmableTrait;
+use UserFrosting\Sprinkle\Core\Database\Migrator\Migrator;
 use UserFrosting\System\Bakery\BaseCommand;
 
 /**
@@ -79,13 +80,13 @@ protected function execute(InputInterface $input, OutputInterface $output)
         try {
             $migrated = $migrator->run(['pretend' => $pretend, 'step' => $step]);
         } catch (\Exception $e) {
-            $this->io->writeln($migrator->getNotes());
+            $this->displayNotes($migrator);
             $this->io->error($e->getMessage());
             exit(1);
         }
 
         // Get notes and display them
-        $this->io->writeln($migrator->getNotes());
+        $this->displayNotes($migrator);
 
         // If all went well, there's no fatal errors and we have migrated
         // something, show some success
@@ -128,4 +129,16 @@ protected function setupMigrator(InputInterface $input)
 
         return $migrator;
     }
+
+    /**
+     * Display migrator notes.
+     *
+     * @param Migrator $migrator
+     */
+    protected function displayNotes(Migrator $migrator)
+    {
+        if (!empty($notes = $migrator->getNotes())) {
+            $this->io->writeln($notes);
+        }
+    }
 }

app/sprinkles/core/src/Bakery/MigrateRefreshCommand.php

@@ -71,13 +71,13 @@ protected function execute(InputInterface $input, OutputInterface $output)
         try {
             $rolledback = $migrator->rollback(['pretend' => false, 'steps' => $steps]);
         } catch (\Exception $e) {
-            $this->io->writeln($migrator->getNotes());
+            $this->displayNotes($migrator);
             $this->io->error($e->getMessage());
             exit(1);
         }
 
         // Get notes and display them
-        $this->io->writeln($migrator->getNotes());
+        $this->displayNotes($migrator);
 
         // Stop if nothing was rolledback
         if (empty($rolledback)) {
@@ -88,7 +88,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
 
         // Run back up again
         $migrated = $migrator->run(['pretend' => false, 'step' => false]);
-        $this->io->writeln($migrator->getNotes());
+        $this->displayNotes($migrator);
 
         // If all went well, there's no fatal errors and we have migrated
         // something, show some success

app/sprinkles/core/src/Bakery/MigrateResetCommand.php

@@ -87,13 +87,13 @@ protected function performReset(InputInterface $input)
         try {
             $resetted = $migrator->reset($pretend);
         } catch (\Exception $e) {
-            $this->io->writeln($migrator->getNotes());
+            $this->displayNotes($migrator);
             $this->io->error($e->getMessage());
             exit(1);
         }
 
         // Get notes and display them
-        $this->io->writeln($migrator->getNotes());
+        $this->displayNotes($migrator);
 
         // Delete the repository
         if (!$pretend && $migrator->repositoryExists()) {

app/sprinkles/core/src/Bakery/MigrateRollbackCommand.php

@@ -80,13 +80,13 @@ protected function execute(InputInterface $input, OutputInterface $output)
                 $migrated = $migrator->rollback(['pretend' => $pretend, 'steps' => $steps]);
             }
         } catch (\Exception $e) {
-            $this->io->writeln($migrator->getNotes());
+            $this->displayNotes($migrator);
             $this->io->error($e->getMessage());
             exit(1);
         }
 
         // Get notes and display them
-        $this->io->writeln($migrator->getNotes());
+        $this->displayNotes($migrator);
 
         // If all went well, there's no fatal errors and we have migrated
         // something, show some success

app/sprinkles/core/src/Mail/Mailer.php

@@ -11,6 +11,8 @@
 namespace UserFrosting\Sprinkle\Core\Mail;
 
 use Monolog\Logger;
+use PHPMailer\PHPMailer\Exception as phpmailerException;
+use PHPMailer\PHPMailer\PHPMailer;
 
 /**
  * Mailer Class.
@@ -37,14 +39,14 @@ class Mailer
      * @param Logger  $logger A Monolog logger, used to dump debugging info for SMTP server transactions.
      * @param mixed[] $config An array of configuration parameters for phpMailer.
      *
-     * @throws \phpmailerException Wrong mailer config value given.
+     * @throws phpmailerException Wrong mailer config value given.
      */
     public function __construct($logger, $config = [])
     {
         $this->logger = $logger;
 
         // 'true' tells PHPMailer to use exceptions instead of error codes
-        $this->phpMailer = new \PHPMailer(true);
+        $this->phpMailer = new PHPMailer(true);
 
         // Configuration options
         switch ($config['mailer']) {
@@ -79,7 +81,7 @@ public function __construct($logger, $config = [])
                 }
                 break;
             default:
-                throw new \phpmailerException("'mailer' must be one of 'smtp', 'mail', 'qmail', or 'sendmail'.");
+                throw new phpmailerException("'mailer' must be one of 'smtp', 'mail', 'qmail', or 'sendmail'.");
         }
 
         // Set any additional message-specific options
@@ -97,7 +99,7 @@ public function __construct($logger, $config = [])
     /**
      * Get the underlying PHPMailer object.
      *
-     * @return \PHPMailer
+     * @return PHPMailer
      */
     public function getPhpMailer()
     {
@@ -113,7 +115,7 @@ public function getPhpMailer()
      * @param MailMessage $message
      * @param bool        $clearRecipients Set to true to clear the list of recipients in the message after calling send().  This helps avoid accidentally sending a message multiple times.
      *
-     * @throws \phpmailerException The message could not be sent.
+     * @throws phpmailerException The message could not be sent.
      */
     public function send(MailMessage $message, $clearRecipients = true)
     {
@@ -163,7 +165,7 @@ public function send(MailMessage $message, $clearRecipients = true)
      * @param MailMessage $message
      * @param bool        $clearRecipients Set to true to clear the list of recipients in the message after calling send().  This helps avoid accidentally sending a message multiple times.
      *
-     * @throws \phpmailerException The message could not be sent.
+     * @throws phpmailerException The message could not be sent.
      */
     public function sendDistinct(MailMessage $message, $clearRecipients = true)
     {