Closed Bug 389517 Opened 17 years ago Closed 17 years ago

CSV export gives resolution and status in English

Categories

(Bugzilla :: Query/Bug List, defect)

defect
Not set
minor

Tracking

()

RESOLVED FIXED
Bugzilla 3.0

People

(Reporter: ycombarnous, Assigned: emmanuel)

Details

Attachments

(2 files, 1 obsolete file)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.5) Gecko/20070713 Firefox/2.0.0.5
Build Identifier: 

When changing the label of status and resolution in field-descs.tmp.html, the HTMLbuglist is displaying the new labels, while the CSV buglist is displaying the DB labels.

Reproducible: Always

Steps to Reproduce:
1.
2.
3.
Version: unspecified → 3.0
My fix:
Add: [% PROCESS "global/field-descs.none.tmpl" %]

      [% IF column == "opendate" OR column == "changeddate" %]
        [% rawcolumn = column.replace("date", "time") %]
        [% bug.$column = date.format(bug.$rawcolumn, "%Y-%m-%d %H:%M:%S") %]
      [% ELSIF column == 'bug_status' %]
        [% bug.$column = status_descs.${bug.$column} %]
      [% ELSIF column == 'resolution' %]
        [%- bug.$column = get_resolution(bug.$column) %]
      [% END %]
This affects 2.22 as well (well, tip of the 2.22 branch in any case).
Status: UNCONFIRMED → NEW
Ever confirmed: true
Yann, attach a patch if you want it implemented.
Taking
Status: NEW → ASSIGNED
Assignee: query-and-buglist → eseyman
Status: ASSIGNED → NEW
OS: Windows XP → All
Hardware: PC → All
Target Milestone: --- → Bugzilla 3.0
Taking, I said
Status: NEW → ASSIGNED
Attached patch Patch to the 2.22 branch (obsolete) — Splinter Review
Attachment #273936 - Flags: review?
Attachment #273937 - Flags: review?
Attachment #273938 - Flags: review?
Comment on attachment 273938 [details] [diff] [review]
Patch to the CVS tip

> [% PROCESS global/variables.none.tmpl %]
>+[% PROCESS "global/field-descs.none.tmpl" %]

FYI, field-descs.none.tmpl automatically calls variables.none.tmpl, so you only can remove the call to variables.none.tmpl here.
Comment on attachment 273938 [details] [diff] [review]
Patch to the CVS tip

> [% PROCESS global/variables.none.tmpl %]
>+[% PROCESS "global/field-descs.none.tmpl" %]

Remove [% PROCESS global/variables.none.tmpl %] on checkin. r=LpSolit
Attachment #273938 - Flags: review? → review+
Comment on attachment 273936 [details] [diff] [review]
Patch to the 2.22 branch

2.22 is restricted to security bugs only.
Attachment #273936 - Attachment is obsolete: true
Attachment #273936 - Flags: review?
Comment on attachment 273937 [details] [diff] [review]
Patch to the 3.0 branch

> [% PROCESS global/variables.none.tmpl %]
>+[% PROCESS "global/field-descs.none.tmpl" %]

Same comment as above, [% PROCESS global/variables.none.tmpl %] can go away. r=LpSolit
Attachment #273937 - Flags: review? → review+
Severity: normal → minor
Flags: approval3.0+
Flags: approval+
tip:

Checking in template/en/default/list/list.csv.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/list/list.csv.tmpl,v  <--  list.csv.tmpl
new revision: 1.6; previous revision: 1.5
done

3.0:

Checking in template/en/default/list/list.csv.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/list/list.csv.tmpl,v  <--  list.csv.tmpl
new revision: 1.5.4.1; previous revision: 1.5
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.