Closed Bug 285403 Opened 20 years ago Closed 20 years ago

LearnAboutColumns does not work on PostgreSQL

Categories

(Bugzilla :: Bugzilla-General, defect, P1)

2.19.2
defect

Tracking

()

RESOLVED FIXED
Bugzilla 2.20

People

(Reporter: mkanat, Assigned: mkanat)

References

Details

Attachments

(1 file)

LearnAboutColumns uses SHOW COLUMNS, which of course doesn't work. Thankfully, this is easy to handle. We can either make a Bugzilla::DB function that calls the Schema to do this, or we can use some code that I wrote (but never posted) when I was re-writing versioncache entirely. :-)
Attachment #176851 - Flags: review?(Tomas.Kopal)
Status: NEW → ASSIGNED
Target Milestone: --- → Bugzilla 2.20
Priority: -- → P1
Comment on attachment 176851 [details] [diff] [review] Change LearnAboutColumns to use the Schema This is obviously correct, but it breaks the assertion we made that the schema will be accessed only by checksetup during upgrade. I suppose it's fine for now, we can take another shot at this later, if neccessary...
Attachment #176851 - Flags: review?(Tomas.Kopal) → review+
(In reply to comment #2) > (From update of attachment 176851 [details] [diff] [review] [edit]) > This is obviously correct, but it breaks the assertion we made that the schema > will be accessed only by checksetup during upgrade. Hrm. I made that assertion only temporarily anyhow. And we're still accessing through Bugzilla::DB. So I think we're OK. :-) > I suppose it's fine for > now, we can take another shot at this later, if neccessary... Eventually it can use the _bz_real_schema from the serialization patches, to be a more useful sub.
Flags: approval?
Flags: approval? → approval+
Checking in globals.pl; /cvsroot/mozilla/webtools/bugzilla/globals.pl,v <-- globals.pl new revision: 1.314; previous revision: 1.313 done Checking in Bugzilla/DB.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/DB.pm,v <-- DB.pm new revision: 1.30; previous revision: 1.29 done
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: