Getting old values of a textbox

I want to get new values from the Text box when i click the save button and then i want to update the database whit the new values of the Text box....I have looked for some solutions but cant figure it out...Here is the code i am using...
<table style="width:100%;">
        <tr>
            <td>
                <asp:Label ID="lblEditPost" runat="server" Text="Edit post:"></asp:Label>
            </td>
            <td>
                &nbsp;</td>
        </tr>
        <tr>
            <td>
                <asp:TextBox ID="txtEditPost" runat="server" Height="202px" 
                    TextMode="MultiLine" Width="501px" EnableViewState="true"></asp:TextBox>
            </td>
            <td>
                &nbsp;</td>
        </tr>
        <tr>
            <td>
                <asp:Button ID="btnUpdatePost" runat="server" Text="Save" />
                <br /><br />
                <asp:RequiredFieldValidator ID="RequiredFieldValidator1" runat="server" 
                    ControlToValidate="txtEditPost" ErrorMessage="The post cant be empty!"></asp:RequiredFieldValidator>
            </td>
            <td>
                &nbsp;</td>
        </tr>
    </table>
------------------------------------------------------------------
Imports System.Web.Security
Imports System.Data
Imports System.Data.OleDb
Imports System.Configuration
Partial Class editPost
    Inherits System.Web.UI.Page
    Protected Sub Page_Load(ByVal sender As Object, ByVal e As System.EventArgs) Handles Me.Load
        getPost()
    End Sub
    Protected Sub btnUpdatePost_Click(ByVal sender As Object, ByVal e As System.EventArgs) Handles btnUpdatePost.Click
 
        updatePost(txtEditPost.Text)
 
    End Sub
 
    Sub updatePost(ByVal strPostContent As String)
 
        Dim id As Integer = Request.QueryString("id")
 
        Dim MyConn As OleDbConnection = New OleDbConnection(ConfigurationManager.AppSettings("strConn"))
        Dim MySQL As String = "UPDATE [tblPostings] SET [PostContent]=@PostContent WHERE [tblPostings.PostID] = @id"
        Dim MyCmd As New OleDbCommand(MySQL, MyConn)
 
        With MyCmd.Parameters
            .Add(New OleDbParameter("@id", id))
            .Add(New OleDbParameter("@PostContent", strPostContent))
        End With
 
        MyConn.Open()
        MyCmd.ExecuteNonQuery()
        'Response.Redirect("viewtopic.aspx?id=", id)
        MyConn.Close()
 
    End Sub
End Class

Open in new window

NesteAsked:
Who is Participating?
 
abelCommented:
In the code you are showing, a few things bear a little attention:

> Dim id As Integer = Request.QueryString("id")

there's no sign of the "id" in your querystring. Since you use a POST method, maybe you meant to get the contents of a hidden field that goes by the name "id"

Same line: QueryString returns a string. You are requesting an integer. You should get an error there. Either way, it is best to convert the string to an integer before assigning it to one.

> getPost()

this method is called in the Page_Load, which also means it will be called before the event of the button_click. You do not show that method, but if it sets any values, i.e. of your txtEditPost, it will reset its value to its original one, which is then the cause of your problem. Can you show us that method too, please?

-- Abel --
0
 
EspavoCommented:
This looks like it should work...
Are you passing a value for Request.QueryString("id")?
0
 
NesteAuthor Commented:
Yeah, iam passing the PostID...

Really cant figure whats the problem is, why its not saving the new value to txtEditPost.Text...Everytime i try and edit any posts and click the Save button i just get the same old value :S
0
Cloud Class® Course: Microsoft Windows 7 Basic

This introductory course to Windows 7 environment will teach you about working with the Windows operating system. You will learn about basic functions including start menu; the desktop; managing files, folders, and libraries.

 
NesteAuthor Commented:
i dont have any other methods for it...i didn't use getPost()

I have a gridview in viewtopic.aspx whit user posts...When i click a specific post i send the PostID to editpost.aspx....

Almost everything works except that when i edit a post i allways and hit the Save button, it posts back the old value of the post...giving txtEditPost.Text the old value...
0
 
NesteAuthor Commented:
Here is the getPost() this code is ok i think...sorry didn't get you, so tired hehe
    Sub getPost()
 
        Dim id As Integer = Request.QueryString("id")
 
        Dim MyConn As OleDbConnection = New OleDbConnection(ConfigurationManager.AppSettings("strConn"))
        Dim MySQL As String = "SELECT * FROM [tblPostings] WHERE [tblPostings.PostID] = @id"
        Dim MyCmd As New OleDbCommand(MySQL, MyConn)
 
        With MyCmd.Parameters
            .Add(New OleDbParameter("@id", id))
        End With
 
        MyConn.Open()
        Dim dataReader As OleDbDataReader = MyCmd.ExecuteReader()
 
        Dim PostContent As String = dataReader.Read()
 
        Dim post As String = dataReader("PostContent")
 
        txtEditPost.Text = post
 
        txtEditPost.DataBind()
 
        MyConn.Close()
 
    End Sub

Open in new window

0
 
NesteAuthor Commented:
hmmm now i know what you mean abel...Can i do something about this...call the getpost() somewhere else ??
0
 
NesteAuthor Commented:
Got it, now it works fine...

        If Not IsPostBack Then
            getPost()
        End If

Thx Abel
0
 
abelCommented:
You were quicker than me! Yes, the Page.IsPostBack is what you should use to differentiate between the first access to a page or any time an action is requested.
0
 
NesteAuthor Commented:
Damn now i got another problem whit the same code...

Now i get the new value from txtEditPost.Text and everything works great BUT it dont update the DB :S

I am not getting any error's but i see in db that nothing is updated...

Any suggestions what problem may be?

0
 
NesteAuthor Commented:
hmmm MyCmd.ExecuteNonQuery() returns 0

Any help ? :)
0
 
abelCommented:
Though that is a completely new question and there may be experts that are better at that subject than I, I can give you a hint of where to look. The ExecuteNonQuery returns the number of rows affected. If it fails, it will raise an error. That it returns zero means that the WHERE-part of the UPDATE query does not select anything.

Try to place a breakpoint in your code and create the statement by hand from this:

        UPDATE [tblPostings] SET [PostContent]=@PostContent WHERE [tblPostings.PostID] = @id

by replacing @id and @PostContent with the values they are holding (you can see that on the mouseover while in break mode). Go to your database (use any SQL client you like) and execute the statement by hand and tweak uit until it works.

Note that the WHERE-part can be used in a SELECT statement to find out whether you are actually selecting anything to be updated (the most common error is that you are not selecting anything):

       SELECT * FROM  tblPostings WHERE tblPostings.PostID = @id

-- Abel --
 
0
 
NesteAuthor Commented:
My parameters were wrong, thx abel :)
0
Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.

All Courses

From novice to tech pro — start learning today.