As a result of #898028: Change heading before Toolbar to "Administrative Toolbar" instead of "management" this variable is no longer being used and can be removed.

CommentFileSizeAuthor
toolbar-unused-variable.patch665 bytesDavid_Rothstein

Comments

tobiasb’s picture

tobiasb
Primary language German
Location Berlin
commented
Status: Needs review » Reviewed & tested by the community
bleen’s picture

bleen commented

If you want cleaner code and one less unnecessary function call I suppose you could go this way. I guess that's just how you roll...

RTBC++

Everett Zufelt’s picture

Everett Zufelt commented

Thanks for finding this, I should have been more judicious in making sure I wasn't making the variable redundant when I rolled the other patch.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.