Problem with Listing 4-9

RichardF · October 15th, 2004, 03:43 PM

I cannot get this listing to run correctly. I first tried typing it myself, then cut and pasted the downloaded listing and I get the same error.

When I click on Edit, change a column and then click on Update I get the error message...

"Must declare the variable '@CustomerID'"

Any ideas?

stu9820 · October 15th, 2004, 03:47 PM

Can you post your code?

Yippie Ki Yea, Mr. Falcon.

RichardF · October 15th, 2004, 05:04 PM

As I mentioned, I cut and pasted the code from the downloadable code on this website. Apart from correcting the quotes around the "VB" at the top of the page, it is exactly as downloaded from wrox.com.

Here it is...

<%@ Page Language="VB" %>

<html xmlns="http://www.w3.org/1999/xhtml" >
<head id="Head1" runat="server">
    <title>SqlDataSource Control Page</title>
</head>
<body>
    <form id="form1" runat="server">
        <asp:GridView ID="GridView1" Runat="server" Datasourceid="SqlDataSource1"
         AutoGenerateColumns="False" DataKeyNames="CustomerID"
         BackColor="White" GridLines="Vertical"
         BorderStyle="Solid" CellPadding="3" ForeColor="Black"
         BorderColor="#999999" BorderWidth="1px">
           <Columns>
              <asp:CommandField ShowEditButton="True">
              </asp:CommandField>
              <asp:BoundField SortExpression="CustomerID" HeaderText="CustomerID"
               ReadOnly="True" DataField="CustomerID">
              </asp:BoundField>
              <asp:BoundField SortExpression="CompanyName"
               HeaderText="Company Name" DataField="CompanyName">
              </asp:BoundField>
              <asp:BoundField SortExpression="Country" HeaderText="Country"
               DataField="Country">
              </asp:BoundField>
              <asp:BoundField SortExpression="Region" NullDisplayText="NO REGION"
               HeaderText="Region" DataField="Region">
              </asp:BoundField>
           </Columns>
           <FooterStyle BackColor="#CCCCCC">
           </FooterStyle>
           <SelectedRowStyle ForeColor="White" BackColor="#000099"
            Font-Bold="True">
           </SelectedRowStyle>
           <PagerStyle ForeColor="Black" HorizontalAlign="Center"
            BackColor="#999999">
           </PagerStyle>
           <HeaderStyle ForeColor="White" BackColor="Black" Font-Bold="True">
           </HeaderStyle>
           <AlternatingRowStyle BackColor="#CCCCCC">
           </AlternatingRowStyle>
        </asp:GridView>
        <asp:SqlDataSource ID="SqlDataSource1" Runat="server"
         SelectCommand="Select * From Customers"
         UpdateCommand="UPDATE Customers SET CompanyName = @CompanyName, Country = @Country, Region = @Region WHERE (CustomerID = @CustomerID)"
         ConnectionString="Server=(local);Trusted_Connectio n=True;Integrated Security=SSPI;Persist Security Info=True;Database=Northwind"
         ProviderName="System.Data.SqlClient">
            <UpdateParameters>
                <asp:Parameter Name="CustomerID" Type="String">
                </asp:Parameter>
                <asp:Parameter Name="CompanyName" Type="String">
                </asp:parameter>
                <asp:parameter Name="Country" Type="String">
                </asp:parameter>
                <asp:parameter Name="Region" Type="String">
                </asp:parameter>
            </UpdateParameters>
        </asp:SqlDataSource>
    </form>
</body>
</html>

RichardF · October 16th, 2004, 11:10 AM

Just in case it wasn't obvious, I am referring to Listing 4-9 on page 88 of the book. It was mentioned in the subject of this topic, but I forgot to include it in the message itself.

RichardF · October 21st, 2004, 09:12 AM

I assume someone else will hit this problem too, so here is the solution, directly from the author Bill Evjen. As is common with Beta products, Microsoft "tweaked" a few things!

Change the Update command to...

UpdateCommand="UPDATE Customers SET CompanyName = @CompanyName, Country = @Country, Region = @Region WHERE (CustomerID = @original_CustomerID)

...note the "original_" prefix on the CustomerID parameter.