BookStackApp / BookStack Public
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Laravel 7.x Shift #3011
Laravel 7.x Shift #3011
Conversation
Shift automatically applies the Laravel coding style - which uses the PSR-2 coding style as a base with some minor additions. You may customize the adopted coding style by adding a [PHP CS Fixer][1] or [PHP CodeSniffer][2] config to your project root. Feel free to use [Shift's Laravel ruleset][3] to help you get started. For more information on customizing the code style applied by Shift, [watch this short video][4]. [1]: https://github.com/FriendsOfPHP/PHP-CS-Fixer [2]: https://github.com/squizlabs/PHP_CodeSniffer [3]: https://gist.github.com/laravel-shift/cab527923ed2a109dda047b97d53c200 [4]: https://laravelshift.com/videos/shift-code-style
PHP 5.5.9+ adds the new static `class` property which provides the fully qualified class name. This is preferred over using class name strings as these references are checked by the parser.
|
Shift recommends changing this to align with Laravel and improve its automation. After doing so, you are welcome to request a rerun of this Shift. |
|
Shift attempted to automate these changes. But you should compare your |
|
Shift automated this change. However, if you receive a |
|
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 |
|
Shift automated this change. However, you should review these changes as well as check your application for any additional instances.
|
|
Shift attempted to automate this change, but you should review any additional environment configuration and update to the new variable name. |
|
Watch dealing with dependencies for tips on handling package incompatibilities. |
|
|
|
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 Shift found the instances of these assertions within:
|
|
|
|
This does not affect how dates are stored. Only how they are serialized when using the If you need to preserve the previous format, you may override the |
|
|
|
|
This pull request includes the changes for upgrading to Laravel 7.x. Feel free to commit any additional changes to the
shift-51217branch.Before merging, you need to:
shift-51217branchcomposer update(if the scripts fail, add--no-scripts)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.
The text was updated successfully, but these errors were encountered: