Двата for цикъла не са вложени във вашия код:
foreach ($groups as $label => $opt) { ?>
<optgroup label="<?php echo $label; ?>">
<?php } <-- wrong here
foreach ($groups[$label] as $id => $name) { ?>
<option value="<?php echo $id; ?>"><?php echo $name; ?></option>
<?php } ?>
Резултатът е, че първо се създават всички opt групи и след това се добавят служителите за последната група (защото $label
и $opt
са налични и след приключване на цикъла).
Трябва да вложите циклите (използвайки алтернативен синтаксис за контрол структури ):
<?php foreach($groups as $label => $opt): ?>
<optgroup label="<?php echo $label; ?>">
<?php foreach ($opt as $id => $name): ?>
<option value="<?php echo $id; ?>"><?php echo $name; ?></option>
<?php endforeach; ?>
</optgroup>
<?php endforeach; ?>
Освен това мисля, че трябва да използвате emp_id
, а не grp_id
при създаване на масива:
while ($qa = $employees->GetRows()) {
$groups[$qa['groupname']][$qa['emp_id']] = $qa['empname'];
}