Drop down list with foreach loop PHP

Question

I'm trying to make a drop down list with PHP using a foreach loop to loop through the data. This works without a drop down when I was displaying the results in a table so the loop does work.

It seems the drop down list gets populated (as the list expands/contracts when I've added new fields in for testing) but no data is actually shown. I only need to show one field, though. Here's my code:

    <select name="language_select">
        <?php foreach($this->getContent('languages') as $language => $value) : ?>
            <option value="<?($language['name']);?>"></option>
        <?php endforeach ?>
    </select>

So it fetches an array and tries to return the data, pretty simple. What am I doing wrong?

Is short PHP tag enabled on your server? Try changing <? to <?php and see if that helps. — Amal
– Amal, Commented Jul 31, 2013 at 16:34
You have to echo the name within the tags as well as in the value like: <option value="<?($language['name']);?>"><?($language['name']);?></option> (are you echoing the variables?) — kalley
– kalley, Commented Jul 31, 2013 at 16:35

Orangepill · Accepted Answer · 2013-08-01 14:16:46Z

1

I'm not to sure of the composition the return of $this->getContent('languages') but I think this is what you need.

<select name="language_select">
    <?php foreach($this->getContent('languages') as $language) : ?>
        <option value="<?= urlencode($language['name']);?>"><?= htmlspecialchars($language["name"]) ?></option>
    <?php endforeach ?>
</select>

edited Aug 1, 2013 at 14:16

answered Jul 31, 2013 at 16:39

Orangepill

24.7k3 gold badges44 silver badges64 bronze badges

Sign up to request clarification or add additional context in comments.

6 Comments

Ciaran Over a year ago

This works! With one slight problem: it is displaying the id field (language_id) instead of the name field, as specified in the loop. Any suggestions?

Ciaran Over a year ago

Nothing gets returned. I've added in the 'languages_id' field to test what comes back, and now each field is duplicated i.e 0,0,1,1,2,2 etc.

Ciaran Over a year ago

I've viewed the array in Chrome, and each element in the array is matched correctly. I.e [0] is matched to [language_id]1, [name]English, [shortcode]en and so on. So I'm not sure why it is displaying just the ID field.

Orangepill Over a year ago

@CiaranReen Again what does print_r($this->getContent('languages')); show

Ciaran Over a year ago

If you cared to read my above comment I said 'nothing gets returned'.

|

Ciaran · Accepted Answer · 2013-08-01 14:48:54Z

0

Managed to get it working, with Orangepills help (thanks), with this:

<option value="<?= ($language['name']);?>"><?= ($language['name']) ?></option>

answered Aug 1, 2013 at 14:48

Ciaran

5554 gold badges11 silver badges27 bronze badges

Collectives™ on Stack Overflow

Drop down list with foreach loop PHP

2 Answers 2

6 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

6 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related