Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Error: Column "id" is an identity column defined as GENERATED ALWAYS #199

Closed
ishanvyas22 opened this issue May 9, 2023 · 1 comment · Fixed by #200
Closed

Error: Column "id" is an identity column defined as GENERATED ALWAYS #199

ishanvyas22 opened this issue May 9, 2023 · 1 comment · Fixed by #200
Assignees
@pabloelcolombiano
Labels
bug Something isn't working

Comments

@ishanvyas22
Copy link
Contributor

ishanvyas22 commented May 9, 2023 •
edited
Loading

Currently postgres jobs are failing on PHP 8.0 & 8.2 with below error:

...

171) CakephpFixtureFactories\Test\TestCase\Scenario\FixtureScenarioTest::testLoadScenario with data set #2 ('TenAustralianAuthors', 10)
CakephpFixtureFactories\Error\FixtureScenarioException: Error in Factory CakephpFixtureFactories\Test\Factory\AuthorFactory.
 Message: SQLSTATE[428C9]: <<Unknown error>>: 7 ERROR:  cannot insert a non-DEFAULT value into column "id"
DETAIL:  Column "id" is an identity column defined as GENERATED ALWAYS.
HINT:  Use OVERRIDING SYSTEM VALUE to override. 

...

This is due to changes of phinx package in version 0.13.0 (PR: cakephp/phinx#2085).
@pabloelcolombiano pabloelcolombiano self-assigned this May 10, 2023
@pabloelcolombiano pabloelcolombiano added the bug Something isn't working label May 10, 2023
@pabloelcolombiano
Copy link
Collaborator

It is impossible with the actual phinx version to overwrite ID values.
See the issue here: cakephp/phinx#2193

A solution would be to add a ->disablePrimaryKeyOffset() to by-pass the random setting of the primary key and let the SQL engine manage the assignment.

ishanvyas22 added a commit to ishanvyas22/cakephp-fixture-factories that referenced this issue May 10, 2023
@ishanvyas22
Add ability to disable primary key generation by factory class
020f312 
Fixes vierge-noire#199
@ishanvyas22 ishanvyas22 mentioned this issue May 10, 2023
Merged
ishanvyas22 added a commit to ishanvyas22/cakephp-fixture-factories that referenced this issue May 10, 2023
@ishanvyas22
Add ability to disable primary key generation by factory class
3b0af36 
Fixes vierge-noire#199
pabloelcolombiano added a commit that referenced this issue May 11, 2023
@pabloelcolombiano
#199 Reduce PHP and CakePHP requirements
cde8991
pabloelcolombiano added a commit that referenced this issue May 11, 2023
@pabloelcolombiano
#199 Reduce PHP and CakePHP requirements
8c79319
pabloelcolombiano added a commit that referenced this issue May 11, 2023
@pabloelcolombiano
Merge pull request #205 from vierge-noire/#199-reduce-requirements
29f22be 
#199 Reduce PHP and CakePHP requirements
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pabloelcolombiano pabloelcolombiano

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add ability to disable primary key generation by factory class ishanvyas22/cakephp-fixture-factories
2 participants
@ishanvyas22 @pabloelcolombiano