fix: insert with missing=default uses column default before using domain default
This commit is contained in:
committed by
Steve Chavez
parent
0d640442b1
commit
56c14474da
@@ -25,6 +25,7 @@ This project adheres to [Semantic Versioning](http://semver.org/).
|
|||||||
- #3727, Clarify "listening" logs - @steve-chavez
|
- #3727, Clarify "listening" logs - @steve-chavez
|
||||||
- #3795, Clarify `Accept: vnd.pgrst.object` error message - @steve-chavez
|
- #3795, Clarify `Accept: vnd.pgrst.object` error message - @steve-chavez
|
||||||
- #3779, Always log the schema cache load time - @steve-chavez
|
- #3779, Always log the schema cache load time - @steve-chavez
|
||||||
|
- #3706, Fix insert with `missing=default` uses default value of domain instead of column - @taimoorzaeem
|
||||||
|
|
||||||
### Changed
|
### Changed
|
||||||
|
|
||||||
|
|||||||
@@ -622,7 +622,7 @@ tablesSqlQuery =
|
|||||||
d.description AS description,
|
d.description AS description,
|
||||||
-- typbasetype and typdefaultbin handles `CREATE DOMAIN .. DEFAULT val`, attidentity/attgenerated handles generated columns, pg_get_expr gets the default of a column
|
-- typbasetype and typdefaultbin handles `CREATE DOMAIN .. DEFAULT val`, attidentity/attgenerated handles generated columns, pg_get_expr gets the default of a column
|
||||||
CASE
|
CASE
|
||||||
WHEN t.typbasetype != 0 THEN pg_get_expr(t.typdefaultbin, 0)
|
WHEN (t.typbasetype != 0) AND (ad.adbin IS NULL) THEN pg_get_expr(t.typdefaultbin, 0)
|
||||||
WHEN a.attidentity = 'd' THEN format('nextval(%L)', seq.objid::regclass)
|
WHEN a.attidentity = 'd' THEN format('nextval(%L)', seq.objid::regclass)
|
||||||
WHEN a.attgenerated = 's' THEN null
|
WHEN a.attgenerated = 's' THEN null
|
||||||
ELSE pg_get_expr(ad.adbin, ad.adrelid)::text
|
ELSE pg_get_expr(ad.adbin, ad.adrelid)::text
|
||||||
|
|||||||
@@ -572,6 +572,15 @@ spec actualPgVersion = do
|
|||||||
, matchHeaders = ["Preference-Applied" <:> "missing=default, return=representation"]
|
, matchHeaders = ["Preference-Applied" <:> "missing=default, return=representation"]
|
||||||
}
|
}
|
||||||
|
|
||||||
|
it "inserts a COLUMN default before a DOMAIN default with missing=default" $
|
||||||
|
request methodPost "/evil_friends_with_column_default?columns=id,name" [("Prefer", "return=representation"), ("Prefer", "missing=default")]
|
||||||
|
[json| { "name": "Demon" } |]
|
||||||
|
`shouldRespondWith`
|
||||||
|
[json| [{"id": 420, "name": "Demon"}] |]
|
||||||
|
{ matchStatus = 201
|
||||||
|
, matchHeaders = ["Preference-Applied" <:> "missing=default, return=representation"]
|
||||||
|
}
|
||||||
|
|
||||||
it "inserts json that has duplicate keys" $ do
|
it "inserts json that has duplicate keys" $ do
|
||||||
request methodPost "/tbl_w_json" [("Prefer", "return=representation")]
|
request methodPost "/tbl_w_json" [("Prefer", "return=representation")]
|
||||||
[json| { "data": { "a": 1, "a": 2 }, "id": 3 } |]
|
[json| { "data": { "a": 1, "a": 2 }, "id": 3 } |]
|
||||||
|
|||||||
Vendored
+5
@@ -3274,6 +3274,11 @@ create table evil_friends(
|
|||||||
, name text
|
, name text
|
||||||
);
|
);
|
||||||
|
|
||||||
|
create table evil_friends_with_column_default(
|
||||||
|
id devil_int default 420
|
||||||
|
, name text
|
||||||
|
);
|
||||||
|
|
||||||
create table bets (
|
create table bets (
|
||||||
id int
|
id int
|
||||||
, data_json json
|
, data_json json
|
||||||
|
|||||||
Reference in New Issue
Block a user