347116 - Move field-definition code from checksetup.pl into a module

Status: RESOLVED FIXED

(Bugzilla :: Installation & Upgrading, enhancement)

Description

[Max Kanat-Alexander]

Right now checksetup.pl does a lot of creation of fields for the fielddefs table in one block.

Because of the way we have it structured, it probably actually should just manage that whole table in one block, before the schema changes.

Anyhow, it should all be moved into a module, probably Bugzilla::Field.

Comment 1

[Max Kanat-Alexander]

Attachment: v1

Okay, here it is. I just moved all the code into Bugzilla::Field, because it seemed to make sense for it to be there, and it's not a gigantic chunk of code.

Comment 2

[Max Kanat-Alexander]

Requesting approval directly, as module owner.

Comment 3

[Max Kanat-Alexander]

Checking in checksetup.pl;
/cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v  <--  checksetup.pl
new revision: 1.535; previous revision: 1.534
done
Checking in Bugzilla/Field.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Field.pm,v  <--  Field.pm
new revision: 1.16; previous revision: 1.15
done
Checking in Bugzilla/Install/DB.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Install/DB.pm,v  <--  DB.pm
new revision: 1.3; previous revision: 1.2
done