[RESOLVED] Method being executed twice.

[creativepragmat]

Hello everyone,

I am developing a standard script to register new users but the INSERT query keeps getting called twice for no apparent reason prompting the following error as a result of the 'email' field being set to unique:

Warning: pg_query() [function.pg-query]: Query failed: ERROR: duplicate key value violates unique constraint "users_email_key" in Register.php on line 30
ERROR: duplicate key value violates unique constraint "users_email_key"

This is a stripped down test version of register().

<?php
class Register
{
public function register()
{
echo "Test";
}
}
?>

This is a simple test of the method:

<?php
require_once('Register.php');

$register = new Register();

$register->register();
?>

The output is 'TestTest'.

I should also note that this happens both on my test server and production server. I worked with PHP for years but never encountered this before. Hopefully, somebody else has had the same problem and can help without effort.

Thank you for any assistance.

[djjjozsi]

Hi,
before you run the query with php, let's test it in phpmyadmin.
Before you write into that table, test if there is that email address in the table. I quess just, but has that table's field a Unique option ?

hi,
josh

http://www.phpcode.hu

creativepragmat;10889508 wrote:

Hello everyone,

I am developing a standard script to register new users but the INSERT query keeps getting called twice for no apparent reason prompting the following error as a result of the 'email' field being set to unique:

Warning: pg_query() [function.pg-query]: Query failed: ERROR: duplicate key value violates unique constraint "users_email_key" in Register.php on line 30
ERROR: duplicate key value violates unique constraint "users_email_key"

This is a stripped down test version of register().

<?php
require_once('Connect.php');

class Register
{
public function register()
{
$conn = new Connect();

    $conn->openConnection();

    $result = pg_query($conn->getConnection(), "INSERT INTO users (email, name, passhash, last_ip, level) VALUES (
            '[e-mail]', 'test', 'test', '" . $_SERVER['REMOTE_ADDR'] . "', '3')") or die(pg_last_error());

    $conn->closeConnection();
}

}
?>[/B]

This is a simple test of the method:

<?php
require_once('Register.php');

$register = new Register();

$register->register();
?>

I should also note that this happens both on my test server and production server. I worked with PHP for years but never encountered this before. Hopefully, somebody else has had the same problem and can help without effort.

Thank you all for any assistance.

[creativepragmat]

Hi Josh,

Thank you for taking the time to respond. When I run the query on phpMyAdmin, it executes once as it should. The original code did check to see that the email address exists before executing the INSERT statement. I just simplified the code for testing and to make it easier for forum users.

[Kudose]

Try removing $result = since you don't actually call it.

[creativepragmat]

Thank you Kudose. I removed $result and it is still inserting twice.

Time for first insertion: 2008-10-13 10:11:21.948655
Time for second insertion: 2008-10-13 10:11:21.981228

[madwormer2]

It could be the fact that your function in the class is the same name as the class. When you do $register = new Register();, php initialises the class and attempts to run register() function from within it. Either remove the () from that line, or leave out the other line calling register.

[creativepragmat]

You hit the nail right on the head 'madwormer2'! Most of my experience with PHP is with 4+ and not 5 so I was not aware of how PHP handles classes. Thank you for taking the time to respond. I really appreciate it.

[Kudose]

::smacks forehead::

duh

The case insensitivity threw me off. Good catch.

Please mark this thread resolved using the thread tools if you have no more questions.

[creativepragmat]

It is resolve but I can't find the thread tools.

[Kudose]

It's at the top of the thread on the right side ... third forum link menu from the top.

[creativepragmat]

Thanks again!