"Jennifer Downey" <[EMAIL PROTECTED]> wrote in message
news:[EMAIL PROTECTED]...
> I have no takers on this one?
You would have more help if you didn't glumph
a whole whack of code in... it takes five minutes
just to sort out what's what.
> if I have on item it is fine. If I have two items it
> won't update the first items price but will the
> second. if I try to enter a price in the first items
> textbox it doesn't update and then deletes the
> second item's price.
It sounds like you are trying to return multiple
values to a single variable - ie, you need to
return an array, then iterate through the array
to process it.
> $uid = $row['uid'];
> $id = $row['id'];
> $name = $row['name'];
> $image = $row['image'];
> $iquantity = $row['quantity'];
> $itype = $row['type'];
> $iprice = $row['price'];
Look up the documentation on extract()
> if($update)
> {
> $eprice = '$price[]';
> $query = "UPDATE {$config["prefix"]}_shop SET price = '$eprice'
> where uid = {$session["uid"]} AND id = '$id'";
> $ret = mysql_query($query) or die(mysql_error());
if(isset($update) and is_array($price))
foreach($price as $id => $newval)
if ($newval != "") {
$query =
"UPDATE {$config['prefix']}_shop "
."SET price='$newval' "
."WHERE uid='{$session['uid']}' AND id='$id' ";
mysql_query($query) or die(mysql_error());
}
> echo "<TABLE BORDER='0' WIDTH='95%' CELLPADDING='0'
> CELLSPACING='0'><TR>";
> echo "<TD width=20%><img src='$image'></TD>";
> echo "<TD width=30%><font size=2>$name</font></TD>";
> echo "<TD width=20%><font
> size=2><CENTER>$iquantity</CENTER></font></TD>";
> echo "<TD width=30%><font size=2><CENTER><a
> href='$PHP_SELF?id=$id&remove=yes'>X</a></CENTER></font></TD>";
> echo "<TD width=30%><font size=2><CENTER><input type=\"text\"
> value=\"\" name=\"price[]\" size='8'
> MAXLENGTH='8'><BR></a></CENTER></font></TD>";
> echo "</TD></TR></TABLE>";
> echo "<input=\"hidden\" name=\"remove\" value=\"yes\">";
> }
(grin) you realize your column widths add to 130% ?
This is not necessarily a problem; rather, I point it out as
a symptom of poorly formatted and hard-to-follow code.
I often find it useful to write simple table-making functions
just so it's easier to follow what's going on... something
like
// adjustable indentation for prettyprinting
define("BEGINTABLE", "\n\t");
define("ENDTABLE", "\n\t");
define("BEGINROW", "\n\t\t");
define("ENDROW", "\n\t\t");
define("BEGINCELL", "\n\t\t\t");
define("ENDCELL", "");
define("BEGINCONTENTS", "");
function makeTable($content, $width="", $border=0) {
return
BEGINTABLE."<table"
.( $width != "" ? " width='$width'" : "")
." border='$border' cellspacing='0' cellpadding='0'>"
.$content
.ENDTABLE."<\table>";
}
function makeRow($content) {
return
BEGINROW."<tr>"
.$content
.ENDROW."<\tr>";
}
function makeCell($content="", $width="") {
return
BEGINCELL."<td".($width != "" ? " width='$width'" : "").">"
.($content != "" ? $content : " ")
.ENDCELL."</td>";
}
Then your code turns into
// separate out the recurrent formatting
$s = "<div style='font-size: larger; align=center;'>";
$e = "</div>";
$content = makeRow(
makeCell($s."Image".$e, "20%")
.makeCell($s."Name".$e, "30%")
.makeCell($s."Quantity".$e, "20%")
.makeCell($s."Remove Item".$e, "30%")
.makeCell($s."Price".$e, "30%")
);
// Wow, the odd total here is a lot more obvious!
$query =
"SELECT uid, id, name, image, type, quantity "
."FROM {$config["prefix"]}_shop "
."WHERE uid = {$session["uid"]}";
$res = mysql_query($query) or die(mysql_error());
while($row = mysql_fetch_array($res)) {
extract($row);
$content .= makeRow(
makeCell("<img src='$image'>")
.makeCell($s.$name.$e)
.makeCell($s.$quantity.$e)
.makeCell($s."<input type='checkbox' "
."name='rem[$id]' value='true'>".$e)
.makeCell($s."<input type='text' value='' "
."name='price[$id]' size='8' maxlength='8'>".$e)
);
}
echo
"\n<form>"
.makeTable($content, "95%")
.$s
."<input type='reset' value='Clear form'>"
."<input type='submit' name='update' "
."value='Update And Remove'>"
.$e
."\n</form>";
... I have changed a few things; for one, instead
of removing items singly, I have refit a set of
checkboxes, returning an array of selected items
for removal.
Also, note my use of single-quotes inside the double-quoted
strings... I find this much easier to follow, instead of umpteen
dozen escaped double-slashes.
I hope this is of some use to you.
Hugh.
--
PHP General Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
