Insert Problem?

MystedVeil

This form submits to two different tables (or rather it is supposed to 😃 ) based on the npc_id.

I am getting the error

Parse error: parse error, unexpected T_VARIABLE in /home/wowdesi/public_html/forms/add_npc.php on line 29

if(isset($_POST['submit'])){// handle the form
// Connect to the database.
require_once ('../incl/mysql_connect.php');
        // Query the database.    

$npc_name= mysql_real_escape_string($_POST['npc_name']);
$sub_name= mysql_real_escape_string($_POST['sub_name']);
$side = mysql_real_escape_string($_POST['side']); 
$npc_level = mysql_real_escape_string($_POST['npc_level']); 
$npc_rank = mysql_real_escape_string($_POST['npc_rank']); 
$npc_type = mysql_real_escape_string($_POST['npc_type']); 
$npc_family = mysql_real_escape_string($_POST['npc_family']); 
$npc_value_gold = mysql_real_escape_string($_POST['npc_value_gold']);  //Line 29 
$npc_value_copper = mysql_real_escape_string($_POST['npc_value_copper']); 
$quest_relations = mysql_real_escape_string($_POST['quest_relations']); 
$updated = mysql_real_escape_string($_POST['updated']); 


// The insert condition.
$query = "INSERT INTO  npc_beastiary (npc_id, npc_name, sub_name, side,  npc_level, npc_rank, npc_type, npc_family, npc_value_gold, npc_value_silver, npc_value_copper, quest_relations,  updated) VALUES
  ('',  '$npc_name', '$sub_name', '$side', '$npc_level', '$npc_rank', '$npc_type',  '$npc_family', '$npc_value_gold', '$npc_value_silver', '$npc_value_copper', '$quest_relations', NOW()) ";  

$result = @mysql_query ($query) or die ("could not execute query."); // Run the query.
$npc_id = mysql_insert_id()

// there can be a total of 4 related npc_id to zone_id/area_id's
$query = "INSERT INTO  npc_locations (npcloc_id, npc_id, zone_id, area_id, coordinate) VALUES
  ('$npcloc_id', ' $npc_id',  '$zone_id',  '$area_id',  '$coordinate'),
  ('$npcloc_id',  '$npc_id',  '$zone_id',  '$area_id',  '$coordinate'),
  ('$npcloc_id',  '$npc_id',  '$zone_id',  '$area_id',  '$coordinate'),
  ('$npcloc_id',  '$npc_id',  '$zone_id',  '$area_id',  '$coordinate')
   ";  

        $result = @mysql_query ($query)or die ("could not execute query."); // Run the query.

   if ($result) { // If it ran OK.


            // Send an email, if desired.
            echo '<table width=95% align=center class=tableOutline>
<tr><td>
<p>Thank you for adding this NPC!</p>

<p><a href=/forms/add_npc.php target=_self>Add another NPC</a></p>

</td></tr></table>';
       include ('../incl/footer.html'); // Include the HTML footer.
              exit();                

        } else { // If it did not run OK.
            // Send a message to the error log, if desired.
            echo '<table width=95% align=center class=tableOutline>
<tr><td><p>Your information could not be registered due to a system error. We apologize for any inconvenience.</p></td></tr></table>
'; 
            }        

    mysql_close(); // Close the database connection.

} // End of the main Submit conditional.

samudasu

The following line in your code is missing the closing semi colon. It's not on line 29 but these error messages don't always give the correct line where the real error occurs.
$npc_id = mysql_insert_id()

MystedVeil

Thank you, that took care of that problem. Kept looking over the code to find a missed } or ;... eyes must be too blurry.

Now there is a new problem, the form pulls up but it only submits information to the npc_locations table (the only info it submits there is npcloc_id and npc_id).

I can add my form if needed, thinking right now it has something to do with the inserts.

samudasu

do some simple debugging. Print out your $querys and see if they look ok, missing any info, etc.

frost110

Well dude, if there can be a total of 4, how can you tell one apart from the other? It seems you need a for loop with a check box or array, or at least different variable names.

You cannot expect 4 inputs to know how to handle themselves...This part of your code should probably look something similiar to this:


  ('$npcloc_id', ' $npc_id',  '$zone_id',  '$area_id',  '$coordinate'),
  ('$npcloc_id2',  '$npc_id2',  '$zone_id2',  '$area_id2',  '$coordinate2'),
  ('$npcloc_id3',  '$npc_id3',  '$zone_id3',  '$area_id3',  '$coordinate3'),
  ('$npcloc_id4',  '$npc_id4',  '$zone_id4',  '$area_id4',  '$coordinate4')

--FrosT

MystedVeil

Thank you, I have adjusted that and am now working on doing this with for loop. I am not real good with loops so I will probably be posting again in a little bit. 🙂