Fixed primary key too long identifier name

Xethron / migrations-generator

Laravel Migrations Generator: Automatically generate your migrations from an existing database schema.

MIT License

3.32k stars 588 forks source link

Fixed primary key too long identifier name #124

Open rtorralba opened 7 years ago

rtorralba commented 7 years ago

When index is primary the index name is always 'PRIMARY', then is not necessary to check isDefaultIndexName. If you don't set index name, try to make a index identifier joining (implode) all fields of the key and if this field is bigger than 64 characters it fail

rtorralba commented 7 years ago

I executed your project phpunit tests locally and fails too:

There was 1 error:

1) Xethron\MigrationsGenerator\MigrationsGeneratorServiceProviderTest::registers_migrations_generator BadMethodCallException: Method Mockery_0_stdClass::singleton() does not exist on this mock object

I can't access to the travis execution log for view what exactly fails.