Closed Bug 232154 Opened 19 years ago Closed 19 years ago
After bmo upgrade, I lose my previous column configuration
My old cookies.txt says: bugzilla.mozilla.org FALSE / FALSE 1877472088 COLUMNLIST severity platform owner status resolution component summary The new one says: bugzilla.mozilla.org FALSE / FALSE 2145916809 COLUMNLIST bug_severity%20assigned_to%20bug_status%20resolution%20component%20short_short_desc
The <space> => %20 is OK, I think - its just extra CGI.pm escaping I'm more interested in the severity => bug_severity change. Someone eeds to stop chaning filed names every release, and add backwards compat code when they do.....
In fact some code was written to deal with old field names when these were changed back in bug 176461, but the code was buggy, removing spaces between columns in columnlist so that some column names got run together and thus didn't match any existing columns. Here's the fix.
Assignee: justdave → myk
Status: NEW → ASSIGNED
Attachment #139887 - Flags: review?(bbaetz)
Comment on attachment 139887 [details] [diff] [review] patch v1: fixes problem OK. Don't all three need a \b at the end? It doesn't make a difference now, but it could one day. r=bbaetz either way.
Attachment #139887 - Flags: review?(bbaetz) → review+
Checking in buglist.cgi; /cvsroot/mozilla/webtools/bugzilla/buglist.cgi,v <-- buglist.cgi new revision: 1.242; previous revision: 1.241 done
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
leading and trailing \b for all column names to improve robustness per bbaetz' suggestion in bug and on IRC.
Attachment #139887 - Attachment is obsolete: true
Is this the bug that causes the "change columns" page to not show any columns as checked the first time you visit it to make a column change?
Nope(In reply to comment #6) > Is this the bug that causes the "change columns" page to not show any columns > as checked the first time you visit it to make a column change? Nope, that's a different bug, and it doesn't look to be filed. File it please. :-)
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.