From 3aa0d738116a06ec9059a69783ce73cf3bf9b964 Mon Sep 17 00:00:00 2001 From: Colin Adler Date: Thu, 18 Apr 2024 18:47:02 -0500 Subject: [PATCH] chore: fix down migration 196 (#13006) It didn't account for null values. --- .../000196_external_auth_providers_jsonb.down.sql | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/coderd/database/migrations/000196_external_auth_providers_jsonb.down.sql b/coderd/database/migrations/000196_external_auth_providers_jsonb.down.sql index e1a9b42cad..7d440dd5b3 100644 --- a/coderd/database/migrations/000196_external_auth_providers_jsonb.down.sql +++ b/coderd/database/migrations/000196_external_auth_providers_jsonb.down.sql @@ -11,11 +11,15 @@ CREATE OR REPLACE FUNCTION revert_migrate_external_auth_providers_to_jsonb(jsonb DECLARE result text[]; BEGIN - SELECT - array_agg(id::text) INTO result - FROM ( - SELECT - jsonb_array_elements($1) ->> 'id' AS id) AS external_auth_provider_ids; + IF jsonb_typeof($1) = 'null' THEN + result := '{}'; + ELSE + SELECT + array_agg(id::text) INTO result + FROM ( + SELECT + jsonb_array_elements($1) ->> 'id' AS id) AS external_auth_provider_ids; + END IF; RETURN result; END; $$;