Frog CMS

I have been trying to find a solution to the question about a vertical menu, so that when you click on a level, the menu on the next level below is revealed. (Asked as post #29 in this thread.)

I think I almost have it. Well, I have it working for Home > Level1 > Level2, but not for Level3.

I get this error message for "Level3": Fatal error: Call to a member function children() on a non-object in [path]\frog\app\frontend\classes\Page.php(170)...

Here is my code:

<?php $subPageId = explode('/', $_SERVER['REQUEST_URI']); $level2=$subPageId[2]; $level3=$subPageId[3]; ?>

<h3>NAVIGATION</h3>
<ul>
    <li><a<?php echo url_match('/') ? ' class="current"': ''; ?> href="<?php echo URL_PUBLIC; ?>">home</a></li>
    <?php foreach($this->find('/')->children() as $menu): ?>
    <li><?php echo $menu->link($menu->title, (in_array($menu->slug, explode('/', $this->url)) ? ' class="current"': null)); ?>

    <?php if ($level2 != '') : ?>
        <?php $page2 = $this->find($level2); ?>
        <?php if (strpos($_SERVER['REQUEST_URI'],$menu->slug) == true) : ?>
        <ul>
        <?php foreach ($page2->children(array()) as $menu2): ?>
        <li><?php echo $menu2->link(); ?>

            <?php if ($level3 != '') : ?>
            <?php if (strpos($_SERVER['REQUEST_URI'],$menu2->slug) == true) : ?>
            <?php $page3 = $this->find('$level3'); ?>

            <ul>
                <?php foreach ($level3->children(array()) as $menu3): ?>
                <li><?php echo $menu3->link(); ?></li>
                <?php endforeach; ?>
            </ul>
            <?php endif; ?>
            <?php endif; ?>

        </li>
        <?php endforeach; ?>
        </ul>
        <?php endif; ?>
        <?php endif; ?>

    </li>
    <?php endforeach; ?> 
    </ul>

A couple notes: (1) the first "explode" line gets me the "slug", and the numerical index would need to be adjusted depending on where Frog lives on your server; (2) the if ($level2 != '') bit tests to see if there is a slug, and if so, continues to process that level.

The line that produces the error is this one: <?php foreach ($level3->children(array()) as $menu3): ?>. There must be a problem calling children-of-chilren this way, but I can not work out how to fix this. You can see it is just a variation for what is working well at "Level2".

Can anyone give some help on this? Many thanks!

David.

Ahh, now I see what you pointed out ... but sadly, I am getting the very same error message. :( Thanks anyway!

Any other ideas?

P.s. You can see this code trying to work in the "GB Pages" and "Poet" sections of the this Frog "test site".

Genius! You sorted it, Philippe! I should have caught the ('$level3') problem myself! But I would not have known to do $this->find($level2.'/'.$level3);. So that is great! This is the menu system that aroundthefur wanted, I think, but down to three levels, not just two.

If anyone wants to try it, here is the exact working code that I have in the sidebar of the Home Page:

<?php $subPageId = explode('/', $_SERVER['REQUEST_URI']); $level2=$subPageId[2]; $level3=$subPageId[3]; ?>

<h3>NAVIGATION</h3>
<ul>
    <li><a<?php echo url_match('/') ? ' class="current"': ''; ?> href="<?php echo URL_PUBLIC; ?>">home</a></li>
    <?php foreach($this->find('/')->children() as $menu): ?>
    <li><?php echo $menu->link($menu->title, (in_array($menu->slug, explode('/', $this->url)) ? ' class="current"': null)); ?>

    <?php if ($level2 != '') : ?>
        <?php $page2 = $this->find($level2); ?>
        <?php if (strpos($_SERVER['REQUEST_URI'],$menu->slug) == true) : ?>
        <ul>
        <?php foreach ($page2->children(array()) as $menu2): ?>
        <li><?php echo $menu2->link(); ?>

            <?php if ($level3 != '') : ?>
            <?php if (strpos($_SERVER['REQUEST_URI'],$menu2->slug) == true) : ?>
            <?php $page3 = $this->find($level2.'/'.$level3); ?>

            <ul>
                <?php foreach ($page3->children(array()) as $menu3): ?>
                <li><?php echo $menu3->link(); ?></li>
                <?php endforeach; ?>
            </ul>
            <?php endif; ?>
            <?php endif; ?>

        </li>
        <?php endforeach; ?>
        </ul>
        <?php endif; ?>
        <?php endif; ?>

    </li>
    <?php endforeach; ?> 
</ul>

Again, you may need to adjust the $subPageId[n] numbers in the first line to match the structure of the URL on your own site. This is presently in action on the "GB Pages" and "Poets" tabs here. (That host is doing file-storage maintenance today, so the site may be inaccessible from time to time.)

Thanks, Philippe! And I hope that helps...

David.

You also have the option to use ".html" as the URL suffix. You should probably change this in config.php, too, so it reads like this:

// add a suffix to pages (simluating static pages '.html')
define('URL_SUFFIX', '');

That ''); is two single ' marks with nothing between them, by the way!

D.

Update: There were complications of two kinds:

1. The "Articles" section had been removed, so the main Home Page code was throwing errors (not the menu code in the sidebar).
2. The ".html" extension was also causing errors in the menu code. :|

There are two solutions, if this technique is to work:

1. Remove the URL suffix, as in post #19 above, in this thread. Or,
2. Try to deal with the .html suffix in the code.

I tried to solve #2 - and with this additional line (2nd line below), it works at "Level 2":

<?php if ($level2 != '') : ?>
<?php $item2 = substr($level2,0,-5); ?>

That strips the URL suffix, and then you use <?php $page2 = $this->find($item2); ?> to get the menu items. But, that does not work at "Level 3", and I cannot work out what to do there!

So -- to use this technique, either (a) get rid of URL suffixes!, or (b) get Philippe to work out a complete solution to #2. :)

David.

If anyone else cares :) there was a further problem with this technique, in that when you did get down to a "Level 3" page, it threw off the truncating technique described in post #23 above. I have managed to find a way around that! So, here (for the record!) is the complete working code on aroundthefur's site:

<?php $subPageId = explode('/', $_SERVER['REQUEST_URI']); $level2=$subPageId[1]; $level3=$subPageId[2]; ?>

<h3>NAVIGATION</h3>
<ul>
    <li><a<?php echo url_match('/') ? ' class="current"': ''; ?> href="<?php echo URL_PUBLIC; ?>">home</a></li>
    <?php foreach($this->find('/')->children() as $menu): ?>
    <li><?php echo $menu->link($menu->title, (in_array($menu->slug, explode('/', $this->url)) ? ' class="current"': null)); ?>

    <?php if ($level3 != '') {$item2 = $level2;} else {$item2 = substr($level2,0,-5);} ?>
        <?php if ($level2 != '') : ?>
        <?php $page2 = $this->find($item2); ?>
        <?php if (strpos($_SERVER['REQUEST_URI'],$menu->slug) == true) : ?>
            <ul>
            <?php foreach ($page2->children(array()) as $menu2): ?>
            <li><?php echo $menu2->link(); ?></li>
            <?php endforeach; ?>
            </ul>
        <?php endif; ?>
    <?php endif; ?>

    </li>
    <?php endforeach; ?> 
</ul>

The if ... else line detects which kind of variable to use. FWIW! David.