search

laravel-shift / factory-generator

MIT License
41 stars 20 forks source link

Fails on PostGIS enabled databases because geography isnt a known type of DBAL #12

Open securit opened 3 years ago

securit commented 3 years ago

Any chance you can add geography and geometry as valid types? There is a good discussion on this here https://laracasts.com/discuss/channels/general-discussion/doctrinedbal-how-to-add-data-types-when-using-xethronmigrations-generator

Doctrine\DBAL\Exception

Unknown database type geography requested, Doctrine\DBAL\Platforms\PostgreSQL100Platform may not support it.

at vendor/doctrine/dbal/lib/Doctrine/DBAL/Platforms/AbstractPlatform.php:454 450▕ 451▕ $dbType = strtolower($dbType); 452▕ 453▕ if (! isset($this->doctrineTypeMapping[$dbType])) { ➜ 454▕ throw new Exception( 455▕ 'Unknown database type ' . $dbType . ' requested, ' . static::class . ' may not support it.' 456▕ ); 457▕ } 458▕

  +28 vendor frames

29 artisan:35 Illuminate\Foundation\Console\Kernel::handle()

laravel on  Eight [?] is 📦 v1.9.4 via ⬢ v15.8.0 via 🐘 v8.0.3 on 🐳 v20.10.5 took 4s

jasonmccreary commented 3 years ago

Sure. I am not familiar with these personally. Do you want to try opening a PR? Or you can add more clarity around the column type and desired Faker data you wish to see this package generate.

securit commented 3 years ago

Geographic and geometry data types are standard types within Laravel. I guess the question should really be, why aren't they supported natively ;) https://laravel.com/docs/8.x/migrations#available-column-types

jasonmccreary commented 3 years ago

Thanks. But that doesn't really answer my question.

[what are] the column type and desired Faker data you wish to see this package generate?