search

mikaeljorhult / hydrofon

Equipment booking system.
MIT License
23 stars 11 forks source link

Laravel 7.x Shift #55

Closed mikaeljorhult closed 4 years ago

mikaeljorhult commented 4 years ago

This pull request includes the changes for upgrading to Laravel 7.x. Feel free to commit any additional changes to the shift-27763 branch.

Before merging, you need to:

If you need help with your upgrade, check out the Human Shifts. You may also join the Shifty Coders Slack workspace to level-up your Laravel skills.

mikaeljorhult commented 4 years ago

:information_source: Laravel 7 upgraded to Symfony 5 which passes instances of the Throwable interface instead of Exception instances to core components like the Exceptions\Handler.

Shift automated this change. However, if you receive a Throwable type mismatch error there may be additional references you need to convert.

mikaeljorhult commented 4 years ago

:information_source: Laravel 7 moved the Authentication components into a separate laravel/ui package, which includes the make commands as well as the traits related to authentication.

Shift added this dependency for convenience. However, if you do not plan to use the Authentication components, you may remove this dependency as well as the Auth controllers from your application.

mikaeljorhult commented 4 years ago

:warning: The MAIL_DRIVER environment variable was renamed in Laravel 7 to MAIL_MAILER in order to support multiple mail configurations.

Shift attempted to automation this change, but you should review any additional environment configuration and update to the new variable name.

mikaeljorhult commented 4 years ago

:information_source: Shift updated your dependencies for Laravel 7. While many of the popular packages are reviewed, you may have to update additional packages in order for your application to be compatible with Laravel 7.

Watch dealing with dependencies for tips on handling package incompatibilities.

mikaeljorhult commented 4 years ago

:information_source: Laravel 7 removed some of the PHPUnit configuration in favor of the defaults. While this is not a required changed, it is a good opportunity to compare your phpunit.xml with the default Laravel 7 version to modernize your configuration.

mikaeljorhult commented 4 years ago

:warning: In Laravel 7, all of the assertSee methods now automatically escape the expected values.

If you were manually escaping the values passed to these methods, you no longer need do so. If you were asserting values containing HTML that should not be escaped, you may set the new escaped argument to false.

Shift found the instances of these assertions within:

mikaeljorhult commented 4 years ago

:warning: Laravel 7 uses a new date format when serializing models. The previous format was 2019-12-02 20:01:00. Now, dates are serialized using an ISO-8601 compatible date format of 2019-12-02T20:01:00.283041Z in UTC.

This does not affect how dates are stored. Only how they are serialized when using the toArray or toJson Eloquent methods.

If you need to preserve the previous format, you may override the serializeDate method on your model. Review the Date Serialization section of the Upgrade Guide for more details.

mikaeljorhult commented 4 years ago

:information_source: All of the underlying Symfony components used by Laravel have been upgraded to Symfony 5. If you are directly interacting with Symfony components, you should review the Symfony change log for additional changes.

mikaeljorhult commented 4 years ago

:tada: Congratulations, you're now running the latest version of Laravel.

The following Shifts may be used ensure your application is fully upgraded and following practices recommended by Laravel and the community:

You may run these Shifts and stay upgraded forever as well by subscribing to a Shifty Plans.