Jump to content

Menu Builder

kongondo

By kongondo,
in Modules/Plugins

 Share

Recommended Posts

kongondo Hero Member

kongondo

Posted
  • Author
Posted (edited)
44 minutes ago, Roych said:

not realy working. The children are not native children of a page, they are made with MenuBuilder drag&drop option. I want those in my header main menu, but in my footer I only want to show the first row.

 

That's exactly what I said wouldn't work ?. You cannot control non-native children. If the children are MB (drag and drop) children, they are always shown (unless you disable them in the backend using the disable menu item option).

I don't understand what you mean by 'not really working'. A little more detail would help.

Edited by kongondo
Link to comment
Share on other sites
  • 3 months later...
Roych Sr. Member

Roych

Posted
Posted

I'have a strange problem, my drag and drop is not working anymore not sure why. All the settings are correct and should be working. It's quite annoying, anybody else have this problem? any ideas how to fix this? Menu is published, unlocked. ...

menu.gif.3a7b50e1cbbf0873fc1596a123bf6c37.gif

Thank you

R

Link to comment
Share on other sites
  • 2 months later...
Clarity Sr. Member

Clarity

Posted
Posted

Hello, @kongondo!

I use the module in this way: I'm creating a site with possibility of creating blank menu without items.

On these lines I'm trying to get menu items:
  

$menuBuilder = $this->modules->get('MarkupMenuBuilder');
$this->menuItems = $menuBuilder->getMenuItems($id);

Then I get an error: "No menu items found! Confirm that such a menu exists and that it has menu items." if items are not present. This is caused by the line 1366:

https://github.com/kongondo/MenuBuilder/blob/main/MarkupMenuBuilder.module#L1366

I tried to fix it manually using the fact that the menu is a page and menu items are being stored in menu_items variable: 

$menuBuilder = $this->modules->get('MarkupMenuBuilder');
$menu = $this->pages->get($id);
if($menu->menu_items) {
    $this->menuItems = $menuBuilder->getMenuItems($id);
} else {
    return;
}

However, it seems to be only a workaround. Am I using module correctly? If no, how can I use it to avoid such an error without adding items?

If it is not possible, I suggest the following fix:

Here https://github.com/kongondo/MenuBuilder/blob/main/MarkupMenuBuilder.module#L1366 return can be replaced from $this->throwError() to [] or array().

I created a new issue here: https://github.com/kongondo/MenuBuilder/issues/50. Can you please see this issue?

Link to comment
Share on other sites
Roych Sr. Member

Roych

Posted
Posted

Maybe something like this? 

$menuBuilder = $this->modules->get('MarkupMenuBuilder');
$menu = $this->pages->get($id);

if ($menu->menu_items->count()) {
    $this->menuItems = $menuBuilder->getMenuItems($id);
} else {
    // Handle the case where there are no menu items, for example:
    $this->menuItems = []; // or any other appropriate action
}

Haven't tested!

This code (should) first checks if there are any menu items by using the count() method on the menu_items field. If there are menu items, it proceeds to fetch and assign them using the getMenuItems method from the MenuBuilder module. If there are no items, you can handle it as needed. In the example above, an empty array is assigned to $this->menuItems, but you can customize it to match your specific requirements.

This approach allows you to use the MenuBuilder module to create menus with or without items, without triggering the error you mentioned.

Cheers,

R

  • Like 1
Link to comment
Share on other sites
Ivan Gretsky Hero Member

Ivan Gretsky

Posted
Posted
On 10/27/2023 at 12:55 PM, Roych said:

Maybe something like this? 

$menuBuilder = $this->modules->get('MarkupMenuBuilder');
$menu = $this->pages->get($id);

if ($menu->menu_items->count()) {
    $this->menuItems = $menuBuilder->getMenuItems($id);
} else {
    // Handle the case where there are no menu items, for example:
    $this->menuItems = []; // or any other appropriate action
}

Haven't tested!

This code (should) first checks if there are any menu items by using the count() method on the menu_items field. If there are menu items, it proceeds to fetch and assign them using the getMenuItems method from the MenuBuilder module. If there are no items, you can handle it as needed. In the example above, an empty array is assigned to $this->menuItems, but you can customize it to match your specific requirements.

This approach allows you to use the MenuBuilder module to create menus with or without items, without triggering the error you mentioned.

Cheers,

R

@Roych, as I can see from his post @Clarity is already doing almost the same as you suggested. But he is looking for a cleaner way to handle it that already exist (or asks @kongondoto update the module to provide it).

  • Like 1
Link to comment
Share on other sites
  • 2 months later...
  • 1 month later...
  • 2 months later...
froot Hero Member

froot

Posted
Posted

how to return an array of the menu items? So far I was not successful. 

$menu = $modules->get('MarkupMenuBuilder');// $menu is an example
echo $menu->getMenuItems('topmenu'); // topmenu is the name of my menu

But it only returns Menu|Menu|Menu

How can that be?
 

Link to comment
Share on other sites
Nicolas Sr. Member

Nicolas

Posted
Posted

You need to iterate over the menu items object. 

<?php
$mainMenuItems = $mb->getMenuItems('Main navigation', 2, $options);

foreach($mainMenuItems as $menu_item) {
	// You know have access to $menu_item->title
  	// You know have access to $menu_item->url
  	// See the doc for available $menu_item properties
        $has_children = '';
        $has_children_dropdown = '';
        $dropdown = '';
        $tag_class = $menu_item->cssClass!=''?$menu_item->cssClass . ' ' . $has_children:$has_children;

        if($menu_item->isParent) {
            $has_children = 'has-submenu';
            $has_children_dropdown = '<span uk-icon="chevron-down"></span>';
            $children = $wire_pages->get($menu_item->pagesID)->children();

            $dropdown = '<div uk-dropdown="animation: uk-animation-slide-top-small; duration: 300; animate-out: true; offset: 0; pos: bottom-left"><ul class="uk-nav uk-navbar-dropdown-nav">';

            foreach($children as $child) {
                 $dropdown .= '<li><a href="' . $child->url .'">'. $child->title . '</a></li>';
            }
            $dropdown .= '</ul></div>';

        }

        if($menu_item->pagesID == $current_page->parent->id || $menu_item->pagesID == $current_page->id) {
          $item =  '<li class="current-item ' . $tag_class . '"><a href="' . $menu_item->url .'">'. $menu_item->title . $has_children_dropdown . '</a>' . $dropdown . '</li>';
        } else {
          $item = '<li class="'. $tag_class .'"><a href="' . $menu_item->url .'">'. $menu_item->title . $has_children_dropdown . '</a>' . $dropdown . '</li>';
        }
        if($menu_item->parentID == 0) {
            echo $item;
        }
    }

 

  • Like 2
Link to comment
Share on other sites
  • 6 months later...
Ivan Gretsky Hero Member

Ivan Gretsky

Posted
Posted

How are you doing, @kongondo!

Your module is a blast! Still finding some really cool ways to use it to control how menus could be controlled from the admin.

The thing that really worries me is that the module seems to be not fully compatible with newer versions of php.

Is there a chance you could find some time to upgrade it anytime soon? Or are you willing to accept PRs with fixes? I can see one that is not dealt with yet, so curious if me or someone else should try to come up with solutions with compatibility problems?

Thanks!

Link to comment
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
 Share
Go to topic listing
×
×
  • Create New...