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?
I wear a lot of hats...

"The solutions and answers provided on Experts Exchange have been extremely helpful to me over the last few years. I wear a lot of hats - Developer, Database Administrator, Help Desk, etc., so I know a lot of things but not a lot about one thing. Experts Exchange gives me answers from people who do know a lot about one thing, in a easy to use platform." -Todd S.

EspavoCommented:
This looks like it should work...
Are you passing a value for Request.QueryString("id")?
0
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

Experts Exchange Solution brought to you by

Your issues matter to us.

Facing a tech roadblock? Get the help and guidance you need from experienced professionals who care. Ask your question anytime, anywhere, with no hassle.

Start your 7-day free trial
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
Exploring SQL Server 2016: Fundamentals

Learn the fundamentals of Microsoft SQL Server, a relational database management system that stores and retrieves data when requested by other software applications.

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
It's more than this solution.Get answers and train to solve all your tech problems - anytime, anywhere.Try it for free Edge Out The Competitionfor your dream job with proven skills and certifications.Get started today Stand Outas the employee with proven skills.Start learning today for free Move Your Career Forwardwith certification training in the latest technologies.Start your trial today
ASP.NET

From novice to tech pro — start learning today.