PHP inside PHP?

[kilbey1]

How might I make this code work when it's already inside PHP code?

print("
<select name='pricerange'>
	<option VALUE='Less than $10' <? if($row[price_range]=='Less than $10') print 'selected';?>>Less than $10
	<option VALUE='$10 - $25' <? if($row[price_range]=='$10 - $25') print 'selected';?>>$10 - 25
	<option VALUE='$25 - $50' <? if($row[price_range]=='$25 - $50') print 'selected';?>>$25 - 50
	<option VALUE='More than $50' <? if($row[price_range]=='More than $50') print 'selected';?>>Over $50
</select>
");

• Eve
  [e-mail]

[diego25]

Don't start in PHP mode then:

<select name='pricerange'>

<option VALUE='Less than $10' <? if($row[price_range]=='Less than $10') print 'selected';?>>Less than $10

<option VALUE='$10 - $25' <? if($row[price_range]=='$10 - $25') print 'selected';?>>$10 - 25

<option VALUE='$25 - $50' <? if($row[price_range]=='$25 - $50') print 'selected';?>>$25 - 50

<option VALUE='More than $50' <? if($row[price_range]=='More than $50') print 'selected';?>>Over $50

</select>

[shu]

Try:

echo '
<select name="pricerange">
    <option VALUE="Less than $10"'.(($row[price_range]=='Less than $10')? 'selected':'').'>Less than $10</option>
    <option VALUE="$10 - $25"'.(($row[price_range]=='$10 - $25')? 'selected':'').'>$10 - $25</option>
    <option VALUE="$25 - $50"'.(($row[price_range]=='$25 - $50')? 'selected':'').'>$25 - $50</option>
    <option VALUE="More than $50"'.(($row[price_range]=='More than $50')? 'selected':'').'>Over $50</option>
</select>';

[kilbey1]

Yay! Thank you, Shu. It was the proper syntax of the if statement that I wasn't sure how to do! It's working great.

• Eve

[kilbey1]

One other question. I am trying to run this with checkboxes, with the values in the database in a Set. This is my setup with only 'selected' changed to 'checked'.

echo '
<input type="checkbox" name="Parking Lot" value="Parking Lot"'.(($row[parking]=='Parking Lot')? 'checked':'').'> Parking Lot 
<input type="checkbox" name="Street" value="Street"'.(($row[parking]=='Street')? 'checked':'').'>Street
<input type="checkbox" name="Valet" value="Valet"'.(($row[parking]=='Valet')? 'checked':'').'>Valet 
<input type="checkbox" name="On/Off Premises" value="On/Off Premises"'.(($row[parking]=='On/Off Premises')? 'checked':'').'>On/Off Premises
';

Is there something else I need to do to get back a set?

• Eve

[Weedpacket]

Give all the checkboxes the same name ending with "[]" (name="parking[]", say), and they'll arrive in PHP as an array. That would be the closest to a Set you'll get. You'll have to write the SQL query to get the rest of the way (course, you can do that without changing the checkbox names - but then it's a pain having to keep track of all those different names).

[kilbey1]

I'm actually not sure what you're on to here. I understand about giving the checkboxes the same name ending - no problem there. thus far, the code looks like so:

echo '
<input type="checkbox" name="parking[]" value="Parking Lot"'.(($row[parking]=='Parking Lot')? ' checked':'').'> Parking Lot
<input type="checkbox" name="parking[]" value="Street"'.(($row[parking]=='Street')? ' checked':'').'> Street
<input type="checkbox" name="parking[]" value="Valet"'.(($row[parking]=='Valet')? ' checked':'').'> Valet
<input type="checkbox" name="parking[]" value="On/Off Premises"'.(($row[parking]=='On/Off Premises')? ' checked':'').'> On/Off Premises
';

But I'm not clear on how to pull back the database values to show that in restaurant ID 1, Parking Lot and Street are selected in the DB Set.

Thanks,
Eve
[e-mail]

[Harbinger]

no don't make the names ALL parking[] , keep them as they were and add [] to the end of the name...

[kilbey1]

Ok, now I'm with you that far.

echo '
<input type="checkbox" name="Parking Lot[]" value="Parking Lot"'.(($row[parking]=='Parking Lot')? 'checked':'').'> Parking Lot 
<input type="checkbox" name="Street[]" value="Street"'.(($row[parking]=='Street')? 'checked':'').'>Street
<input type="checkbox" name="Valet[]" value="Valet"'.(($row[parking]=='Valet')? 'checked':'').'>Valet 
<input type="checkbox" name="On/Off Premises[]" value="On/Off Premises"'.(($row[parking]=='On/Off Premises')? 'checked':'').'>On/Off Premises
';

