Closed
Bug 563128
Opened 14 years ago
Closed 14 years ago
$dynamic_left_menu and $include_left_nav_menu are unused
Categories
(www.mozilla.org :: General, enhancement)
www.mozilla.org
General
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: abuchanan, Assigned: abuchanan)
Details
Attachments
(1 file)
|
1.53 KB,
patch
|
sgarrity
:
review+
pascalc
:
feedback+
|
Details | Diff | Splinter Review |
$dynamic_left_menu and $include_left_nav_menu are unused
[abuchanan@khan trunk]$ grep -r include_left_nav_menu * | grep -v .svn
includes/footer-portal-pages.inc.php:if (isset($include_left_nav_menu) && $include_left_nav_menu) {
includes/footer.inc.php:if (isset($include_left_nav_menu) && $include_left_nav_menu) {
should remove this unused code
|
Assignee | |
Comment 1•14 years ago
|
||
Assignee: nobody → abuchanan
|
|
Assignee | |
Comment 2•14 years ago
|
||
Comment on attachment 442888 [details] [diff] [review]
patch, remove unused code
could you two review and confirm this isn't used anywhere please? thanks
Attachment #442888 -
Flags: review?(steven)
Attachment #442888 -
Flags: feedback?(pascalc)
|
||
Comment 3•14 years ago
|
||
Comment on attachment 442888 [details] [diff] [review]
patch, remove unused code
good for me
Attachment #442888 -
Flags: feedback?(pascalc) → feedback+
|
||
Comment 4•14 years ago
|
||
Comment on attachment 442888 [details] [diff] [review]
patch, remove unused code
Rest in peace, dynamic_left_menu...
Attachment #442888 -
Flags: review?(steven) → review+
|
|
Assignee | |
Comment 5•14 years ago
|
||
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
|
||
Updated•14 years ago
|
Status: RESOLVED → VERIFIED
|
||
Updated•12 years ago
|
Component: www.mozilla.org/firefox → www.mozilla.org
|
|
||
Updated•12 years ago
|
Component: www.mozilla.org → General
Product: Websites → www.mozilla.org
You need to log in
before you can comment on or make changes to this bug.
Description
•