Skip to content
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 5.2 Shift #17

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Laravel 5.2 Shift #17

wants to merge 7 commits into from

Conversation

laravel-shift
Copy link
Owner

This pull request includes the changes for upgrading to Laravel 5.2. Feel free to commit any additional changes to the laravel-5.2-shift branch.

Before merging, you should:

  • Checkout the laravel-5.2-shift branch
  • Review all pull request comments for additional changes
  • Run composer update (if the pre-commands fail, add --no-scripts)
  • Thoroughly test your application

If you would like more help with your upgrade, check out the new human services from Laravel Shift.

The Laravel framework adopts the PSR-2 coding style in version 5.1.
Laravel apps *should* adopt this coding style as well. Read the
[PSR-2 coding style guide][1] for more details and check out [PHPCS][2]
to use as a code formatting tool.

[1]: https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-2-coding-style-guide.md
[2]: https://github.com/squizlabs/PHP_CodeSniffer
Laravel 5.2 adjusts the `Guard` object used within middleware. In
addition, new `can` and `throttles` middleware were added.
Laravel 5.2 introduces a *Guard* object to the Authentication
component. In doing so, the configuration options and default
Middleware changed.
@laravel-shift
Copy link
Owner Author

⚠ Laravel 5.2 adds Middleware Groups for convenience. Since this may affect your routes and controllers, I did not automate this upgrade. You should read about Middleware Groups to understand this change.

When you are ready to upgrade to Middleware Groups, you should:

  • Compare your app/Http/Kernel.php with the 5.2 version
  • Compare your app/Providers/RouteServiceProvider.php with the 5.2 version
  • Review any usages of the web middleware within your application

@laravel-shift
Copy link
Owner Author

ℹ Laravel 5.2 changed the value of $redirectTo from /home to / in app/Http/Controllers/Auth/AuthController.php. I did not make this change since you may have customized it for your app. However, you may wish to change this if you are not using Auth or if you want to follow the latest Laravel conventions.

@laravel-shift
Copy link
Owner Author

ℹ Laravel 5.2 updated the database session driver. If you are using the database driver, you need to add the user_id (nullable integer), ip_address (nullable string), and user_agent (text) columns to your session database table.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant