Laravel 8/9

[bryanrscott]

Suggested updates for laravel 8. I am suggesting a new branch as laravel 8 upgrades to symphony 5.
Includes edits by @npostman
Welcome any additional suggestions.

[bryanrscott]

Not yet ready

[bryanrscott]

Confirmed functional. Misread configuration

[npostman]

Implemented and confirmed to work with L8.6.0. Thanks @bryanrscott

[bryanrscott]

Acknowledgement link for @npostman
#57

[ahosker]

Up and running in Laravel 8. Thanks @bryanrscott

[JorisDebonnet]

This line is a bit confusing because of the .x at the start.
I recommend changing it into "For Laravel 7.0 <= version <= 8.x" 😊
And while you're at it, I recommend also changing that second line into "For Laravel 5.3 <= version ..." then...

Less brain cells wasted for developers trying to figure out which branch they need 😊

[bryanrscott]

@JorisDebonnet Agree and have updated README as suggested

[JorisDebonnet]

@JorisDebonnet Agree and have updated README as suggested

Okay, uh, but it's not 100% as I meant now 😄
My suggestion for the 'mathematically most correct' version was this:

#### For Laravel <= 5.2, please use the [v1 branch](https://github.com/squareboat/sneaker/tree/v1)!
#### For Laravel 5.3 <= version <= 6.x, please use the [v5 branch](https://github.com/squareboat/sneaker/tree/v5)!
#### For Laravel 7.0 <= version <= 8.x, please use the [v7 branch](https://github.com/squareboat/sneaker/tree/v7)!

Alternatively, 'plain English' could be even easier to read:

#### For Laravel up to 5.2, please use the [v1 branch](https://github.com/squareboat/sneaker/tree/v1)!
#### For Laravel 5.3 to 6.x, please use the [v5 branch](https://github.com/squareboat/sneaker/tree/v5)!
#### For Laravel 7 and up, please use the [v7 branch](https://github.com/squareboat/sneaker/tree/v7)!

Or did I misunderstand, and does Laravel 7 require a separate branch from Laravel 8+? Then I guess the last line would need to say "for Laravel 7.x, please use ..."

[bryanrscott]

Yes Laravel 8 would need a seperate branch.

[jvandervalk]

Hopefully Laravel support for version 8 can be added soon.

[JorisDebonnet]

Hopefully Laravel support for version 8 can be added soon.

For what it's worth, here's how to use bryanrscott's fork in your own project (that's what I'm doing at the moment) in order to use Sneaker with Laravel 8:

composer.json

{
    "require": {
        "squareboat/sneaker": "dev-master"
    },
    "repositories": [ { "type": "vcs", "url": "https://github.com/bryanrscott/sneaker" } ],
}

[michaelheumann]

Hi, I just ported my application to Laravel 8 and Sneaker does not seem to work. I have run into this same issue when porting to Laravel 6 and 7, and it was due to the composer version (I had to downgrade composer to 1.10). But now this happens both with composer 1 and 2.

This is the message at the end of composer install:

Generating optimized autoload files
> Illuminate\Foundation\ComposerScripts::postAutoloadDump
> @php artisan package:discover

In Container.php line 835:
                                          
  Target class [sneaker] does not exist.  
                                          

In Container.php line 833:
                                
  Class sneaker does not exist  

Do ypu have a workaround for this?

[bryanrscott]

@michaelheumann I would recommend
a) use composer 2
b) uninstall sneaker, including commenting out the lines in app/Exceptions/Handler.php
c) composer update
d) install sneaker via composer
e) complete install by uncommenting lines in app/Exceptions/Handler.php

[npostman]

@bryanrscott Thanks for keeping this nice package up to date!

[eugenefvdm]

Curious if this project has stalled or if it's possible to install in Laravel 8, or even better, Laravel 9?

[bryanrscott]

@ eugenevdm the forked version supports Laravel 9

[bryanrscott]

@ eugenevdm the forked version supports Laravel 9