• Eve
  [e-mail]

[shu]

You only want to use [] if you have multiple sets of checkboxes on one page! In that case, having the [] is not enough, how are you gonna to distinguish one set from the other?

echo '
<input type="checkbox" name="ParkingLot['.$row['set_id'].']" value="Parking Lot"'.(($row[parking]=='Parking Lot')? 'checked':'').'> Parking Lot 
<input type="checkbox" name="Street['.$row['set_id'].']" value="Street"'.(($row[parking]=='Street')? 'checked':'').'>Street
<input type="checkbox" name="Valet['.$row['set_id'].']" value="Valet"'.(($row[parking]=='Valet')? 'checked':'').'>Valet 
<input type="checkbox" name="OnOffPremises['.$row['set_id'].']" value="On/Off Premises"'.(($row[parking]=='On/Off Premises')? 'checked':'').'>On/Off Premises';

PS: Using special characters in your variable names such as ( ) spaces and (/) slashes is a BAD BAD BAD practice!

[kilbey1]

Interesting. My PHP code now looks like this:

$parking = "
<input type=\"checkbox\" name=\"Parking Lot['.$row['parking'].']\" value=\"Parking Lot\"'.(($row[parking]=='Parking Lot')? 'checked':'').'> Parking Lot 
<input type=\"checkbox\" name=\"Street['.$row['parking'].']\" value=\"Street\"'.(($row[parking]=='Street')? 'checked':'').'>Street
<input type=\"checkbox\" name=\"Valet['.$row['parking'].']\" value=\"Valet\"'.(($row[parking]=='Valet')? 'checked':'').'>Valet 
<input type=\"checkbox\" name=\"On/Off Premises['.$row['parking'].']\" value=\"On/Off Premises\"'.(($row[parking]=='On/Off Premises')? 'checked':'').'>On/Off Premises
";
$frm->freerow("<B>Parking</B>",$parking);

And it's spitting back an error:
Parse error: parse error, expecting T_STRING' orT_VARIABLE' or `T_NUM_STRING' in /livesite/www/admin/dining_admin.php on line 307

I really have no idea how to clean this up, and still allow me to show which items are selected from the set. My set is set up like so:

Field name = parking
Set values: 'Parking Lot','Street','Valet','On/Off Premises'

Perhaps instead of using checkboxes as form elements, it may be better to use a multiple select form?

• Eve

[Weedpacket]

Originally posted by Harbinger
no don't make the names ALL parking[] , keep them as they were and add [] to the end of the name...

Well that sounds like a fantastic waste of time.... instad of a bunch of variables all with different names you end up with a bunch of arrays all with different names and onle ony element each.

Name the checkboxes "parking[]" and the values of all the checked checkboxes will be in the array $_POST['parking'][].

What the syntax for a MySQL Set (if that's what's being talked about) insertion looks like, I don't know; but if it's a comma-delimited list of strings, then you can get that from the $_POST array by going

$list="'".join("','",$_POST['parking'])."'";

But however it's written (should be in the MySQL documentation, I suppose), making up ever more baroque names for the checkboxes in the form is not gong to help at all.

[kilbey1]

Like this?

$list="'".join("','",$_POST['parking'])."'"; 
$parking = "<input type=\"checkbox\" name=\"parking[]\" value=\"Parking Lot\"";
$parking .= (($list)? ' checked':'');
$parking .= "> Parking Lot";

$parking .= " <input type=\"checkbox\" name=\"parking[]\" value=\"Street\"";
$parking .= (($list)? ' checked':'');
$parking .= ">Street";

$parking .= " <input type=\"checkbox\" name=\"parking[]\" value=\"Valet\"";
$parking .= (($list)? ' checked':'');
$parking .= ">Valet";

$parking .= " <input type=\"checkbox\" name=\"parking[]\" value=\"On/Off Premises\"";
$parking .= (($list)? 'checked':'');
$parking .= ">On/Off Premises";

I'm sure this is wrong, since it's not pulling any checks back. 🙂

[Weedpacket]

No; this like

$list="'".join("','",$_POST['parking'])."'"; 

goes in the page that is processing the form. Like I said, I guess you're wanting to put the list of selected checkboxes into the database - I figured that the above code would get them into a form suitable for sticking into a database query to do that.

To turn checkboxes on and off when the form is displayed - well, I presume you've got your array of different parking types that are to be shown as "checked" - which I'll call $row, following your initial code, and containing things like 'Street', 'Valet' and so on.

$parking .='<input type="checkbox" name="Valet[]" value="Valet"'.in_array($row,'Valet')? 'checked':'').'>Valet '
[/php