-
-
Notifications
You must be signed in to change notification settings - Fork 80
Closed
Labels
Description
Postgres sequence can use other schema than public. In this case NEXTVAL and CURRVAL should include Schema name like NEXTVAL("schema"."sequence_name"). In standard approach result of SequenceFeature class was NEXTVAL("schema.sequence_name") which is inproper behavior. I added checking, if the $sequenceName is instance of TableIdentifier. In this case NEXTVAL and CURRVALL is modified.
Provide a narrative description of what you are trying to accomplish:
-
Are you fixing a bug?
- Detail how the bug is invoked currently.
- Detail the original, incorrect behavior.
- Detail the new, expected behavior.
- Base your feature on the
master
branch, and submit against that branch. - Add a regression test that demonstrates the bug, and proves the fix.
- Add a
CHANGELOG.md
entry for the fix.
-
Are you creating a new feature?
- Why is the new feature needed? What purpose does it serve?
- How will users use the new feature?
- Base your feature on the
develop
branch, and submit against that branch. - Add only one feature per pull request; split multiple features over multiple pull requests
- Add tests for the new feature.
- Add documentation for the new feature.
- Add a
CHANGELOG.md
entry for the new feature.
-
Is this related to quality assurance?
-
Is this related to documentation?
Originally posted by @tkonop at zendframework/zend-db#378