Here is my code:

PHP Code:

                $bandListSql = "SELECT * FROM bands ORDER BY name";
        $bandListSql = mysql_query($bandListSql);
                
        function output_band_list($pos){
            echo '<option value="-1" selected>---------</option>';
            while($r = mysql_fetch_array($bandListSql)) {
                $bandList .= '<option value="'.$r['id'].'">'.$r['name'].'</option>';
            }//end while
            mysql_data_seek($r, 0); //reset pointer 
        }//end function 


It out puts to a dropdown list but all that i am getting is:
---------

its not doing the rest can anyone see why?

$bandListSql is not in the scope of your function, so essentially empty. You either need to pass it in as a parameter to your function, or put the line

global $bandListSql;

as the first line of function code - this way it will refer to the $bandListSql variable outside your function.

ok that was part of the problem, but i think i have fixed it, i wasnt outputting the list i was just assigning it to a variable

ok, i have another problem, here is my function:

PHP Code:

function output_band_list($pos){
          global $bandListSql;
            global $bandGigArray;
            echo '<option value="-1">---------</option>';
            
            while($r = mysql_fetch_array($bandListSql)) {
                if($r['id'] == $bandGigArray[$pos]){
                    echo '<option value="'.$r['id'].'" selected>'.$r['name'].'</option>';
                }else{
                    echo '<option value="'.$r['id'].'">'.$r['name'].'</option>';
                }//end if
            }//end while
            mysql_data_seek($r, 0); //reset pointer 
        }//end function 


It works perfectly on the first call of it, but after that it doesnt work, it just outputs ---------
i assume its something to do with the while loop, but i thought mysql_data_Seek would fix that

I would try extracting $r outside the function, and declaring it global.

PHP Code:

global $bandListSql;
global $bandGigArray; 
global $r 


Reset the pointer at the beginning of the function just to be sure it's good to go, then go over $r with a foreach() loop.