Closed Bug 410823 Opened 17 years ago Closed 17 years ago

Bugzilla fails when editing or deleting a multi-select field value

Categories

(Bugzilla :: Administration, task)

3.1.1
task
Not set
major

Tracking

()

RESOLVED FIXED
Bugzilla 3.2

People

(Reporter: romaia, Assigned: romaia)

References

Details

Attachments

(1 file, 1 obsolete file)

When trying to edit the value of a multi-select option this error appears:

DBD::mysql::db do failed: Unknown column 'cf_multi' in 'where clause' [for Statement "UPDATE bugs SET cf_multi = ? WHERE cf_multi = ?"]
Attachment #295398 - Flags: review?(LpSolit)
Comment on attachment 295398 [details] [diff] [review]
check if the field is a multi-select and handle it properly

This fixes the problem when editing a value, but deleting one still fails:

DBD::mysql::db selectrow_array failed: Unknown column 'cf_audience' in 'where clause' [for Statement "SELECT COUNT(*) FROM bugs WHERE cf_audience = ?"] at /var/www/html/bugzilla/editvalues.cgi line 275

Please fix that in this bug as well.
Attachment #295398 - Flags: review?(LpSolit) → review+
Flags: blocking3.1.3+
Summary: Multi-select fields values are not editable. → Bugzilla fails when editing or deleting a multi-select field value
Depends on: 287330
We need a Selenium script testing all custom field formats.
Flags: testcase?
Attachment #295398 - Attachment is obsolete: true
Attachment #295404 - Flags: review?(LpSolit)
Comment on attachment 295404 [details] [diff] [review]
v2 - also fix error when deleting a value.

Works fine. Note that these || 0 are useless and can be safely removed. I will remove them on checkin. r=LpSolit
Attachment #295404 - Flags: review?(LpSolit) → review+
Flags: approval+
Checking in editvalues.cgi;
/cvsroot/mozilla/webtools/bugzilla/editvalues.cgi,v  <--  editvalues.cgi
new revision: 1.27; previous revision: 1.26
done
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: