RT-3.5 possible Header/Footer fixes

It seems to me there is some cleanup of table tags missing from the
Elements/Footer page. Conditional on the Menu argument, some extra
closing tags are inserted into the HTML that don’t seem to fit anymore.
Attached is a patch that simply removes those tags, since they seem
unnecessary to me.

The only remaining use of the Menu argument seems to be the output of
some extra information on the Login page. Since that has nothing to do
with the menu, I’d suggest using a better name.

Also, the Elements/Header page apparently got mixed up, because the test
for $LoggedIn is now completely missing. As a result, even on the Login
page, I’m getting "Logged in as " (not the missing name). I’m including
another patch that reverts the logic to what’s used in version 3.4, which
fixes the problem for me. However, I might be missing something which
prompted that change in the first place.

Finally, I’m unable to figure out, why above mentioned extra information on
the login page doesn’t get displayed in its proper place below the login
box. I’ve checked the HTML, which is correct (with those patches applied).
Still it’s completely misplaced. Does anyone else see this and does anyone
have an idea what’s wrong? I’m stumped.

Rolf

footer.patch (593 Bytes)

header.patch (1.11 KB)