- Notifications
You must be signed in to change notification settings - Fork109
Open
Description
Bug report
I switched to using the vscode extension that relies on this server from UniqueVision's unmaintained extension and despite initial hiccups, it is working right now and I am very grateful for it.
- I confirm this is a bug with Supabase, not with my own application.
- I confirm I have searched theDocs, GitHubDiscussions, andDiscord.
Describe the bug
Most of my extension code is interpreted correctly and good feedback is given by the server and the syntax checker except for one instance: (an auditing trigger function)
create or replacefunctionstellenangebote_update_audit_handler () returns trigger language plpgsqlas $audit$declare column_namestext[] := array['refnr','beruf','titel','ort','plz','region','land','lat','lon','arbeitgeber','veroeffentlichungsdatum','modifikationstimestamp','eintrittsdatum','kundennummerhash','externeurl' ]; ttext :=null; is_equalboolean;begin foreach tin array column_names loop execute format('select ($1).%1$I is not distinct from ($2).%1$I', t) using old, new into is_equal; if not is_equal then execute format('insert into stellenangebote_update_audit''(job_refnr, column_name, old_value, new_value) values''($1,''%1$s'', ($2).%1$I::text, ($3).%1$I::text)', t) usingold.refnr, old, new; end if; end loop; return new;end $audit$;
Note that thestellenangebote and the audit tables are defined as follows:
createtableif not exists stellenangebote_update_audit ( createdtimestampprimary key defaultcurrent_timestamp, job_refnrtextnot null, column_nametextnot null, old_valuetext, new_valuetext);createtableif not exists stellenangebote ( refnrvarchar(32)not null, beruftextnull, titeltextnull, orttextnull, plz ValidPLZnull, regiontextnull, landtextnull, latnumericnull, lonnumericnull, arbeitgebertextnull, veroeffentlichungsdatumdatenull, modifikationsTimestamptimestampnull, eintrittsdatumdatenull, kundennummerHashtextnull, externeUrltextnull, createdtimestampnot null defaultcurrent_timestamp, updatedtimestampnot null defaultcurrent_timestamp, deletedtimestampnull, archive_partitionsmallintnot null default0check (archive_partition>=0)) partition by range(archive_partition);
To Reproduce
NA
Expected behavior
The code works great so there should not be an error. It seems as if the syntax checker things that the array of column names is a column name which is not the case. This may have also to do with the fact I am using a partitioned table, not entirely sure.
Screenshots

System information
- OS: Linux
Additional context
NA