a href issue with mysql fetch array

[scott815]

I am trying to make a the code below that works to show up inside a hyperlink for each listing. I tried wraping with a href but with no luck. I am new but trying to figure this out without any luck.

while($row = mysql_fetch_array($result, MYSQL_ASSOC))
{
echo $row['Rows'] . " " .$row['field_rental_state_value']. " <br>";
}

[grallis]

Hi scott815 -

Would help to see a bit more code, but I imagine you're trying to have the href in a link be dynamically populated with each id from a DB result set? In that case it would be -

while($row = mysql_fetch_assoc($result)){
echo "<a href=\"someScript.php?id=$row[id]\">$row[field_rental_state_value]</a> <br />";
}

Please verify this is what you need. Thanks

[scott815]

while($row = mysql_fetch_assoc($result)){
echo "<a href=\"someScript.php?id=$row[id]\">$row['Rows'] . " " .$row['field_rental_state_value']</a> <br />";
}

trying something like this. Trying to put what I have in the first and link it to site links like this www.website.com/state/[field_rental_state_value]

[PHP_Chimp]

echo "<a href=\"someScript.php?id=$row[id]\">{$row['Rows']} {$row['field_rental_state_value']}</a> <br />"; 

Values from arrays need to have {}'s around them.

Give that a try and see how it works.

[grallis]

Hi scott815 -

If you're going by this snippet of code you left

echo "<a href=\"someScript.php?id=$row[id]\">$row['Rows'] . " " .$row['field_rental_state_value']</a> <br />";

You'll need to jump out of your double quotes properly, and if you're not going to jump out of your double quotes, drop the single quotes around the array key, like one of these:

Jumping out to concatenate ...

echo "<a href=\"someScript.php?id=$row[id]\">" . $row['Rows'] . " " . $row['field_rental_state_value'] . "</a> <br />";

or this

Not jumping out ...

echo "<a href=\"someScript.php?id=$row[id]\">$row[Rows] $row[field_rental_state_value]</a> <br />";

[g1smd]

I always sanitise the data when using it as a URL... remove spaces (convert word breaks to hyphens or periods), make all lower-case, remove other punctuation, etc, and repurpose it for the anchor text: retain spaces, Camel Case the words etc.

[grallis]

Good call g1smd - I usually do it on the way in, unless it's all user-entered data, at which point i cleanse it on the way in, then take some precautionary measures on the way out.

[g1smd]

Sure, "garbage in, garbage out" is the first rule of computers.