View Single Post
  #2 (permalink)  
Old August 24th, 2008, 03:53 AM
Imar's Avatar
Imar Imar is offline
Wrox Author
Points: 72,055, Level: 100
Points: 72,055, Level: 100 Points: 72,055, Level: 100 Points: 72,055, Level: 100
Activity: 0%
Activity: 0% Activity: 0% Activity: 0%
Join Date: Jun 2003
Location: Utrecht, Netherlands.
Posts: 17,086
Thanks: 80
Thanked 1,587 Times in 1,563 Posts

Hi there,

Why do you need imgStatus as an array of ImageButton controls? Why not just declare a single instance of it inside your loop, just as you do with your other controls? Inside the loop you only seem to use imgStatus(i) and I don't see imgStatus being used elsewhere.

Also, based on what do you think a ImageButton will behave differently for each row? All I see is this:

imgStatus(i).OnClientClick = "'Review.aspx');"

Which means that each button simply opens Review.aspx with no additional state. Maybe you need to pass a query string parameter to Review.aspx?



Imar Spaanjaars
Everyone is unique, except for me.
Author of Beginning ASP.NET 3.5 : in C# and VB, ASP.NET 2.0 Instant Results and Dreamweaver MX 2004
Want to be my colleague? Then check out this post.