Skip to content
  • Categories
  • Recent
  • Tags
  • Popular
  • World
  • Users
  • Groups
Skins
  • Light
  • Cerulean
  • Cosmo
  • Flatly
  • Journal
  • Litera
  • Lumen
  • Lux
  • Materia
  • Minty
  • Morph
  • Pulse
  • Sandstone
  • Simplex
  • Sketchy
  • Spacelab
  • United
  • Yeti
  • Zephyr
  • Dark
  • Cyborg
  • Darkly
  • Quartz
  • Slate
  • Solar
  • Superhero
  • Vapor

  • Default (No Skin)
  • No Skin
Collapse
Code Project
  1. Home
  2. General Programming
  3. C#
  4. Update stored procedure doesn't update, it keeps on saying info modification fails, please help

Update stored procedure doesn't update, it keeps on saying info modification fails, please help

Scheduled Pinned Locked Moved C#
sharepointdatabasehelpannouncement
4 Posts 4 Posters 0 Views 1 Watching
  • Oldest to Newest
  • Newest to Oldest
  • Most Votes
Reply
  • Reply as topic
Log in to reply
This topic has been deleted. Only users with topic management privileges can see it.
  • U Offline
    U Offline
    User 10307097
    wrote on last edited by
    #1

    private void Update_Click(object sender, EventArgs e) { try { if (connect.State == ConnectionState.Open) { connect.Close(); } connect.Open(); SqlCommand cmd = connect.CreateCommand(); cmd = new SqlCommand("sp_UpdateDepartStaff", connect); cmd.CommandType = CommandType.StoredProcedure; cmd.Parameters.AddWithValue("@StaffID", SqlDbType.Int).Value = logInIDTextBox.Text.ToString(); cmd.Parameters.AddWithValue("@LogInID", SqlDbType.Int).Value = staffIDTextBox.Text.ToString(); cmd.Parameters.AddWithValue("@StaffRoleID", SqlDbType.Int).Value = staffRoleIDTextBox.Text.ToString(); cmd.Parameters.AddWithValue("@FirstName", SqlDbType.NVarChar).Value = firstNameTextBox.Text; cmd.Parameters.AddWithValue("@LastName", SqlDbType.NVarChar).Value = lastNameTextBox.Text; cmd.Parameters.AddWithValue("@UserName", SqlDbType.NVarChar).Value = userNameTextBox.Text; cmd.Parameters.AddWithValue("@Password", SqlDbType.NVarChar).Value = passwordTextBox.Text; cmd.Parameters.AddWithValue("@Email", SqlDbType.NVarChar).Value = emailTextBox.Text; cmd.Parameters.AddWithValue("@Phone", SqlDbType.NVarChar).Value = phoneTextBox.Text; cmd.Parameters.AddWithValue("@StaffRole", SqlDbType.NVarChar).Value = staffRoleTextBox.Text; cmd.Parameters.AddWithValue("@Department", SqlDbType.NVarChar).Value = departmentTextBox.Text; // cmd.Parameters.Add(new SqlParameter("@ProfilePic", SqlDbType.Image)); MemoryStream MemStream = new MemoryStream(); byte[] DataPic_Update = null; this.profilePicPictureBox.Image.Save(MemStream, ImageFormat.Png); DataPic_Update = MemStream.GetBuffer(); MemStream.Read(DataPic_Update, 0, DataPic_Update.Length); // image content SqlParameter photo = new SqlParameter("@profilePic", SqlDbType.Image); photo.Value = DataPic_Update; cmd.Parameters.Add(photo); int temp = 0; temp = cmd.ExecuteNonQuery(); if (temp > 0) { connect.Close(); MessageBox.Show("Your info upda

    C S T 3 Replies Last reply
    0
    • U User 10307097

      private void Update_Click(object sender, EventArgs e) { try { if (connect.State == ConnectionState.Open) { connect.Close(); } connect.Open(); SqlCommand cmd = connect.CreateCommand(); cmd = new SqlCommand("sp_UpdateDepartStaff", connect); cmd.CommandType = CommandType.StoredProcedure; cmd.Parameters.AddWithValue("@StaffID", SqlDbType.Int).Value = logInIDTextBox.Text.ToString(); cmd.Parameters.AddWithValue("@LogInID", SqlDbType.Int).Value = staffIDTextBox.Text.ToString(); cmd.Parameters.AddWithValue("@StaffRoleID", SqlDbType.Int).Value = staffRoleIDTextBox.Text.ToString(); cmd.Parameters.AddWithValue("@FirstName", SqlDbType.NVarChar).Value = firstNameTextBox.Text; cmd.Parameters.AddWithValue("@LastName", SqlDbType.NVarChar).Value = lastNameTextBox.Text; cmd.Parameters.AddWithValue("@UserName", SqlDbType.NVarChar).Value = userNameTextBox.Text; cmd.Parameters.AddWithValue("@Password", SqlDbType.NVarChar).Value = passwordTextBox.Text; cmd.Parameters.AddWithValue("@Email", SqlDbType.NVarChar).Value = emailTextBox.Text; cmd.Parameters.AddWithValue("@Phone", SqlDbType.NVarChar).Value = phoneTextBox.Text; cmd.Parameters.AddWithValue("@StaffRole", SqlDbType.NVarChar).Value = staffRoleTextBox.Text; cmd.Parameters.AddWithValue("@Department", SqlDbType.NVarChar).Value = departmentTextBox.Text; // cmd.Parameters.Add(new SqlParameter("@ProfilePic", SqlDbType.Image)); MemoryStream MemStream = new MemoryStream(); byte[] DataPic_Update = null; this.profilePicPictureBox.Image.Save(MemStream, ImageFormat.Png); DataPic_Update = MemStream.GetBuffer(); MemStream.Read(DataPic_Update, 0, DataPic_Update.Length); // image content SqlParameter photo = new SqlParameter("@profilePic", SqlDbType.Image); photo.Value = DataPic_Update; cmd.Parameters.Add(photo); int temp = 0; temp = cmd.ExecuteNonQuery(); if (temp > 0) { connect.Close(); MessageBox.Show("Your info upda

      C Offline
      C Offline
      Chris Quinn
      wrote on last edited by
      #2

      What value is returned in temp? Do you have the stored procedure code? A message that something didn't work is not very helpful - you need to debug to find any return messages or error codes

      ========================================================= I'm an optoholic - my glass is always half full of vodka. =========================================================

      1 Reply Last reply
      0
      • U User 10307097

        private void Update_Click(object sender, EventArgs e) { try { if (connect.State == ConnectionState.Open) { connect.Close(); } connect.Open(); SqlCommand cmd = connect.CreateCommand(); cmd = new SqlCommand("sp_UpdateDepartStaff", connect); cmd.CommandType = CommandType.StoredProcedure; cmd.Parameters.AddWithValue("@StaffID", SqlDbType.Int).Value = logInIDTextBox.Text.ToString(); cmd.Parameters.AddWithValue("@LogInID", SqlDbType.Int).Value = staffIDTextBox.Text.ToString(); cmd.Parameters.AddWithValue("@StaffRoleID", SqlDbType.Int).Value = staffRoleIDTextBox.Text.ToString(); cmd.Parameters.AddWithValue("@FirstName", SqlDbType.NVarChar).Value = firstNameTextBox.Text; cmd.Parameters.AddWithValue("@LastName", SqlDbType.NVarChar).Value = lastNameTextBox.Text; cmd.Parameters.AddWithValue("@UserName", SqlDbType.NVarChar).Value = userNameTextBox.Text; cmd.Parameters.AddWithValue("@Password", SqlDbType.NVarChar).Value = passwordTextBox.Text; cmd.Parameters.AddWithValue("@Email", SqlDbType.NVarChar).Value = emailTextBox.Text; cmd.Parameters.AddWithValue("@Phone", SqlDbType.NVarChar).Value = phoneTextBox.Text; cmd.Parameters.AddWithValue("@StaffRole", SqlDbType.NVarChar).Value = staffRoleTextBox.Text; cmd.Parameters.AddWithValue("@Department", SqlDbType.NVarChar).Value = departmentTextBox.Text; // cmd.Parameters.Add(new SqlParameter("@ProfilePic", SqlDbType.Image)); MemoryStream MemStream = new MemoryStream(); byte[] DataPic_Update = null; this.profilePicPictureBox.Image.Save(MemStream, ImageFormat.Png); DataPic_Update = MemStream.GetBuffer(); MemStream.Read(DataPic_Update, 0, DataPic_Update.Length); // image content SqlParameter photo = new SqlParameter("@profilePic", SqlDbType.Image); photo.Value = DataPic_Update; cmd.Parameters.Add(photo); int temp = 0; temp = cmd.ExecuteNonQuery(); if (temp > 0) { connect.Close(); MessageBox.Show("Your info upda

        S Offline
        S Offline
        Simon_Whale
        wrote on last edited by
        #3

        1. Your first 3 parameters are defined as integer but you are passing a string. 2. Why are you converting a Textbox.Text to string? e.g Textbox.Text.ToString() Have you tried to run the stored procedure directly on the server with some dummy values? if so did it work sucessfully or did it throw an error?

        Every day, thousands of innocent plants are killed by vegetarians. Help end the violence EAT BACON

        1 Reply Last reply
        0
        • U User 10307097

          private void Update_Click(object sender, EventArgs e) { try { if (connect.State == ConnectionState.Open) { connect.Close(); } connect.Open(); SqlCommand cmd = connect.CreateCommand(); cmd = new SqlCommand("sp_UpdateDepartStaff", connect); cmd.CommandType = CommandType.StoredProcedure; cmd.Parameters.AddWithValue("@StaffID", SqlDbType.Int).Value = logInIDTextBox.Text.ToString(); cmd.Parameters.AddWithValue("@LogInID", SqlDbType.Int).Value = staffIDTextBox.Text.ToString(); cmd.Parameters.AddWithValue("@StaffRoleID", SqlDbType.Int).Value = staffRoleIDTextBox.Text.ToString(); cmd.Parameters.AddWithValue("@FirstName", SqlDbType.NVarChar).Value = firstNameTextBox.Text; cmd.Parameters.AddWithValue("@LastName", SqlDbType.NVarChar).Value = lastNameTextBox.Text; cmd.Parameters.AddWithValue("@UserName", SqlDbType.NVarChar).Value = userNameTextBox.Text; cmd.Parameters.AddWithValue("@Password", SqlDbType.NVarChar).Value = passwordTextBox.Text; cmd.Parameters.AddWithValue("@Email", SqlDbType.NVarChar).Value = emailTextBox.Text; cmd.Parameters.AddWithValue("@Phone", SqlDbType.NVarChar).Value = phoneTextBox.Text; cmd.Parameters.AddWithValue("@StaffRole", SqlDbType.NVarChar).Value = staffRoleTextBox.Text; cmd.Parameters.AddWithValue("@Department", SqlDbType.NVarChar).Value = departmentTextBox.Text; // cmd.Parameters.Add(new SqlParameter("@ProfilePic", SqlDbType.Image)); MemoryStream MemStream = new MemoryStream(); byte[] DataPic_Update = null; this.profilePicPictureBox.Image.Save(MemStream, ImageFormat.Png); DataPic_Update = MemStream.GetBuffer(); MemStream.Read(DataPic_Update, 0, DataPic_Update.Length); // image content SqlParameter photo = new SqlParameter("@profilePic", SqlDbType.Image); photo.Value = DataPic_Update; cmd.Parameters.Add(photo); int temp = 0; temp = cmd.ExecuteNonQuery(); if (temp > 0) { connect.Close(); MessageBox.Show("Your info upda

          T Offline
          T Offline
          toyfercho
          wrote on last edited by
          #4

          The cmd.ExecuteNonQuery should return the number of rows affected by the command on UPDATE statement in this case if you are getting this message "Your info modification failed " means that for some reason the SP is not updating or is passing null values in the parameters could ya please share the content of the SP that way we can give a more precise solution.

          1 Reply Last reply
          0
          Reply
          • Reply as topic
          Log in to reply
          • Oldest to Newest
          • Newest to Oldest
          • Most Votes


          • Login

          • Don't have an account? Register

          • Login or register to search.
          • First post
            Last post
          0
          • Categories
          • Recent
          • Tags
          • Popular
          • World
          • Users
          • Groups