Closed
Bug 308150
Opened 19 years ago
Closed 19 years ago
admin/products/deleted.html.tmpl doesn't include admin/products/footer.html.tmpl
Categories
(Bugzilla :: User Interface, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.20
People
(Reporter: LpSolit, Assigned: LpSolit)
Details
Attachments
(2 files, 2 obsolete files)
|
1.87 KB,
patch
|
bugzilla
:
review+
|
Details | Diff | Splinter Review |
|
2.89 KB,
patch
|
bugzilla
:
review+
|
Details | Diff | Splinter Review |
admin/products/deleted.html.tmpl should have links to most common product pages, i.e. should include admin/products/footer.html.tmpl.
| Assignee | ||
Updated•19 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → Bugzilla 2.20
| Assignee | ||
Comment 2•19 years ago
|
||
I backported the relevant part of the patch from bug 306242 about admin/products/footer.html.tmpl which is still broken in 2.20 when classifications are used.
Attachment #195732 -
Flags: review?(bugzilla)
Comment on attachment 195730 [details] [diff] [review] patch for the tip, v1 Just one minor issue: we need to set the classification template variable, so the classification links are presented in the footer.
Attachment #195730 -
Flags: review?(bugzilla) → review-
| Assignee | ||
Comment 4•19 years ago
|
||
fix Gavin's comment.
Attachment #195730 -
Attachment is obsolete: true
Attachment #195756 -
Flags: review?(bugzilla)
| Assignee | ||
Comment 5•19 years ago
|
||
Gavin's comment applies to the backport too.
Attachment #195732 -
Attachment is obsolete: true
Attachment #195757 -
Flags: review?(bugzilla)
| Assignee | ||
Updated•19 years ago
|
Attachment #195732 -
Flags: review?(bugzilla)
Comment on attachment 195756 [details] [diff] [review] patch for the tip, v2 Yes, tested, works. bug#308183 will refine the useclassification checks
Attachment #195756 -
Flags: review?(bugzilla) → review+
Comment 7•19 years ago
|
||
Holding approval for the 2.20 release (the tree is now locked down for release), this'll go in 2.20.1.
Comment on attachment 195757 [details] [diff] [review] backport for 2.20, v2 And this one is OK as well
Attachment #195757 -
Flags: review?(bugzilla) → review+
| Assignee | ||
Updated•19 years ago
|
Flags: approval2.20?
Comment 9•19 years ago
|
||
ok, Fréd says we can take this one for 2.20 because they haven't started the release testing yet.
Flags: approval?
Flags: approval2.20?
Flags: approval2.20+
Flags: approval+
| Assignee | ||
Comment 10•19 years ago
|
||
tip: Checking in editproducts.cgi; /cvsroot/mozilla/webtools/bugzilla/editproducts.cgi,v <-- editproducts.cgi new revision: 1.98; previous revision: 1.97 done Checking in template/en/default/admin/products/deleted.html.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/admin/products/deleted.html.tmpl,v <-- deleted.html.tmpl new revision: 1.2; previous revision: 1.1 done Checking in template/en/default/admin/products/footer.html.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/admin/products/footer.html.tmpl,v <-- footer.html.tmpl new revision: 1.5; previous revision: 1.4 done 2.20rc2: Checking in editproducts.cgi; /cvsroot/mozilla/webtools/bugzilla/editproducts.cgi,v <-- editproducts.cgi new revision: 1.85.2.5; previous revision: 1.85.2.4 done Checking in template/en/default/admin/products/deleted.html.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/admin/products/deleted.html.tmpl,v <-- deleted.html.tmpl new revision: 1.1.2.1; previous revision: 1.1 done Checking in template/en/default/admin/products/footer.html.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/admin/products/footer.html.tmpl,v <-- footer.html.tmpl new revision: 1.3.2.1; previous revision: 1.3 done
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•