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)

2.20
defect
Not set
trivial

Tracking

()

RESOLVED FIXED
Bugzilla 2.20

People

(Reporter: LpSolit, Assigned: LpSolit)

Details

Attachments

(2 files, 2 obsolete files)

admin/products/deleted.html.tmpl should have links to most common product pages,
i.e. should include admin/products/footer.html.tmpl.
Status: NEW → ASSIGNED
Target Milestone: --- → Bugzilla 2.20
Attached patch patch for the tip, v1 (obsolete) — Splinter Review
Attachment #195730 - Flags: review?(bugzilla)
Attached patch backport for 2.20, v1 (obsolete) — Splinter Review
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-
fix Gavin's comment.
Attachment #195730 - Attachment is obsolete: true
Attachment #195756 - Flags: review?(bugzilla)
Gavin's comment applies to the backport too.
Attachment #195732 - Attachment is obsolete: true
Attachment #195757 - Flags: review?(bugzilla)
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+
Flags: approval?
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+
Flags: approval2.20?
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+
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.

Attachment

General

Created:
Updated:
Size: