Closed Bug 171127 Opened 19 years ago Closed 18 years ago is broken


(Bugzilla :: Bugzilla-General, defect)

Windows 2000
Not set



Bugzilla 2.18


(Reporter: jussi, Assigned: alex)



(Keywords: helpwanted, regression)


(1 file)

The schema change from the new groups system broke The relevant
part from the script output is:

select bit from groups where name='ReadInternal': Unknown column 'bit' in 'field
list' at line 246.

There is propably more references to the old groups table.
Can we consider this a feature? Please? :)
Target Milestone: --- → Bugzilla 2.18
Same problem here. Any news on this bug?
ccing the people that have been working with the new groups schema...
CCing Dan Berlin, who from stuff I've seen around, I know is actually using the
bug_email stuff.  I'd appreciate any input or fixes you'd be willing to provide
on these bugs :-)
I've found that if I change all occurances of "bit" to "id" in each of the SQL
statements in, it will work. Well, I didn't test it very well, but
at least it accepts my emails and puts them in the database.
T.R. Fullhart had a partial fix which I have now completed.  His work-around
simply allowed the bug to be entered into the database but would not handle any
groupset nor fix the processmail issue.  My proposed patch fixes both.

Only one caveat, and that is the bug may be entered into a group which the user
has no access rights to.  I did not bother adding this check since this
interface is via email and email is dead easy to fake up.  If the user does
enter an incorrect group, then the groups they are listed in will be given, so
if anyone needs this additional check, it should be easy enough to add.
Attachment #129362 - Flags: review?
Comment on attachment 129362 [details] [diff] [review]
Fixes group and mail problem

patch applies and it compiles.	Generally looks good, but haven't tested, but
this is only contrib anyway. :)
Attachment #129362 - Flags: review? → review+
Checking in contrib/;
/cvsroot/mozilla/webtools/bugzilla/contrib/,v  <--
new revision: 1.16; previous revision: 1.15
Closed: 18 years ago
Flags: approval+
Resolution: --- → FIXED
Assignee: justdave → alex
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.