Everybody! This is important. In a few days, these forums will be moving over to using the totally sweet Discourse platform. To ensure this migration happens smoothly with no loss of content, these forums are currently in a read-only mode. I do apologize for the inconvenience.

There is never a good time to turn the forums off for an extended period of time, but I promise the new forums will be a billion times better. I'm pretty sure of it.

See you all on the other side in a few days, and if you have any (non-technical) questions, please e-mail me at kirupa@kirupa.com. For technical questions, try to find a tutorial that corresponds to what you are looking for and post in the comments section of that page.

Cheers,
Kirupa

Results 1 to 6 of 6

Thread: Query adds two records

  1. #1

    Query adds two records

    Hi folks,

    Have simple query to MYSQL database, executes fine, but adds two records instead of just the one.

    Any ideas?

  2. #2

  3. #3

    Here's my code

    This is what I'm using.

    PHP Code:
    <?php
    $hostname 
    ="*******";
    $username="*******";
    $password="*******";
    $database="*******";
    mysql_connect($hostname,$username,$password);
    @
    mysql_select_db($database) or die( "Unable to select database"); 

    $region=$_POST['prop_Region'];
    $name=$_POST['prop_Name'];
    $banner=$_POST['prop_Banner'];

    $query="INSERT INTO properties 

    (prop_Name, prop_Region, prop_Banner)
    VALUES 
    ('
    $name', '$region', '$banner')";

    mysql_query ($query);

    if (!
    mysql_query($query))
      {
      die(
    'Error: ' mysql_error());
      }
    echo 
    "success";
    mysql_close()

    ?>

  4. #4
    your query is running twice:
    PHP Code:
    mysql_query ($query); // 1st time

    if (!mysql_query($query)) // 2nd time
      
    {
      die(
    'Error: ' mysql_error());
      } 
    so use something like:
    PHP Code:
    $result mysql_query$query ) or die( mysql_error() ); 
    also, this method of taking $_POST variables straight into a query is ripe for a SQL injection attack. i highly advise against this.

  5. #5
    That's great man, thanks.

    re: the form variables, how would you advise handling them.

  6. #6

Thread Information

Users Browsing this Thread

There are currently 1 users browsing this thread. (0 members and 1 guests)

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •  

Home About kirupa.com Meet the Moderators Advertise

 Link to Us

 Credits

Copyright 1999 - 2012