Mark Scherer
Mark Scherer
Ping @MasterOdin
I wonder if we might need a test case here to back things up.
Postgres is actually already failing on this: LINE 1: ... "sort" INTEGER NOT NULL DEFAULT '0', "lat" REAL (10) NULL,... So, yeah, scale and precision should be omitted for float,...
Note: If https://github.com/cakephp/migrations/issues/647 gets traction, then this is actually very likely going to be possible. Just wanted to cross-link this for reference.
I started the default fields stuff: https://github.com/vierge-noire/cakephp-fixture-factories/pull/243 Feel free to chime in for more cool things.
I am not sold on the singular part however bin/cake bake fixture_factory Events It also uses plural input (table, sure), and it creates a factory that can create multiple ones....
But wouldnt other specific CakePHP elements like Helpers also need the same adjustments? Only adjusting one element seems a bit inconsistent.
Rest seems fine Some few States dont have geo data however My report says ~60 of 5000: ``` Missing lat/lng for Saint George Parish of country Antigua And Barbuda, skipping...
Another small inconsistency I found when importing, was: - City `Ibiza` geocodes to state `Balearic Islands` (`IB`) But it is different here both in name and code in the data...
Is there no way to fix the actual phpcs rule to not throw wrong error?