Quick question.. thanks :)

[Gaheris]

return (isset($_POST[$_index])) ? $_POST[$_index] : ''; 

I had missed a parenthese.

Your select box markup is a bit messed up. Do you want to let the user choose more then the current entry, which hasn't even got a name in your code?

[mak-uk]

Hi

I simply want the previous selected option to be selected once again. If no option is selected, then it be blank (again) so they can select it.

The original code for that part was:

<td><select name="createdby_select" id="createdBy">
      <OPTION SELECTED VALUE="">---------------</OPTION>
      <?php
  while ($createdby_q = mysql_fetch_array($createdby_query)) {
  $createdby_id = $createdby_q["createdbyID"];
  $createdby_name = $createdby_q["createdbyName"];
  echo("<OPTION VALUE='$createdby_id'>$createdby_name<BR>\n");
   }
       ?>
    </select></td>

So I was just wondering how to adjust this using your code.

Thanks.

Mak

[Gaheris]

/* Ignore this line */ ?>
<td>
    <select name="createdby_select" id="createdBy">
        <option value="">---------------</option>
        <?php
            $preselection = safe_post('createdby_select');
            while ($createdby_q = mysql_fetch_array($createdby_query)) {
                $selected = ($preselection == $createdby_q['createdbyID']) ? 'selected' : '';
                echo '<option '.$selected.' value="'.$createdby_q['createdbyID'].'>'.
                      $createdby_q['createdbyName'].'<br>'."\n";
           }
        ?>
    </select>
</td>

[mak-uk]

Hi

I tried that but something odd happens.

The options I can select are:

1. Myself
2. Parent
3. Guardian
4. Other family member
5. Friend

When I select 'Myself' and post the form with the rest blank, it brings up the appropriate error messages and I would expect it to have 'Myself' selected again. Instead, it has:

"--------------
Parent
Other family member
Forename"

Equally, similar odd stuff happens when I select any other option. What I don't understand is where it is getting 'Forename' from? Is something wrong with the way I am accessing the $_POST array?

Thanks.

Mak

[Gaheris]

In the while loop echo out $createdby_q['createdbyName'], $createdby_q['createdbyID'] and $selected after the echo please.

[mak-uk]

Hi

Sorry for being thick but do you mean:

<?php 
            $preselection = safe_post('createdby_select'); 
            while ($createdby_q = mysql_fetch_array($createdby_query)) { 
      $selected = ($preselection == $createdby_q['createdbyID']) ? 'selected' : ''; 
               //echo '<option '.$selected.' value="'.$createdby_q['createdbyID'].'>'.$createdby_q['createdbyName'].'<br>'."\n"; 
               echo '<option '.$selected.' value="">'\n"; 
               
           } 
        ?>

?

Thanks.

Mak

[Gaheris]

echo 'Name: ' , $createdby_q['createdbyName'] , ' ID: ' , $createdby_q['createdbyID'] , ' Selected: ' , $selected' , ' Preselection: ' , $preselection , '<br />'; 

after the echo of the option tag. This is for debugging reasons, to know that is fetched from the database and what the data generates.

[mak-uk]

Hi

With that I get:

Parse error: parse error, unexpected T_CONSTANT_ENCAPSED_STRING, expecting ',' or ';'

Any ideas?

Thanks.

Mak

[Gaheris]

Sorry one quote to much.

echo 'Name: ' , $createdby_q['createdbyName'] , ' ID: ' , $createdby_q['createdbyID'] , ' Selected: ' , $selected , ' Preselection: ' , $preselection , '<br />'; 

[mak-uk]

Hi

Then I get (in the re-displayed select box):

->Parent Name: Parent ID: 2: Preselection: 1
->Other family member Name: Other family member ID: 4 Selected: Preselection: 1
->Forename

Those are the three options it shows. What's odd is the 'Forename' as that is, as yet, just the next field:

<tr>
    <td width="223">Forename</td>
    <td width="241">
     <input name="mForename" type="text" id="mForename" size="25" maxlength="25">
    </td>
</tr> 

Hope that helps?

Oh, btw, the database code is:

# Table structure for table `createdby`
#

CREATE TABLE `createdby` (
  `createdbyID` int(1) NOT NULL auto_increment,
  `createdbyName` varchar(20) default NULL,
  PRIMARY KEY  (`createdbyID`)
) TYPE=MyISAM AUTO_INCREMENT=6 ;

#
# Dumping data for table `createdby`
#

INSERT INTO `createdby` VALUES (1, 'Myself');
INSERT INTO `createdby` VALUES (2, 'Parent');
INSERT INTO `createdby` VALUES (3, 'Guardian');
INSERT INTO `createdby` VALUES (4, 'Other family member');
INSERT INTO `createdby` VALUES (5, 'Friend');

Thanks.

Mak

[Gaheris]

Can I have the full source, I would need everything to be able to successfully debug it. Thanks.

[mak-uk]

Hi

It was a bit long to post so I have PM'd you it in sections.

Thanks.

Mak

[Gaheris]

Yes, I will look into it tommorrow.

[mak-uk]

Hi

Look foward to your reply

Thanks.

Mak

[Gaheris]

Okay, I attached the PHP file, I hope it helps you.

[mak-uk]

Hi

Where is the attached PHP file?

Thanks.

Mak

[Gaheris]

You have to wait till it is approved.

[mak-uk]

Hi

I still can't see anything.. either here or in private messages.. and it has been a couple of days?

Thanks.

Mak

[Gaheris]

I see it now.

[mak-uk]

Hi

Thanks for that!

I just tried it and get this:

Parse error: parse error, unexpected ';' in d:\inetpub\wwwroot\site\join_form.php on line 6

I think it is referring to this line:

return (isset($_POST[$_index]) ? $_POST[$_index] : ''; 

Thanks.

Mak

[Jump]

Should be

return (isset($_POST[$_index])) ? $_POST[$_index] : ''; 

[mak-uk]

Hi

I fixed that but then I get:

Parse error: parse error, unexpected T_CONSTANT_ENCAPSED_STRING in d:\inetpub\wwwroot\site\join_form.php on line 39

$selected     = ($preselected == $createdby_id) 'selected' : ''; 

I thought it may be some missing parentheses but that didn't work.

Any ideas?

Thanks.

Mak

[Gaheris]

Must be

$selected     = ($preselected == $createdby_id) ? 'selected' : ''; 

[mak-uk]

Hi

What about:

Parse error: parse error, unexpected '=' in d:\inetpub\wwwroot\site\join_form.php on line 84

$preselection = safe_post('day_select'); 

Thanks.

Mak

[Gaheris]

I hopefully cleared out some of the errors, see the attached file.