Solved

Insert Records create Duplicate Records

Posted on 2010-09-10
3
257 Views
Last Modified: 2012-05-10
The code below is inserting additional data in another personal user information table.  however it keeps creating more than one record upon insert.  Why? and How do I fix it?
Protected Sub CreateUserWizard1_CreatedUser(ByVal sender As Object, ByVal e As System.EventArgs)

        'Insert Data into table and Add other pertinent data

        Dim UserNameTextBox As TextBox = CreateUserWizardStep1.ContentTemplateContainer.FindControl("UserName")

        Dim createDate As Date = DateTime.Now.ToString("MM/dd/yy")



        Dim rowsAffected As Integer

        Dim rIdentity As Integer = -1

        Try

            Dim myConnectionString As String

            Dim myQuery As String

            Dim myConnection As SqlConnection

            Dim cmd As SqlCommand





            myConnectionString = "Data Source=ServerIP;Initial Catalog=dbname; User Id=username; Password=password;"

            myQuery = "INSERT INTO aspnet_UserName (UserName, Firstname, Lastname, Prefixname, Suffixname, Nickname, " _

                & "CreateDate, AccountStatus, OrgNameID) VALUES ('" _

                & UserNameTextBox.Text & "', '" & Firstname.Text & "', '" & Lastname.Text & "', '" & PrefixnameDDL.Text & "', '" _

                & Suffix.Text & "', '" & Nickname.Text & "', '" & createDate & "', 'Enabled', '" _

                & getOrgNameID(OrganizationDDL.Text) & "') SELECT SCOPE_IDENTITY();"

            myConnection = New SqlConnection(myConnectionString)

            myConnection.Open()

            cmd = New SqlCommand(myQuery, myConnection)

            rowsAffected = cmd.ExecuteNonQuery()

            rIdentity = Integer.Parse(cmd.ExecuteScalar().ToString()) 'Get the @Identity Column



            myConnection.Close()

        Catch ex As Exception

            ClientScript.RegisterStartupScript(Me.GetType(), "alert", "<script language='javascript'>alert('CreateUserWizard1_CreatedUser produce the following error: " + ex.Message + "');</script>")

            Response.Write(ex.Message)

        End Try

    End Sub

Open in new window

0
Comment
Question by:Robert Treadwell
3 Comments
 
LVL 65

Expert Comment

by:rockiroads
ID: 33651225
one of two ways perhaps

create a unique index so that the operation fails

or

read the table, check the record exists or not
if it exists then update else insert
0
 
LVL 30

Expert Comment

by:MlandaT
ID: 33651245
1. You are using the CreateUserWizard.
2. When you use this wizard, the CreateUserWizard1_CreatedUser event only fires AFTER the user has already been created.
3. You are doing an insert there... which is creating the duplicate user.

With the CreateUserWizard, the user is automatically created for you, you dont need to execute an INSERT.

Comment out all the code in this method, compile and attempt to create a user. You will see that the duplicate users are gone.
0
 
LVL 5

Accepted Solution

by:
David Christal CISSP earned 500 total points
ID: 33651712
If MlandaT's suggestion doesn't work out try it with line 24 commented out.
Looks like you're executing the insert on both 24 & 25
0

Featured Post

Control application downtime with dependency maps

Visualize the interdependencies between application components better with Applications Manager's automated application discovery and dependency mapping feature. Resolve performance issues faster by quickly isolating problematic components.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Suggested Solutions

SQL Server engine let you use a Windows account or a SQL Server account to connect to a SQL Server instance. This can be configured immediatly during the SQL Server installation or after in the Server Authentication section in the Server properties …
This article explains how to reset the password of the sa account on a Microsoft SQL Server.  The steps in this article work in SQL 2005, 2008, 2008 R2, 2012, 2014 and 2016.
This tutorial gives a high-level tour of the interface of Marketo (a marketing automation tool to help businesses track and engage prospective customers and drive them to purchase). You will see the main areas including Marketing Activities, Design …
Windows 10 is mostly good. However the one thing that annoys me is how many clicks you have to do to dial a VPN connection. You have to go to settings from the start menu, (2 clicks), Network and Internet (1 click), Click VPN (another click) then fi…

895 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

16 Experts available now in Live!

Get 1:1 Help Now