User

Welcome, Guest. Please login or register.
Did you miss your activation email?

Login with username, password and session length
Select language:

Community


Donate

Donate for PortaMx !
Your donation is safe and helps support the issues and causes you care most about.

Stats

Members
  • *Total Members: 4010
  • *Latest: kuki78
Stats
  • *Total Posts: 15656
  • *Total Topics: 2322
  • *Online Today: 23
  • *Most Online: 157
(27 Aug 09, 03:54:53)
Users online
  • *Users: 0
  • *Guests: 14
  • *Total: 14
« previous next »
Pages: [1]   Go Down

Author Topic: show option on 2nd sub buttons  (Read 2420 times)

0 Members and 1 Guest are viewing this topic.

Offline feline

  • CO PortaMx corp.
  • Administrator
  • *
  • Posts: 5224
  • Gender: Female
show option on 2nd sub buttons
« on: 18 Jan 10, 02:48:07 »
Today I found a heavy bug on SMF 2 RC2 ...
If you have 2nd subbuttons, the 'show' option is't handled  :o
To fix that, edit /Sources/Subs.php
find:
Code: [Select]
// Go through the sub buttons if there are any.
if (!empty($button['sub_buttons']))
foreach ($button['sub_buttons'] as $key => $subbutton)
{
if (empty($subbutton['show']))
unset($button['sub_buttons'][$key]);
}

replace with:
Code: [Select]
// Go through the sub buttons if there are any.
if (!empty($button['sub_buttons']))
foreach ($button['sub_buttons'] as $key => $subbutton)
{
if (empty($subbutton['show']))
unset($button['sub_buttons'][$key]);

// 2nd level subbutton
if(!empty($subbutton['sub_buttons']))
{
foreach($subbutton['sub_buttons'] as $key2 => $sub2_button)
{
if(empty($sub2_button['show']))
unset($button['sub_buttons'][$key]['sub_buttons'][$key2]);
}
}
}

Now the 2nd subbuttons show option works  ;)

Fel
Logged
Many are stubborn in relation to the path, a few in relation to the target.

Offline antechinus

  • PortaMx Supporter
  • *
  • Posts: 677
Re: show option on 2nd sub buttons
« Reply #1 on: 19 Jan 10, 01:53:35 »
Interesting. I'm not sure if that is deliberate but I'll file a bug report anyway.
Logged
Using Internet Explorer 6 on the internet is like urinating in a public swimming pool.
It's rude, there's no excuse for it, and anyone who does it should be ashamed of themselves.

Offline feline

  • CO PortaMx corp.
  • Administrator
  • *
  • Posts: 5224
  • Gender: Female
Re: show option on 2nd sub buttons
« Reply #2 on: 19 Jan 10, 02:08:27 »
On the index.template.php the code for 2nd subbuttons are exist .. so it's necessary to get the 'show' option to work (I think)

Fel
Logged
Many are stubborn in relation to the path, a few in relation to the target.
Pages: [1]   Go Up
« previous next »