Closed
Bug 171127
Opened 22 years ago
Closed 21 years ago
bug_email.pl is broken
Categories
(Bugzilla :: Bugzilla-General, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.18
People
(Reporter: jussi, Assigned: alex)
References
Details
(Keywords: helpwanted, regression)
Attachments
(1 file)
9.45 KB,
patch
|
justdave
:
review+
|
Details | Diff | Splinter Review |
The schema change from the new groups system broke bug_email.pl. The relevant part from the script output is: select bit from groups where name='ReadInternal': Unknown column 'bit' in 'field list' at globals.pl line 246. There is propably more references to the old groups table.
Comment 1•22 years ago
|
||
Can we consider this a feature? Please? :)
Keywords: helpwanted,
regression
Target Milestone: --- → Bugzilla 2.18
Comment 2•22 years ago
|
||
Same problem here. Any news on this bug?
Comment 3•22 years ago
|
||
ccing the people that have been working with the new groups schema...
Comment 4•21 years ago
|
||
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 :-)
Comment 5•21 years ago
|
||
I've found that if I change all occurances of "bit" to "id" in each of the SQL statements in bug_email.pl, it will work. Well, I didn't test it very well, but at least it accepts my emails and puts them in the database.
Assignee | ||
Comment 6•21 years ago
|
||
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.
Assignee | ||
Updated•21 years ago
|
Attachment #129362 -
Flags: review?
Comment 7•21 years ago
|
||
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+
Comment 8•21 years ago
|
||
Checking in contrib/bug_email.pl; /cvsroot/mozilla/webtools/bugzilla/contrib/bug_email.pl,v <-- bug_email.pl new revision: 1.16; previous revision: 1.15 done
Status: NEW → RESOLVED
Closed: 21 years ago
Flags: approval+
Resolution: --- → FIXED
Updated•18 years ago
|
Assignee: justdave → alex
Updated•12 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•