Skip to:
Content
Pages
Categories
Search
Top
Bottom

bp mag classic skips ”else if” statement in optionsbar

  • Avatar of creaturis
    creaturis
    Participant

    @creaturis

    I’m using bp mag and bp mag classic as my main theme. brajesh recently updated his themes and everything works fine. however if i activate bp mag classic i have a problem in this else if statement. the code is this

    if(bp_is_home()){
    do_action(“bpmag_member_options_nav”);
    }
    else if(bp_is_member ())
    bp_get_displayed_user_nav ();
    else
    bp_get_options_nav ();

    i have an issue with this:

    else if(bp_is_member ())
    bp_get_displayed_user_nav ();

    when I go to another member of my site it skips this and immediately goes to

    else
    bp_get_options_nav ();

    is there a bug in this code or something else.

    here is an image of what i’m talking about

    http://i1060.photobucket.com/albums/t451/Bastani/bpmagclassicbug-1.jpg

    buddypress 1.5.5 wordpress 3.3.2. latests versions of bp mag and bp classic

    thanks in advance. hope someone can help me with this.

    p.s. yes i asked Brajesh also on his forum but just want to know if its a problem with this code or if it is more complicated.

Viewing 5 replies - 1 through 5 (of 5 total)
  • Avatar of Hugo
    Hugo
    Moderator

    @hnla

    The question would be how does this behaviour stack up against bp-defaults behaviour answer that and you answer the question as to whether it’s an issue with this theme. To be honest it’s really a question that should have been answered one way or another by Brajesh? You say you asked him but neglect to say what his response was!

    Avatar of creaturis
    creaturis
    Participant

    @creaturis

    nothing yet asked him almost 3 days ago but didn’t get a reply here: http://buddydev.com/forums/topic/coupld-of-bugs-with-bp-mag-classic. so i had nothing better to do then looking into it myself, and I found this problem. read some tutorials about php codes. (else if) tried some different ways to implement it all failed.

    just wondering if the code looks okay like that. so i can search for the issue somewhere else. i will take a look in the bp default templates didn’t do that yet thanks.

    I was also afraid since it works on his bp mag classic demo its got nothing to do with the theme. and its once again an issue with my wordpress installation or host.

    Avatar of creaturis
    creaturis
    Participant

    @creaturis

    aha thanks Hugo. took a look at plugins/buddypress/bp-core/deprecated/1.5.php tried changing bp_is_member to bp_is_user and it works now. I only hope this is the right way to do it. oO

    Avatar of Hugo
    Hugo
    Moderator

    @hnla

    yep will be, I would pass that on to Brajesh so he can update, it was a good spot I scanned through that code snippet but missed that, however the point of the deprecated function is to allow the older function to still work but just return the newer form so not sure why it would correct anything?

    Avatar of creaturis
    creaturis
    Participant

    @creaturis

    have no idea i googled “bp_is_member” cause i knew this would probably be the main reason that it wouldn’t work. found a site called http://phpxref.ftwr.co.uk/buddypress/nav.html?_functions/index.html
    went to my cpanel of the location it gave me and just changed it in the optionsbar.php.

    past picture was from a previous install but with this new installation i had the same issue until i changed it to user.

    http://i1060.photobucket.com/albums/t451/Bastani/bpmagoptionsbarfixed.png

Viewing 5 replies - 1 through 5 (of 5 total)

You must be logged in to reply to this topic.