Looping

[1Unreal]

Im trying to get my head round more of this plugin stuff.

In vB options I have a textarea. I explode that out into an array then I want to loop each item to display in an unordered list where ever on the template.

How can I make this more compatable with vB?

PHP Code:

if($vbulletin->options['enabled'] == 1)
    {
        $items = explode("\n", $vbulletin->options['html']);
        echo '<ul>';
        foreach($items as $item)
        {
            echo '<li>'.$item.'</li>';    
        }
        echo '</ul>';
    } 


Thanks

[TigerC10]

Your only issue is the second <ul> tag. It should be a closing </ul> instead. Aside from that it should work, as long as the $vbulletin->options['html'] variable has newline characters in it. Otherwise it will all display on one line.

As a matter of convension, you should change those option names, go with a prefix like...
$vbulletin->options['mymod_enabled']
$vbulletin->options['mymod_html']


That way nothing interferes with one another later on down the road.

[1Unreal]

Sorry that was a problem with my syntax but the original problem still exists.

How can I display this loop in a template?

Would it be best to write whatever the loop outputs to a template then include that template on another template?

If so, how would I do that?

[Lynne]

You need to put that php in a plugin - php cannot go into a template.

[1Unreal]

Yeah I know, I've got trouble showing the contents of this plugin though.

[TigerC10]

To get it in a template, you shouldn't echo. You need to stuff it into a variable like $output_list.

PHP Code:

if($vbulletin->options['enabled'] == 1)
    {
        $items = explode("\n", $vbulletin->options['html']);
        $output_list = '<ul>';
        foreach($items as $item)
        {
            $output_list .= '<li>'.$item.'</li>';    
        }
        $output_list .= '</ul>';
    } 


Then, in your template just put the $output_list variable where you want it to display. Tah-dah! The template engine will just output any variable values if you just call it..

<if condition="$vbulletin->options[mymod_enabled]">
$output_list
</if>

Can you dig it?

[1Unreal]

That doesn't work :|

[Dismounted]

You may want to check if items exist in $items first.

[1Unreal]

Yeah there is.

[Dismounted]

I meant add a PHP code check, and then you can just do (in the template):

<if condition="$output_list">$output_list</if>