[Solved] Is the Id Bug Fixed?

[Adam Gritt]

We are currently using the 2010.2.825 release of the Telerik MVC controls for which the Release Notes state:
Fixed: Id property of ViewComponentBase does not replace "[" and "]" with "_"

but when I look at the code and in using the components the fix is no longer included.  It was fixed in the Beta release but in the current full release the source code is as follows: 

                return HtmlAttributes.ContainsKey("id") ?

                       HtmlAttributes["id"].ToString() :

                       (!string.IsNullOrEmpty(Name) ? Name.Replace(".", HtmlHelper.IdAttributeDotReplacement) : null);

Did this bug fix get rolled out in the change from 2010.2.713 or is the download for the source just not correct?

[Atanas Korchev]

Hello Adam Gritt,

This is the right code. The . in the ID must be replaced with _ as with any of the built-in ASP.NET MVC extensions. Why do you think this code is wrong?

The fix for the bug you are mentioning is in the Start method of the ClientSideObjectWriter class.

Regards,
Atanas Korchev
the Telerik team

[Adam Gritt]

Ok, I see what the problem is now.  

When we were using the Beta release (2010.2.713) the fix was to update the Id value to 
replace the "[" and "]" with "_".  The Release (2010.2.825) changed this so that it 
references items by using the "[" and "]" instead of replacing them with an underscore.  
We had some javascript code that was looking up the controls we had on a page expecting 
the id to be different based on how the previous beta was implemented.