Menu Builder

[kongondo]

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 May 10, 2023 by kongondo

[kongondo]

42 minutes ago, Roych said:

Ohhh, with a lot of thinking, this is somehow working But not as expected 😉

Is your issue resolved then? I see your edit as well but I am still confused.

[Roych]

No, still not resolved, I tought that the code above works but when I looked closer it wasn't. So I guess the best approach would be to create another menu for the footer.

 

[DV-JF]

When I sort an existing menu and try to save it I get an error on PHP 8.1.x PW latest stable version:

method_exists(): Argument #1 ($object_or_class) must be of type object|string, null given search

@kongondo It would be nice if you could fix this based on this issue: https://github.com/kongondo/MenuBuilder/issues/48

 

[Roych]

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. ...

Thank you

R

[Clarity]

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?

[Roych]

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

[Ivan Gretsky]

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).

[DV-JF]

Hi @kongondo any news on this one:

 I'de run in this error again. Do you have any plans to fix this issue?

Greets!

[Roych]

Is it possible to upgrade module to work with PHP 8.2 or 8.3 maybe?

It only works on PHP 7.4 at least here ...

Thank you
R