Declaration of BackupManager\Laravel\Laravel51Compatibility::table(array $headers, $rows, $style = 'default') should be compatible with Illuminate\Consol e\Command::table($headers, $rows, $style = 'default')

[4sh3]

Hi friends,

I am using windows 10 with php 7.1, I am trying to update laravel 5.4* to 5.5 and when I do composer update to my app:

[ErrorException] Declaration of BackupManager\Laravel\Laravel51Compatibility::table(array $headers, $rows, $style = 'default') should be compatible with Illuminate\Consol e\Command::table($headers, $rows, $style = 'default')

Script php artisan optimize handling the post-update-cmd event returned with error code 1

What can i do?

Thanks for your help

[4sh3]

Solved, I just changed the function: public function table($headers, $rows, $style = 'default') in BackupManager\Laravel\Laravel51Compatibility

[ShawnMcCool]

I think that perhaps a better solution might be to create Laravel 55 compatibility.

[SergeyMiracle]

@ShawnMcCool PR was made several days ago...

[SergeyMiracle]

@ShawnMcCool any news on PR #87 What else needs to be done?

[whoacowboy]

Ironically I get this error upgrading from 5.4 to 5.5.

Declaration of BackupManager\Laravel\Laravel55Compatibility::table($headers, $rows, 
$style = 'default') should be compatible with Illuminate\Console\Command::table($headers, 
$rows, $tableStyle = 'default', array $columnStyles = Array)

if I change this

public function table($headers, $rows, $style = 'default')

to

public function table($headers, $rows, $style = 'default', array $columnStyles = [])

it works.

[xembill]

in Laravel55Compatibility public function table($headers, $rows, $style = 'default', array $columnStyles = [])

that works fine...

[matthewscalf]

Hoping this gets pushed in soon, thanks!

[ShawnMcCool]

Submit a PR =) I'll merge it in.

[whoacowboy]

@ShawnMcCool It's here. Can you tag it too?

[4sh3]

Please solve it in the repository because when someone works with heroku should do composer updates on the server and is so hard change this each time you push the code correct this...

[4sh3]

Error on Laravel55Compatibility:

[ErrorException] Declaration of BackupManager\Laravel\Laravel55Compatibility::table($headers , $rows, $style = 'default') should be compatible with Illuminate\Console\C ommand::table($headers, $rows, $tableStyle = 'default', array $columnStyles = Array)

[ShawnMcCool]

Ok, I'm not sure why anyone would do composer update on a server. But, that's beside the point.

@4sh3 did you see this? https://github.com/backup-manager/laravel/pull/91

[croucour]

do the same for lumen (5.5) can be a great thing no ?

[ShawnMcCool]

I'll honestly merge anything at this point.

[scottybo]

Still having this issue on 5.4.. For anyone else wanting to modify vendor files (shudder), change Laravel51Compatibility to:

public function table($headers, $rows, $style = 'default', array $columnStyles = [])

[ShawnMcCool]

There are different compatibility files for each version of Laravel's API change. This ticket is about 5.1. What happens when you use the 54 compatibility file?

[dennisvandenende]

I changed following files: Laravel5DbListCommand Laravel5DbBackupCommand Laravel5DbRestoreCommand from use Laravel51Compatibility; into use Laravel55Compatibility;

And works like a charm

cheers