Closed Bug 163299 Opened 21 years ago Closed 21 years ago

Can't change cc accessible checkbox

Categories

(Bugzilla :: Creating/Changing Bugs, defect)

2.17
x86
Linux
defect
Not set
blocker

Tracking

()

RESOLVED FIXED
Bugzilla 2.18

People

(Reporter: bbaetz, Assigned: bbaetz)

Details

(Keywords: regression)

Attachments

(1 file)

I stuffed up the fielddefs fieldname for bug 161865. Patch coming
Attached patch oopsSplinter Review
Who wants to review this?
Keywords: patch, regression, review
Target Milestone: --- → Bugzilla 2.18
Comment on attachment 95733 [details] [diff] [review]
oops

"oops."

r=preed
Attachment #95733 - Flags: review+
Comment on attachment 95733 [details] [diff] [review]
oops

2xr from Joel assuming that bbaetz saw this fix the original problem. 
It's clear that this is the right thing to do.
Attachment #95733 - Flags: review+
Sorry I didn't get here sooner. Hmm. How annoying. It's stuff like this that
makes checksetup.pl 3221 lines long. :-|

It's only been like this for a week. Can we leave the fixup hack in for a month
and then remove it, on the basis that if someone updated this week, that are
likely to be a developer, and will update again in the next month?

Gerv
gerv: I dunno - see what developers@ thinks? HAving it there won't harm
anything; it just shows up on the query checkbox, then.

Checked in, but leaving open to possibly remove the hack later.
Status: NEW → ASSIGNED
OK, this got checked in ages ago. Lets keep the line in - its not doing any harm.
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.