theme hack works – partially

  • Resolved nikdow

    (@nikdow)


    10 years, 1 month ago

    My problem was the theme iMedica (you can add that to your list of incompatible themes).
    I added do_action( 'wp_nav_menu_item_custom_fields', $item_id, $item, $depth, $args ); to the start_el function in class FW_Admin_Menu_Walker extends Walker_Nav_Menu in file /wp-content/plugins/imedica-core/framework/extensions/megamenu/includes/class-fw-admin-menu-walker.php
    and now I can see some, but not all of the nav-menu-roles options. I get this much:
    Display Mode
    Logged In Users Logged Out Users Everyone
    but I don’t get the checkboxes for the roles.

    Is there some other reason why nav-menu-roles wouldn’t be showing the roles to select? Or is this theme’s plugin just too gnarly?

    And yes I will be contacting the theme author, in the meantime, trying to make it work.

    https://wordpress.org/plugins/nav-menu-roles/

Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Author HelgaTheViking

    (@helgatheviking)

    10 years, 1 month ago

    The checkboxes only appear if you’ve selected “Logged In Users”. So NMR’s javascript has to have loaded (nav-menu-roles.js or nav-menu-roles.min.js depending on whether you have SCRIPT_DEBUG defined). Check the page source to be sure it has loaded, and additionally check your dev tools console for script errors.

    Thread Starter nikdow

    (@nikdow)

    10 years, 1 month ago

    Thanks for clearing up my elementary error, working well now.

    Now I just have to get that plugin author to toe the line, as I have my plugins on auto-update, but from your “plan B” I gather the option once set up will still work even if the errant plugin updates.

    Thanks for a great plugin.

    Plugin Author HelgaTheViking

    (@helgatheviking)

    10 years, 1 month ago

    Good luck. It’s a *very* easy fix for the theme author, especially if you send them the patched file. Hopefully it works out. Yes, plan B will work even if the theme updates and breaks your patch. There actually is some movement on the core trac issues holding this up, so we might actually see this resolved at the core level eventually.

    Glad you dig NMR!

    sfxfreak

    (@sfxfreak)

    10 years ago

    Hi,

    @nikdow – did you get any response from BrainstormForce?
    I was just hit with the same problem, but in a different file and no matter what I try to do – I can’t get my web site to work again.

    Thanks.

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

The topic ‘theme hack works – partially’ is closed to new replies.