Closed
Bug 131136
Opened 22 years ago
Closed 18 years ago
putting all SQL-statements into one file
Categories
(Bugzilla :: Bugzilla-General, enhancement, P5)
Bugzilla
Bugzilla-General
Tracking
()
VERIFIED
WONTFIX
People
(Reporter: kiril.sramko, Assigned: nobody)
Details
I'm just working on bug# 106592 and I've got change a lot of the SQL-statements. Bugzilla has 564 SendSQL()-calls, distributed thru all files. And all tables and fields are in plain-text and not as variables writen. And so I've got the idea to put all statements into one files as methods, whitch could be call. This got the advanteges: - this file could be generate, depending on the database, the user has - all statemens are central and if you change the database, you jousst have to work in one file. So, got anybody any other ideas or critics?
Comment 1•22 years ago
|
||
-> bugzilla, but I'd just like to say that that would make everything impossible to follow, with no real benefit that grep can't give you.
Assignee: gerv → justdave
Component: Bugzilla Helper → Bugzilla-General
Product: Webtools → Bugzilla
Version: other → unspecified
Reporter | ||
Comment 2•22 years ago
|
||
But if you want to change Bugzilla that way, that it work on any DBMS you have to make specific statements to take benefit of the different DBMS's. So why don't put the statements in significant labelled procedures, put these in one file and make the rest of the Bugzilla-sorce independen of the DBMS?
Comment 3•22 years ago
|
||
I'm definitely not convinced. I think the DB-independence work is all we need in this area. Gerv
Updated•22 years ago
|
Priority: -- → P5
Target Milestone: --- → Future
Comment 4•22 years ago
|
||
Reassigning all of my "future" targetted bugs to indicate that I'm not presently working on them, and someone else could feel free to work on them.
Comment 5•22 years ago
|
||
Reassigning all of my "future" targetted bugs to indicate that I'm not presently working on them, and someone else could feel free to work on them. (sorry for the spam if you got this twice, it didn't take right the first time)
Assignee: justdave → nobody
Updated•18 years ago
|
QA Contact: mattyt-bugzilla → default-qa
Comment 6•18 years ago
|
||
We are moving most (all?) SQL statements into .pm modules. And our code is already compatible with at least 2 DB servers: MySQL and PostgreSQL. WONTIFX!
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → WONTFIX
Target Milestone: Future → ---
Comment 7•18 years ago
|
||
Agreed--this will not be done, because we're moving that code mostly into modules, which makes more sense.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•