brady.k — Sat, 17 Jul 2004 20:00:49 +0000

Yeah it works...thanks to abhishek

Abhishek Reddy — Sat, 17 Jul 2004 03:55:39 +0000

Busy wrote: your missing the database table name, $location isn't working

Yeah. In fact, the whole switch() wasn't working -- the data sent to $group was malformed. The form fields were contructed as: descriptive_text. It needed to be value="intended_code">descriptive_text. Otherwise, the default $group value was descriptive_text.

Additionally, the if() statements were using the assignment symbol (=), not the comparison (==) operator.

Just for reference.

Busy — Fri, 16 Jul 2004 05:39:01 +0000

Eskater05 wrote: Yes, i just did...and here's the results
UPDATE SET name = 'test', time = '5:25.11', year = '1235' WHERE event = '25 Free''

your missing the database table name, $location isn't working

vexcom — Fri, 16 Jul 2004 05:16:45 +0000

Not sure if this will help at all, but i get problems whenever i use column names that are also SQL reserved functions or language reserved words.

For example you should avoid naming columns "group" because SQL gets confused when it sees the word group, Ive had similar problems with USER, URL and someother weird ones. When in doubt add My.. so try MyGroup or Group1 as the column name...

This may not help in this case, but its good practice yo

GL!

- oh, one more thing, try breakingit down to the simplest esence, like one case, or use hardcodd values before you start using cariables make sure you basic SQL code is solid...

brady.k — Wed, 07 Jul 2004 14:58:20 +0000

Yes, i just did...and here's the results
UPDATE SET name = 'test', time = '5:25.11', year = '1235' WHERE event = '25 Free''

Busy — Thu, 01 Jul 2004 21:47:42 +0000

have you tried echoing your values after the switch statement, or echoing the $sql value before the query

brady.k — Thu, 01 Jul 2004 16:48:10 +0000

ok, i have spent some time on this, and modified it a little. it still doesnt work.

heres the new php:

<?php
/* Get input */
$group = $_POST['group'];
$name = addslashes($_POST['name']);
$time = addslashes($_POST['time']);
$year = $_POST['year'];
$event = $_POST['event'];

/* Find right case */
switch ($group) {
   case 'Under6Girls':
    $location = \"Under6Girls\";
   break;

   case 'Under6Boys':
    $location = \"Under6Boys\";
   break;

   case 'MixedUnder6':
    $location = \"relay\";
    $event = \"100 Free Mixed Under6\";
   break;
   
   case 'Under8Girls':
       $location = \"Under8Girls\";
   break;

   case 'Under8Boys':
       $location = \"Under8Boys\";
   break;

   case 'MixedUnder8':
    $location = \"relay\";
    $event = \"100 Free Mixed Under8\";
   break;
   
   case '910Girls':
       $location = \"910Girls\";
    if ($event = \"200 Free Relay\") {
    $location = \"relay\";
    $event = \"200 Free Girls 910\";
    };
   break;

   case '910Boys':
     $location = \"910Boys\";
    if ($event = \"200 Free Relay\") {
    $location = \"relay\";
    $event = \"200 Free Boys 910\";
    };
   break;
   
   case '1112Girls':
     $location = \"1112Girls\";
    if ($event = \"200 Free Relay\") {
    $location = \"relay\";
    $event = \"200 Free Girls 1112\";
    };
   break;
   
   case '1112Boys':
     $location = \"1112Boys\";
    if ($event = \"200 Free Relay\") {
    $location = \"relay\";
    $event = \"200 Free Boys 1112\";
    };
   break;
   
   case '1314Girls':
     $location = \"1314Girls\";
    if ($event = \"200 Free Relay\") {
    $location = \"relay\";
    $event = \"200 Free Girls 1314\";
    };
   break;
   
   case '1314Boys':
     $location = \"1314Boys\";
    if ($event = \"200 Free Relay\") {
    $location = \"relay\";
    $event = \"200 Free Boys 1314\";
    };
   break;
   
   case '1518Girls':
     $location = \"1518Girls\";
   break;
   
   case '1518Boys':
     $location = \"1518Boys\";
   break;
   
   case 'Mixed1518':
    $location = \"relay\";
    $event = \"200 Free Mixed 1518\";
   break;
};
  
/* Update */
$sql  = \"UPDATE \" . $location . \" SET \";
$sql .= \"name = '\" . $name . \"', \";
$sql .= \"time = '\" . $time . \"', \";
$sql .= \"year = '\" . $year . \"' \";
$sql .= \"WHERE event = '\" . $event. \"'\"; 
$query = mysql_query($sql) or die(mysql_error());

/* Disconnect */
disconnect_dbh();

echo \"Done.  <a href=\\"javascript:go_back()\\">Go Back</a>\";
?>

You can see the page at dscpool.com/records

ShawnK — Sun, 20 Jun 2004 04:19:01 +0000

lol...

The problem is from 9-10 year old boys and down you put a ; instead of a : after the case.

Busy — Tue, 01 Jun 2004 10:13:32 +0000

write & like this instead amp;
the gaps in the names could be a problem too

Greg K — Tue, 01 Jun 2004 05:12:12 +0000

The "yea" was most likely it cutting off the SQL statement in the middle of the "year = '..." part.

What is the structure of your database? What are the field types?

Try just for testing to remove the & and then see if it works. If it works then, you will know that is the problem. If you still get the error without the &, then that works, and something else must be wrong, but that is where I would start first.

-Greg