c# manually binding data to gridview via stored procedure + parameter

Question

I've been getting the error

System.Data.SqlClient.SqlException: Procedure or function 'GetActiveComissions' expects parameter '@param1', which was not supplied.

on the code line da.Fill(dt). When I try to bind data to my GridView based on a stored procedure, clearly supplying the parameter.

Code:

ApplicationDbContext context = new ApplicationDbContext();
var UserManager = new UserManager<ApplicationUser>(new UserStore<ApplicationUser>(context));
string user = User.Identity.GetUserId();

string storedprocedure = "GetActiveComissions";
SqlConnection cn = GetSqlCon.GetCon();

DataTable dt = new DataTable();

SqlCommand cmd = new SqlCommand(storedprocedure, cn); 
cmd.CommandType = System.Data.CommandType.StoredProcedure;
cmd.Parameters.AddWithValue("@param1", user);

SqlDataAdapter da = new SqlDataAdapter(cmd);
da.Fill(dt);

GridViewActiveComissions.DataSource = dt;
GridViewActiveComissions.DataBind();

Could I be passing the parameter in a wrong way?

The stored procedure:

CREATE PROCEDURE [dbo].[GetActiveComissions]
    @param1 nvarchar(128)
AS
    SELECT 
        Comissions.CreationDate, Comissions.Accepted, 
        Comissions.BicycleModel, Comissions.BicycleType, 
        Comissions.YearOfAquisition, Comissions.Details, 
        AspNetUsers.WorkshopName 
    FROM 
        Comissions 
    INNER JOIN 
        AspNetUsers ON Comissions.WorkshopId = AspNetUsers.Id 
                    AND Comissions.Accepted = 1 
                    AND Comissions.ClientId = @param1
    RETURN 0

I've tested the stored procedure separately and it does work as intended, given the parameter is provided.

Slobodzyan Taras · Accepted Answer · 2016-12-11 22:29:17Z

2

remove '@' from parameter name, must be like this

cmd.Parameters.AddWithValue("param1", user);

answered Dec 11, 2016 at 22:29

Slobodzyan Taras

1792 silver badges12 bronze badges

Sign up to request clarification or add additional context in comments.

6 Comments

Slobodzyan Taras Over a year ago

cmd.Parameters.Add("param1", user);

Slobodzyan Taras Over a year ago

Comissions.ClientId, varchar or INT?

daniel hernandez Over a year ago

Same with that one.

daniel hernandez Over a year ago

It's a nvarchar(128)

Slobodzyan Taras Over a year ago

check "user" for value, maybe "user" is empty or null

|

Collectives™ on Stack Overflow

c# manually binding data to gridview via stored procedure + parameter

1 Answer 1

6 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

6 Comments

Your Answer

Sign up or log in

Post as a guest

Related